changes to be compliant with Eff C++ rules
authormasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 20 Oct 2006 17:11:57 +0000 (17:11 +0000)
committermasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 20 Oct 2006 17:11:57 +0000 (17:11 +0000)
ITS/AliITSPidParItem.cxx
ITS/AliITSPident.cxx
ITS/AliITSPident.h
ITS/AliITSReconstructor.cxx
ITS/AliITSReconstructor.h
ITS/AliITSSteerPid.cxx
ITS/AliITSSteerPid.h
ITS/AliITSpidESD1.cxx
ITS/AliITSpidESD2.cxx
ITS/AliITSpidESD2.h

index 0154155..3b41c97 100644 (file)
 
 ClassImp(AliITSPidParItem)
 //____________________________________________________________________
-AliITSPidParItem::AliITSPidParItem(){
+AliITSPidParItem::AliITSPidParItem():
+fPCenter(0),
+fPWidth(0)
+{
   // default constructor
-  fPCenter=0;
-  fPWidth=0;
   for(Int_t i=0;i<39;i++){
   fBuff[i]=0;
   }
 }//____________________________________________________________________
-AliITSPidParItem::AliITSPidParItem(Float_t center,Float_t width,Double_t *buff){
+AliITSPidParItem::AliITSPidParItem(Float_t center,Float_t width,Double_t *buff):
+fPCenter(center),
+fPWidth(width)
+{
   // standard constructor
-  fPCenter=center;
-  fPWidth=width;
   for (Int_t i=0;i<39;i++) fBuff[i]=buff[i];
 
 }
index 3b51d09..fcdc421 100644 (file)
@@ -9,18 +9,19 @@
 
 ClassImp(AliITSPident)
   //_______________________________________________________________________
-AliITSPident::AliITSPident(){
+AliITSPident::AliITSPident():
+fMom(0),
+fdEdx(0),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(0),
+fPPriorik(0),
+fPPrioripi(0),
+fPPriorie(0),
+fInvPt(0)
+{
   // default constructor
-  fMom=0;
-  fdEdx=0;
-  fInvPt=0;
-  fPBayesp=0;
-  fPBayesk=0;
-  fPBayespi=0;
-  fPPriorip=0;
-  fPPriorik=0;
-  fPPrioripi=0;
-  fPPriorie=0;
   for (Int_t i=0;i<4;i++){
     fCondFunProLay[i]=0;
     fCondFunKLay[i]=0;
@@ -32,28 +33,45 @@ AliITSPident::~AliITSPident(){
   // destructor
 }
 //______________________________________________________________________
-AliITSPident::AliITSPident(const AliITSPident &ob) :TObject(ob) {
+AliITSPident::AliITSPident(const AliITSPident &ob) :TObject(ob),
+fMom(ob.fMom),
+fdEdx(ob.fdEdx),
+fPBayesp(ob.fPBayesp),
+fPBayesk(ob.fPBayesk),
+fPBayespi(ob.fPBayespi),
+fPPriorip(ob.fPPriorip),
+fPPriorik(ob.fPPriorik),
+fPPrioripi(ob.fPPrioripi),
+fPPriorie(ob.fPPriorie),
+fInvPt(ob.fInvPt)  
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSPident","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSPident& AliITSPident::operator=(const AliITSPident& /* ob */){
+AliITSPident& AliITSPident::operator=(const AliITSPident&  ob){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSPident();
+  new(this) AliITSPident(ob);
   return *this;
 }
 
 
 //_______________________________________________________________________
-AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie){
+AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie):
+fMom(mom),
+fdEdx(dEdx),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(priorip),
+fPPriorik(priorik),
+fPPrioripi(prioripi),
+fPPriorie(priorie),
+fInvPt(invPt){
 
   //test
 
-  fMom=mom;
-  fInvPt=invPt;
   for(Int_t la=0;la<4;la++){//loop on layers
     Double_t parp[3];Double_t park[3];Double_t parpi[3];
     sp->GetParFitLayer(la,fMom,parp,park,parpi);
@@ -72,11 +90,6 @@ AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteer
       CookFunItsLay(la,2,parpi,Qlay[la],fMom,range[4],range[5],"fPi");
     
   }
-  fPPriorip=priorip;
-  fPPriorik=priorik;
-  fPPrioripi=prioripi;
-  fPPriorie=priorie;
-
 
   Float_t prior[4];Double_t condFun[4][3];
 
@@ -94,12 +107,22 @@ AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteer
   fPBayesp=CookCombinedBayes(condFun,prior,0);
   fPBayesk=CookCombinedBayes(condFun,prior,1); 
   fPBayespi=CookCombinedBayes(condFun,prior,2); 
-  fdEdx=dEdx;
-}
+ }
 
 
 //_______________________________________________________________________
-AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie){
+AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie):
+fMom(0),
+fdEdx(0),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(priorip),
+fPPriorik(priorik),
+fPPrioripi(prioripi),
+fPPriorie(priorie),
+fInvPt(0)
+{
   //
   Double_t xr;
   Double_t par[5];
@@ -127,11 +150,6 @@ AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Q
       CookFunItsLay(la,1,park,Qlay[la],fMom,range[2],range[3],"fKao");
       CookFunItsLay(la,2,parpi,Qlay[la],fMom,range[4],range[5],"fPi");
   }
-  fPPriorip=priorip;
-  fPPriorik=priorik;
-  fPPrioripi=prioripi;
-  fPPriorie=priorie;
-
 
   Float_t prior[4];Double_t condFun[4][3];
 
index a122728..123b744 100644 (file)
@@ -70,7 +70,7 @@ class AliITSPident : public TObject{
   Float_t CookProd(Double_t condfun[][3],Int_t part) const;
   Float_t CookSum(Double_t condfun[][3],Float_t *prior) const;
   AliITSPident(const AliITSPident &ob); // copy constructor
-  AliITSPident& operator=(const AliITSPident & /* source */); // ass. op.
+  AliITSPident& operator=(const AliITSPident & ob); // ass. op.
 
   Float_t fMom;                   // Particle momentum
   Double_t fdEdx;                 // Particle dE/dx
index dabc5c1..fbb7288 100644 (file)
 ClassImp(AliITSReconstructor)
 
 //___________________________________________________________________________
-AliITSReconstructor::AliITSReconstructor() : AliReconstructor(){
+AliITSReconstructor::AliITSReconstructor() : AliReconstructor(),
+fItsPID(0)
+{
   // Default constructor
-  fItsPID=0;
 }
  //___________________________________________________________________________
 AliITSReconstructor::~AliITSReconstructor(){
@@ -55,21 +56,21 @@ AliITSReconstructor::~AliITSReconstructor(){
   delete fItsPID;
 } 
 //______________________________________________________________________
-AliITSReconstructor::AliITSReconstructor(const AliITSReconstructor &ob) :AliReconstructor(ob) {
+AliITSReconstructor::AliITSReconstructor(const AliITSReconstructor &ob) :AliReconstructor(ob),
+fItsPID(ob.fItsPID) 
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSpidESD2","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSReconstructor& AliITSReconstructor::operator=(const AliITSReconstructor& /* ob */){
+AliITSReconstructor& AliITSReconstructor::operator=(const AliITSReconstructor&  ob ){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSReconstructor();
+  new(this) AliITSReconstructor(ob);
   return *this;
 }
 //______________________________________________________________________
-void AliITSReconstructor::Init(AliRunLoader *runLoader){
+void AliITSReconstructor::Init(AliRunLoader *runLoader) const{
     // Initalize this constructor bet getting/creating the objects
     // nesseary for a proper ITS reconstruction.
     // Inputs:
index 0f81836..814f74a 100644 (file)
@@ -23,8 +23,8 @@ public:
   AliITSReconstructor();
   virtual ~AliITSReconstructor();
   AliITSReconstructor(const AliITSReconstructor &ob); // copy constructor
-  AliITSReconstructor& operator=(const AliITSReconstructor & /* source */); // ass. op.
-  virtual void Init(AliRunLoader* runLoader);
+  AliITSReconstructor& operator=(const AliITSReconstructor & ob); // ass. op.
+  virtual void Init(AliRunLoader* runLoader) const;
   
   virtual void         Reconstruct(AliRunLoader* runLoader) const;
   virtual void         Reconstruct(AliRunLoader* runLoader,
index 64e7a42..0aee419 100644 (file)
 ClassImp(AliITSSteerPid)
 
   //______________________________________________________________
-  AliITSSteerPid::AliITSSteerPid(){
+AliITSSteerPid::AliITSSteerPid():
+fClonarr2(0),
+fVect2(0),
+fVect2lay1(0),
+fVect2lay2(0),
+fVect2lay3(0),
+fVect2lay4(0),
+fFitTree(0),
+fItem(0),
+fPCenter(0),
+fPWidth(0)
+{
   // default constructor
-  fClonarr2=0;
-  fVect2=0;
-  fItem=0;
-  fFitTree=0;
-}
+ }
 //______________________________________________________________
 AliITSSteerPid::~AliITSSteerPid(){
   // destructor
@@ -26,17 +33,25 @@ AliITSSteerPid::~AliITSSteerPid(){
 }
 
 //______________________________________________________________________
-AliITSSteerPid::AliITSSteerPid(const AliITSSteerPid &ob) :TObject(ob) {
+AliITSSteerPid::AliITSSteerPid(const AliITSSteerPid &ob) :TObject(ob),
+fClonarr2(ob.fClonarr2),
+fVect2(ob.fVect2),
+fVect2lay1(ob.fVect2lay1),
+fVect2lay2(ob.fVect2lay2),
+fVect2lay3(ob.fVect2lay3),
+fVect2lay4(ob.fVect2lay4),
+fFitTree(ob.fFitTree),
+fItem(ob.fItem),
+fPCenter(ob.fPCenter),
+fPWidth(ob.fPWidth) {
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSSteerPid","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSSteerPid& AliITSSteerPid::operator=(const AliITSSteerPid& /* ob */){
+AliITSSteerPid& AliITSSteerPid::operator=(const AliITSSteerPid& ob){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSSteerPid();
+  new(this) AliITSSteerPid(ob);
   return *this;
 }
 
index c4b4fa4..075d61e 100644 (file)
@@ -34,7 +34,7 @@ class AliITSSteerPid : public TObject{
 
   AliITSPidParItem* Item(TClonesArray *Vect,Float_t mom);
   AliITSSteerPid(const AliITSSteerPid &ob); // copy constructor
-  AliITSSteerPid& operator=(const AliITSSteerPid & /* source */); // ass. op.
+  AliITSSteerPid& operator=(const AliITSSteerPid & ob); // ass. op.
 
 
   TFile *fClonarr2;              // file of TClones Array
index 7a3cb0d..a9228bf 100755 (executable)
 
 ClassImp(AliITSpidESD1)
 
-AliITSpidESD1::AliITSpidESD1(): AliITSpidESD() {
+AliITSpidESD1::AliITSpidESD1(): AliITSpidESD(),
+fMIP(0),
+fRes(0),
+fRange(0) 
+{
   //Default constructor
-  fMIP=0;
-  fRes=0;
-  fRange=0;
 }
 //_________________________________________________________________________
-AliITSpidESD1::AliITSpidESD1(Double_t *param): AliITSpidESD()
+AliITSpidESD1::AliITSpidESD1(Double_t *param): AliITSpidESD(),
+fMIP(param[0]),
+fRes(param[1]),
+fRange(param[2])
 {
   //
   //  The main constructor
   //
-  fMIP=param[0];
-  fRes=param[1];
-  fRange=param[2];
 }
 
 
index 1236f40..b21a084 100644 (file)
 
 ClassImp(AliITSpidESD2)
 //_________________________________________________________________________
-AliITSpidESD2::AliITSpidESD2():AliITSpidESD()
+  AliITSpidESD2::AliITSpidESD2():AliITSpidESD(),
+fTracker(0),
+fLoader(0),
+fSp(0)
 { //
   //  The main constructor
-  fTracker=0;
-  fLoader=0;
-  fSp=0;
 }
 //_________________________________________________________________________
-AliITSpidESD2::AliITSpidESD2(AliITStrackerMI* tracker,AliITSLoader* loader):AliITSpidESD()
+AliITSpidESD2::AliITSpidESD2(AliITStrackerMI* tracker,AliITSLoader* loader):AliITSpidESD(),
+fTracker(0),
+fLoader(0),
+fSp(0)
 { //
   //  The main constructor
   fTracker=tracker;
@@ -60,17 +63,20 @@ AliITSpidESD2::~AliITSpidESD2(){
 
 }
 //______________________________________________________________________
-AliITSpidESD2::AliITSpidESD2(const AliITSpidESD2 &ob) :AliITSpidESD(ob) {
+AliITSpidESD2::AliITSpidESD2(const AliITSpidESD2 &ob) :AliITSpidESD(ob),
+fTracker(ob.fTracker),
+fLoader(ob.fLoader),
+fSp(ob.fSp) 
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSpidESD2","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSpidESD2& AliITSpidESD2::operator=(const AliITSpidESD2& /* ob */){
+AliITSpidESD2& AliITSpidESD2::operator=(const AliITSpidESD2& ob ){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSpidESD2();
+  new(this) AliITSpidESD2(ob);
   return *this;
 }
 
index 14e5eb2..f8c9168 100755 (executable)
@@ -25,7 +25,7 @@ public:
   virtual ~AliITSpidESD2();
   virtual Int_t MakePID(AliESD *event);
   AliITSpidESD2(const AliITSpidESD2 &ob); // copy constructor
-  AliITSpidESD2& operator=(const AliITSpidESD2 & /* source */); // ass. op.
+  AliITSpidESD2& operator=(const AliITSpidESD2 & source); // ass. op.
 
 private:
   AliITStrackerMI *fTracker; //!tracker MI