RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ItEr59S08CUAsignacionRecursosLimitantesItEr58S10: Since expression is too complicated use if-else instead

Browse source
This commit is contained in:
Óscar González Fernández 2010-06-07 00:45:25 +02:00
parent 1d8b04d5be
commit cd0cfc04b6
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
navalplanner-business/src/main/java/org/navalplanner/business/planner/limiting/entities/Gap.java
View file

@ -99,11 +99,14 @@ public class Gap implements Comparable<Gap> {
}
private Integer calculateHoursInGap(Resource resource, DateAndHour startTime, DateAndHour endTime) {
// this method will be removed; when hours in gap is really used
// startTime is never null
return (endTime == null || startTime == null) ? Integer.MAX_VALUE
: calculateHoursInGap(resource, startTime.getDate(), startTime
.getHour(), endTime.getDate(), endTime.getHour());
// TODO remove this method. Use GapRequirements instead
if (endTime == null || startTime == null) {
// startTime is never null when hours in gap is really use
return Integer.MAX_VALUE;
} else {
return calculateHoursInGap(resource, startTime.getDate(), startTime
.getHour(), endTime.getDate(), endTime.getHour());
}
}
public int getHoursInGap() {