[PHP5] Objecten in een Array

[jverdeyen]

Hoi,

Ik ben een kleine applicatie aan het schrijven. Nu heb ik al even zitten piekeren over een oplossing voor volgend "probleem".

Heel concreet:

Twee soorten objecten

• Rekening
• Verkoop

Een Rekening bestaat uit:

• Een Array van Verkoop Objecten
• ... nog andere gegevens

on

Een Verkoop Object bestaat uit artikels, prijs, korting etc...

Aangezien ik een functie heb onder de klasse Rekening, nl. addVerkoop,
die ervoor zorg dat de Array aangevuld met het meegegeven Verkoop Object.

addVerkoop basis :

PHP:

function addVerkoop($verkoop)
    {
        array_push($this->verkopen,$verkoop);
    }

Is het nu mogelijk om deze objecten opnieuw te recupureren (op te halen) uit dit Rekening Object ?

Ik heb al volgende methode gebruikt,

PHP:

$verkoop1 = new Verkoop();
$verkoop1->setAantal(1);
$verkoop1->setKorting(10);
$verkoop1->setArtikel(56);

$verkoop2 = new Verkoop();
$verkoop2->setAantal(5);
$verkoop2->setKorting(20);
$verkoop2->setArtikel(53);

$rekening = new Rekening();
$rekening->addVerkoop($verkoop1);
$rekening->addVerkoop($verkoop2);

$i=0;
$lijst = $rekening->getVerkopen();

while ( $i < sizeof($lijst) )
{
    echo $lijst[$i]->getArtikel()."<br/>";
    $i++;
}

Dit is maar een demo code om te zien hoe ik dit later correct kan aanpakken.

Nu, deze manier werkt, maar is dit wel de juiste manier van werken ? Of zit ik hier maar te klooien aan iets dat later voor probs kan zorgen ?

Thx

[ Voor 4% gewijzigd door jverdeyen op 01-06-2007 17:36 ]

[SchizoDuckie]

gaat op zich goed zo toch?

Ik ben alleen zelf niet zon voorstander van nutteloze getters en setters:

PHP:

$Rekening = new Rekening();
$Rekening->verkopen[] = $verkoop1; // [] zorgt voor gelijk adden aan array

print_r($rekening->verkopen);

[ Voor 8% gewijzigd door SchizoDuckie op 01-06-2007 17:34 ]

[jverdeyen]

SchizoDuckie schreef op vrijdag 01 juni 2007 @ 17:33:
gaat op zich goed zo toch?

Ik ben alleen zelf niet zon voorstander van nutteloze getters en setters:

PHP:

$Rekening = new Rekening();
$Rekening->verkopen[] = $verkoop1; // [] zorgt voor gelijk adden aan array

print_r($rekening->verkopen);

Ok thx.
Dus zo mag ik effectief verderwerken ?

De addVerkoop functie gaat nog vanalles nakijken ... gewoon toevoegen aan de array maakt dit wrs niet mogelijk ?

[Grijze Vos]

SchizoDuckie schreef op vrijdag 01 juni 2007 @ 17:33:
gaat op zich goed zo toch?

Ik ben alleen zelf niet zon voorstander van nutteloze getters en setters:

Ik ben zelf niet zo'n voorstander van openstellen van code onderdelen waar andere componenten geen fuck mee te maken hebben.

Ooit gehoord van encapsulatie, design by contract, en aanverwanten?

[KompjoeFriek]

Getters en Setters zijn zeker niet nutteloos.

Zeker als hij ook nog dit zegt:

jverdeyen schreef op vrijdag 01 juni 2007 @ 17:37:
[...]
De addVerkoop functie gaat nog vanalles nakijken ... gewoon toevoegen aan de array maakt dit wrs niet mogelijk ?

Je demo code ziet er in mijn ogen wel netjes uit.

ipv

PHP:

$verkoop1 = new Verkoop();
$verkoop1->setAantal(1);
$verkoop1->setKorting(10);
$verkoop1->setArtikel(56);

zou je nog kunnen overwegen om die dingen als parameter in je "constructor" mee te nemen zodat die aanroep bv zoiets word:

PHP:

$verkoop1 = new Verkoop(1, 10, 56);

[PrisonerOfPain]

Grijze Vos schreef op zaterdag 02 juni 2007 @ 00:49:
[...]

Ik ben zelf niet zo'n voorstander van openstellen van code onderdelen waar andere componenten geen fuck mee te maken hebben.

Ooit gehoord van encapsulatie, design by contract, en aanverwanten?

Daar gaan accessors/mutators je niet bij helpen. Encapsulatie word impliciet nog steeds gebroken omdat je alsnog de ingewanden van je object blootstelt aan de buitenwereld, zei het via een workaround. Wat design by contract er mee te maken heeft is me vaag, als je correctheid wilt garanderen moet je je properties überhaupt moet bekend maken.

Ooit gehoord van het Hollywood principle, Inversion of control of gewoonweg goed object georiënteerd ontwerpen?

[Spockz]

Ehm. Als jij nog extra checks wilt uitvoeren voordat je een item aan dat array toevoegt zul je toch echt een setter moeten gebruiken aangezien je geen properties hebt zoals je die in Delphi hebt.

[Alex Picard]

In plaats van

PHP:

$i=0;
$lijst = $rekening->getVerkopen();

while ( $i < sizeof($lijst) )
{
    echo $lijst[$i]->getArtikel()."<br/>";
    $i++;
}

Zou ik aanraden:

PHP:

$lijst = $rekening->getVerkopen();
foreach ($lijst as $verkoop) {
    echo $verkoop->getArtikel()."<br/>";
}

Tenzij de index i belangrijk is, maar dat is deze meestal niet.

[jverdeyen]

Alex Picard schreef op zaterdag 02 juni 2007 @ 15:42:
In plaats van

PHP:

$i=0;
$lijst = $rekening->getVerkopen();

while ( $i < sizeof($lijst) )
{
    echo $lijst[$i]->getArtikel()."<br/>";
    $i++;
}

Zou ik aanraden:

PHP:

$lijst = $rekening->getVerkopen();
foreach ($lijst as $verkoop) {
    echo $verkoop->getArtikel()."<br/>";
}

Tenzij de index i belangrijk is, maar dat is deze meestal niet.

Owh nice, hij gaat effectief het object gebruiken dan.

@kompjoefriek

In mijn contructor(s) doe ik meestal een unset van alle variabelen,
een reden kan ik niet meteen geven ... gewoonte wrs.
Nu gebruik ik wel een init() functie waar 'k de paramters dan in meegeef,
dat is ongeveer dezelfde manier van werken.

Verder thx voor de info,
hierbij heb ik weer massas bijgeleerd.

[Spockz]

Ik zou eigenlijk nog wel zo ver willen gaan dat je beter:

PHP:

foreach ($rekening->getVerkopen() as $verkoop) { /*...*/ }

kan gebruiken omdat je nu niet een hele lange lijst kopieert. En het zou nog mooier zijn als je met een reference werkt, maar die zijn bij PHP in de praktijk trager.

[remcotolsma]

Alex Picard schreef op zaterdag 02 juni 2007 @ 15:42:
Tenzij de index i belangrijk is, maar dat is deze meestal niet.

Ook als de index belangrijk is zou ik de 'foreach' constructie aanraden. En het is denk ik niet echt zinvol om eerst 2 strings aan elkaar te plakken en daarna te echo'en. Beide strings achter elkaar aan echo'en werkt ook prima.

PHP:

foreach ($lijst as $key => $verkoop) {
    echo $key, ': ', $verkoop->getArtikel(), '<br/>'; 
}

[jverdeyen]

de echo speelt niet zoveel rol,
dat was puur om de code te testen.

[ACM]

Alex Picard schreef op zaterdag 02 juni 2007 @ 15:42:
Tenzij de index i belangrijk is, maar dat is deze meestal niet.

In welk geval je altijd nog dit kan doen:

PHP:

foreach($lijst as $i => $object)
{}

Spockz schreef op zaterdag 02 juni 2007 @ 15:54:
kan gebruiken omdat je nu niet een hele lange lijst kopieert. En het zou nog mooier zijn als je met een reference werkt, maar die zijn bij PHP in de praktijk trager.

Je kopieert de lijst helemaal niet, PHP werkt standaard intern met referenties die volgens het copy-on-change principe aangepast worden zodra dat nodig is. Zolang je de array dus niet wijzigt maar enkel uitleest maakt ie er ook geen echte copy van, maar bij de reference die jij liever hebt moet ie wel wat andere administratie bijhouden.

[Spockz]

Je kopieert de lijst helemaal niet, PHP werkt standaard intern met referenties die volgens het copy-on-change principe aangepast worden zodra dat nodig is. Zolang je de array dus niet wijzigt maar enkel uitleest maakt ie er ook geen echte copy van, maar bij de reference die jij liever hebt moet ie wel wat andere administratie bijhouden.

Ok. Sorry, my bad.