Auswahlliste springt auf ersten Eintrag und nicht auf Datenbankeintarg: php, Mysql

ingrid_mari

Update Seite: update funktioniert auch, doch kehre ich auf diese update-Seite von einer anderen Seite zurück, ist der erste Eintrag der Liste ausgewählt und nicht der Eintrag, der in der Datenbank steht. Wo hakts hier, kann mir da jmd helfen? Danke, Ingrid

PHP

if($_POST['Submit'])
{
$XX=$_POST['XX'];
$update=mysql_query("update account set accountinhaber='$XX where ID_account='$a'") or die ("Ungültige Anweisung ");
}
$ergebnis=mysql_query("select * from account where ID_account='$a'") or die ("Ungültige Anweisung ");
while($DS = mysql_fetch_array( $ergebnis )) {
 $accountinhaber=$DS['accountinhaber'] ;
}
<form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">
 
<?php  
$ergebnis= mysql_query("SELECT nachname FROM user") OR die(mysql_error()); 
echo "<select name='XX'>";
while($row = mysql_fetch_assoc($ergebnis)) 
{ 
$selected = isset($XX) && $XX == $row['nachname] ? ' selected="selected"' : '';
echo '<option'.$selected.'>' . $row['nachname'] . '</option>';
}
?>
</select>
 
<input type="submit" name="Submit" value="ändern" />
</form>

Alles anzeigen

Thonixx

Da fällt dir nix auf?

PHP

$selected = isset($XX) && $XX == $row['nachname] ? ' selected="selected"' : '';

Da fehlt ein Hochkommata, richtig wärs so:

PHP

$selected = isset($XX) && $XX == $row['nachname'] ? ' selected="selected"' : '';

Ob das das Problem löst, bin ich mir nicht sicher.

ingrid_mari

nein, löst das Problem nicht, ich hab offensichtlich beim Kopieren das ' verloren ...

Unregistriert

Zitat

Wo hakts hier

An allen Ecken und Enden, angefangen bei nicht eingerücktem Quellcode!

PHP

if($_POST['Submit'])

Schreibe

PHP

if(isset($_POST['Submit']))

Wobei bedacht sein sollte, dass der Submit-Button nicht unbedingt mitgesendet wird!

PHP

$XX=$_POST['XX'];

$XX, sehr aussagekräftiger Variablenname, da kannst du auch $willibald schreiben

PHP

$update=mysql_query("update account set accountinhaber='$XX where ID_account='$a'") or die ("Ungültige Anweisung ");

Ohne ein mysql_real_escape_string Werte in die DB zu schreiben ist fahrlässig

PHP

$ergebnis=mysql_query("select * from account where ID_account='$a'") or die ("Ungültige Anweisung ");

Kein "Select *" benutzen und wo kommt $a her?

PHP

while($DS = mysql_fetch_array( $ergebnis )) {
 $accountinhaber=$DS['accountinhaber'] ;
}

Wenn es nur einen Treffer geben kann, ist die Schleife Blödsinn, oder aber $accountinhaber wird immer wieder überschrieben, was ebenfalls Blödsinn wäre

PHP

<form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">

In google mal PHP_SELF XSS eingeben

PHP

$selected = isset($XX) && $XX == $row['nachname] ? ' selected="selected"' : '';

ändern in

PHP

$selected = (!empty($XX) && $XX == $row['nachname]) ? ' selected="selected"' : '';

ingrid_mari

danke dir für die Hinweise, - aber alles egal, der Fehler hält sich, springt immer auf den ersten Eintrag der Auswahlliste, obwohl in der DB was andres steht

Unregistriert

Dann würde ich mal alle Variablen per echo ausgeben, mit anderen Worten: einfach mal debuggen. Und an den Anfang des Scripted diese beiden Zeilen schreiben:

PHP

error_reporting(E_ALL);
ini_set('display_errors', true);

ingrid_mari

dankdank hilft a nix

Unregistriert

Joah mei, wos haißt des jetza

ingrid_mari

geht ned, wenn ich mit echo den datenbankeintrag ausgeben lass is alls ok, da hats was mit dem option select klumpat, dem unnedigen, ...norddeutscher freund, i bin nicht aus bayern

crAzywuLf

Ich nehme an du hast die Verbesserungen des netten unregistrierten Users nicht mal übernommen... Bei so einem Code ist es klar das nichts funktioniert. An deiner Stelle würd ich das Programmieren an den Nagel hängen, schon allein die Variablennamen die du benutzt bringen einen ja schon zum reiern... Geschweige von Einrückungen oder dass der Code überhaupt einen Syntax Check überleben würde. Das Ding hat mehr Syntaxfehler, wie ein Kinderpuzzle Puzzleteile hat. Da kann man nur hoffen, dass du sowas nicht beruflich machst.

PHP

<?php
if($_SERVER['REQUEST_METHOD'] == 'POST' && isset($_POST['Submit']))
{
    $nachname = $_POST['nachname'];
    mysql_query("UPDATE `account` SET `accountinhaber` = '" . mysql_real_escape_string($nachname) . "' WHERE `ID_account` = '$a' LIMIT 1") or die ("Ungültige Anweisung");
}


$query_accountinhaber = mysql_query("SELECT `accountinhaber` FROM `account` WHERE `ID_account` = '$a' LIMIT 1") or die ("Ungültige Anweisung ");
$accountinhaber = mysql_result($query_accountinhaber, 0);
?>
<form id="form1" name="form1" method="post" action="<?php echo $_SERVER['PHP_SELF']; ?>"><?php /* PHP_SELF IST ÜBRIGENS DAS SCHLIMMSTE WAS MAN MACHEN KANN... da gehört der Scriptname rein, z.B. ./script.php */ ?>
    <select name="nachname">
<?php
$query_nachnamen = mysql_query("SELECT `nachname` FROM `user`") OR die(mysql_error());
while($row = mysql_fetch_assoc($query_nachnamen)) 
{ 
    $selected = ($row['nachname'] == $accountinhaber) ? ' selected="selected"' : '';
    echo '<option' . $selected . '>' . $row['nachname'] . '</option>';
}
?>
    </select>
<input type="submit" name="Submit" value="ändern" />
</form>

Alles anzeigen

Welchen Sinn deine Datenbankstruktur hat weiss ich auch noch nicht so recht... So wie ich das Script interpretieren konnte, kann man sich einen Nachnamen aus einer Nachnamenliste aussuchen, den dann ein bestimmter Account $a erhält. Logisch ergibt das für mich zwar überhaupt kein Sinn aber wie auch immer... Ich hab dir dein Script mal einigermaßen überarbeitet, so dass es auch lesbar ist und auch funktionieren könnte.

ingrid_mari

dein didaktischerstil ist a bissl wild .. aber danke. ich habe variablennamen geändert für das Forum weil ichs so für übersichtlicher gehalten habe, scheint nicht gut zu kommen. ansonsten geht das update, schwierigkeiten gibts wie gesagt bei dem option select, wobei ich glaub, dass ich da in der DB einen Hund hab, dem geh ich jetzt nach, danke lg ingrid

crAzywuLf

Aber dir ist aufgefallen das ich unter anderem diese select Zeile geändert habe?

PHP

$selected = ($row['nachname'] == $accountinhaber) ? ' selected="selected"' : '';

Hast du mal über phpMyAdmin nachgeschaut, ob es in der Datenbank alles richtig drin steht?

ingrid_mari

ja, das stimmt alles, ich hab auch schon x versionen von dem teil ausprobiert, auch wird der nachname in das Feld in der Tabelle account eingetragen, update funktioniert ja ...

aber ich glaub das problem liegt daran, dass bei meinem update form die Verbindung über die ID von der tabelle account (wo der nachname eingetragen wird)
zur tabelle user (die sämtliche nachnamen enthält) fehlt
und da torkle ich noch herum, dazwischen ist nämlich
die Zwischentabelle account_user und da sollen ja beim update die IDs eingetragen werden, das bring ich mit mehr zeit zusammen.
Könnte das sein, dass das der Grund ist weshalb nach dem update der wert in der auswahlliste wieder auf den ersten wert springt?
(ich bin neu bei php. ich bau nur ein paar musterseiten zusammen).
Wieder danke für deine hilfe! lg i

crAzywuLf

Dann ändere doch mal diese Zeile hier:

PHP

echo '<option'.$selected.'>' . $row['nachname'] . '</option>';

in

PHP

echo '<option'.$selected.'>"' . $row['nachname'] . '" == "' . $accountinhaber . '".....' . $row['nachname'] . '</option>';

Dann siehst du bei jedem Eintrag der Dropdown was verglichen wird...
Wird der Accountinhaber richtig ausgeben? Gibt es da vll Leerzeichen zwischen den Anführungszeichen und dem Wort...?

ingrid_mari

Hallo crAzywuLf ,
Du hast den Fehler bereits in deiner vorletzten Antwort gefunden:
(ich habs nicht gleich bemerkt weil ich dazwischen noch was falsches eingebaut hatte)

$selected = ($row['nachname'] == $accountinhaber) ? ' selected="selected"' : '';
meine ursprüngliche version war
$selected = ($row['nachname'] == $XX) ? ' selected="selected"' : ''; wobei $XX=$_POST['XX'];

das war falsch

DAS WARS, ich danke herzlichst, ingrid

Auswahlliste springt auf ersten Eintrag und nicht auf Datenbankeintarg: php, Mysql

Tags

Benutzer online in diesem Thema