Program Model setTrajects

[fsuarezg]

de functie setTrajects neemt een list maar dit zou een set moeten zijn. (of er moet een extra variant toegevoegd worden die een set neemt) Ik neem aan dat ik dat zelf niet mag aanpassen? @nicocarraggi @svdvonde

[Timbo925]

Ben niet zeker of het aanpassen iets gaat kapot maken maar je kan gewoonn de funcitie setTraject(Set<Traject?) aanmaken in de service die exact het zelfde doet als deze met de list. Makkelijkste en veiligste optie.

[fsuarezg]

ik zal hem zelf gewoon aanmaken zonder die andere methode te verwijderen :)

[Timbo925]

👍

[MPieter]

Ik denk dat we best dit issue laten openstaan, zodanig dat we weten dat we die dubbele methode nog moeten verwijderen.

Een algemene oplossing zou zijn:

public void setTrajects(Collection<Traject> trajects) {
    this.trajects.addAll(trajects);
}

Vermits set en list allebei overerven van collectie zou dit moeten gaan denk ik.

[nicocarraggi]

public void setTrajects(Set<Traject> newTrajects) {
    this.trajects.clear();
    this.trajects.addAll(newTrajects);
}

Dit heb ik er van gemaakt! Elke keer dat een Set wordt ge-set moeten de elementen van newTrajects worden toegevoegd aan de bestaande Set van trajects. De reden hiervan is om het removen ook goed te laten werken.

[MPieter]

@fsuarezg Let hierbij als je merged, je neemt nu best een pull van calzone.