mrTsjolder / SWOP-KULAK

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

omtrent Task.setAlternativeTask #28

Closed Xenolel closed 9 years ago

Xenolel commented 9 years ago

We willen Task.setAlternativeTask(altTask, project) hebben en in deze methode hetvolgende checken:

project.hasTask(this) && project.hasTask(altTask)

Dit heeft als gevolg dat de volgorde van de volgende regels in Project.createTask als volgt moet zijn:

addTask(t);
if(altFor != Project.NO_ALTERNATIVE)
    getTask(altFor).setAlternativeTask(t, this);

Het probleem is nu dat de task kan aangemaakt worden zelfs als de alternativeFor koppeling mislukt

oplossingen:

nu weet ik niet echt wat de beste oplossing is

isostorm commented 9 years ago

Het 2de lijkt het best ma dan is er wss wel een dubbele check? Wrm mag die taak eigenlijk niet toegevoegd worden, het systeem wordt eigenlijk niet in een verkeerde staat gebracht hé.

Xenolel commented 9 years ago

ja we kunnen da inderdaad eigl. gewoon toelaten

isostorm commented 9 years ago

Het moet gwn duidelijk vermeld staan in de commentaar.