From 1131c504c693d0042ba403cd9a014ee7af15c0b4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?= Date: Thu, 16 Sep 2010 20:31:11 +0200 Subject: [PATCH] Remove resetGenericAssignments method It was confusing since generic is a name also used in GenericResourceAllocation and GenericDayAssignment. FEA: ItEr60S19TimeUnitDataType --- .../planner/entities/ResourceAllocation.java | 22 +++------------ ...orForSpecifiedResourcesPerDayAndHours.java | 28 +++++++++++-------- 2 files changed, 21 insertions(+), 29 deletions(-) diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java index 07d6236a0..74d04e890 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java @@ -231,11 +231,10 @@ public abstract class ResourceAllocation extends } @Override - protected void setNewDataForAllocation( - ResourceAllocation allocation, TaskDate end, - ResourcesPerDay resourcesPerDay, - List dayAssignments) { - allocation.resetGenericAssignmentsTo(dayAssignments, end); + protected void setNewDataForAllocation( + ResourceAllocation allocation, TaskDate end, + ResourcesPerDay resourcesPerDay, List dayAssignments) { + allocation.resetAssignmentsTo(dayAssignments, end); allocation.updateResourcesPerDay(); } @@ -812,19 +811,6 @@ public abstract class ResourceAllocation extends protected abstract ICalendar getCalendarGivenTaskCalendar( ICalendar taskCalendar); - private void resetGenericAssignmentsTo(List assignments, - TaskDate end) { - resetAssignmentsTo(cast(assignments), end); - } - - private List cast(List value) { - List result = new ArrayList(); - for (DayAssignment dayAssignment : value) { - result.add(getDayAssignmentType().cast(dayAssignment)); - } - return result; - } - protected abstract Class getDayAssignmentType(); public ResourceAllocation copy(Scenario scenario) { diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/allocationalgorithms/AllocatorForSpecifiedResourcesPerDayAndHours.java b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/allocationalgorithms/AllocatorForSpecifiedResourcesPerDayAndHours.java index 803057154..233d8e63a 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/allocationalgorithms/AllocatorForSpecifiedResourcesPerDayAndHours.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/allocationalgorithms/AllocatorForSpecifiedResourcesPerDayAndHours.java @@ -110,20 +110,26 @@ public abstract class AllocatorForSpecifiedResourcesPerDayAndHours { private void setAssignmentsForEachAllocation(TaskDate end) { for (Entry> entry : resultAssignments .entrySet()) { - ResourceAllocation allocation = entry.getKey() - .getBeingModified(); - ResourcesPerDay resourcesPerDay = entry.getKey() - .getGoal(); - List value = entry.getValue(); - setNewDataForAllocation(allocation, end, resourcesPerDay, - value); + setNewDataForAllocation(entry, end); } } - protected abstract void setNewDataForAllocation( - ResourceAllocation allocation, TaskDate explicitEnd, - ResourcesPerDay resourcesPerDay, - List dayAssignments); + private void setNewDataForAllocation( + Entry> entry, + TaskDate end) { + @SuppressWarnings("unchecked") + ResourceAllocation allocation = (ResourceAllocation) entry + .getKey().getBeingModified(); + ResourcesPerDay resourcesPerDay = entry.getKey().getGoal(); + @SuppressWarnings("unchecked") + List value = (List) entry.getValue(); + setNewDataForAllocation(allocation, end, resourcesPerDay, + value); + } + + protected abstract void setNewDataForAllocation( + ResourceAllocation allocation, TaskDate explicitEnd, + ResourcesPerDay resourcesPerDay, List dayAssignments); protected abstract List createAssignmentsAtDay( ResourcesPerDayModification allocation, LocalDate day,