mrTsjolder / SWOP-KULAK

Source code for project Software design
GNU General Public License v2.0
0 stars 1 forks source link

Task constructor -status #15

Closed Xenolel closed 9 years ago

Xenolel commented 9 years ago

Klopt het dat we niet langer willen dat het mogelijk is de status van een task mee te geven in de constructor?

Dit betekent dat de volgende 2 constructors overblijven:

isostorm commented 9 years ago

We hebben die constructor nog nodig voor het inleesbestand, want sommmige taken staan op finished daar blijkbaar.

Xenolel commented 9 years ago

Moeten we daarmee rekening houden? We kunnen ook de taak aanmaken en ze daarna updaten in de parser

isostorm commented 9 years ago

Ja inderdaad da kan ook natuurlijk en dat is mss het best, dus ja dan mag die constructor idd weg

mrTsjolder commented 9 years ago

nog een voorstel: extra constructor Task(String description, long estDur, int accDev, Task altFor, Task[] prereq) zodat setAlternativeTask() in die constructor kan gebeuren en private wordt?

isostorm commented 9 years ago

Ja dat is een zeer goed idee

Xenolel commented 9 years ago

Als je setAlternativeTask op private wilt zetten, betekent het ook dat je niet langer bestaande taken kunt kiezen als alternative voor een andere taak

mrTsjolder commented 9 years ago

misschien is het niet slecht als we die abstractie maken... ?

Xenolel commented 9 years ago

sgoed, kzal setAlternativeTask private maken