Misschien hebben jullie er wat aan <option> vanuit database

[bluesbrother]

1. Maak een database tabel aan met een kolom optienaam bijvoorbeeld "Mijn website", en een kolom met optieurl met daarin "../index.php"

2. een functie met daarin het aanmaken van je select box

function make_menu($menu_subject)
{
switch ($menu_subject)
{
case ("mainsubjects"):
$query = "SELECT optienaam , optieurl FROM optietabel";
$vartxt = "optienaam";
$varurl = "optieurl";
break;
}
// hier kan je nog meer case's neerzetten
{
$result = connectdb($query);
// connectdb is je connectfunctie die een query kan ontvangen.
$menu = ("<option>Maak een keuze</option>");
while (list ($vartxt, $varurl) = mysql_fetch_row ($result))
{
$menu .= ("<option value=\"$varurl\">$vartxt</option>");
}
}
return $menu;
}

roep dan vervolgens aan de functie aan met:

$mainmenu = make_menu("mainmenu");

en dan print"al je html spul enzo en dan je select:

<select name=mainmenu onchange=window.open(this.options[this.selectedIndex].value)>$mainmenu

Als je verbeteringen hebt dan graag !!

[Grum]

1/ [php] tags
2/ break in een switch ?
3/ default case in een switch/default voor $menu_subject
4/ geen ()'s om simpele assignments
5/ geen "'s (double quotes) om strings zonder variabelen erin
6/ onnodig gebruik van list( ..., ... ) (kost weer cpu he )
7/ liever fetch_array gebruiken (wat sneller - zie ook hierboven)
8/ niet onnodig "'s escapen maar gewoon de string tussen single quotes zetten en deze met variabelen concatten.
9/ 'vergeten' "'s naar " om te zetten bij $varurl
10/ 'vergeten' </>/& om te zetten naar </>/& bij $vartxt
11/ je sluit je select tag niet af ...

Genoeg commentaar ? .. verbeteringen kan je zelf wel doorvoeren i suppose

[SchizoDuckie]

PHP:

function createSelect($name, $query)
{
    $output = '<select class="form" name="'.$name.'">';
    $result = mysql_query($query);
    while ($array = mysql_fetch_array($result))
    {
        $extratext = ($array['value'] == $_POST["$name"]) ? ' selected': '';
        $output .= "\t<option value='".addslashes($array['value'])."'".$extratext.">".$array['text'])."</option>\n";    
    }
    $output .= '</select>';
    return($return);
}

* SchizoDuckie heeft hem zoiets gewoon de query direct meegeven aan de functie. da's veel flexibeler Hij checkt ook of de waarde al in $_POST[] voorkomt, en zo ja, dan selecteert ie die

[D2k]

misschien hebben we er wel wat aan maar aan threads als dit hebben we niet zo veel. hiervoor zijn de codebase threads die (met een beetje mazzel) in de search te vinden zijn