fixes for null pointers exceptions

raffaelladevita · raffaelladevita · commit 1fdbc5f82a01 · 2021-09-23T23:51:13.000+02:00
diff --git a/build-coatjava.sh b/build-coatjava.sh
@@ -98,8 +98,8 @@ cp external-dependencies/jclara-4.3-SNAPSHOT.jar coatjava/lib/utils
 mkdir -p coatjava/lib/services
 
 ### clean up any cache copies ###
-rm -rf ~/.m2/repository/org/hep/hipo
-rm -rf ~/.m2/repository/org/jlab
+#rm -rf ~/.m2/repository/org/hep/hipo
+#rm -rf ~/.m2/repository/org/jlab
 cd common-tools/coat-lib; $mvn clean; cd -
 
 unset CLAS12DIR
diff --git a/reconstruction/dc/src/main/java/org/jlab/rec/dc/track/TrackCandListFinder.java b/reconstruction/dc/src/main/java/org/jlab/rec/dc/track/TrackCandListFinder.java
@@ -749,7 +749,7 @@ private Track FindBestTrack(List<Track> trkList) {
     public void matchHits(List<StateVec> stateVecAtPlanesList, Track trk,
             DCGeant4Factory DcDetector, Swim dcSwim) {
 
-        if (stateVecAtPlanesList == null) {
+        if (stateVecAtPlanesList == null || trk.get_Vtx0()==null) {
             return;
         }
 
@@ -819,9 +819,6 @@ public void setHitDoubletsInfo(Segment seg) {
                     h.set_AssociatedHBTrackID(-1);
             }          
         }
-        else {
-            System.out.println("setHitDoubletsInfo : this method should be used only for segments on track");
-        }
     }
 
     private double calcCurvSign(Track cand) {
diff --git a/reconstruction/dc/src/main/java/org/jlab/service/dc/DCTBEngine.java b/reconstruction/dc/src/main/java/org/jlab/service/dc/DCTBEngine.java
@@ -219,9 +219,11 @@ public boolean processDataEvent(DataEvent event) {
             return true; // HB tracks not saved correctly
         }
         for(Segment seg : segments) {
-            TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].get_ListOfHBSegments().add(seg); 
-            if(seg.get_Status()==1)
-                TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].set_Status(1);
+            if(seg.get(0).get_AssociatedHBTrackID()>0) {
+                    TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].get_ListOfHBSegments().add(seg); 
+                    if(seg.get_Status()==1)
+                        TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].set_Status(1);
+            }
         }
         
         //6) find the list of  track candidates

Original file line number	Diff line number	Diff line change
`@@ -749,7 +749,7 @@ private Track FindBestTrack(List<Track> trkList) {`
`749`	`749`	`public void matchHits(List<StateVec> stateVecAtPlanesList, Track trk,`
`750`	`750`	`DCGeant4Factory DcDetector, Swim dcSwim) {`
`751`	`751`
`752`		`- if (stateVecAtPlanesList == null) {`
	`752`	`+ if (stateVecAtPlanesList == null \|\| trk.get_Vtx0()==null) {`
`753`	`753`	`return;`
`754`	`754`	`}`
`755`	`755`
`@@ -819,9 +819,6 @@ public void setHitDoubletsInfo(Segment seg) {`
`819`	`819`	`h.set_AssociatedHBTrackID(-1);`
`820`	`820`	`}`
`821`	`821`	`}`
`822`		`- else {`
`823`		`- System.out.println("setHitDoubletsInfo : this method should be used only for segments on track");`
`824`		`- }`
`825`	`822`	`}`
`826`	`823`
`827`	`824`	`private double calcCurvSign(Track cand) {`
Original file line number	Diff line number	Diff line change
`@@ -219,9 +219,11 @@ public boolean processDataEvent(DataEvent event) {`
`219`	`219`	`return true; // HB tracks not saved correctly`
`220`	`220`	`}`
`221`	`221`	`for(Segment seg : segments) {`
`222`		`- TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].get_ListOfHBSegments().add(seg);`
`223`		`- if(seg.get_Status()==1)`
`224`		`- TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].set_Status(1);`
	`222`	`+ if(seg.get(0).get_AssociatedHBTrackID()>0) {`
	`223`	`+ TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].get_ListOfHBSegments().add(seg);`
	`224`	`+ if(seg.get_Status()==1)`
	`225`	`+ TrackArray[seg.get(0).get_AssociatedHBTrackID()-1].set_Status(1);`
	`226`	`+ }`
`225`	`227`	`}`
`226`	`228`
`227`	`229`	`//6) find the list of track candidates`