Vælg en "option" med php

Hej Emil,

Umiddelbart virker det som om at du gør det på en HELT skæv måde . Jeg har i hvert fald svært ved at se ideen i dit array som du så tilsyneladende starter tomt og fylder noget ind i en plads der svarer til dit kategori ID. Jeg vil i stedet foreslå at du har et array med kategorier og deres ID'er, enten fra databasen eller i en central fil (de fleste har fx en functions.php eller lignende, som de includer i alle deres filer). Dette array bruger du så til at lave en forbindelse imellem et tal og et kategori navn.

Hvis dine kategorier ikke har noget ID, men blot er en tekststreng der ligger direkte i din database sammen med forumtråden (hvilket som regel er en dårlig måde at gøre tingene på!), så har jeg svært ved at se hvad du skal med det array og de omgang if statements du bruger i toppen . Men i så fald kan du blot lave et array af tekststrenge som du så løber igennem når du vil oprette din select boks. Det vigtigste er sådan set at du får samlet dine kategorier ét sted, ellers vil du helt sikkert få lavet rod i det på et tidspunkt.

Prøv at forklare lidt mere, så kan det være vi bedre kan hjælpe dig.

Mvh.

Kasper (TSW)

Emil... når du er begyndt at være god til PHP, så er det jo netop at du skal være herinde for at hjælpe andre med deres problemer.

Kasper er inde på noget af det rigtig. Det ekstra array $options er lidt overflødigt, men kan agere workaround hvis man har fuld error_reporting slået til.

Har skrevet et forslag til hvordan jeg ville strukturere koden for at opnå et lignende resultat:

<?php
// ALTID PHP FØRST :)
$access = $_SESSION['brugernavn'] ? true : false;
// HVis vi ikke har access - så bare stop scriptet og send til en login form.
if(!$access) {
	header('Location: /');
	die("<h3>Upss..</h3><p>Du skal være logget ind for at oprette en tråd</p>");
}

// Ok - klar til business!
if(isset($_POST['submit'])) {
	if ($_POST['post_kategori']){
		$selected_category = in_array(array('minecraft', 'masterminerz-net', 'serveren', 'off-topic'), $_POST['post_kategori']) ?
	    	$_POST['post_kategori'] :
			'';
	}
     
    $titel = $_POST['titel'];
    $kategori = $_POST['kategori'];
    $tekst = $_POST['tekst'];
    $forfatter = $_SESSION['brugernavn'];
    $dato = date("d-m-Y"); 
         
    $message_type = $message = '';
    if($titel && $kategori !=='ingenting' && $tekst &&  $selected_category) {
    	// DB Connect (den ville jeg flytte til sin egen fil)
		mysql_connect("*****", "*****", "******");
		mysql_select_db("*******");
		
		// Query (hvordan ved du om det sket?)
		mysql_query("INSERT INTO traade (`titel`, `indhold`, `forfatter`, `kategori`, `dato`) VALUES ('$titel', '$tekst', '$forfatter', '$kategori', '$dato')") or die (mysql_error());
 
		$message_type = 'success';
		$message = 'Din tråd blev succesfuldt oprettet!';
	} else {
		$message_type = 'error';
		$message = 'Vær venlig at udfylde alle felterne, samt vælge hvilken kategori din tråd er.'; 
	}
}
?>
<!-- Så må vi gerne skrive html. PHP skal kun eksistere som PHTML. :) -->
<h3>Opret en ny tråd</h3>
<?php if($message && $message_type): ?>
	<p class="<?php echo $message_type?>"><?php echo $message; ?></p>
<?php endif; ?>
<form method="post" action="">
  <table style="margin:5px 10px 5px 10px;color:#fff;">
     <tr>
         <td>
         Titel:<br />
         <input style="width:640px;" id="titel" name="titel" value="<?php echo $titel; ?>" />
         </td>
     </tr>
     <tr>
         <td>
	         Kategori:
	         <select name="kategori">
	             <option name="ingenting" value="ingenting">... </option>
	             <option value="minecraft"<?php if($selected_category == 'minecraft'):?> selected="true"<?php endif;?>>Minecraft</option>
	             <option value="masterminerz-net"<?php if($selected_category == 'masterminerz-net'):?> selected="true"<?php endif;?>>Masterminerz.net</option>
	             <option value="serveren"<?php if($selected_category == 'serveren'):?> selected="true"<?php endif;?>>Serveren</option>
	             <option value="off-topic"<?php if($selected_category == 'off-topic'):?> selected="true"<?php endif;?>>Off topic</option>
	         </select>
         </td>
     </tr>
     <tr>
         <td>
         Skriv dit indlæg her:<br />
         <textarea style="width: 640px;" name="tekst" rows="8"><?php echo $tekst; ?></textarea>
         </td>
     </tr>
     <tr>
         <td>
         	<input type="submit" value=" Opret " name="submit">
         </td>
     </tr>
  </table>
</form>

Hehe, tak for jeres svar . Så går jeg igang med at prøve det ud

kig i øvrigt lidt på php switch i stedet for at lave mange "if" statements... det kan være nemmere...

nedenstående kode lånt fra w3schools

 $x = 2;
 
 switch ($x)
 {
 case 1:
   echo "Number 1";
   break;
 case 2:
   echo "Number 2";
   break;
 case 3:
   echo "Number 3";
   break;
 default:
   echo "No number between 1 and 3";
 }