From 94631b2fb999a25e4372c1ec6d3fc3d113a8dac5 Mon Sep 17 00:00:00 2001 From: masera Date: Fri, 20 Oct 2006 17:11:57 +0000 Subject: [PATCH] changes to be compliant with Eff C++ rules --- ITS/AliITSPidParItem.cxx | 14 ++++--- ITS/AliITSPident.cxx | 84 ++++++++++++++++++++++--------------- ITS/AliITSPident.h | 2 +- ITS/AliITSReconstructor.cxx | 19 +++++---- ITS/AliITSReconstructor.h | 4 +- ITS/AliITSSteerPid.cxx | 39 +++++++++++------ ITS/AliITSSteerPid.h | 2 +- ITS/AliITSpidESD1.cxx | 17 ++++---- ITS/AliITSpidESD2.cxx | 28 ++++++++----- ITS/AliITSpidESD2.h | 2 +- 10 files changed, 127 insertions(+), 84 deletions(-) diff --git a/ITS/AliITSPidParItem.cxx b/ITS/AliITSPidParItem.cxx index 015415527f9..3b41c9706e9 100644 --- a/ITS/AliITSPidParItem.cxx +++ b/ITS/AliITSPidParItem.cxx @@ -10,18 +10,20 @@ 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]; } diff --git a/ITS/AliITSPident.cxx b/ITS/AliITSPident.cxx index 3b51d0985c1..fcdc421b753 100644 --- a/ITS/AliITSPident.cxx +++ b/ITS/AliITSPident.cxx @@ -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]; diff --git a/ITS/AliITSPident.h b/ITS/AliITSPident.h index a1227283545..123b7449147 100644 --- a/ITS/AliITSPident.h +++ b/ITS/AliITSPident.h @@ -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 diff --git a/ITS/AliITSReconstructor.cxx b/ITS/AliITSReconstructor.cxx index dabc5c10569..fbb7288670b 100644 --- a/ITS/AliITSReconstructor.cxx +++ b/ITS/AliITSReconstructor.cxx @@ -45,9 +45,10 @@ 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: diff --git a/ITS/AliITSReconstructor.h b/ITS/AliITSReconstructor.h index 0f818367df5..814f74aa96b 100644 --- a/ITS/AliITSReconstructor.h +++ b/ITS/AliITSReconstructor.h @@ -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, diff --git a/ITS/AliITSSteerPid.cxx b/ITS/AliITSSteerPid.cxx index 64e7a428aa5..0aee419ed9f 100644 --- a/ITS/AliITSSteerPid.cxx +++ b/ITS/AliITSSteerPid.cxx @@ -11,13 +11,20 @@ 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; } diff --git a/ITS/AliITSSteerPid.h b/ITS/AliITSSteerPid.h index c4b4fa42d79..075d61ecb51 100644 --- a/ITS/AliITSSteerPid.h +++ b/ITS/AliITSSteerPid.h @@ -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 diff --git a/ITS/AliITSpidESD1.cxx b/ITS/AliITSpidESD1.cxx index 7a3cb0d2ecb..a9228bf9736 100755 --- a/ITS/AliITSpidESD1.cxx +++ b/ITS/AliITSpidESD1.cxx @@ -26,21 +26,22 @@ 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]; } diff --git a/ITS/AliITSpidESD2.cxx b/ITS/AliITSpidESD2.cxx index 1236f40191f..b21a084ef99 100644 --- a/ITS/AliITSpidESD2.cxx +++ b/ITS/AliITSpidESD2.cxx @@ -36,15 +36,18 @@ 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; } diff --git a/ITS/AliITSpidESD2.h b/ITS/AliITSpidESD2.h index 14e5eb2f512..f8c91680c4e 100755 --- a/ITS/AliITSpidESD2.h +++ b/ITS/AliITSpidESD2.h @@ -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 -- 2.43.0