Je maakt enkele fouten, die later lastig kunnen worden:

- Functies zijn public, private of protected, dit komt voor het function keyword te staan.
- Members declareer je in de klasse, om meer overzicht te houden, deze zijn ook public, private of protected.
- Kijk eens naar de magic method toString().
- Je plaatst je accolades wat vreemd?

Heb hem ook even voor je geupdate.;)

private moet vergete, wordt nooit gebruikt.

beide functies die je nu hebt worden BUITEN de class gebruikt dus public.

Wel cool topic Misschien kan je het nog wat uitbreiden door van die $candy een array te maken en altijd snoepjes toe te voegen?

Ja, je kunt de constructor normaal gesproken niet op private of protected zetten, tenzij je met singleton werkt.;) Constructors zijn nogmaal gesproken dus public. Wanneer je een instantie aanmaakt wordt de constructor namelijk aangeroepen, van buiten de class, tenzij je binnen de class de instantie maakt!

protected kan je gebruiken bij een functie die je enkel in de class gebruikt bijvoorbeeld om telkens het meest gewaardeerde snoepje te berekenen en dan laat je de uitkomst daarvan returnen door een functie die je van buiten afroept

je moet van die null array() maken en dan van die regel waar je candies bijdoet doe je $this->candy[] = $new_candy;

de constructor hoeft dan niet echt meer aangezien je snoepjes later add met die functie

Het is sowieso al geen OOP Wat je nu doet zijn gewoon classes, je moet ze nog leren verbinden met elkaar etc.

Voila en wij geven je tips om je class verder uit te breiden Zo leer je meer hé :p

je hebt er wss geen array van gemaakt?

Een array er van maken is al helemaal geen OOP.;) Als je OOP te werk wilt gaan kun je beter een CandyFactory maken, en een Candy class. Deze Factory maakt dus de Candy classes en houd ze bij. Dat komt dichter in de buurt van OOP!

Komt door je constructor Doe die weg en doe die "dropje" ook weg en dan heb je het niet meer