[PHP] siteontwerp netter maken

[MBV]

Excuses voor de vage titel, ik probeer het in de TS beter te doen

Ik zit hier met een vrij slecht uitbreidbare site. Een beheeromgeving waarbij verschillende beheerders voor hun klanten rechten kunnen toekennen e.d., heeft vaak dezelfde gegevens nodig. Dus kom je, nadat je bij index.php op een van de links hebt geklikt, eerst bij klantkeuze uit. Daarna ga je soms verder naar kies gebruiker om iets te doen.

structuur is dus als volgt:

code:

voegtoe verwijder keuze3 keuze4
    |       |        |        |
     -------------------------
                |
           kies klant
                |------------ 
            kies gebruiker   |
     ---------------         |
    |       |       |        |
voegtoe verwijder keuze3 keuze4

Hoe kan ik dat overzichtelijk maken? In de code is het nog een geweldige if-else ladder, omdat er niet 4 maar 20 knoppen zijn die langs 'kies klant' gaan, en sommige ook langs 'kies gebruiker'. Case-structuur lijkt mij een verbetering, maar ook niet geweldig. Heeft iemand een beter idee?

[BtM909]

Zo zonder code en ontwerp vraag ik me af waarom je een 'geweldige if-else ladder' hebt

Desnoods geef je ze eerst de keuze om een klant te kiezen, zodat je nog maar 4 opties hebt na stap 1?

Maar ik blijf erbij dat het veel simpeler kan, mits je wat meer info hier neerplempt

[MBV]

ok, ik zal even wat meer info verzamelen in de vorm van code. Mij lijkt dat alleen niet zo noodzakelijk, omdat het om de structuur gaat.
een gedeelte van kiesklant.php:

PHP:

<?
if ($_GET['action'])=delete)
{
   $next="kiesgebruiker.php";
   $nexturl="delete.php";
}

if ($_GET['action'])=add)
{
   $next="kiesgebruiker.php";
   $nexturl="add.php";
}
....
?>
<form target="<?php echo $next;?>" method="POST">
<input type="hidden" name="nexturl" value="<?php echo $nexturl; ?>" />
<select name="usercode">
<?
while (mysql_fetch_array($fp))
{
......
}
?>
</select>
</form>

De werkelijke code zou de opmaak erg verpesten

[ripexx]

Waarom doe je die selectie zo, kan het niet in een scherm? Wat beheer je enz

Ik zou een geheel ander structuur toepassen en veel meer actie/module gericht gaan werken. Je logt in en kiest een actie, klant of gebruiker. Vanwaar je weer naar andere stappen kan enz enz. Meer kan ik er op het moment ook niet van maken

[Verwijderd]

MBV schreef op 19 april 2004 @ 15:44:
ok, ik zal even wat meer info verzamelen in de vorm van code. Mij lijkt dat alleen niet zo noodzakelijk, omdat het om de structuur gaat.
een gedeelte van kiesklant.php:

PHP:

<?
if ($_GET['action'])=delete)
{
   $next="kiesgebruiker.php";
   $nexturl="delete.php";
}

if ($_GET['action'])=add)
{
   $next="kiesgebruiker.php";
   $nexturl="add.php";
}
....
?>
<form target="<?php echo $next;?>" method="POST">
<input type="hidden" name="nexturl" value="<?php echo $nexturl; ?>" />
<select name="usercode">
<?
while (mysql_fetch_array($fp))
{
......
}
?>
</select>
</form>

De werkelijke code zou de opmaak erg verpesten

PHP:

if ($_GET['action'])=delete)

Gebruik twee keer == en aanhalingstekens om delete.

Verder kan je een switch statement gebruiken, dat is al een stuk overzichtelijker.

[ Voor 15% gewijzigd door Verwijderd op 19-04-2004 16:10 ]

[simon]

Waarom niet met een array werken en die uitlezen, nadat je keurig een isset hebt gedaan op die get variabele

Dat werkt imho veel makkelijker, krijg je veel kortere code door

[MBV]

@ripexx: ik wil graag een 'wizard' interface houden, dus je kiest een gegeven, en gaat door naar de volgende. Ik weet niet precies hoe je mijn situatie wilt doen, aangezien klant afhankelijk is van gebruiker.
@dannyhaak: sorry, ik heb net even dat voorbeeldje ingetypt. niet gecopy-paste ofzo, even uit mijn blote hoofd. Typovoutje is snel gemaakt
@Simon: ik vat jouw niet helemaal. Bedoel je een array met constanten, die ik dan in een aparte include zou stoppen? Dat zou iig een stuk inzichtelijker zijn.
array['delete'] = kiesgebruiker.php;
array['voegtoe'] = kiesgebruiker.php; enz....

edit:
post gerust door, maar ik kan de komende paar uur niet reageren. vanavond laat pas weer

[ Voor 12% gewijzigd door MBV op 19-04-2004 16:41 ]

[JochemK]

Kun je niet mbv een listbox een klant kiezen, waarna je bepaald of een user verificatie nodig is zo ja -> ga je dat doen -> zo nee iets anders.

[MBV]

Dat zit er nu dus in. Het gaat niet om verficatie, maar om beheerstaken. Je kan bijvoorbeeld een klant verwijderen, of een gebruiker van een klant. Ik zoek een manier om dat netjes op te lossen, en dat idee van de array lijkt mij (tot nu toe) het beste. Maar andere ideeën hoor ik graag

[simon]

Ik had gister een voorbeeld je voor je, maar die was ik vergeten te posten

PHP:

<?php
$paginas['delete'] = 'delete.php';
$paginas['omfg']   = 'omfg.php'  ;

if (isset($_GET['action']) AND isset($paginas[  $_GET['action']  ]))
{
   include_once ($paginas[  $_GET['action']  ]);
}
else
{
   echo 'Error: pagina niet gevonden';
}
?>

[ Voor 32% gewijzigd door simon op 20-04-2004 08:54 ]

[Helmet]

misschien een beetje algemene opmerking, maar het gebruik van een template engine als templatePower of Smarty geeft wel veel meer structuur in je code

[MBV]

ik zat aan iets anders te denken:

PHP:

<?
$paginas['add']    = array("kiesgebruiker.php","add.php");
$paginas['delete'] = array("kiesgebruiker.php","delete.php");

if (isset($_GET['action'] && isset($pagina[$_GET['action'])
{
   $next    = $pagina[$_GET['action']][1];
   $nexturl = $pagina[$_GET['action']][2];
}
else error("Index.php","foutje"); //error is een functie met een redirect enzo...
....
?>
<form target="<?php echo $next;?>" method="POST">
<input type="hidden" name="nexturl" value="<?php echo $nexturl; ?>" />
<select name="usercode">
<?
while (mysql_fetch_array($fp))
{
......
}
?>
</select>
</form>

edit:

en over template engines: Dat staat los van mijn vraag. Daarmee zou ik het form los kunnen koppelen (en ja, ik zou willen dat degene die met dit project is begonnen dat ook had gedaan), maar dat heeft niets met een slimmere, onderhoudbare structuur te maken voor navigeren door forms heen.

[ Voor 36% gewijzigd door MBV op 20-04-2004 09:29 ]