- GetModuleSegmentation(moduleId) moved to private to prevent from
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 19 Oct 2006 09:33:06 +0000 (09:33 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 19 Oct 2006 09:33:06 +0000 (09:33 +0000)
  potentional errors due to passing chamberId instead of mofuleId
  (as they are now different)
- Using AliMpDEManager for detElemId -> moduleId

MUON/AliMUONSegmentation.cxx
MUON/AliMUONSegmentation.h

index 147f446..65886ba 100644 (file)
@@ -28,9 +28,9 @@
 #include "AliMUONSegmentation.h"
 #include "AliMUONVGeometryDESegmentation.h"
 #include "AliMUONGeometrySegmentation.h"
-#include "AliMUONGeometryStore.h"
-
+\
 #include "AliMpVSegmentation.h"
+#include "AliMpDEManager.h"
 
 #include "AliLog.h"
 
@@ -94,6 +94,37 @@ AliMUONSegmentation::~AliMUONSegmentation()
 }
 
 //
+// private functions
+//
+
+//_____________________________________________________________________________
+AliMUONGeometrySegmentation* 
+AliMUONSegmentation::GetModuleSegmentation(
+                        Int_t moduleId, Int_t cathod, Bool_t warn) const
+{
+/// Return the geometry module segmentation specified by moduleId
+
+  if (cathod < 0 || cathod >= 2) {
+    if (warn) {
+      AliWarningStream() 
+        << "Cathod: " << cathod << " outside limits" << std::endl;
+    }                   
+    return 0;  
+  }  
+
+  if (moduleId < 0 || moduleId >= fModuleSegmentations[cathod]->GetEntriesFast()) {
+    if (warn) {
+      AliWarningStream() 
+        << "Index: " << moduleId << " outside limits" << std::endl;
+    }                   
+    return 0;  
+  }  
+
+  return (AliMUONGeometrySegmentation*) 
+            fModuleSegmentations[cathod]->At(moduleId);
+}    
+
+//
 // public functions
 //
 
@@ -170,40 +201,13 @@ void  AliMUONSegmentation::Init()
                            
 //_____________________________________________________________________________
 AliMUONGeometrySegmentation* 
-AliMUONSegmentation::GetModuleSegmentation(
-                        Int_t moduleId, Int_t cathod, Bool_t warn) const
-{
-/// Return the geometry module segmentation specified by moduleId
-
-  if (cathod < 0 || cathod >= 2) {
-    if (warn) {
-      AliWarningStream() 
-        << "Cathod: " << cathod << " outside limits" << std::endl;
-    }                   
-    return 0;  
-  }  
-
-  if (moduleId < 0 || moduleId >= fModuleSegmentations[cathod]->GetEntriesFast()) {
-    if (warn) {
-      AliWarningStream() 
-        << "Index: " << moduleId << " outside limits" << std::endl;
-    }                   
-    return 0;  
-  }  
-
-  return (AliMUONGeometrySegmentation*) 
-            fModuleSegmentations[cathod]->At(moduleId);
-}    
-
-//_____________________________________________________________________________
-AliMUONGeometrySegmentation* 
 AliMUONSegmentation::GetModuleSegmentationByDEId(
                         Int_t detElemId, Int_t cathod, Bool_t warn) const
 {
 /// Return the geometry module specified by detElemId/cathod
 
   // Get moduleId 
-  Int_t moduleId = AliMUONGeometryStore::GetModuleId(detElemId);
+  Int_t moduleId = AliMpDEManager::GetGeomModuleId(detElemId);
 
   return GetModuleSegmentation(moduleId, cathod, warn);
 }    
index 608953e..b8f9d74 100644 (file)
@@ -56,8 +56,6 @@ class AliMUONSegmentation : public TObject
     
     // Geometry segmentations
     //
-    AliMUONGeometrySegmentation* GetModuleSegmentation(
-                     Int_t moduleId, Int_t cathod, Bool_t warn = true) const;
 
     AliMUONGeometrySegmentation* GetModuleSegmentationByDEId(
                      Int_t detElemId, Int_t cathod, Bool_t warn = true) const;
@@ -85,6 +83,9 @@ class AliMUONSegmentation : public TObject
     AliMUONSegmentation&  operator = (const AliMUONSegmentation& right);
      
   private:
+    AliMUONGeometrySegmentation* GetModuleSegmentation(
+                     Int_t moduleId, Int_t cathod, Bool_t warn = true) const;
+
     // data members
     TObjArray*  fMpSegmentations;        ///< array of mapping segmentations
     TObjArray*  fDESegmentations;        ///< array of DE segmentations