Radio buttons

Tijdelijk in een sessie opslaan.

echo echo echo echo :). Tis allemaal statische HTML hoor.

Er zijn (ten minste) twee manieren om dit te onthouden:
1. quick and dirty, je POST naar dezelfde pagina en gebruikt $_POST om eventueel waarden/selecties terug te plakken/plaatsen. Nadelen: rommelige code en mogelijk "dubbelposts"
2. je hebt een aparte verwerkstap waarin je tijdelijk gegevens opslaat in een sessie. Indien in deze stap blijkt dat er een fout in de invoer zit of dat deze onvolledig is stuur je iemand terug naar het formulier alwaar de reeds eerder ingevulde gegevens (die je in de sessie onthouden hebt) weer terugzet in het formulier.
Nadelen: wat meer overhead (maar wel zijn alle stappen dan beter gescheiden)

Dus dan komt er, als ik het goed begrijp, te staan:
in de head:
$_SESSION['aanhef'] = $_POST['aanhef'];

en in de body:
<?php
echo "<input name=\"geslacht\" type=\"radio\" id=\"geslacht\" value=\"man\"";
if ($_SESSION['aanhef'] == 1) {
echo "checked=\"checked\"";
}
echo "\>dhr.";
echo "<input name=\"geslacht\" type=\"radio\" id=\"geslacht\" value=\"vrouw\"";
if ($_SESSION['aanhef'] == 0) {
echo "checked=\"checked\"";
}
echo "\>mevr.";
?>

Heb ik dat zo goed begrepen?

Euh, niet helemaal. Kijk eens naar de values :). Daarnaast doe je hier twee keer min of meer hetzelfde. Don't Repeat Yourself. Hier heb je loops voor :). Ook kan het zijn dat het formulier nog niet eerder is ingevuld, in dat geval bestaat $_SESSION['aanhef'] helemaal niet. En dan doe je er verstandig aan om dit enigszins te "namespacen" in je sessie zodat alle formulierdata een beetje bij elkaar staat, ik zou dit op zijn minst in een subarray "form" zetten ofzo.

Aldus krijg je zoiets:

<?php
// vergeet niet ergens je sessie van tevoren te starten/voort te zetten met
session_start();

// ...

// definitie
$aanhef = array(
    array(
        'label' => 'dhr.',
        'value' => 'man',
    ),
    array(
        'label' => 'mevr.',
        'value' => 'vrouw',
    ),
);

// draai de velden uit
foreach ($aanhef as $row) {
    $checked = isset($_SESSION['form']['geslacht']) && $_SESSION['form']['geslacht'] == $row['value'] ? ' checked="checked"' : '';
    $id = 'geslacht_'.$row['value'];
    ?><input type="radio" name="geslacht" id="<?php echo $id ?>" value="<?php echo $row['value'] ?>"<?php echo $checked ?> />
    <label for="<?php echo $id ?>"><?php echo $row['label'] ?></label><?php
}
?>

Ook doe je er verstandig aan op gezette plaatsen gebruik te maken van output escaping.

Thomas,
Bedankt De session_start had ik er al in zitten.
Ik ga het op deze manier proberen.
Bedankt voor de moeite.

Kleine wijziging, wellicht is het handiger om het id als volgt te genereren, dan is dit minder afhankelijk van de waarden van radio buttons, het bovenstaande werkt, maar wellicht is het volgende wat breder toepasbaar:

<?php
// overige code gelijk
// ...
foreach ($aanhef as $index => $row) { // toevoeging $index
    $checked = isset($_SESSION['form']['geslacht']) && $_SESSION['form']['geslacht'] == $row['value'] ? ' checked="checked"' : '';
    $id = 'geslacht_'.$index; // maak gebruik van $index voor generiek id
    ?><input type="radio" name="geslacht" id="<?php echo $id ?>" value="<?php echo $row['value'] ?>"<?php echo $checked ?> />
    <label for="<?php echo $id ?>"><?php echo $row['label'] ?></label><?php
}
?>

Zie je ook dat, wanneer je je code wat gestructureerder opzet, je dit soort wijzigingen heel snel kunt doorvoeren?

Thomas,
Ik zie het, de tweede optie werkt idd gemakkelijker. Was er zelf zo niet opgekomen.
Bedankt.