Refactoring of constants handling classes.
authorphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 5 May 2010 17:14:07 +0000 (17:14 +0000)
committerphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 5 May 2010 17:14:07 +0000 (17:14 +0000)
1) Moved some constants that is the for PHOS and
EMCAL up to the Calo baseclass.
2) Removed redudnant Getter functions.
3) Moved the declarations of variables common to  PHOS and EMCAL
up to the Calo baseclass.

HLT/CALO/AliHLTCaloConstants.cxx
HLT/CALO/AliHLTCaloConstants.h
HLT/EMCAL/AliHLTEMCALConstants.cxx
HLT/EMCAL/AliHLTEMCALConstants.h
HLT/EMCAL/AliHLTEMCALRcuCellAccumulatedEnergyDataStruct.h
HLT/PHOS/AliHLTPHOSConstants.cxx
HLT/PHOS/AliHLTPHOSConstants.h

index 526949f..cdc863b 100644 (file)
 
 ClassImp(AliHLTCaloConstants);
 
-AliHLTCaloConstants::AliHLTCaloConstants()
+AliHLTCaloConstants::AliHLTCaloConstants() : fkNALTROS(4),
+                                            fkNALTROCHANNELS(16),
+                                            fkNBRANCHES(2),
+                                            fkCSPSPERFEE(32),
+                                            fkNDATATYPES(10),
+                                            fkMAXHOSTS(20),
+                                            fkDEFAULTEVENTPORT(42001), 
+                                            fkCELLSTEP(-1),
+                                            fkMAXCELLSTEPETA(-1),
+                                            fkMINCELLSTEPETA(-1),
+                                            fkCELLSTEPPHI(-1),
+                                            fkCELLHEIGHT(-1),
+                                            fkCELLANGLE(-1)
+                                            fkRADLENGTH(-1),
+                                            fkCRITICENERGY(-1),
+                                            fkCJ(-1),
+                                            fkDDLOFFSET(-1)
 {
   //Default constructor
 }
index 5c07aba..2f3c433 100644 (file)
@@ -35,15 +35,15 @@ public:
 
   AliHLTCaloConstants();
   virtual ~AliHLTCaloConstants();
-  virtual Int_t GetMAXHOSTS() const = 0;                                       
-  virtual Int_t GetDEFAULTEVENTPORT() const = 0;                              
-  static  Int_t GetALTROMAXSAMPLES()   { return fgkALTROMAXSAMPLES; }; 
+  virtual Int_t GetMAXHOSTS() const  { return fkMAXHOSTS; } ;                                  
+  virtual Int_t GetDEFAULTEVENTPORT()  const { return fkDEFAULTEVENTPORT; };                          
+  static  Int_t GetALTROMAXSAMPLES()    { return fgkALTROMAXSAMPLES; }; 
   static  Int_t GetALTROMAXPRESAMPLES()  { return fgkALTROMAXPRESAMPLES; } ; 
   
-  static  Int_t GetNGAINS()    {  return fgkNGAINS; };         
-  static  Int_t GetHIGHGAIN()  { return fgkHIGHGAIN; };
-  static  Int_t GetLOWGAIN()   { return fgkLOWGAIN; }; 
-  static  Int_t GetMAXBINVALUE() {return fkMAXBINVALUE; };     
+  static  Int_t GetNGAINS()     {  return fgkNGAINS; };        
+  static  Int_t GetHIGHGAIN()   { return fgkHIGHGAIN; };
+  static  Int_t GetLOWGAIN()    { return fgkLOWGAIN; }; 
+  static  Int_t GetMAXBINVALUE()  {return fkMAXBINVALUE; };    
   
   
   // virtual Int_t GetNZROWSMOD() const { return fgkNZROWSMOD;} 
@@ -53,51 +53,58 @@ public:
   virtual Int_t GetNXCOLUMNSRCU() const = 0;
   virtual Int_t GetNXCOLUMNSMOD() const = 0;                           
   //  virtual Int_t GetNGAINS() const = 0;                             
-  virtual Int_t GetNDATATYPES() const = 0;                                     
+  virtual Int_t GetNDATATYPES() const { return fkNDATATYPES; } ;                                       
   virtual Int_t GetPFMAXPATHLENGTH() const = 0;
   virtual Int_t GetPFDEFAULTNSAMPLES() const = 0;                              
   virtual Int_t GetPFDEFAULTSTARTINDEX() const = 0;                            
   virtual Double_t GetDEFAULTTAU() const = 0;                                  
   virtual Int_t GetDEFAULTFS() const = 0;                                     
+  Int_t GetCSPSPERFEE() const {  return fkCSPSPERFEE; };                                       
  
-  
-  virtual Int_t GetCSPSPERFEE() const = 0;                                     
-  // virtual Int_t GetRCU0() const = 0;                                                
-  // virtual Int_t GetRCU1() const = 0;                                                
-  // virtual Int_t GetRCU2() const = 0;                                                
-  // virtual Int_t GetRCU3() const = 0; 
-  // virtual Int_t GetZ0() const = 0;          
-  // virtual Int_t GetZ1() const = 0;                                          
-  // virtual Int_t GetX0() const = 0;                                          
-  // virtual Int_t GetX1() const = 0;                                         
-  
-
-
   virtual Int_t GetNMODULES() const = 0;                                       
   virtual Int_t GetNRCUS() const = 0;                                          
   virtual Int_t GetNRCUSPERMODULE() const = 0;                                         
   virtual Int_t GetNRCUSPERTOTAL() const = 0;                                  
   virtual Int_t GetNFEECS() const = 0;                                                 
-  virtual Int_t GetNALTROS() const = 0;                                        
-  virtual Int_t GetNALTROCHANNELS() const = 0;                                         
-  virtual Int_t GetNBRANCHES() const = 0;                                      
-  virtual Float_t GetCELLSTEP() const = 0;
-  // EMCAL specific
-  virtual Float_t GetMAXCELLSTEPETA() const { return 0; };  //FR
-  virtual Float_t GetMINCELLSTEPETA() const  { return 0;  }//FR
-  virtual Float_t GetCELLSTEPPHI() const { return 0; }          //FR
-  virtual Float_t GetCELLHEIGHT() const { return 0; }       //FR
-  virtual Float_t GetCELLANGLE() const { return 0; }      //FR
-  virtual Float_t GetRADLENGTH() const { return 0; }       //FR
-  virtual Float_t GetCRITICENERGY() const { return 0; };        //FR
-  virtual Float_t GetCJ() const { return 0; }
-  // end
-  virtual Int_t GetNRCUSPERSECTOR() const = 0;                                         
+  Int_t GetNALTROS() {  return fkNALTROS; };                                   
+  Int_t GetNALTROCHANNELS() const { return fkNALTROCHANNELS; };                                        
+  Int_t GetNBRANCHES() const { return fkNBRANCHES; };                                  
+   // EMCAL specific
+  Float_t GetCELLSTEP() const { return fkCELLSTEP; }
+  Float_t GetMAXCELLSTEPETA() const { return fkMAXCELLSTEPETA; }  //FR
+  Float_t GetMINCELLSTEPETA() const { return fkMINCELLSTEPETA; }  //FR
+  Float_t GetCELLSTEPPHI() const { return fkCELLSTEPPHI; }        //FR
+  Float_t GetCELLHEIGHT() const { return fkCELLHEIGHT; }        //FR
+  Float_t GetCELLANGLE() const { return fkCELLANGLE; }        //FR
+  Float_t GetRADLENGTH() const { return fkRADLENGTH; }        //FR
+  Float_t GetCRITICENERGY() const { return fkCRITICENERGY; }        //FR
+  Float_t GetCJ() const { return fkCJ;} //FR
   virtual Int_t GetDDLOFFSET() const = 0;
-  virtual TString GetDETNAME() const = 0;
+  TString GetDETNAME() { return fkDETNAME; };
+  
+  
+protected:
+  TString fkDETNAME;
+  Float_t fkCELLSTEP; //Constant
+  Float_t fkMAXCELLSTEPETA;
+  Float_t fkMINCELLSTEPETA;
+  Float_t fkCELLSTEPPHI;
+  Float_t fkCELLHEIGHT;
+  Float_t fkCELLANGLE;
+  Float_t fkRADLENGTH;
+  Float_t fkCRITICENERGY;
+  Float_t fkCJ;
+  Int_t fkDDLOFFSET;   //Constant
   
- public:
+private:
+  const Int_t fkNALTROS;  /**<Number of ALTROs per frontend card*/
+  const Int_t fkNALTROCHANNELS;  //Constant
+  const Int_t fkNBRANCHES; //Constant
+  const Int_t fkCSPSPERFEE; //Constant
+  const Int_t fkNDATATYPES;          //Constant
+  const Int_t fkMAXHOSTS;                  //Constant
+  const Int_t fkDEFAULTEVENTPORT;          //Constant
   ClassDef(AliHLTCaloConstants, 1);
 };
 
index 779f87c..189d00b 100644 (file)
@@ -29,50 +29,30 @@ ClassImp(AliHLTEMCALConstants);
 
 AliHLTEMCALConstants::AliHLTEMCALConstants() :
   AliHLTCaloConstants(),
-  fkMAXHOSTS(20),
-  fkDEFAULTEVENTPORT(42001),
   fkNZROWSRCU(56),
   fkNXCOLUMNSRCU(16),
   fkNZROWSMOD(48),
   fkNXCOLUMNSMOD(24),
-  fkNDATATYPES(10),
-  fkPFMAXPATHLENGTH(256),
-  fkPFDEFAULTNSAMPLES(70),
-  fkPFDEFAULTSTARTINDEX(0),
-  fkDEFAULTTAU(0.2),
-  fkDEFAULTFS(10),
-  fkCSPSPERFEE(32),
-
-  // fkRCU0(0),
-  // fkRCU1(1),
-  // fkRCU2(2),
-  // fkRCU3(3),
-  // fkZ0(0),
-  // fkZ1(1),
-  // fkX0(0),
-  // fkX1(1),
   fkNMODULES(10),
   fkNRCUS(4),
   fkNRCUSPERMODULE(2),
   fkNRCUSPERTOTAL(fkNMODULES*fkNRCUSPERMODULE),
-  fkNFEECS(9),
-  fkNALTROS(4),
-  fkNALTROCHANNELS(16),
-  fkNBRANCHES(2), 
-  fkCELLSTEP(6.0),
-  fkMAXCELLSTEPETA(6.32),      // FR: max tower dimension along eta
-  fkMINCELLSTEPETA(5.99),      // FR: min tower dimension along eta
-  fkCELLSTEPPHI(6.04667),      // FR: tower dimension along phi
-  fkCELLHEIGHT(27.74),         // FR: tower height
-  fkCELLANGLE(1.50),           // FR: tower tapeiring angle (DEG)
-  fkRADLENGTH(1.23),
-  fkCRITICENERGY(8),
-  fkCJ(0.5),
-  fkNRCUSPERSECTOR(4),
-  fkDDLOFFSET(4608),
-  fkDETNAME("EMCAL")
+  fkNFEECS(9)
 {
+  fkDETNAME = "EMCAL";
+    //  fkNBRANCHES(2), 
+  fkCELLSTEP = 6.0;
+  fkMAXCELLSTEPETA =6.32;      // FR: max tower dimension along eta
+  fkMINCELLSTEPETA = 5.99;     // FR: min tower dimension along eta
+  fkCELLSTEPPHI = 6.04667;     // FR: tower dimension along phi
+  fkCELLHEIGHT =  27.74;       // FR: tower height
+  fkCELLANGLE  =  1.50;                // FR: tower tapeiring angle (DEG)
+  fkRADLENGTH  =  1.23;
+  fkCRITICENERGY =  8;
+  fkCJ  =   0.5;
+  //  fkNRCUSPERSECTOR(4),
+  fkDDLOFFSET =  4608;
+
   //Default constructor
 }
 
index 8a6467c..8128037 100644 (file)
@@ -30,30 +30,18 @@ class AliHLTEMCALConstants : public AliHLTCaloConstants
 
 
 public:
-  
   AliHLTEMCALConstants();
   ~AliHLTEMCALConstants();
-  Int_t GetMAXHOSTS() const { return fkMAXHOSTS;}
-  Int_t GetDEFAULTEVENTPORT() const { return fkDEFAULTEVENTPORT; }
   Int_t GetNZROWSRCU() const { return fkNZROWSRCU;}
   Int_t GetNXCOLUMNSRCU() const { return fkNXCOLUMNSRCU;} 
   Int_t GetNZROWSMOD() const { return fkNZROWSMOD;} 
   Int_t GetNXCOLUMNSMOD() const { return fkNXCOLUMNSMOD;} 
-  Int_t GetNDATATYPES() const { return fkNDATATYPES;} 
-  Int_t GetPFMAXPATHLENGTH() const { return fkPFMAXPATHLENGTH;} 
-  Int_t GetPFDEFAULTNSAMPLES() const{ return fkPFDEFAULTNSAMPLES;} 
-  Int_t GetPFDEFAULTSTARTINDEX() const { return fkPFDEFAULTSTARTINDEX;} 
-  Double_t GetDEFAULTTAU() const { return fkDEFAULTTAU;} 
-  Int_t GetDEFAULTFS() const { return fkDEFAULTFS;} 
-  Int_t GetCSPSPERFEE() const { return fkCSPSPERFEE;} 
   Int_t GetNMODULES() const { return fkNMODULES;} 
   Int_t GetNRCUS() const { return fkNRCUS;} 
   Int_t GetNRCUSPERMODULE() const { return fkNRCUSPERMODULE;} 
   Int_t GetNRCUSPERTOTAL() const { return fkNRCUSPERTOTAL;} 
   Int_t GetNFEECS() const { return fkNFEECS;} 
-  Int_t GetNALTROS() const { return fkNALTROS;} 
-  Int_t GetNALTROCHANNELS() const { return fkNALTROCHANNELS;} 
-  Int_t GetNBRANCHES() const { return fkNBRANCHES;} 
+  
   Float_t GetCELLSTEP() const { return fkCELLSTEP; }
   Float_t GetMAXCELLSTEPETA() const { return fkMAXCELLSTEPETA; }  //FR
   Float_t GetMINCELLSTEPETA() const { return fkMINCELLSTEPETA; }  //FR
@@ -62,51 +50,20 @@ public:
   Float_t GetCELLANGLE() const { return fkCELLANGLE; }        //FR
   Float_t GetRADLENGTH() const { return fkRADLENGTH; }        //FR
   Float_t GetCRITICENERGY() const { return fkCRITICENERGY; }        //FR
-  Float_t GetGJ() const { return fkCJ;} //FR
-  Int_t GetNRCUSPERSECTOR() const { return fkNRCUSPERSECTOR; }                                         
+  Float_t GetCJ() const { return fkCJ;} //FR
   Int_t GetDDLOFFSET() const { return fkDDLOFFSET; }
-  TString GetDETNAME() const { return fkDETNAME; }
 
 private:
   /** Constant members */
-  const Int_t fkMAXHOSTS;//Constant
-  const Int_t fkDEFAULTEVENTPORT;//Constant
   const Int_t fkNZROWSRCU; /**<Number of rows per module*/ 
   const Int_t fkNXCOLUMNSRCU;//Constant
   const Int_t fkNZROWSMOD;  /**<Number of rows per module*/ 
   const Int_t fkNXCOLUMNSMOD;  /**<Number of columns per module*/ 
-  const Int_t fkNDATATYPES;//Constant
-  const Int_t fkPFMAXPATHLENGTH;//Constant
-  const Int_t fkPFDEFAULTNSAMPLES;//Constant
-  const Int_t fkPFDEFAULTSTARTINDEX;//Constant
-  const Double_t fkDEFAULTTAU; /**<Assume that the signal rise time of the altrp pulses is 2 us (nominal value of the electronics)*/
-  const Int_t fkDEFAULTFS; /**<Assume that the signal is samples with 10 MHZ samle rate*/
-  
-  const Int_t fkCSPSPERFEE;//Constant
-  
   const Int_t fkNMODULES;   /**<Number of modules of the EMCAL detector*/
   const Int_t fkNRCUS;   /**<Number of RCUs per Module*/
   const Int_t fkNRCUSPERMODULE;   /**<Number of RCUs per Module*/
   const Int_t fkNRCUSPERTOTAL; /**<Total number of RCUs for EMCAL*/
   const Int_t fkNFEECS;  /**<Number of Frontend cards per branch*/
-  const Int_t fkNALTROS;  /**<Number of ALTROs per frontend card*/
-  const Int_t fkNALTROCHANNELS; //Constant
-  const Int_t fkNBRANCHES; //Constant
-
-  const Float_t fkCELLSTEP; //Constant
-  const Float_t fkMAXCELLSTEPETA;
-  const Float_t fkMINCELLSTEPETA;
-  const Float_t fkCELLSTEPPHI;
-  const Float_t fkCELLHEIGHT;
-  const Float_t fkCELLANGLE;
-  const Float_t fkRADLENGTH;
-  const Float_t fkCRITICENERGY;
-  const Float_t fkCJ;
-
-  const Int_t fkNRCUSPERSECTOR; //Constant
-  const Int_t fkDDLOFFSET;   //Constant
-  const TString fkDETNAME;  //Constant
-
   ClassDef(AliHLTEMCALConstants, 1)
 
 };
index 89c2298..ece0366 100644 (file)
  * provided "as is" without express or implied warranty.                   *
  **************************************************************************/
 
-//#include "AliHLTPHOSCommonDefs.h"
-//#include "AliHLTCaloConstant.h"
-// using namespace PhosHLTConst;
-//using namespace CaloHLTConst;
 
 #include "AliHLTEMCALConstant.h"
 
index fdf2b98..3eaaeef 100644 (file)
@@ -29,38 +29,20 @@ ClassImp(AliHLTPHOSConstants);
 
 AliHLTPHOSConstants::AliHLTPHOSConstants() :
   AliHLTCaloConstants(),
-  fkMAXHOSTS(20),
-  fkDEFAULTEVENTPORT(42001),
   fkNZROWSRCU(56),
   fkNXCOLUMNSRCU(16),
   fkNZROWSMOD(56),
   fkNXCOLUMNSMOD(64),
-  fkNDATATYPES(10),
-  fkCSPSPERFEE(32),
-  
-  // fkRCU0(0),
-  // fkRCU1(1),
-  // fkRCU2(2),
-  // fkRCU3(3),
-  // fkZ0(0),
-  // fkZ1(1),
-  // fkX0(0),
-  // fkX1(1),
-
   fkNMODULES(5),
   fkNRCUS(4),
   fkNRCUSPERMODULE(4),
   fkNRCUSPERTOTAL(fkNMODULES*fkNRCUSPERMODULE),
-  fkNFEECS(14),
-  fkNALTROS(4),
-  fkNALTROCHANNELS(16),
-  fkNBRANCHES(2),
-  fkCELLSTEP(2.255),
-  //  fkNRCUSPERSECTOR(-9999),
-  fkDDLOFFSET(1792),
-  fkDETNAME("PHOS")
+  fkNFEECS(14)
 {
   //Default constructor
+  fkDETNAME = "PHOS";
+  fkCELLSTEP = 2.255;
+  fkDDLOFFSET = 1792;
 }
 
 AliHLTPHOSConstants::~AliHLTPHOSConstants()
index 1b66042..7b71db0 100644 (file)
@@ -22,51 +22,29 @@ class AliHLTPHOSConstants : public AliHLTCaloConstants
 public:
   AliHLTPHOSConstants();
   ~AliHLTPHOSConstants();
-  Int_t GetMAXHOSTS() const { return fkMAXHOSTS;}                              
-  Int_t GetDEFAULTEVENTPORT() const { return fkDEFAULTEVENTPORT; }             
   Int_t GetNZROWSRCU() const { return fkNZROWSRCU;}                            
   Int_t GetNXCOLUMNSRCU() const { return fkNXCOLUMNSRCU;}                      
   Int_t GetNZROWSMOD() const { return fkNZROWSMOD;}                            
   Int_t GetNXCOLUMNSMOD() const { return fkNXCOLUMNSMOD;}                      
-  Int_t GetNDATATYPES() const { return fkNDATATYPES;}                          
-  Int_t GetCSPSPERFEE() const { return fkCSPSPERFEE;}                          
   Int_t GetNMODULES() const { return fkNMODULES;}                              
   Int_t GetNRCUS() const { return fkNRCUS;}                                    
   Int_t GetNRCUSPERMODULE() const { return fkNRCUSPERMODULE;}                  
   Int_t GetNRCUSPERTOTAL() const { return fkNRCUSPERTOTAL;}                    
   Int_t GetNFEECS() const { return fkNFEECS;}                                  
-  Int_t GetNALTROS() const { return fkNALTROS;}                                
-  Int_t GetNALTROCHANNELS() const { return fkNALTROCHANNELS;}                  
-  Int_t GetNBRANCHES() const { return fkNBRANCHES;}                            
-  Float_t GetCELLSTEP() const { return fkCELLSTEP; }                           //   Int_t GetNRCUSPERSECTOR() const { return fkNRCUSPERSECTOR; }                         Int_t GetDDLOFFSET() const { return fkDDLOFFSET; }
-  TString GetDETNAME() const { return fkDETNAME; }
+  //  Float_t GetCELLSTEP() const { return fkCELLSTEP; }                               
   
 private:
   /** Constant members */
-  const Int_t fkMAXHOSTS;                  //Constant
-  const Int_t fkDEFAULTEVENTPORT;          //Constant
   const Int_t fkNZROWSRCU; /**<Number of rows per module*/ 
   const Int_t fkNXCOLUMNSRCU;          //Constant
   const Int_t fkNZROWSMOD;  /**<Number of rows per module*/ 
   const Int_t fkNXCOLUMNSMOD;  /**<Number of columns per module*/ 
-  const Int_t fkNDATATYPES;          //Constant
-  const Int_t fkCSPSPERFEE; //Constant
-  
   const Int_t fkNMODULES;   /**<Number of modules of the PHOS detector*/
   const Int_t fkNRCUS;   /**<Number of RCUs per Module*/
   const Int_t fkNRCUSPERMODULE;   /**<Number of RCUs per Module*/
   const Int_t fkNRCUSPERTOTAL; /**<Total number of RCUs for PHOS*/
   const Int_t fkNFEECS;  /**<Number of Frontend cards per branch*/
-  const Int_t fkNALTROS;  /**<Number of ALTROs per frontend card*/
-  const Int_t fkNALTROCHANNELS;  //Constant
-  const Int_t fkNBRANCHES; //Constant
-
-  const Float_t fkCELLSTEP;  //Constant
-  //  const Int_t fkNRCUSPERSECTOR;  //Constant
-  const Int_t fkDDLOFFSET;  //Constant
-  const TString fkDETNAME;  //Constant
-   
+  
   ClassDef(AliHLTPHOSConstants, 1);
 
 };