diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java index 913c5eea..ce3f9cad 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java @@ -81,7 +81,7 @@ public class VehicleRoutingProblem { private Map tentativeJobs = new LinkedHashMap(); - private Set jobsInInitialRoutes = new HashSet(); + private Set jobsInInitialRoutes = new HashSet<>(); private Map tentative_coordinates = new HashMap(); @@ -308,7 +308,7 @@ public class VehicleRoutingProblem { incActivityIndexCounter(); if (act instanceof TourActivity.JobActivity) { Job job = ((TourActivity.JobActivity) act).getJob(); - jobsInInitialRoutes.add(job.getId()); + jobsInInitialRoutes.add(job); addLocationToTentativeLocations(job); registerJobAndActivity(abstractAct, job); } @@ -429,7 +429,7 @@ public class VehicleRoutingProblem { transportCosts = new CrowFlyCosts(getLocations()); } for (Job job : tentativeJobs.values()) { - if (!jobsInInitialRoutes.contains(job.getId())) { + if (!jobsInInitialRoutes.contains(job)) { addJobToFinalJobMapAndCreateActivities(job); } } @@ -438,8 +438,8 @@ public class VehicleRoutingProblem { for (Job job : jobs.values()) { ((AbstractJob)job).setIndex(jobIndexCounter++); } - for (String jobId : jobsInInitialRoutes) { - ((AbstractJob)tentativeJobs.get(jobId)).setIndex(jobIndexCounter++); + for (Job job : jobsInInitialRoutes) { + ((AbstractJob)job).setIndex(jobIndexCounter++); } boolean hasBreaks = addBreaksToActivityMap();