Fixing coding conventions (RC22)
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Jun 2011 19:56:10 +0000 (19:56 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Jun 2011 19:56:10 +0000 (19:56 +0000)
28 files changed:
MUON/AliMUONCheck.cxx
MUON/AliMUONCheck.h
MUON/AliMUONGMSSubprocessor.cxx
MUON/AliMUONGMSSubprocessor.h
MUON/AliMUONGeometryBuilder.cxx
MUON/AliMUONGeometryBuilder.h
MUON/AliMUONGeometryDetElement.cxx
MUON/AliMUONGeometryDetElement.h
MUON/AliMUONGeometryModuleTransformer.cxx
MUON/AliMUONGeometryModuleTransformer.h
MUON/AliMUONGeometryTransformer.cxx
MUON/AliMUONGeometryTransformer.h
MUON/mapping/AliMpDDLStore.cxx
MUON/mapping/AliMpDDLStore.h
MUON/mapping/AliMpDEManager.cxx
MUON/mapping/AliMpDEManager.h
MUON/mapping/AliMpDataProcessor.cxx
MUON/mapping/AliMpDataProcessor.h
MUON/mapping/AliMpExMap.cxx
MUON/mapping/AliMpExMap.h
MUON/mapping/AliMpFiles.cxx
MUON/mapping/AliMpFiles.h
MUON/mapping/AliMpSectorReader.cxx
MUON/mapping/AliMpSectorReader.h
MUON/mapping/AliMpStringObjMap.cxx
MUON/mapping/AliMpStringObjMap.h
MUON/mapping/AliMpTriggerReader.cxx
MUON/mapping/AliMpTriggerReader.h

index 531d46f..c39f75e 100644 (file)
 ClassImp(AliMUONCheck)
 /// \endcond
 
-const TString AliMUONCheck::fgkDefaultOutFileName = "output.txt"; //!< default output file name 
+//_____________________________________________________________________________
+const TString& AliMUONCheck::GetDefaultOutFileName()
+{
+  /// Default output file name 
+  static const TString kDefaultOutFileName = "output.txt";
+  return kDefaultOutFileName;
+}  
 
+//_____________________________________________________________________________
 AliMUONCheck::AliMUONCheck(const char* galiceFile, const char* esdFile,Int_t firstEvent, Int_t lastEvent,const char* outDir) 
 : TObject(),
 fFileName(galiceFile),
 fFileNameSim(),
 fesdFileName(esdFile),
 fkOutDir(outDir),
-fOutFileName(fgkDefaultOutFileName),
+fOutFileName(GetDefaultOutFileName()),
 fFirstEvent(firstEvent),
 fLastEvent(lastEvent)
 {
@@ -87,7 +94,7 @@ fFileName(galiceFile),
 fFileNameSim(galiceFileSim),
 fesdFileName(esdFile),
 fkOutDir(outDir),
-fOutFileName(fgkDefaultOutFileName),
+fOutFileName(GetDefaultOutFileName()),
 fFirstEvent(firstEvent),
 fLastEvent(lastEvent)
 {
index 57b75ca..c37f8c0 100644 (file)
@@ -47,7 +47,7 @@ private:
   AliMUONCheck& operator=(const AliMUONCheck& rhs);
   
 private:
-  static const TString fgkDefaultOutFileName; //!< default output file name 
+  static const TString& GetDefaultOutFileName(); 
 
   TString fFileName;   //!< File (galice.root) to read from fro reconstructed data
   TString fFileNameSim; //!< File (galiceSim.root) for simulated data
index ac9a035..8eca844 100644 (file)
 ClassImp(AliMUONGMSSubprocessor)
 /// \endcond
 
-const Int_t    AliMUONGMSSubprocessor::fgkSystem = AliPreprocessor::kDCS;
-const TString  AliMUONGMSSubprocessor::fgkDataId = "GMS";
-const TString  AliMUONGMSSubprocessor::fgkMatrixArrayName = "GMSarray";
+const Int_t AliMUONGMSSubprocessor::fgkSystem = AliPreprocessor::kDCS;
+
+//
+// static methods
+//
+
+//______________________________________________________________________________
+const TString& AliMUONGMSSubprocessor::GetDataId()
+{
+  /// The data Id
+  static const TString kDataId = "GMS";
+  return kDataId;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMUONGMSSubprocessor::GetMatrixArrayName()
+{
+  /// The fixed matrix array name
+  static const TString kMatrixArrayName = "GMSarray";
+  return kMatrixArrayName;
+}  
+
+//
+// ctor, dtor
+//
 
 //______________________________________________________________________________
 AliMUONGMSSubprocessor::AliMUONGMSSubprocessor(AliMUONPreprocessor* master) 
@@ -69,7 +91,6 @@ AliMUONGMSSubprocessor::~AliMUONGMSSubprocessor()
 // private methods
 //
 
-
 //______________________________________________________________________________
 Bool_t  AliMUONGMSSubprocessor::Initialize(Int_t /*run*/, 
                                          UInt_t /*startTime*/, UInt_t /*endTime*/)
@@ -98,7 +119,7 @@ UInt_t AliMUONGMSSubprocessor::ProcessFile(const TString& fileName)
   }  
   
   // Get array with matrices
-  TClonesArray* array = (TClonesArray*)f.Get(fgkMatrixArrayName);
+  TClonesArray* array = (TClonesArray*)f.Get(GetMatrixArrayName());
   if ( ! array ) {
     Master()->Log(Form("TClonesArray not found in file %s",fileName.Data()));
     return 2;
@@ -274,11 +295,11 @@ UInt_t AliMUONGMSSubprocessor::Process(TMap* /*dcsAliasMap*/)
 /// Return failure (0) in case procession of some file has failed
 
   UInt_t result = 1;
-  TList* sources = Master()->GetFileSources(fgkSystem, fgkDataId);
+  TList* sources = Master()->GetFileSources(fgkSystem, GetDataId());
   TIter next(sources);
   TObjString* o(0x0);
   while ( ( o = static_cast<TObjString*>(next()) ) ) {
-    TString fileName(Master()->GetFile(fgkSystem, fgkDataId, o->GetName()));
+    TString fileName(Master()->GetFile(fgkSystem, GetDataId(), o->GetName()));
     result *= ProcessFile(fileName);
   }
   delete sources;
index 143c048..26d3fef 100644 (file)
@@ -38,13 +38,16 @@ class AliMUONGMSSubprocessor : public AliMUONVSubprocessor
     /// Not implemented
     AliMUONGMSSubprocessor& operator=(const AliMUONGMSSubprocessor&);
 
+    // static methods
+    static const TString& GetDataId();         
+    static const TString& GetMatrixArrayName();
+
+    // methods
     UInt_t ProcessFile(const TString& filename);
 
     // static data members
     static const Int_t    fgkSystem;           ///< The data system
-    static const TString  fgkDataId;           ///< The data Id
-    static const TString  fgkMatrixArrayName;  ///< The fixed matrix array name
-  
+
     // data members
     AliMUONGeometryTransformer* fTransformer; ///< Geometry transformer
 
index 4da5cf4..74cfb99 100644 (file)
 #include <TVirtualMC.h>
 #include <TGeoManager.h>
 
-// static data members
-const TString  AliMUONGeometryBuilder::fgkDefaultVolPathsFileName = "volpath.dat";   
-const TString  AliMUONGeometryBuilder::fgkDefaultTransformFileName = "transform.dat";   
-const TString  AliMUONGeometryBuilder::fgkDefaultSVMapFileName = "svmap.dat";    
-const TString  AliMUONGeometryBuilder::fgkOutFileNameExtension = ".out";    
-
 /// \cond CLASSIMP
 ClassImp(AliMUONGeometryBuilder)
 /// \endcond
 
+//
 // static functions
+//
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryBuilder::GetDefaultTransformFileName()
+{
+  ///< default transformations file name                                     
+  static const TString kDefaultTransformFileName = "transform.dat";   
+  return kDefaultTransformFileName;
+}    
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryBuilder::GetDefaultSVMapFileName() 
+{
+  ///< default svmaps file name                                      
+  static const TString kDefaultSVMapFileName = "svmap.dat";    
+  return kDefaultSVMapFileName;
+}    
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryBuilder::GetOutFileNameExtension()
+{
+  ///< default output file name extension                                    
+  static const TString kOutFileNameExtension = ".out";    
+  return kOutFileNameExtension;
+}    
+
 
 //______________________________________________________________________________
 TGeoHMatrix AliMUONGeometryBuilder::Multiply(const TGeoMatrix& m1, 
@@ -113,13 +133,17 @@ TGeoHMatrix AliMUONGeometryBuilder::Multiply(const TGeoMatrix& m1,
   return m1 * m2 * m3 * m4;
 }
 
+//
+// ctors, dtor
+//
+
 //______________________________________________________________________________
 AliMUONGeometryBuilder::AliMUONGeometryBuilder(AliModule* module)
   : TObject(),
     fModule(module),
     fAlign(false),
-    fTransformFileName(fgkDefaultTransformFileName),
-    fSVMapFileName(fgkDefaultSVMapFileName),
+    fTransformFileName(GetDefaultTransformFileName()),
+    fSVMapFileName(GetDefaultSVMapFileName()),
     fGlobalTransformation(), 
     fGeometryBuilders(0),
     fGeometry(0)
index 5a47bef..a55311a 100644 (file)
@@ -66,6 +66,11 @@ class AliMUONGeometryBuilder : public TObject
     AliMUONGeometryBuilder&  operator = (const AliMUONGeometryBuilder& right);
  
   private:
+    // static methods
+    static const TString& GetDefaultTransformFileName();    
+    static const TString& GetDefaultSVMapFileName();    
+    static const TString& GetOutFileNameExtension();        
+
     // method
     void PlaceVolume(const TString& name, const TString& mName, Int_t copyNo, 
              const TGeoHMatrix& matrix, Int_t npar, Double_t* param,
@@ -74,12 +79,6 @@ class AliMUONGeometryBuilder : public TObject
     void CreateGeometryWithoutTGeo();
     void SetAlignToBuilder(AliMUONVGeometryBuilder* builder) const;         
 
-    // static data members
-    static const TString  fgkDefaultVolPathsFileName;  ///< default volume paths file name                                        
-    static const TString  fgkDefaultTransformFileName; ///< default transformations file name                                     
-    static const TString  fgkDefaultSVMapFileName;     ///< default svmaps file name                                      
-    static const TString  fgkOutFileNameExtension;     ///< default output file name extension                                    
-
     // data members
     AliModule*       fModule;              ///< the AliRoot module
     Bool_t           fAlign;               ///< \brief option to read transformations 
@@ -102,7 +101,7 @@ inline void  AliMUONGeometryBuilder::InitGeometry()
 
 /// Write sensitive volume maps
 inline void  AliMUONGeometryBuilder::WriteSVMaps()
-{ WriteSVMaps(fSVMapFileName + fgkOutFileNameExtension); }
+{ WriteSVMaps(fSVMapFileName + GetOutFileNameExtension()); }
 
 /// Return geometry parametrisation
 inline 
index be6b5b0..a423554 100644 (file)
 ClassImp(AliMUONGeometryDetElement)
 /// \endcond
 
-const TString AliMUONGeometryDetElement::fgkDENamePrefix = "DE";
+//
+// static methods
+//
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryDetElement::GetDENamePrefix()
+{
+  ///< Geometry DE name prefix
+  static const TString kDENamePrefix = "DE";
+  return kDENamePrefix;
+}  
 
 //______________________________________________________________________________
 TString AliMUONGeometryDetElement::GetDEName(Int_t detElemId)
 {
 /// Return the module name for given moduleId
 
-  TString deName(fgkDENamePrefix);
+  TString deName(GetDENamePrefix());
   deName += detElemId;
   return deName;
 }   
index 768e663..6b07be1 100644 (file)
@@ -26,7 +26,7 @@ class AliMUONGeometryDetElement : public TObject
     virtual ~AliMUONGeometryDetElement();
 
     // static methods
-    static TString GetDENamePrefix();
+    static const TString& GetDENamePrefix();
     static TString GetDEName(Int_t detElemId);
 
     // methods
@@ -72,9 +72,6 @@ class AliMUONGeometryDetElement : public TObject
     // methods
     void PrintTransform(const TGeoHMatrix* transform) const;
  
-     // static data members
-    static const TString  fgkDENamePrefix;///< Geometry module name prefix
     // data members
     TString       fDEName;     ///< detection element name
     TString       fVolumePath; ///< \brief the full path of aligned volume
@@ -87,10 +84,6 @@ class AliMUONGeometryDetElement : public TObject
 
 // inline functions
 
-/// Return module name prefix
-inline TString AliMUONGeometryDetElement::GetDENamePrefix()
-{ return fgkDENamePrefix; }
-
 /// Set the full path of the aligned volume or envelope in geometry
 inline void AliMUONGeometryDetElement::SetVolumePath(const TString& volumePath)
 { fVolumePath = volumePath; }
index 2f9bf0f..4b771a4 100644 (file)
 ClassImp(AliMUONGeometryModuleTransformer)
 /// \endcond
 
-const TString AliMUONGeometryModuleTransformer::fgkModuleNamePrefix = "GM";
+//
+// static methods
+//
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryModuleTransformer::GetModuleNamePrefix()
+{
+  /// Geometry module name prefix
+  static const TString& kModuleNamePrefix = "GM";
+  return kModuleNamePrefix;
+}  
 
 //______________________________________________________________________________
 TString AliMUONGeometryModuleTransformer::GetModuleName(Int_t moduleId)
 {
 /// Return the module name for given moduleId
 
-  TString moduleName(fgkModuleNamePrefix);
+  TString moduleName(GetModuleNamePrefix());
   moduleName += moduleId;
   return moduleName;
 }   
 
+//
+// ctor, dtor
+//
+
 //______________________________________________________________________________
 AliMUONGeometryModuleTransformer::AliMUONGeometryModuleTransformer(Int_t moduleId)
  : TObject(),
index 2393541..2df899c 100644 (file)
@@ -37,8 +37,8 @@ class AliMUONGeometryModuleTransformer : public TObject
     virtual ~AliMUONGeometryModuleTransformer();
 
     // static methods
-    static TString GetModuleNamePrefix();
-    static TString GetModuleName(Int_t moduleId);
+    static const TString& GetModuleNamePrefix();
+    static       TString  GetModuleName(Int_t moduleId);
 
     // methods
     void Global2Local(Int_t detElemId,
@@ -81,9 +81,6 @@ class AliMUONGeometryModuleTransformer : public TObject
       operator = (const AliMUONGeometryModuleTransformer& rhs);
 
   private:
-    // static data members
-    static const TString  fgkModuleNamePrefix; ///< Geometry module name prefix
-
     // data members
     Int_t                 fModuleId;   ///< the module Id
     TString               fModuleName; ///< the module name
@@ -98,11 +95,6 @@ class AliMUONGeometryModuleTransformer : public TObject
 
 // inline functions
 
-/// Return module name prefix
-inline TString 
-AliMUONGeometryModuleTransformer::GetModuleNamePrefix()
-{ return fgkModuleNamePrefix; }
-
 /// Set the full path of aligned module volume or envelope in geometry
 inline void 
 AliMUONGeometryModuleTransformer::SetVolumePath(const TString& volumePath)
index e7ac07f..76d656f 100644 (file)
 ClassImp(AliMUONGeometryTransformer)
 /// \endcond
 
-const TString  AliMUONGeometryTransformer::fgkDefaultDetectorName = "MUON";
+//
+// static private methods
+//
+
+//______________________________________________________________________________
+const TString& AliMUONGeometryTransformer::GetDefaultDetectorName()
+{ 
+  /// Default detector name
+  static const TString kDefaultDetectorName = "MUON";
+  return kDefaultDetectorName;
+}  
  
+//
+// ctor, dtor
+//
+
 //______________________________________________________________________________
 AliMUONGeometryTransformer::AliMUONGeometryTransformer()
 
   : TObject(),
-    fDetectorName(fgkDefaultDetectorName),
+    fDetectorName(GetDefaultDetectorName()),
     fModuleTransformers(0),
     fMisAlignArray(0),
     fDEAreas(0x0)
index dcff23d..8d85eef 100644 (file)
@@ -104,9 +104,12 @@ class AliMUONGeometryTransformer : public TObject
     AliMUONGeometryTransformer&  operator = (const AliMUONGeometryTransformer& right);
  
   private:
+    // static methods
+    static const TString&  GetDefaultDetectorName(); 
+
     // methods
       
-      void CreateDEAreas() const;
+    void CreateDEAreas() const;
     
     Bool_t LoadMapping() const;
     AliMUONGeometryModuleTransformer* GetModuleTransformerNonConst(
@@ -136,10 +139,6 @@ class AliMUONGeometryTransformer : public TObject
     
     TString GetModuleSymName(Int_t moduleId) const;
     TString GetDESymName(Int_t detElemId) const;
-    
-    // static data members
-    static const TString  fgkDefaultDetectorName; ///< Default detector name
-    
 
     // data members
     TString        fDetectorName;       ///< Detector name
index 2e3805e..d5c579f 100644 (file)
@@ -65,14 +65,28 @@ ClassImp(AliMpDDLStore)
 AliMpDDLStore* AliMpDDLStore::fgInstance = 0;
 const Int_t    AliMpDDLStore::fgkNofDDLs = 20;
 const Int_t    AliMpDDLStore::fgkNofTriggerDDLs = 2;
-const TString  AliMpDDLStore::fgkRevertKeyword = "REVERT"; 
-const TString  AliMpDDLStore::fgkExplicitKeyword = "EXPLICIT";
 
 //
 // static methods
 //
 
 //______________________________________________________________________________
+const TString&  AliMpDDLStore::GetRevertKeyword()
+{
+  /// A keyword for ReadBusPatchSpecial()
+  static const TString kRevertKeyword = "REVERT"; 
+  return kRevertKeyword;
+}  
+
+//______________________________________________________________________________
+const TString&  AliMpDDLStore::GetExplicitKeyword()
+{
+  /// A keyword for ReadBusPatchSpecial()
+  static const TString  kExplicitKeyword = "EXPLICIT";
+  return kExplicitKeyword;
+}  
+
+//______________________________________________________________________________
 AliMpDDLStore* AliMpDDLStore::Instance(Bool_t warn) 
 {
     /// Create the DDL store if it does not yet exist
@@ -516,14 +530,14 @@ Bool_t AliMpDDLStore::ReadBusPatchSpecial()
           return kFALSE;
         }
      
-        if ( sKey == fgkRevertKeyword ) {
+        if ( sKey == GetRevertKeyword() ) {
           AliDebugStream(3)
             << "Reverting readout of bus patch " << busPatchID  << endl;
         
           // Now revert the manus in this bus patch
           busPatch->RevertReadout();     
         }
-        else if ( sKey == fgkExplicitKeyword ) {
+        else if ( sKey == GetExplicitKeyword() ) {
         
           busPatch->ResetReadout();
 
index d9645c2..e61cbe7 100644 (file)
@@ -81,6 +81,10 @@ class AliMpDDLStore : public  TObject {
     /// Not implemented
     AliMpDDLStore& operator=(const AliMpDDLStore& rhs);
 
+    // static methods
+    static const TString&  GetRevertKeyword();
+    static const TString&  GetExplicitKeyword();
+
     // methods
     Int_t  GetManuListIndex(Int_t detElemId) const;
     Int_t  GetBusPatchIndex(Int_t detElemId, Int_t manuId) const;
@@ -97,8 +101,6 @@ class AliMpDDLStore : public  TObject {
     static AliMpDDLStore* fgInstance; ///< Singleton instance
     static const Int_t    fgkNofDDLs; ///< Total number of DDLs
     static const Int_t    fgkNofTriggerDDLs; ///< Total number of trigger DDLs
-    static const TString  fgkRevertKeyword; ///< A keyword for ReadBusPatchSpecial()
-    static const TString  fgkExplicitKeyword; ///< A keyword for ReadBusPatchSpecial()
 
     // data members    
     const AliMpDataStreams&  fkDataStreams;  //!< Data streams
index 0b308e1..8ee81aa 100644 (file)
@@ -42,17 +42,20 @@ ClassImp(AliMpDEManager)
 /// \endcond
 
 const Int_t AliMpDEManager::fgkCoefficient = 100;
-TArrayI     AliMpDEManager::fgNofDEPerChamber;
 
+//
+// static private methods
+//
 //______________________________________________________________________________
-
-AliMpDEManager::~AliMpDEManager()
+TArrayI& AliMpDEManager::GetNofDEPerChamber() 
 {
-/// Destructor
+  /// number of detElemId per chamber
+  static TArrayI nofDEPerChamber;
+  return nofDEPerChamber;
 }
 
 //
-// static private methods
+// static public methods
 //
 
 //______________________________________________________________________________
@@ -231,17 +234,17 @@ Int_t AliMpDEManager::GetNofDEInChamber(Int_t chamberId, Bool_t warn)
   if ( ! IsValidChamberId(chamberId,warn) ) return 0;
 
   // Fill array if it is empty
-  if ( ! fgNofDEPerChamber.GetSize() ) {
-    fgNofDEPerChamber.Set(AliMpConstants::NofChambers());
+  if ( ! GetNofDEPerChamber().GetSize() ) {
+    GetNofDEPerChamber().Set(AliMpConstants::NofChambers());
     AliMpDEIterator it;
     for ( Int_t i=0; i<AliMpConstants::NofChambers(); i++ ) {
       Int_t counter = 0;
       for ( it.First(i); ! it.IsDone(); it.Next() ) ++counter;
-      fgNofDEPerChamber[i] = counter;
+      GetNofDEPerChamber()[i] = counter;
     }  
   }
   
-  return fgNofDEPerChamber[chamberId];    
+  return GetNofDEPerChamber()[chamberId];    
 
 }
 
@@ -259,3 +262,13 @@ MpPair_t AliMpDEManager::GetDetElemIdRange(Int_t chamberId)
   
 }
 
+//
+// ctors, dtor
+//
+
+//______________________________________________________________________________
+AliMpDEManager::~AliMpDEManager()
+{
+/// Destructor
+}
+
index 74afdd5..5d31330 100644 (file)
@@ -64,10 +64,12 @@ class AliMpDEManager : public  TObject {
     AliMpDEManager(const AliMpDEManager& rhs);
     /// Not implemented
     AliMpDEManager& operator=(const AliMpDEManager& rhs);
+
+    // static methods
+    static  TArrayI&  GetNofDEPerChamber();
       
     // static data members     
-    static const Int_t   fgkCoefficient; ///< Coefficient used in DE Id <-> station
-    static    TArrayI fgNofDEPerChamber; ///< Number of detElemId per chamber
+    static const Int_t  fgkCoefficient; ///< Coefficient used in DE Id <-> station
 
   ClassDef(AliMpDEManager,0)  // The manager class for definition of detection element types
 };
index 6326706..9cc456a 100644 (file)
 #include <sstream>
 #include <map>
 
-const TString AliMpDataProcessor::fgkHeaderFileName = "add.h";
-const TString AliMpDataProcessor::fgkImplFileName = "add.cxx";
+//
+// static private methods
+//
+
+//_____________________________________________________________________________
+const TString& AliMpDataProcessor::GetHeaderFileName() 
+{
+  /// Return the default name for generated header file
+  static const TString kHeaderFileName = "add.h";
+  return kHeaderFileName;
+}    
 
+//_____________________________________________________________________________
+const TString& AliMpDataProcessor::GetImplFileName() 
+{
+  /// Return the default name for generated implementation file
+  static const TString kImplFileName = "add.cxx";
+  return kImplFileName;
+}  
+
+//
+// ctors, dtor
+//
 
 //_____________________________________________________________________________
 AliMpDataProcessor::AliMpDataProcessor()
@@ -52,8 +72,8 @@ AliMpDataProcessor::AliMpDataProcessor()
 {
 /// Default and standar constructor
 
-  fHeaderFile.open(fgkHeaderFileName.Data(), std::ios::out);
-  fImplFile.open(fgkImplFileName.Data(), std::ios::out);
+  fHeaderFile.open(GetHeaderFileName().Data(), std::ios::out);
+  fImplFile.open(GetImplFileName().Data(), std::ios::out);
   
   // Add check
 
index cb80a6b..e376445 100644 (file)
@@ -32,7 +32,10 @@ class AliMpDataProcessor : public TObject
     Bool_t        GenerateCode(AliMpDataMap* dataMap);
     
   private:
-  
+    // static methods
+    static const TString& GetHeaderFileName();
+    static const TString& GetImplFileName();  
     // methods  
     void ProcessDirectory(const TString& path, AliMpDataMap* map);
     void ProcessFile(const TString& path, AliMpDataMap* map );
@@ -40,10 +43,6 @@ class AliMpDataProcessor : public TObject
     void GenerateFileCode(const TString& path);
     void GenerateFill();
 
-    // static data
-    static const TString fgkHeaderFileName;///< default name for generated header file
-    static const TString fgkImplFileName;  ///< default name for generated implementation file
-
     // data
     Int_t     fCounter;    ///< data files counter
     ofstream  fHeaderFile; ///< header file
index 08b2955..7a295db 100644 (file)
@@ -49,14 +49,20 @@ const Bool_t  AliMpExMap::fgkDefaultOwnership = true;
 const Int_t AliMpExMap::fgkSeparator1 = 10000;
 const Int_t AliMpExMap::fgkSeparator2 = 100;
 
-const TString  AliMpExMap::fgkCharacterMap 
-  = " 1234567890abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ.-";
-
 //
 // static methods
 //
 
 //_____________________________________________________________________________
+const TString&  AliMpExMap::GetCharacterMap()
+{
+  /// Return the string mapping characters to integers
+  static const TString kCharacterMap 
+    = " 1234567890abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ.-";
+  return kCharacterMap;  
+}
+
+//_____________________________________________________________________________
 Long_t  AliMpExMap::GetIndex(const TString& s)
 {
 /// Convert the TString to integer.
@@ -68,7 +74,7 @@ Long_t  AliMpExMap::GetIndex(const TString& s)
 
   Long_t index = 0;
   for (Int_t i=s.Length()-1; i>=0; --i)  
-    index = index*fgkSeparator2 + fgkCharacterMap.First(s(i));
+    index = index*fgkSeparator2 + GetCharacterMap().First(s(i));
   
   return index;
 }
@@ -80,7 +86,7 @@ TString  AliMpExMap::GetString(Long_t index)
 
   TString s;
   while (index >0) {
-    Char_t c = fgkCharacterMap(index%fgkSeparator2);
+    Char_t c = GetCharacterMap()(index%fgkSeparator2);
     s += c;
     index = index/fgkSeparator2;
   }
index b965e05..51cebab 100644 (file)
@@ -65,6 +65,9 @@ class AliMpExMap : public TObject
     AliMpExMapIterator* CreateIterator() const;
     
   private:  
+    // static methods
+    static const TString&  GetCharacterMap();
+
     // methods
 
     // conversion between Int_t pair key and Long_t
@@ -84,8 +87,6 @@ class AliMpExMap : public TObject
                                          ///  of Int_t pair to Int_t
     static const Int_t    fgkSeparator2; ///< \brief the separator used for conversion
                                          ///  of TString to Int_t
-    static const TString  fgkCharacterMap; ///< \brief the string mapping characters 
-                                           ///  to integers 
     
     // data members
     mutable TExMap  fMap;     //!<  Transient map class
index f53ffbe..3e89533 100755 (executable)
@@ -44,40 +44,180 @@ ClassImp(AliMpFiles)
 /// \endcond
 
 //
-// static
-
-// static data members
-
-const TString AliMpFiles::fgkDataDir = "/data";
-const TString AliMpFiles::fgkDataRunDir = "/data_run";
-const TString AliMpFiles::fgkStationDir = "/station";
-const TString AliMpFiles::fgkBendingDir = "bending_plane/";
-const TString AliMpFiles::fgkNonBendingDir = "non-bending_plane/";
-const TString AliMpFiles::fgkDENames = "denames"; 
-const TString AliMpFiles::fgkSector  = "zones"; 
-const TString AliMpFiles::fgkSectorSpecial = "zones_special";
-const TString AliMpFiles::fgkSectorSpecial2 = "zones_special_outer";
-const TString AliMpFiles::fgkMotifPrefix   = "motif";  
-const TString AliMpFiles::fgkMotifSpecialPrefix ="motifSpecial";
-const TString AliMpFiles::fgkManuToSerial ="_manu";
-const TString AliMpFiles::fgkPadPosPrefix  = "padPos"; 
-const TString AliMpFiles::fgkDataExt = ".dat";      
-const TString AliMpFiles::fgkBergToGCFileName = "bergToGC"; 
-const TString AliMpFiles::fgkTriggerLocalBoards = "RegionalCrate";
-const TString AliMpFiles::fgkTriggerGlobalBoards = "GlobalCrate";
-const TString AliMpFiles::fgkBusPatchFileName = "DetElemIdToBusPatch";
-const TString AliMpFiles::fgkBusPatchInfoFileName = "BusPatchInfo";
-const TString AliMpFiles::fgkBusPatchSpecialFileName = "BusPatchSpecial";
-const TString AliMpFiles::fgkSerialToBinFileName = "ManuSerialToBin";
+// static private methods
+//
 
 //______________________________________________________________________________
-AliMpFiles::~AliMpFiles() 
+const TString& AliMpFiles::GetDataDir() 
 {
-/// Destructor
+  /// data directory
+  static const TString kDataDir = "/data";
+  return kDataDir;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetDataRunDir() 
+{
+  /// directory for run dependent data
+  static const TString kDataRunDir = "/data_run";
+  return kDataRunDir;
+}
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetStationDir() 
+{
+  /// station directory
+  static const TString kStationDir = "/station";
+  return kStationDir;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetBendingDir() 
+{
+  /// bending plane directory
+  static const TString kBendingDir = "bending_plane/";
+  return kBendingDir;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetNonBendingDir() 
+{
+  /// non-bending plane directory
+  static const TString kNonBendingDir = "non-bending_plane/";
+  return kNonBendingDir;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetDENames() 
+{
+  /// DE names data file name
+  static const TString kDENames = "denames"; 
+  return kDENames;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetSector() 
+{
+  /// sector data file name
+  static const TString kSector  = "zones"; 
+  return kSector;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetSectorSpecial() 
+{
+  /// sector special data file name
+  static const TString kSectorSpecial = "zones_special";
+  return kSectorSpecial;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetSectorSpecial2() 
+{
+  /// sector special data file name
+  static const TString kSectorSpecial2 = "zones_special_outer";
+  return kSectorSpecial2;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetMotifPrefix() 
+{
+  /// motif data file name
+  static const TString kMotifPrefix   = "motif"; 
+  return kMotifPrefix;
+}   
+    
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetMotifSpecialPrefix() 
+{
+  /// special motif data file name 
+  static const TString kMotifSpecialPrefix ="motifSpecial";
+  return kMotifSpecialPrefix;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetManuToSerial() 
+{
+  /// manu to serial file name suffix
+  static const TString kManuToSerial ="_manu";
+  return kManuToSerial;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetPadPosPrefix() 
+{  
+  /// pad position data file name
+  static const TString kPadPosPrefix  = "padPos"; 
+  return kPadPosPrefix;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetDataExt() 
+{
+  /// file extension
+  static const TString kDataExt = ".dat"; 
+  return kDataExt;
 }
+       
+//______________________________________________________________________________
+const TString& AliMpFiles::GetBergToGCFileName() 
+{
+  /// BergToGC mapping file name
+  static const TString kBergToGCFileName = "bergToGC"; 
+  return kBergToGCFileName;
+}  
 
+//______________________________________________________________________________
+const TString& AliMpFiles::GetTriggerLocalBoards() 
+{
+  ///  local board name to id mapping
+  static const TString kTriggerLocalBoards = "RegionalCrate";
+  return kTriggerLocalBoards;
+}  
+
+//______________________________________________________________________________
+const TString& AliMpFiles::GetTriggerGlobalBoards() 
+{
+  ///  global board name to id mapping
+  static const TString kTriggerGlobalBoards = "GlobalCrate";
+  return kTriggerGlobalBoards;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetBusPatchFileName() 
+{
+  /// DetElemIdToBusPatch file name
+  static const TString kBusPatchFileName = "DetElemIdToBusPatch";
+  return kBusPatchFileName;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetBusPatchInfoFileName() 
+{
+  /// BusPatch length file name
+  static const TString kBusPatchInfoFileName = "BusPatchInfo";
+  return kBusPatchInfoFileName;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetBusPatchSpecialFileName() 
+{
+  /// BusPatch special file name
+  static const TString kBusPatchSpecialFileName = "BusPatchSpecial";
+  return kBusPatchSpecialFileName;
+}  
+  
+//______________________________________________________________________________
+const TString& AliMpFiles::GetSerialToBinFileName() 
+{
+  /// serial to bin  number file name
+  static const TString kSerialToBinFileName = "ManuSerialToBin";
+  return kSerialToBinFileName;
+}  
+  
 //
-// public methods
+// static public methods
 //
 
 //______________________________________________________________________________
@@ -92,16 +232,16 @@ TString AliMpFiles::PlaneDataDir(AliMp::StationType station,
   case AliMp::kStation12:
     switch (plane) {
     case AliMp::kBendingPlane:
-      return GetTop() + fgkDataDir + StationDataDir(station, station12Type) + fgkBendingDir;
+      return GetTop() + GetDataDir() + StationDataDir(station, station12Type) + GetBendingDir();
       ;;
     case AliMp::kNonBendingPlane:   
-      return GetTop() + fgkDataDir + StationDataDir(station, station12Type) + fgkNonBendingDir;
+      return GetTop() + GetDataDir() + StationDataDir(station, station12Type) + GetNonBendingDir();
       ;;
     }   
     break;
   case AliMp::kStation345:
   case AliMp::kStationTrigger:  
-    return GetTop() + fgkDataDir + StationDataDir(station, AliMq::kNotSt12);
+    return GetTop() + GetDataDir() + StationDataDir(station, AliMq::kNotSt12);
     break;
   default:  
     AliFatalClass("Incomplete switch on AliMp::PlaneType");
@@ -116,7 +256,7 @@ TString AliMpFiles::StationDataDir(AliMp::StationType station,
 {
 /// Returns the station directory name for the specified station number.
 
-  TString stationDataDir(fgkStationDir);
+  TString stationDataDir(GetStationDir());
   switch (station) {
   case AliMp::kStation12: 
     switch (station12Type) {
@@ -154,7 +294,7 @@ TString AliMpFiles::BusPatchFilePath()
 {
 /// Return path to data file with bus patch mapping.
 
-  return GetTop() + fgkDataDir + "/" + fgkBusPatchFileName + fgkDataExt;
+  return GetTop() + GetDataDir() + "/" + GetBusPatchFileName() + GetDataExt();
 }  
 
 //______________________________________________________________________________
@@ -162,7 +302,7 @@ TString AliMpFiles::BusPatchInfoFilePath()
 {
 /// Return path to data file with bus patch mapping.
 
-  return GetTop() + fgkDataDir + "/" + fgkBusPatchInfoFileName + fgkDataExt;
+  return GetTop() + GetDataDir() + "/" + GetBusPatchInfoFileName() + GetDataExt();
 }  
 
 //______________________________________________________________________________
@@ -170,7 +310,7 @@ TString AliMpFiles::BusPatchSpecialFilePath()
 {
 /// Return path to data file with special bus patch mapping.
 
-  return GetTop() + fgkDataDir + "/" + fgkBusPatchSpecialFileName + fgkDataExt;
+  return GetTop() + GetDataDir() + "/" + GetBusPatchSpecialFileName() + GetDataExt();
 }  
 
 //______________________________________________________________________________
@@ -178,7 +318,7 @@ TString AliMpFiles::SerialToBinFilePath()
 {
 /// Return path to data file containing manu serial numbers with their bin.
 
-  return GetTop() + fgkDataDir + "/" + fgkSerialToBinFileName + fgkDataExt;
+  return GetTop() + GetDataDir() + "/" + GetSerialToBinFileName() + GetDataExt();
 }  
 
 
@@ -188,8 +328,8 @@ TString AliMpFiles::DENamesFilePath(AliMp::StationType station,
 {
 /// Return path to data file with DE names for given station.
  
-  return GetTop() + fgkDataDir + StationDataDir(station, station12Type) 
-           + fgkDENames + fgkDataExt;
+  return GetTop() + GetDataDir() + StationDataDir(station, station12Type) 
+           + GetDENames() + GetDataExt();
 }
 
 //______________________________________________________________________________
@@ -197,9 +337,9 @@ TString AliMpFiles::LocalTriggerBoardMapping()
 {
 /// Return path to data file with local trigger board mapping.
 
-  return GetTop() + fgkDataDir 
+  return GetTop() + GetDataDir() 
           + StationDataDir(AliMp::kStationTrigger, AliMq::kNotSt12) 
-          + fgkTriggerLocalBoards + fgkDataExt;;
+          + GetTriggerLocalBoards() + GetDataExt();;
 }
 
 //______________________________________________________________________________
@@ -207,9 +347,9 @@ TString AliMpFiles::GlobalTriggerBoardMapping()
 {
 /// Return path to data file with local trigger board mapping.
 
-  return GetTop() + fgkDataDir 
+  return GetTop() + GetDataDir() 
       + StationDataDir(AliMp::kStationTrigger, AliMq::kNotSt12) 
-      + fgkTriggerGlobalBoards + fgkDataExt;;
+      + GetTriggerGlobalBoards() + GetDataExt();;
 }
 
 //_____________________________________________________________________________
@@ -243,7 +383,7 @@ TString AliMpFiles::SectorFilePath(AliMq::Station12Type station12Type,
 /// Return path to data file with sector description.
  
   return TString(PlaneDataDir(AliMp::kStation12, station12Type, plane) 
-                 + fgkSector + fgkDataExt);
+                 + GetSector() + GetDataExt());
 }
     
 //______________________________________________________________________________
@@ -253,7 +393,7 @@ TString AliMpFiles::SectorSpecialFilePath(AliMq::Station12Type station12Type,
 /// Return path to data file with sector special description (irregular motifs).
 
   return TString(PlaneDataDir(AliMp::kStation12, station12Type, plane) 
-                 + fgkSectorSpecial + fgkDataExt);
+                 + GetSectorSpecial() + GetDataExt());
 }
     
 //______________________________________________________________________________
@@ -263,7 +403,7 @@ TString AliMpFiles::SectorSpecialFilePath2(AliMq::Station12Type station12Type,
 /// Returns path to data file with sector special description (irregular motifs).
 
   return TString(PlaneDataDir(AliMp::kStation12, station12Type, plane) 
-                 + fgkSectorSpecial2 + fgkDataExt);
+                 + GetSectorSpecial2() + GetDataExt());
 }
 
 //______________________________________________________________________________
@@ -271,7 +411,7 @@ TString AliMpFiles::MotifFileName(const TString& motifTypeID)
 {
   /// Returns name of data file for a given motif type.
   
-  return TString(fgkMotifPrefix +  motifTypeID + fgkDataExt);
+  return TString(GetMotifPrefix() +  motifTypeID + GetDataExt());
 }
 
 //______________________________________________________________________________
@@ -291,7 +431,7 @@ TString AliMpFiles::PadPosFileName(const TString& motifTypeID)
 {
   /// Returns name of data file with pad positions for a given motif type.
   
-  return TString(fgkPadPosPrefix +  motifTypeID + fgkDataExt);
+  return TString(GetPadPosPrefix() +  motifTypeID + GetDataExt());
 }
 
 //______________________________________________________________________________
@@ -311,7 +451,7 @@ TString AliMpFiles::MotifSpecialFileName(const TString& motifID)
 {
   /// Returns name of data file with pad dimensions for a given motif ID.
   
-  return TString(fgkMotifSpecialPrefix + motifID + fgkDataExt);
+  return TString(GetMotifSpecialPrefix() + motifID + GetDataExt());
   
 }
 
@@ -334,8 +474,8 @@ TString AliMpFiles::BergToGCFilePath(AliMp::StationType station,
 /// Returns the path of the file which describes the correspondance between
 /// the berg number and the gassiplex channel.
 
-  return GetTop() + fgkDataDir + StationDataDir(station, station12Type)
-              + fgkBergToGCFileName + fgkDataExt;
+  return GetTop() + GetDataDir() + StationDataDir(station, station12Type)
+              + GetBergToGCFileName() + GetDataExt();
 }
 
 //______________________________________________________________________________ 
@@ -345,8 +485,8 @@ TString AliMpFiles::ManuToSerialPath(const TString& deName,
 {
 /// Returns the path of the file for the manu id to their serial number
 
-  return  GetTop() + fgkDataRunDir + StationDataDir(station, station12Type)
-              + deName + fgkManuToSerial + fgkDataExt; 
+  return  GetTop() + GetDataRunDir() + StationDataDir(station, station12Type)
+              + deName + GetManuToSerial() + GetDataExt(); 
 }
 
 
@@ -378,3 +518,13 @@ TString AliMpFiles::GetTop()
   return ntop;
 }
 
+//
+// ctors, dtor
+//
+
+//______________________________________________________________________________
+AliMpFiles::~AliMpFiles() 
+{
+/// Destructor
+}
+
index d30e18b..0f6f072 100755 (executable)
@@ -110,30 +110,30 @@ class AliMpFiles : public TObject
     AliMpFiles(const AliMpFiles& right);
     /// Not implemented
     AliMpFiles& operator=(const AliMpFiles& right);    
-
-    // static data members  
-    static const TString fgkDataDir;       ///< data directory
-    static const TString fgkDataRunDir;    ///< directory for run dependent data
-    static const TString fgkStationDir;    ///< station directory
-    static const TString fgkBendingDir;    ///< bending plane directory
-    static const TString fgkNonBendingDir; ///< non-bending plane directory
-    static const TString fgkDENames;       ///< DE names data file name
-    static const TString fgkSector;        ///< sector data file name
-    static const TString fgkSectorSpecial; ///< sector special data file name
-    static const TString fgkSectorSpecial2;///< sector special data file name
-    static const TString fgkMotifPrefix;   ///< motif data file name
-    static const TString fgkMotifSpecialPrefix; ///< special motif data file name 
-    static const TString fgkManuToSerial;  ///< manu to serial file name suffix
-    static const TString fgkPadPosPrefix;  ///< pad position data file name
-    static const TString fgkDataExt;       ///< file extension
-    static const TString fgkBergToGCFileName;  ///< BergToGC mapping file name
-    static const TString fgkTriggerLocalBoards;///<  local board name to id mapping
-    static const TString fgkTriggerGlobalBoards;///<  global board name to id mapping
-    static const TString fgkBusPatchFileName;  ///< DetElemIdToBusPatch file name
-    static const TString fgkBusPatchInfoFileName;///< BusPatch length file name
-    static const TString fgkBusPatchSpecialFileName;///< BusPatch special file name
-    static const TString fgkSerialToBinFileName; ///< serial to bin  number file name
     
+    // static methods
+    static const TString& GetDataDir();       
+    static const TString& GetDataRunDir();    
+    static const TString& GetStationDir();    
+    static const TString& GetBendingDir();    
+    static const TString& GetNonBendingDir(); 
+    static const TString& GetDENames();       
+    static const TString& GetSector();        
+    static const TString& GetSectorSpecial(); 
+    static const TString& GetSectorSpecial2();
+    static const TString& GetMotifPrefix();   
+    static const TString& GetMotifSpecialPrefix();
+    static const TString& GetManuToSerial();
+    static const TString& GetPadPosPrefix();
+    static const TString& GetDataExt();     
+    static const TString& GetBergToGCFileName();  
+    static const TString& GetTriggerLocalBoards();
+    static const TString& GetTriggerGlobalBoards();
+    static const TString& GetBusPatchFileName();
+    static const TString& GetBusPatchInfoFileName();
+    static const TString& GetBusPatchSpecialFileName();
+    static const TString& GetSerialToBinFileName();
+
   ClassDef(AliMpFiles, 0) //File names and paths 
 };  
 
index 83ef4d6..bcaa59b 100755 (executable)
 ClassImp(AliMpSectorReader)
 /// \endcond
 
-const TString  AliMpSectorReader::fgkSectorKeyword  = "SECTOR_DATA";
-const TString  AliMpSectorReader::fgkZoneKeyword    = "ZONE";
-const TString  AliMpSectorReader::fgkSubZoneKeyword = "SUBZONE";
-const TString  AliMpSectorReader::fgkRowKeyword     = "ROW_SEGMENT";
-const TString  AliMpSectorReader::fgkEofKeyword     = "EOF";
-const TString  AliMpSectorReader::fgkSectorSpecialKeyword  = "SECTOR_SPECIAL_DATA";
-const TString  AliMpSectorReader::fgkMotifKeyword          = "MOTIF";
-const TString  AliMpSectorReader::fgkRowSpecialKeyword     = "ROW";
-const TString  AliMpSectorReader::fgkPadRowsKeyword        = "PAD_ROWS";
-const TString  AliMpSectorReader::fgkPadRowSegmentKeyword  = "PAD_ROW_SEGMENT";
+//
+// static private methods
+//
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetSectorKeyword()
+{
+  /// sector keyword
+  static const TString kSectorKeyword  = "SECTOR_DATA";
+  return kSectorKeyword;
+}  
 
 //_____________________________________________________________________________
+const TString& AliMpSectorReader::GetZoneKeyword()
+{
+  /// zone keyword
+  static const TString kZoneKeyword = "ZONE";
+  return kZoneKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetSubZoneKeyword() 
+{
+  /// subzone keyword
+  static const TString kSubZoneKeyword = "SUBZONE";
+  return kSubZoneKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetRowKeyword()
+{
+  /// row keyword
+  static const TString kRowKeyword = "ROW_SEGMENT";
+  return kRowKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetSectorSpecialKeyword()
+{
+  /// sector special keyword
+  static const TString kSectorSpecialKeyword = "SECTOR_SPECIAL_DATA";
+  return kSectorSpecialKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetMotifKeyword()
+{
+  /// motif keyword
+  static const TString kMotifKeyword = "MOTIF";
+  return kMotifKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetRowSpecialKeyword()
+{
+  /// row special keyword
+  static const TString kRowSpecialKeyword = "ROW";
+  return kRowSpecialKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetPadRowsKeyword()
+{
+  /// pad rows keyword
+  static const TString kPadRowsKeyword = "PAD_ROWS";
+  return kPadRowsKeyword;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpSectorReader::GetPadRowSegmentKeyword()
+{
+  /// pad row segment keyword
+  static const TString kPadRowSegmentKeyword = "PAD_ROW_SEGMENT";
+  return kPadRowSegmentKeyword;
+}  
+
+//
+// ctors, dtor
+//
+  
+//_____________________________________________________________________________
 AliMpSectorReader::AliMpSectorReader(const AliMpDataStreams& dataStreams,
                                      AliMq::Station12Type station, 
                                      AliMp::PlaneType plane) 
@@ -112,7 +181,7 @@ void  AliMpSectorReader::ReadSectorData(istream& in)
   
   AliDebugStream(2) << keyword << endl;
 
-  if (keyword != fgkSectorKeyword) {
+  if (keyword != GetSectorKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }   
@@ -143,7 +212,7 @@ void  AliMpSectorReader::ReadSectorData(istream& in)
   TString nextKeyword;
   in >> nextKeyword;
     
-  if (nextKeyword != fgkZoneKeyword) {
+  if (nextKeyword != GetZoneKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }      
@@ -163,7 +232,7 @@ void AliMpSectorReader::ReadZoneData(istream& in)
   in >> sizex;
   in >> sizey;
   AliDebugStream(2)
-     << fgkZoneKeyword << " " <<  zoneID << "  " 
+     << GetZoneKeyword() << " " <<  zoneID << "  " 
      << sizex << " " << sizey << endl;
   
   AliMpZone* zone =  fSector->GetZone(zoneID);
@@ -172,7 +241,7 @@ void AliMpSectorReader::ReadZoneData(istream& in)
   TString nextKeyword;
   in >> nextKeyword;
     
-  if (nextKeyword != fgkSubZoneKeyword) {
+  if (nextKeyword != GetSubZoneKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }  
@@ -186,7 +255,7 @@ void AliMpSectorReader::ReadSubZoneData(istream& in, AliMpZone* zone)
 /// Read subzone input data;
 /// create subzone and its to the specified zone.
 
-  AliDebugStream(2) << fgkSubZoneKeyword << endl;
+  AliDebugStream(2) << GetSubZoneKeyword() << endl;
 
   AliMpVMotif* motif = ReadMotifData(in, zone);
   AliMpSubZone* subZone = new AliMpSubZone(motif); 
@@ -195,7 +264,7 @@ void AliMpSectorReader::ReadSubZoneData(istream& in, AliMpZone* zone)
   TString nextKeyword;
   in >> nextKeyword;
     
-  if (nextKeyword != fgkRowKeyword) {
+  if (nextKeyword != GetRowKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }  
@@ -266,7 +335,7 @@ void AliMpSectorReader::ReadRowSegmentsData(istream& in,
     firstMotifPositionId |= AliMpConstants::ManuMask(fPlaneType);
     
     AliDebugStream(2)
-      << fgkRowKeyword << " " 
+      << GetRowKeyword() << " " 
       << offX << " " << offY << " " << inRow << " " << nofMotifs << " " 
       << firstMotifPositionId << " " << firstMotifPositionDId
       << endl;
@@ -287,14 +356,14 @@ void AliMpSectorReader::ReadRowSegmentsData(istream& in,
     subZone->AddRowSegment(rowSegment);
     row->AddRowSegment(rowSegment);
   }
-  while (!in.eof() && (nextKeyword == fgkRowKeyword));
+  while (!in.eof() && (nextKeyword == GetRowKeyword()));
 
   if (in.eof()) return;
 
-  if (nextKeyword == fgkZoneKeyword) {
+  if (nextKeyword == GetZoneKeyword()) {
     ReadZoneData(in);
   }
-  else if (nextKeyword == fgkSubZoneKeyword) {
+  else if (nextKeyword == GetSubZoneKeyword()) {
     ReadSubZoneData(in, zone);
   }   
   else {
@@ -313,7 +382,7 @@ void AliMpSectorReader::ReadSectorSpecialData(istream& in, AliMp::XDirection dir
 
   AliDebugStream(2) << keyword << endl;
 
-  if (keyword != fgkSectorSpecialKeyword) {
+  if (keyword != GetSectorSpecialKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }   
@@ -323,7 +392,7 @@ void AliMpSectorReader::ReadSectorSpecialData(istream& in, AliMp::XDirection dir
 
   AliDebugStream(2) << keyword << endl;
     
-  if (nextKeyword != fgkMotifKeyword) {
+  if (nextKeyword != GetMotifKeyword()) {
     AliErrorStream() << "Wrong file format." << endl;
     return;
   }  
@@ -337,7 +406,7 @@ void AliMpSectorReader::ReadMotifsSpecialData(istream& in)
 {
 /// Read the special (irregular) motifs input data.
 
-  AliDebugStream(2) << fgkMotifKeyword << endl;
+  AliDebugStream(2) << GetMotifKeyword() << endl;
 
   TString nextKeyword;
   do {
@@ -351,9 +420,9 @@ void AliMpSectorReader::ReadMotifsSpecialData(istream& in)
 
     AliDebugStream(2) << nextKeyword << endl;      
   }
-  while (nextKeyword == fgkMotifKeyword);
+  while (nextKeyword == GetMotifKeyword());
     
-  if (nextKeyword != fgkRowSpecialKeyword) {
+  if (nextKeyword != GetRowSpecialKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }      
@@ -397,7 +466,7 @@ void AliMpSectorReader::ReadRowSpecialData(istream& in, AliMp::XDirection direct
   
   AliDebugStream(2) << nextKeyword << endl;
     
-  if (nextKeyword != fgkPadRowsKeyword) {
+  if (nextKeyword != GetPadRowsKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }  
@@ -443,7 +512,7 @@ void AliMpSectorReader::ReadRowSegmentSpecialData(istream& in,
 
   AliDebugStream(2) << keyword << endl;
     
-  if (keyword != fgkPadRowSegmentKeyword) {
+  if (keyword != GetPadRowSegmentKeyword()) {
      AliErrorStream() << "Wrong file format." << endl;
      return;
   }  
@@ -505,14 +574,14 @@ void AliMpSectorReader::ReadRowSegmentSpecialData(istream& in,
                                motifPositionId, nofPadsInRow);
     }  
   }
-  while (!in.eof() && (nextKeyword == fgkPadRowSegmentKeyword));
+  while (!in.eof() && (nextKeyword == GetPadRowSegmentKeyword()));
   
   if (in.eof()) return;
 
-  if (nextKeyword == fgkPadRowsKeyword) {
+  if (nextKeyword == GetPadRowsKeyword()) {
     ReadRowSegmentSpecialData(in, segment, direction);
   }
-  else if (nextKeyword == fgkRowSpecialKeyword) {
+  else if (nextKeyword == GetRowSpecialKeyword()) {
     ReadRowSpecialData(in, direction);
   }   
   else {
index d44ba95..1c22f5a 100755 (executable)
@@ -52,7 +52,18 @@ class AliMpSectorReader : public TObject
     /// Not implemented
     AliMpSectorReader& operator = (const AliMpSectorReader& right);
 
-    // methods
+     // static methods
+    static const TString&  GetSectorKeyword();       
+    static const TString&  GetZoneKeyword();         
+    static const TString&  GetSubZoneKeyword();      
+    static const TString&  GetRowKeyword();          
+    static const TString&  GetSectorSpecialKeyword();
+    static const TString&  GetMotifKeyword();        
+    static const TString&  GetRowSpecialKeyword();   
+    static const TString&  GetPadRowsKeyword();      
+    static const TString&  GetPadRowSegmentKeyword();
+  
+   // methods
     void  ReadSectorData(istream& in);
     void  ReadZoneData(istream& in);
     void  ReadSubZoneData(istream& in, AliMpZone* zone);
@@ -65,18 +76,6 @@ class AliMpSectorReader : public TObject
     void  ReadRowSegmentSpecialData(istream& in,
                           AliMpVRowSegmentSpecial* segment,
                          AliMp::XDirection direction);
-    
-    // static data members
-    static const TString  fgkSectorKeyword;        ///< sector keyword
-    static const TString  fgkZoneKeyword;          ///< zone keyword
-    static const TString  fgkSubZoneKeyword;       ///< subzone keyword
-    static const TString  fgkRowKeyword;           ///< row keyword
-    static const TString  fgkEofKeyword;           ///< eof keyword
-    static const TString  fgkSectorSpecialKeyword; ///< sector special keyword
-    static const TString  fgkMotifKeyword;         ///< motif keyword
-    static const TString  fgkRowSpecialKeyword;    ///< row special keyword
-    static const TString  fgkPadRowsKeyword;       ///< pad rows keyword
-    static const TString  fgkPadRowSegmentKeyword; ///< pad row segment keyword
   
     // data members  
     const AliMpDataStreams&  fkDataStreams; ///< data streams
index fdd6eef..1c0d4b8 100644 (file)
 ClassImp(AliMpStringObjMap)
 /// \endcond
 
-const TString AliMpStringObjMap::fgkUndefinedKey = "Undefined";
+//
+// static private methods
+//
+
+//______________________________________________________________________________
+const TString& AliMpStringObjMap::GetUndefinedKey()
+{
+  /// Undefined key string
+  static const TString kUndefinedKey = "Undefined";
+  return kUndefinedKey;
+}  
+
+//
+// ctors, dtor
+//
 
 //______________________________________________________________________________
 AliMpStringObjMap::AliMpStringObjMap(Bool_t isOwner)
@@ -178,7 +192,7 @@ TString AliMpStringObjMap::CurrentKey()
 {
 /// Set iterator to the first item and return its object
 
-  if ( fCurrentIndex >= fNofItems ) return fgkUndefinedKey;
+  if ( fCurrentIndex >= fNofItems ) return GetUndefinedKey();
   
   return ((TObjString*)fFirstArray.At(fCurrentIndex))->GetString();
 }  
index 6639153..c27b002 100644 (file)
@@ -45,8 +45,8 @@ class AliMpStringObjMap : public TObject
     /// Not implemented
     AliMpStringObjMap& operator = (const AliMpStringObjMap& rhs);
     
-    // static data members
-    static const TString fgkUndefinedKey; ///< Undefined key string
+    // static methods
+    static const TString& GetUndefinedKey(); 
 
     // data members
     Int_t      fNofItems;     ///<  number of items
index d43bccd..25d7a4f 100644 (file)
 ClassImp(AliMpTriggerReader)
 /// \endcond
 
-const TString AliMpTriggerReader::fgkKeywordLayer("LAYER");
-const TString AliMpTriggerReader::fgkKeywordScale("SCALE");
-const TString AliMpTriggerReader::fgkKeywordPcb("PCB");  
-const TString AliMpTriggerReader::fgkKeywordFlipX("FLIP_X");
-const TString AliMpTriggerReader::fgkKeywordFlipY("FLIP_Y");
+//
+// static private methods
+//
+
+//_____________________________________________________________________________
+const TString& AliMpTriggerReader::GetKeywordLayer()
+{
+  /// Keyword: LAYER
+  static const TString kKeywordLayer("LAYER");
+  return kKeywordLayer;
+}  
+
+//_____________________________________________________________________________
+const TString& AliMpTriggerReader::GetKeywordScale()
+{
+  /// Keyword: SCALE
+  static const TString kKeywordScale("SCALE");
+  return kKeywordScale;
+}
+
+//_____________________________________________________________________________
+const TString& AliMpTriggerReader::GetKeywordPcb()
+{
+  /// Keyword : PCB
+  static const TString kKeywordPcb("PCB");  
+  return kKeywordPcb;
+}    
+  
+//_____________________________________________________________________________
+const TString& AliMpTriggerReader::GetKeywordFlipX()
+{
+  /// Keyword : FLIPX
+  static const TString kKeywordFlipX("FLIP_X");
+  return kKeywordFlipX;
+}  
+  
+//_____________________________________________________________________________
+const TString& AliMpTriggerReader::GetKeywordFlipY()
+{
+  /// Keyword : FLIPY
+  static const TString kKeywordFlipY("FLIP_Y");
+  return kKeywordFlipY;
+}  
+
+//
+// ctors, dtor
+//
 
 //_____________________________________________________________________________
 AliMpTriggerReader::AliMpTriggerReader(const AliMpDataStreams& dataStreams, AliMpSlatMotifMap* motifMap) 
@@ -115,7 +157,7 @@ AliMpTriggerReader::BuildSlat(const char* slatName,
     
     TString& keyword = ((TObjString*)tokens->At(0))->String();
     
-    if ( keyword == fgkKeywordPcb )
+    if ( keyword == GetKeywordPcb() )
     {
       if ( tokens->GetEntriesFast() != 3 )
       {
@@ -202,7 +244,7 @@ AliMpTriggerReader::GetBoardNameFromPCBLine(const TString& s)
   
   TString& keyword = ((TObjString*)tokens->At(0))->String();
 
-  if ( keyword == fgkKeywordPcb &&
+  if ( keyword == GetKeywordPcb() &&
        tokens->GetEntriesFast() == 3 )
   {
     boardName = ((TObjString*)tokens->At(2))->String();
@@ -267,7 +309,7 @@ AliMpTriggerReader::FlipLines(TList& lines, Bool_t flipX, Bool_t flipY,
         
         TString& s = oline->String();
         
-        if ( !s.Contains(fgkKeywordPcb) )
+        if ( !s.Contains(GetKeywordPcb()) )
         {
           // Only consider PCB lines.
           continue;
@@ -341,7 +383,7 @@ AliMpTriggerReader::IsLayerLine(const TString& sline) const
 {
   /// Whether sline contains LAYER keyword
 
-  if ( sline.BeginsWith(fgkKeywordLayer) )
+  if ( sline.BeginsWith(GetKeywordLayer()) )
   {
     return 1;
   }
@@ -364,10 +406,10 @@ AliMpTriggerReader::DecodeFlipLine(const TString& sline,
   
   TString keyword(sline(0,blankPos));
   
-  if ( keyword == fgkKeywordFlipX )
+  if ( keyword == GetKeywordFlipX() )
   {
     flipX = kTRUE;
-  } else if ( keyword == fgkKeywordFlipY )
+  } else if ( keyword == GetKeywordFlipY() )
   {
     flipY = kTRUE;
   }
@@ -387,10 +429,10 @@ AliMpTriggerReader::DecodeScaleLine(const TString& sline,
 {
   /// Decode sline containing SCALE keyword
 
-  if ( sline(0,fgkKeywordScale.Length()) == fgkKeywordScale )
+  if ( sline(0,GetKeywordScale().Length()) == GetKeywordScale() )
   {
-    TString tmp(sline(fgkKeywordScale.Length()+1,
-                      sline.Length()-fgkKeywordScale.Length()-1));
+    TString tmp(sline(GetKeywordScale().Length()+1,
+                      sline.Length()-GetKeywordScale().Length()-1));
     Ssiz_t blankPos = tmp.First(' ');
     if ( blankPos < 0 )
     {
@@ -481,11 +523,11 @@ AliMpTriggerReader::ReadLines(const char* slatType,
     if ( sline.Length() == 0 || sline[0] == '#' ) continue;
     
     Bool_t isKeywordThere = 
-      sline.Contains(fgkKeywordPcb) || 
-      sline.Contains(fgkKeywordLayer) ||
-      sline.Contains(fgkKeywordScale) || 
-      sline.Contains(fgkKeywordFlipX) || 
-      sline.Contains(fgkKeywordFlipY);
+      sline.Contains(GetKeywordPcb()) || 
+      sline.Contains(GetKeywordLayer()) ||
+      sline.Contains(GetKeywordScale()) || 
+      sline.Contains(GetKeywordFlipX()) || 
+      sline.Contains(GetKeywordFlipY());
     
     if ( !isKeywordThere ) 
     {
@@ -503,7 +545,7 @@ AliMpTriggerReader::ReadLines(const char* slatType,
 
     if ( isScaleLine < 0 )
     {
-      AliFatalClass(Form("Syntax error near %s keyword\n",fgkKeywordScale.Data()));
+      AliFatalClass(Form("Syntax error near %s keyword\n",GetKeywordScale().Data()));
     }
     else if ( isScaleLine > 0 && slatType2 != slatType )
     {
@@ -758,7 +800,7 @@ AliMpTriggerReader::ReadSlat(const char* slatType, AliMp::PlaneType planeType)
   if ( !IsLayerLine(firstLine) ) 
   {
     std::ostringstream s;
-    s << fgkKeywordLayer;
+    s << GetKeywordLayer();
     lines.AddFirst(new TObjString(s.str().c_str()));
   }
   
index 5f9440c..e60cedf 100644 (file)
@@ -88,17 +88,19 @@ private:
   /// Not implemented
   AliMpTriggerReader& operator=(const AliMpTriggerReader& rhs);
     
+  // static methods
+  static const TString& GetKeywordLayer();
+  static const TString& GetKeywordScale();
+  static const TString& GetKeywordPcb();
+  static const TString& GetKeywordFlipX();
+  static const TString& GetKeywordFlipY();
+  
+  // data members
   const AliMpDataStreams&  fkDataStreams; //!< data streams
   AliMpSlatMotifMap* fMotifMap; //!< storage for motifTypes and motifs...
   
   TMap fLocalBoardMap; //!< map of TObjString to TObjString
 
-  static const TString fgkKeywordLayer; //!< Keyword: LAYER
-  static const TString fgkKeywordScale; //!< Keyword: SCALE
-  static const TString fgkKeywordPcb; //!< Keyword : PCB
-  static const TString fgkKeywordFlipX; //!< Keyword : FLIPX
-  static const TString fgkKeywordFlipY; //!< Keyword : FLIPY
-  
   ClassDef(AliMpTriggerReader,0) // Reader for trigger slats mapping files 
 };