Problem mit PHP IF ELSE

Tacqila · 17. Mai 2013

Hallo,

ich bin seit einiger Zeit dabei mich in PHP und MySQL einzuarbeiten. Aktuell bin ich dabei eine einfache Profilseite zu erstellen die auch ermöglicht die Daten zu ändern.
Gewollt ist, dass man wenn man das Passwort nicht ändern möchte die Felder "Aktuelles Passwort", "Neues Passwort" und "Neues Passwort wiederholen" leer lässt. In diesem Fall sollen einfach nur die anderen Daten wie Vorname, Email usw. geändert werden.

PHP

if (!$actualpw) {
$query = "UPDATE user SET name = '$name', givenname = '$givenname', adress = '$adress', country = '$country', zip = '$zip', city = '$city', email = '$email', telephone = '$telephone', fax = '$fax', voip = '$voip', room = '$room' WHERE guid = '$userid'";
$sql = mysql_query($query) or die(mysql_error());
echo '<meta http-equiv="refresh" content="0; URL=?view=user&action=edit">
<img src="_websystem/themes/v3.default/images/loader/pacman.gif" style="float:left;padding-right:10px">'.$LangUserChangesSaved.'';
}
else {
$LogQuery = "SELECT password FROM user WHERE guid = '$userid'";
$LogSql = mysql_query($LogQuery);
$LogRow = mysql_fetch_object($LogSql);
if($LogRow->password == $actualpw) {
if ($newpw1 == $newpw2) {
$query = "UPDATE user SET name = '$name', givenname = '$givenname', adress = '$adress', country = '$country', zip = '$zip', city = '$city', email = '$email', telephone = '$telephone', fax = '$fax', voip = '$voip', room = '$room', password = '$newpw2' WHERE guid = '$userid'";
$sql = mysql_query($query) or die(mysql_error());
echo '<meta http-equiv="refresh" content="0; URL=?view=user&action=edit">
<img src="_websystem/themes/v3.default/images/loader/pacman.gif" style="float:left;padding-right:10px">'.$LangUserChangesSaved.'';
}
else {
echo $LangUserTwicePwdFail;
}
}
else {
echo $LangUserActualPwdFail;
}
}

Alles anzeigen

Lässt man nun die Felder leer, springt das Script zum letzten else also zu $LangUserActualPwdFail. Gebe ich allerdings das aktuelle Passwort ein macht das Script was es soll. Es prüft auch ob PW1 und PW2 gleich sind.

Ich weis nicht wo ich hier einen Fehler gemacht habe. Vielleicht übersehe ich einfach nur eine Kleinigkeit.

StreetFighter · 17. Mai 2013

Meiner Meinung nach springt dein Script niemals in das erste If rein.

Ich halte die Bedingung !$actualpw für fragwürdig. Meine letzten Programmierarbeiten sind leider schon paar Monate her, aber nach meiner Auffassung kann das nicht funktionieren.

//EDIT: Versuch mal if($actualpw == NULL $actualpw == "")

Tacqila · 17. Mai 2013

Vor dem IF kommt nichts mehr außer die $_POST die nötig sind.
Normalerweise müsste er ja dann direkt das abfragen und nicht erst irgend wo anders rein springen.

PHP

$name = mysql_real_escape_string($_POST['name']);
...

Das !$VAR benutze ich an vielen Stellen erfolgreich um zu prüfen ob eine Variable leer oder voll ist.

Verwende ich if($actualpw == NULL $actualpw == "") statt !$actualpw tut sich anschließend leider gar nichts mehr.

StreetFighter · 17. Mai 2013

Was heißt es "tut sich gar nichts mehr"? Schreib mal testweise in jede mögliche Verzweigung ein echo mit anderer Nachricht, um zu überprüfen was das Script gemacht hat.

Kann gut sein das !$VAR funktioniert, ist bei mir schon bisschen her^^

Wenn meine Vermutung mit Zeile 1 falsch ist, dann habe ich leider keine Ahnung wo der Fehler liegt.

Tacqila · 17. Mai 2013

Kein Content. Header und Navigation laden, aber der Content bleibt weis.

Ich hab die Echos in der Reihenfolge wie oben durch nummeriert 1 - 4.
Wenn ich wie oben beschrieben vorgehe kommt direkt echo #4. Ändere ich das Passwort ohne Fehler bei #2 und mache ich beim neuen Passwort einen Fehler kommt echo #3. Soweit geht das ja auch.

**virtualmarc** · 17. Mai 2013

guck mal in die error.log von deinem webserver ob da was drin steht.

Tacqila · 17. Mai 2013

Hätte ich eigentlich auch direkt drauf kommen können.
Komme ich aber jetzt nicht dran. Das Script läuft nicht im Internet.

Tacqila · 21. Mai 2013

Nun funktioniert es. ich habe es ganz einfach mit

PHP

if ($actualpw == 0)

probiert und schon läufts. Ist ja eigentlich nix anderes wie oben nur ohne das oder.
Wenn ich allerdings aus der 0 ein NULL mache gehts wieder nicht...

Warum es aber mit

PHP

if (!$actualpw)

nicht geht ist mir ein absolutes Rätsel. Wie gesagt. An anderer Stelle läuft das so auch...

StreetFighter · 21. Mai 2013

Naja...hauptsache es läuft

Willkommen! Melden Sie sich an oder registrieren Sie sich.

Problem mit PHP IF ELSE

Teilen