HTML <select> till PHP If-sats

Är det inte så att man får ut värdet genom att kolla $_POST['amdkort']?

Jo.

<?php 
    if(isset($_POST['amdkort'])){ 
        echo $_POST['amdkort']. ' chosen'; 
    } 
?>

Något sånt kanske?..

<select name="amdkort">
	<option value="6950">6950</option>
	<option value="6970">6970</option>
<input type="submit" name="amdsubmit" value="Submit" />
</select>

<?php 
    if($_POST["amdkort"] == "6950"){ 
        print '6950 chosen'; 
    } 
    else if($_POST["amdkort"] == "6970"){  
    echo "6970 chosen"; 
    } 
    else{ 
    print '1'; 
    exit; 
    } 
?>

En notis: Du ska inte ha en input inne i select, det är felaktig html.

<form>
<select>
...
</select>
<input/>
</form>

Snyggare lösning:

<?php $sel = $_POST['amdkort']; ?>
<form method='post' action='<?=$_SERVER['PHP_SELF'];?>'>
   <select name="amdkort">
	<option value="6950" <?=($sel == "6950" ? "selected='selected'" : "")?>>6950</option>
   	<option value="6970" <?=($sel == "6970" ? "selected='selected'" : "")?>>6970</option>
   </select>
   <input type="submit" name="amdsubmit" value="Submit" />
</form>

<?php if(isset($sel)){ echo "Du har valt $sel"; } ?>

<?php $sel = $_POST['amdkort']; ?>
<form method='post' action='<?=$_SERVER['PHP_SELF'];?>'>
   <select name="amdkort">
	<option value="6950" <?=($sel == "6950" ? "selected='selected'" : "")?>>6950</option>
   	<option value="6970" <?=($sel == "6970" ? "selected='selected'" : "")?>>6970</option>
   </select>
   <input type="submit" name="amdsubmit" value="Submit" />
</form>

<?php if(isset($sel)){ echo "Du har valt $sel"; } ?>

Hur är det en snyggare lösning? Verkar otroligt mycket krångligare.

Den markerar valt alternativ i listan, och använder inte tre if-satser för att skriva ut vad man valt. Vad är det som är krångligt menar du?
Jag passade på att snygga till hans html också, flyttade ut submiten från selecten och la till form runtom, kanske det som förvirrar dig?

Du kan även använda "isset" och "switch" istället om du vill för optimera koden liten eftersom "if" är långsammare.

	$amdkort = isset($_POST['amdkort']) ? $_POST['amdkort'] : ""; //Ingen varning om du kör PHP i strict mode
	switch($amdkort)
	{
		//listan av dom korten som är tillåtna
		case 6950:
		case 6970:
			break;
		default:
			$amdkort = "";
	}
	
	if($amdkort !== "")
	{
		//$amdkort har ett värdet du vill ha
	}

	$amdkort = isset($_POST['amdkort']) ? $_POST['amdkort'] : ""; //Ingen varning om du kör PHP i strict mode
	switch($amdkort)
	{
		//listan av dom korten som är tillåtna
		case 6950:
		case 6970:
			break;
		default:
			$amdkort = "";
	}
	
	if($amdkort !== "")
	{
		//$amdkort har ett värdet du vill ha
	}

Ville bara säga att "optimera" kod genom att byta ut kod mot switch och isset är så sjukt onödigt (inte för att det är något fel med dessa, tvärtom använd de gärna). Det är bara så meningslöst att börja optimera sin kod för sånna grejer. Och innan man ens börjar optimera ska man kolla VAD som tar tid, och om det är något problem över huvud taget. Sorry ville ranta lite..

Tänkte precis säga detta, switch statementet fungerar lite lättare när man lärt sig hur det fungerar.

Okej gör som du vill men jag tycker det ser mycket bättre ut sådär istället för att använda flera "else if" för varje kort. Annars kan man också göra en array istället och använda "in_array". Menar inte att det går tusen gånger fortare eller något vad jag menade också är att det ser bättre och enklare ut med "switch".

Sen är jag i alla fall glad om någon säger om något är fel eller om det finns bättre sätt.