[PHP] OO : sterk stukje code schrijven

[whitehouse]

ik wil de volgende code object georienteerd schrijven .. ik heb het idee (maar weet nog niet hoe ik moet beginnen) dat ik nogal een omslachtige manier gebruik..
hebben jullie tips / hints ?

PHP:

$module='mod';
for ($i=2;$i<=14;$i++)
{
    $m=$module.$i;
    
    if (isset($_GET[$m]))
    {
        $count=$count+1;
        echo "<b>gepost : $m</b><br>";
        $mod=$_GET[$m]; 
        switch ($m)
         {
        case 'mod2':
            $mod2=$m;
            break;
        case 'mod3':
            $mod3=$m;
            break;
        case 'mod4':
           $mod4=$m;
           break;
        case 'mod5':
           $mod5=$m;
           break;
        case 'mod6':
           $mod6=$m;
           break;
        case 'mod7':
           $mod7=$m;
           break;
        case 'mod8':
           $mod8=$m;
           break;
        case 'mod9':
           $mod9=$m;
           break ;
        case 'mod10':
           $mod10=$m;
           break ;
        case 'mod11':
           $mod11=$m;
           break ;
        case 'mod12':
           $mod12=$m;
           break ;
        case 'mod13':
           $mod13=$m;
           break ;
        case 'mod14':
           $mod14=$m;
           break ;
        }
    }

}

[Verwijderd]

Ik snap niet echt wat je bedoeld, wat wil je objectgeorienteerd hebben??
Misschien een wat duidelijkere uitleg...

[Verwijderd]

Ik zie hier geen enkele vorm van OO in eerlijk gezegd. Wat wil je bereiken?

[Super_ik]

hij wil deze brakke code in OO geschreve hebbe,
of iig weten hoe dat zou moeten aanpakken

[gorgi_19]

Nog een vraag: wat moet het kunnen? Een beetje abstractie is niet erg, maar kan er ook iets meer bij gedocumenteerd worden?

[whitehouse]

ik heb nu al die cases uitgeschreven .. kan dit ook op een iets meer oo-manier ??

ik wil dus in feite maar 1 keer een case uittypen die voor alle 14 mods werkt..

ik heb een pagina waar 14 modules in staan, die je kunt aanvinken met een checkbox.. nu wil ik als een (of meerdere) checkboxes is aangevinkt, deze op de volgende pagina opvragen ...
(ik post deze middels een GET )..

nu heb ik dat dus gedaan middels die cases, maar ik wil dit wat compacter maken en in feite, als er i.p.v. 14 modules ineens 20 zijn, dat het script het gewoon blijft doen..

[ Voor 56% gewijzigd door whitehouse op 20-01-2003 23:06 ]

[GrimaceODespair]

Een vraagje: waarom moet het eigenlijk OO? Om OO te leren, of gewoon omdat OO "in" is?

edit:
Sorry, no flame intended

[ Voor 24% gewijzigd door GrimaceODespair op 20-01-2003 23:06 ]

[Verwijderd]

Werk met een array()!

_{Of roep ik nu iets doms?}

[Gomez12]

Ikke niet snappe wat je hier nou doet, als je switch als input krijgt 'mod2', dan zet hij $mod2 naar 'mod2'???

[Verwijderd]

je zou zoiets als een variabele variabele kunnen maken dus iets als dit:
$mod{$i} = $m; ipv $mod2 = $m;
of dit verstandig is weet ik niet, de meningen zijn hierover verdeeld (zie andere topics hierover)

[GrimaceODespair]

Tja... mijn ervaring met PHP is dat om volwaardig te OO-en (polymorfisme etc), je erg kunstmatig te werk moet gaan. Precies de uitgebreide array opties zijn inderdaad de kracht van php. Vandaar ook mijn eerdere vraag: "Waarom OO"? Meestal werken arrays immers gemakkelijker.

[Gomez12]

checkboxen, bitwyze dan maar, gewoon 20 bitjes achter elkaar en gewoon zetten wat je nodig hebt. dus 10111110000000... etc.

[whitehouse]

in feite wil ik het volgende doen :

if (isset($mod2=$_POST['mod2'];)
{
$mod2= $_POST['mod2'];
}

maar dan wil ik dit niet 14 keer neerzetten...
ik wil dus dat het nummer hierboven variabel is..

[Spider.007]

volgens mij bedoel je niet echt OO; maar zoek je meer een manier om dit geen 14 keer te hoeven uitschrijven; ga eens op PHP.net kijken naar Arrays enzo

edit:
Ga asjeblieft PHP leren

[ Voor 13% gewijzigd door Spider.007 op 20-01-2003 23:21 ]

[gorgi_19]

Ik zou eens kijken naar een array. Lees gewoon alle waarden in en stop de geldige waarden in een array.

Vervolgens kan je later door middel van een functie controleren of een waarde in de array zit. (kan geen PHP-code hiervan laten zien, sorry)

[Verwijderd]

whitehouse schreef op 20 januari 2003 @ 23:12:

PHP:

if (isset($mod2=$_POST['mod2'];)
{
   $mod2= $_POST['mod2'];
}

heb je enig idee hoe brak deze code is? en dan vooral de eerste regel? waarschijnlijk wel, maar anders zou ik beginnen met een php boek aan te schaffen ofzo...

[whitehouse]

PHP:

<?
if (isset($_POST['mod2'];) 
{ 
   $mod2= $_POST['mod2']; 
}
?>  

bedoelde eigenlijk dit ..

[ Voor 23% gewijzigd door whitehouse op 20-01-2003 23:21 ]

[gorgi_19]

whitehouse schreef op 20 January 2003 @ 23:20:

PHP:

<?
if (isset($_POST['mod2'];) 
{ 
   $mod2= $_POST['mod2']; 
}
?>  

bedoelde eigenlijk dit ..

Als je echt met deze methode door wilt gaan, dan kan je ook eens [rml][ php]Variabele in een variabele[/rml] kijken.

Je probleem is echter niet OO; ik kom hier vooralsnog weinig van tegen. Je wilt gewoon een eenvoudige functie schrijven voor iets.

De tweede optie, zoals al vaker is gezegd, is het gebruik van een array. Dit zijn imho de enige twee nette oplossingen (waarbij ik een voorkeur heb voor de array). Ik denk dat je met deze info weer een stuk verder moet kunnen...

[Verwijderd]

whitehouse schreef op 20 januari 2003 @ 23:20:

PHP:

<?
if (isset($_POST['mod2'];) 
{ 
   $mod2= $_POST['mod2']; 
}
?>  

bedoelde eigenlijk dit ..

leuk en aardig hoor, maar nog steeds eeb ; teveel ... *zucht*
als je het aanpast doe het dan ook goed!

[Nielsz]

En een ) te weinig

[drm]

en een manual te weinig ook, kennelijk

Zoals al een aantal maal gezegd, ga je even verdiepen in arrays. Alles en nog veel meer over te lezen in de PHP manual