VerwachteInvoerImpl.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1486] Fouten in UT TestData verminderen
[ICT-1486] Fouten in UT TestData verminderen
  • More
  • CR-712
  • finished reviewing
idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we dez...

idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we deze property vijrgeven zodat deze rechtstreeks aangepast kan worden vanuit de code, met alle mogelijke gevolgen van dien. Ook hier kan je beter voorkomen dat deze aangepast kan worden.

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een proper...

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het ProductID te wijzigen.

IDem als ProductID : VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu ...

IDem als ProductID :

VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het veld te wijzigen.

[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.
[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.