Immer die zuletzt gespeicherte option selecten

[shunt]

In unserem Memberscript gibts eine Sache die mir überhaupt nich passt . Wenn man ein Profil editieren will steht das dem Member zugewiesene Team immer auf derselben option. Hier ist der Code des Feldes:

<?php
	$query_teams = @mysql_query("SELECT * FROM ".$table."_members_teams ORDER BY id");
	while ($query_teams_data = @mysql_fetch_array($query_teams)) {
?>

<option><?php echo $query_teams_data['name'] ?></option>
		
<?php
	}
?>

Ich hatte mir sowas überlegt wie:

if ($query_teams_data['name'] == $team) {
<option selected><?php echo $query_teams_data['name'] ?></option>
}
else { 
<option><?php echo $query_teams_data['name'] ?></option>
}
$query_teams_data['name']

Das hat aber nicht so recht geklappt und alle Teams wurden doppelt aufgeführt? Weiss jemand warum? mfg

 

[Sven Petruschke]

Weiss jemand warum?

Ja, weil Du die IF-Abfrage in den HTML-Teil geschrieben hast und nicht innerhalb der PHP-Tags. Sollte man anhand des Syntax-Highlightings sehr gut sehen.

snuu

 

[shunt]

So? O o

<?php
	$query_teams = @mysql_query("SELECT * FROM ".$table."_members_teams ORDER BY id");
	while ($query_teams_data = @mysql_fetch_array($query_teams)) {
?>

<?php
if ($query_teams_data['name'] == $team) { 
?>

<option selected><?php echo $query_teams_data['name'] ?></option> 

<?php
} 
else { 
?>

<option><?php echo $query_teams_data['name'] ?></option> 

<?php
} 
?>
<?php
	}
?>

 

[js-mueller]

Jo aber dadurch hast du 2 HTML stellen, für die gleiche Sache. Versuch es mal zu vereinfachen

<?php
    $query_teams = @mysql_query("SELECT * FROM ".$table."_members_teams ORDER BY id");
    while ($query_teams_data = @mysql_fetch_array($query_teams)) {
?>

<?php
$selected = "";
if ($query_teams_data['name'] == $team)
    $selected = "selected=\"selected\"";
?>
<option <?php echo $selected ?>><?php echo $query_teams_data['name'] ?></option>
<?php
}
?>

nochma was anderes:

Der Topic ist zwar k "Immer die zuletzt gespeicherte option selecten", aber mich stört das selecten nen wenig. Tut das nicht weh die deutsche Sprache zu verschandeln Im Prinzip hört sich doch selektieren/aktivieren besser an als selecten

 

[shunt]

Ok ich habs jetzt war eh noch ein Fehler drin bei der einen Variable von daher konnte es eh nicht funktionieren hehe. Danke für eure Hilfe beim selecten