]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - ZDC/AliZDCDigitizer.cxx
Fixes for coverity
[u/mrichter/AliRoot.git] / ZDC / AliZDCDigitizer.cxx
index 79fc6933ade8204d87ef4fca3e17dc859af93a04..838903c4fd41077724810d0e81f9016339a35847 100644 (file)
@@ -64,7 +64,8 @@ AliZDCDigitizer::AliZDCDigitizer() :
   fBeamEnergy(0.),
   fBeamType(""),
   fIspASystem(kFALSE),
-  fIsRELDISgen(kFALSE)
+  fIsRELDISgen(kFALSE),
+  fSpectatorData(0)
 {
   // Default constructor    
   for(Int_t i=0; i<2; i++) fADCRes[i]=0.;
@@ -81,7 +82,8 @@ AliZDCDigitizer::AliZDCDigitizer(AliDigitizationInput* digInput):
   fBeamEnergy(0.),
   fBeamType(""),
   fIspASystem(kFALSE),
-  fIsRELDISgen(kFALSE)
+  fIsRELDISgen(kFALSE),
+  fSpectatorData(NULL)
 {
   // Get calibration data
   if(fIsCalibration!=0) printf("\n\t AliZDCDigitizer -> Creating calibration data (pedestals)\n");
@@ -95,7 +97,10 @@ AliZDCDigitizer::AliZDCDigitizer(AliDigitizationInput* digInput):
 AliZDCDigitizer::~AliZDCDigitizer()
 {
 // Destructor
-// Not implemented
+   if(fSpectatorData!=NULL){
+      fSpectatorData->Close();
+      delete fSpectatorData;
+   } 
 }
 
 
@@ -110,7 +115,8 @@ AliZDCDigitizer::AliZDCDigitizer(const AliZDCDigitizer &digitizer):
   fBeamEnergy(digitizer.fBeamEnergy),
   fBeamType(digitizer.fBeamType),
   fIspASystem(digitizer.fIspASystem),
-  fIsRELDISgen(digitizer.fIsRELDISgen)
+  fIsRELDISgen(digitizer.fIsRELDISgen),
+  fSpectatorData(digitizer.fSpectatorData)
 {
   // Copy constructor
 
@@ -170,7 +176,7 @@ Bool_t AliZDCDigitizer::Init()
   /*if(fIspASystem){
     fBeamType = "p-A";
     AliInfo(" AliZDCDigitizer -> Manually setting beam type to p-A\n");
-    }*/
+  }*/
   
   // Setting beam type for spectator generator and RELDIS generator
   if(((fBeamType.CompareTo("UNKNOWN")) == 0) || fIsRELDISgen){
@@ -178,7 +184,7 @@ Bool_t AliZDCDigitizer::Init()
      AliInfo(" AliZDCDigitizer -> Manually setting beam type to A-A\n");
   }    
   printf("\n\t  AliZDCDigitizer ->  beam type %s  - beam energy = %f GeV\n", fBeamType.Data(), fBeamEnergy);
-  if(fSpectators2Track) printf("\t  AliZDCDigitizer ->  spectator signal added at digit level\n");
+  if(fSpectators2Track) printf("\n\t  AliZDCDigitizer ->  spectator signal added at digit level\n\n");
   
   if(fBeamEnergy>0.1){
     ReadPMTGains();
@@ -323,10 +329,10 @@ void AliZDCDigitizer::Digitize(Option_t* /*option*/)
       specPProj = hijingHeader->ProjSpectatorsp();
       specNTarg = hijingHeader->TargSpectatorsn();
       specPTarg = hijingHeader->TargSpectatorsp();
-      AliInfo(Form("\t AliZDCDigitizer: b = %1.2f fm\n"
+      /*printf("\t AliZDCDigitizer: b = %1.2f fm\n"
       " \t    PROJECTILE:  #spectator n %d, #spectator p %d\n"
       " \t    TARGET:  #spectator n %d, #spectator p %d\n", 
-      impPar, specNProj, specPProj, specNTarg, specPTarg));
+      impPar, specNProj, specPProj, specNTarg, specPTarg);*/
     
       // Applying fragmentation algorithm and adding spectator signal
       Int_t freeSpecNProj=0, freeSpecPProj=0;
@@ -335,10 +341,10 @@ void AliZDCDigitizer::Digitize(Option_t* /*option*/)
       if(specNTarg!=0 || specPTarg!=0) Fragmentation(impPar, specNTarg, specPTarg, freeSpecNTarg, freeSpecPTarg);
       if(freeSpecNProj!=0) SpectatorSignal(1, freeSpecNProj, pm);
       if(freeSpecPProj!=0) SpectatorSignal(2, freeSpecPProj, pm);
-      AliInfo(Form("\t AliZDCDigitizer -> Adding spectator signal for PROJECTILE: %d free  n and %d free p\n",freeSpecNProj,freeSpecPProj));
+      //printf("\t AliZDCDigitizer -> Adding spectator signal for PROJECTILE: %d free  n and %d free p\n",freeSpecNProj,freeSpecPProj);
       if(freeSpecNTarg!=0) SpectatorSignal(3, freeSpecNTarg, pm);
       if(freeSpecPTarg!=0) SpectatorSignal(4, freeSpecPTarg, pm);
-      AliInfo(Form("\t AliZDCDigitizer -> Adding spectator signal for TARGET: %d free  n and %d free p\n",freeSpecNTarg,freeSpecPTarg));
+      //printf("\t AliZDCDigitizer -> Adding spectator signal for TARGET: %d free  n and %d free p\n",freeSpecNTarg,freeSpecPTarg);
     }
   }
 
@@ -378,7 +384,7 @@ void AliZDCDigitizer::Digitize(Option_t* /*option*/)
 
        //Ch. debug
        //printf("\t DIGIT added -> det %d quad %d - digi[0,1] = [%d, %d]\n",
-       //     sector[0], sector[1], digi[0], digi[1]); // Chiara debugging!
+         //   sector[0], sector[1], digi[0], digi[1]); // Chiara debugging!
        
     }
   } // Loop over detector
@@ -462,15 +468,15 @@ void AliZDCDigitizer::ReadPMTGains()
        read = fscanf(fdata,"%f ",&data[ic]);
        if(read==0) AliDebug(3, " Error in reading PMT gains from external file ");
     }
-    beam[ir] = data[0];
-    det[ir] = data[1];
+    beam[ir] = (int) (data[0]);
+    det[ir]  = (int) (data[1]);
     gain[ir] = data[2];
     aEne[ir] = data[3];
     bEne[ir] = data[4];
   }
   fclose(fdata);
   
-  if(((fBeamType.CompareTo("P-P")) == 0)){
+  if(((fBeamType.CompareTo("P-P")) == 0) || ((fBeamType.CompareTo("p-p")) == 0)){
     for(int i=0; i<12; i++){
       if(beam[i]==0 && fBeamEnergy!=0.){
         if(det[i]!=31 && det[i]!=32){
@@ -635,13 +641,12 @@ void AliZDCDigitizer::Fragmentation(Float_t impPar, Int_t specN, Int_t specP,
 }
 
 //_____________________________________________________________________________
-void AliZDCDigitizer::SpectatorSignal(Int_t SpecType, Int_t numEvents, 
-                                      Float_t pm[5][5]) const
+void AliZDCDigitizer::SpectatorSignal(Int_t SpecType, Int_t numEvents, Float_t pm[5][5]) 
 {
 // add signal of the spectators
   
-  TFile *specSignalFile = TFile::Open("$ALICE_ROOT/ZDC/SpectatorSignal.root");
-  if(!specSignalFile || !specSignalFile->IsOpen()) {
+  if(!fSpectatorData) fSpectatorData = TFile::Open("$ALICE_ROOT/ZDC/SpectatorSignal.root");
+  if(!fSpectatorData || !fSpectatorData->IsOpen()) {
     AliError((" Opening file $ALICE_ROOT/ZDC/SpectatorSignal.root failed\n"));
     return;
   }
@@ -652,43 +657,43 @@ void AliZDCDigitizer::SpectatorSignal(Int_t SpecType, Int_t numEvents,
   //
   if(TMath::Abs(sqrtS-5500) < 100.){
     AliInfo(" Extracting signal from SpectatorSignal/energy5500 ");
-    specSignalFile->cd("energy5500");
+    fSpectatorData->cd("energy5500");
     //
     if(SpecType == 1) {           // --- Signal for projectile spectator neutrons
-      specSignalFile->GetObject("energy5500/ZNCSignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy5500/ZNCSignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZNCSignal from SpectatorSignal.root file");
     } 
     else if(SpecType == 2) { // --- Signal for projectile spectator protons
-      specSignalFile->GetObject("energy5500/ZPCSignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy5500/ZPCSignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZPCSignal from SpectatorSignal.root file");
     }
     else if(SpecType == 3) { // --- Signal for target spectator neutrons
-      specSignalFile->GetObject("energy5500/ZNASignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy5500/ZNASignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZNASignal from SpectatorSignal.root file");
     }
     else if(SpecType == 4) { // --- Signal for target spectator protons
-      specSignalFile->GetObject("energy5500/ZPASignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy5500/ZPASignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZPASignal from SpectatorSignal.root file");
     }
   }
   else if(TMath::Abs(sqrtS-2760) < 100.){
     AliInfo(" Extracting signal from SpectatorSignal/energy2760 ");
-    specSignalFile->cd("energy2760");
+    fSpectatorData->cd("energy2760");
     //
     if(SpecType == 1) {           // --- Signal for projectile spectator neutrons
-      specSignalFile->GetObject("energy2760/ZNCSignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy2760/ZNCSignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZNCSignal from SpectatorSignal.root file");
     } 
     else if(SpecType == 2) { // --- Signal for projectile spectator protons
-      specSignalFile->GetObject("energy2760/ZPCSignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy2760/ZPCSignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZPCSignal from SpectatorSignal.root file");
     }
     else if(SpecType == 3) { // --- Signal for target spectator neutrons
-      specSignalFile->GetObject("energy2760/ZNASignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy2760/ZNASignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZNASignal from SpectatorSignal.root file");
     }
     else if(SpecType == 4) { // --- Signal for target spectator protons
-      specSignalFile->GetObject("energy2760/ZPASignal;1",zdcSignal);
+      fSpectatorData->GetObject("energy2760/ZPASignal;1",zdcSignal);
       if(!zdcSignal) AliError("  PROBLEM!!! Can't retrieve ZPASignal from SpectatorSignal.root file");
     }
   }
@@ -743,8 +748,6 @@ void AliZDCDigitizer::SpectatorSignal(Int_t SpecType, Int_t numEvents,
      }
   }while(iev<numEvents);
   
-  specSignalFile->Close();
-  delete specSignalFile;
 }
 
 
@@ -766,7 +769,7 @@ Int_t AliZDCDigitizer::Pedestal(Int_t Det, Int_t Quad, Int_t Res) const
 {
   // Returns a pedestal for detector det, PM quad, channel with res.
   //
-  Float_t pedValue;
+  Float_t pedValue=0.;
   // Normal run
   if(fIsCalibration == 0){
     Int_t index=0, kNch=24;
@@ -779,13 +782,18 @@ Int_t AliZDCDigitizer::Pedestal(Int_t Det, Int_t Quad, Int_t Res) const
     }
     else index = (Det-1)/3+22+kNch*Res; // Reference PMs
     //
-    Float_t meanPed = fPedData->GetMeanPed(index);
-    Float_t pedWidth = fPedData->GetMeanPedWidth(index);
-    pedValue = gRandom->Gaus(meanPed,pedWidth);
-    //
-    /*printf("\t  AliZDCDigitizer::Pedestal -> det %d quad %d res %d - Ped[%d] = %d\n",
-       Det, Quad, Res, index,(Int_t) pedValue); // Chiara debugging!
-    */
+    if(fPedData){
+      Float_t meanPed = fPedData->GetMeanPed(index);
+      Float_t pedWidth = fPedData->GetMeanPedWidth(index);
+      pedValue = gRandom->Gaus(meanPed,pedWidth);
+      //
+      /*printf("\t  AliZDCDigitizer::Pedestal -> det %d quad %d res %d - Ped[%d] = %d\n",
+       Det, Quad, Res, index,(Int_t) pedValue); // Chiara debugging!
+      */
+    }
+    else{
+      printf("  AliZDCDigitizer::Pedestal -> No valid pedestal calibration object loaded!\n\n");
+    }
   }
   // To create calibration object
   else{
@@ -826,13 +834,13 @@ AliZDCPedestals* AliZDCDigitizer::GetPedData() const
 {
 
   // Getting pedestal calibration object for ZDC set
-
+  AliZDCPedestals *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/Pedestals");
   if(!entry) AliFatal("No calibration data loaded!");  
-
-  AliZDCPedestals *calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
-
+  else{
+    calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }