header (Location...

komma

Hi @ all!
Hab ein Problem. Ich programmiere gerade ein schlichtes Loginsystem und ich scheitere gerade an der Anmeldung. Und zwar soll er zuerst überprüfen, ob alle Daten vorhanden sind und nicht etwas ausgelassen wurde. Danach legt er den User an. Dann soll er die seite complete.php ansteuern. Problem, wenn man jetzt zum beispiel nichts ausfüllt. sollte ereigendlich zeigen, dass ein fehler vorhanden ist. Trotzdem macht das Programm munter weiter und geht anschließen auf complete.php. Warum wird header ("Location: register.php?fehler=1"); einfach ignoriert?
Hab das programm mal verkürzt hier reingesetzt:

Code

[...]


if ($username == "")
{
header ("Location: register.php?fehler=1");
}
//Anlegen des Users


[...]


  if (mysql_affected_rows ($connectionid) > 0) 
  { 
    header ("Location: complete.php");
  }
[...]

Alles anzeigen

hoffe ihr könnt mir helfen.

DER_Brain

zeig bitte mal das ganze Script..

lass dir mal $username ausgeben, vlt. steht nicht das drinnen, was du eigentlich möchtest..

..

lg
Andi

komma

Hier der ganze Code:

Code

$username=$_POST["user"];
$passwort=$_POST["pass"];
$repasswort=$_POST["repass"];
$homepage=$_POST["page"];
$email=$_POST["email"];
$tool=$_GET["tool"];
if($_POST) { 
$benutzer["name"] ="$username"; 
$benutzer["passwort"] = "$passwort";
$benutzer["email"] = "$email"; 
$benutzer["tool"] = "$tool";  
$benutzer["homepage"] = "$homepage"; 


if ($username == "")
{
header ("Location: register.php?fehler=1&tool=$tool");
}
if ($passwort == "")
{
header ("Location: register.php?fehler=1&tool=$tool");
}
if ($passwort != $repasswort)
{
header ("Location: register.php?fehler=2&tool=$tool");
}
if ($email == "")
{
header ("Location: register.php?fehler=1&tool=$tool");
}
if ($homepage == "")
{
header ("Location: register.php?fehler=1&tool=$tool");
}
$sql = "INSERT INTO ".
    "im_user (name, passwort, email, tool, homepage) ".
  "VALUES ('".$value["name"]."', '".
                       md5 ($value["passwort"])."', '".
                       $value["email"]."', '".
                       $value["tool"]."', '".
                       $value["homepage"]."')"; 
  mysql_query ($sql); 
  if (mysql_affected_rows ($connectionid) > 0) 
  { 
    header ("Location: complete.php");
  }
}

Alles anzeigen

K.Y.L.T.

Du fängst den Fehler nicht ab

mach um deine SQL - Query mal

Code

if($_GET['fehler']!= 1)
{


}

Ich würde dir allerdings zu einer sichereren Variante raten, wo du wenn ein Fehler kommt eine locale Session oder andere Variable setzt, nicht per get.

LG

DER_Brain

ersetz mal in header("Location:...) teil die & durch &

weiß ned obs hilft.. mal schaun

K.Y.L.T.

ne das muss er nicht machen, dass ist hier keine Nennenswerteveränderung seines Problems, eher die Tatsache, das er wenn er die Location auch aufruft, die Get-Angabe fehler nichts bringt da nicht abgefangen....

komma

Ich glaube ihr habt mich falsch verstanden. Trotzdem, bin durch den Beitrag von K.Y.L.T. auf eine Idee gekommen. Funktioniert jetzt auch.

Danke!

Wirklich schön wenn user zwar eine Lösung für Ihr Problem gefunden haben un es wert finden dies kundzutun- allerdings dann die eigendliche Problemlösung "geheimhalten"!

Also bitte auch so fair sein und zu einem Gelösten Problem auch die Lösung dazuschreiben- wäre toll und der Mühe wohl wert!?

header (Location...

Tags

Benutzer online in diesem Thema