Problem mit Register Script

**LittleGee** · 18.07.2008, 01:40

Du hast in der if Anweisung eine Klammer zuviel.

mfg

**UzumakiNaruto** · 18.07.2008, 09:58

PHP-Code:


if(

   ($UserName=='') OR 

   ($Userpass=='') OR 

   ($UserMail=='')

)

{

ich üersönlich würde das eher so schreiben:

PHP-Code:


if($UserName=='' OR $Userpass=='' OR $UserMail=='') {

**H2O** · 18.07.2008, 10:23

Den zweiten Fehler siehst du ja schon an der Färbung des Codes hier im Forum (und hoffentlich auch in deinem Editor). Ersetze mal alle «\"» ausser dem letzten - dort nimmst du nur den Backslash «\» weg - mit «'» und schon werden die Farben ändern und die Fehlermeldung verschwindet.

**candys** · 18.07.2008, 14:37

ok die Fehler hätte nwir soweit aus gemerzt jetzt kommt immer beim eintragen diese Fehlermeldung

Code:

Notice: Undefined index: UserPass2 in C:\xampp\htdocs\vrh\neu\formular_send.php on line 18

Notice: Undefined variable: UserID in C:\xampp\htdocs\vrh\neu\formular_send.php on line 32

Notice: Undefined variable: UserPass in C:\xampp\htdocs\vrh\neu\formular_send.php on line 32
Fehler!

das ist die bemägelte code stelle
$Userpass2= $_POST['UserPass2'];

$eintrag = "INSERT INTO `users` ( `UserID` , `UserName` , `UserPass` , `UserMail` )
VALUES ('', '$UserID', '$UserName', '$UserPass', '$UserMail' )";

PHP-Code:


<?php

error_reporting(E_ALL); 



//Datenbankverbindung einfügen!

   include('db.php');

//Variablen festlegen

   $UserName = $_POST['UserName'];

   $Userpass = $_POST['UserPass'];

   $Userpass2= $_POST['UserPass2'];

   $Userpass3= md5('UserPass');

   $UserMail = $_POST['UserMail'];

//wenn eine der Variablen name, pass oder mail leer ist, dann [if]

   if($UserName=='' OR $Userpass=='' OR $UserMail=='') {



//Ausgabe der Fehlermeldung

      echo "<b>Error</b>

      Bitte fülle die Felder Username, Passwort, und Email aus!<br><br>

      <a href=\"javascript:%20history.back(-2)\" target=\"_self\">Zurück</a>";

//wenn alles richtig ist: 

   } else {

//hier steht, was Eintrag ist und was damit gemacht werden soll. Welche Spalten sollen ausgefüllt werden?

      $eintrag = "INSERT INTO `users` ( `UserID` ,  `UserName` ,  `UserPass` ,  `UserMail`    ) 

      VALUES ('', '$UserID', '$UserName', '$UserPass', '$UserMail' )"; 

//hier wird entschieden, dass $eintrag über die Datenbankverbindung eingetragen werden soll.

      $eintragen = mysql_query($eintrag);

//Ausgabe der Erfolgsmeldung

      if($eintragen==true) {

         echo "<b>Eintrag erfolgreich</b>

         Der Eintrag in die Tabelle war erfolgreich.";

      } else {

         echo "Fehler!";

         include('formular.php');

      }

   }

//PHP schließen

?>

Wo soll da bitte der Fehler sein? Die Variablen stehen genauso in der DB.

**H2O** · 18.07.2008, 15:12

Im Formular, von dem die Daten herkommen, gibt es kein Feld UserPass2
Es existiert keine Variable $UserID (brauchst du auch nicht)
Es existiert keine Variable $UserPass (PHP beachtet Gross/Kleinschreibung bei Variablennamen
Du sollst nicht das das verschlüsselte Passwort ablegen, nicht den Klartext
Wenn du diese Fehler beseitigt hast, wirst du einen Datenbankfehler #1136 - Anzahl der Felder stimmt nicht mit der Anzahl der Werte in Zeile 1 überein erhalten. Warum wohl?
Falls die UserID mit AUTOINCREMENT definiert ist, brauchst du dafür nichts anzugeben.

**candys** · 18.07.2008, 15:27

Im Formular gibt es ein Feld mit UserPass

die datenbank einträge stammen von eurem Logintut ( http://www.php-resource.de/tutorials/read/38/2/
)
CREATE TABLE users ( UserID int(11) PRIMARY KEY auto_increment,
UserName varchar(30) NOT NULL default '', UserPass varchar(32) NOT NULL
default '', UserSession varchar(32), UserMail varchar(150) NOT NULL default
'', UNIQUE KEY NickName (UserName), UNIQUE KEY UserMail (UserMail) )
Ich stehe jetzt gerade etwas auf dem Schlauch. Da wird bemängelt das es keine Variablen gibt aber die sind doch vorhanden.
Wie geht das denn mit dem verschlüsselten ablegen von dem PW.
Möchte ja auch das die Passwörter verglichen werden.
wie baue ich das ein?

Habe in der DB noch Usersession Stehen aber das ist ja nicht wichtig beim registrieren oder stehe ich da jetzt aufm schlauch?
lg
Claudia

**H2O** · 18.07.2008, 15:54

Lese bitte meine Bemerkungen etwas genauer, ich habe nicht gesagt, dass es kein Feld UserPass gibt, sondern keines mit dem Namen UserPass2.
Und die genannten Variablen existieren in deinem Script nun mal nicht.
Das verschlüsselte Passwort wäre dann jenes, dass bei dir in $Userpass3 steht.
Mit der Datenbank haben diese Meldungen (noch) nichts zu tun.

**candys** · 18.07.2008, 17:10

ok habs geschafft. Eingetragen wird der Datensatz.
so jetzt möchte ich a) das die passwörte überprüft werden ob sie gleich sind
und
B) ob der gewünschte name schon vorhanden ist.

Das einloggen klappt nicht ob wohl ich nach eurem tut gearbeitet habe.

**TobiaZ** · 18.07.2008, 17:19

so jetzt möchte ich a) das die passwörte überprüft werden ob sie gleich sind und B) ob der gewünschte name schon vorhanden ist.

Woran scheitert es denn?

Das einloggen klappt nicht ob wohl ich nach eurem tut gearbeitet habe.

Das ist doch mal ne Aussage!

**candys** · 18.07.2008, 17:27

also ich weis das ich die Passwort variablen mit if vergleichen kann weis aber jetzt nicht wie ich das schreiben muss und wo im script das reinkommt.
Genau wie bei überprüfung des usernamens.
Also wenn ich mich mit dem Script aus dem tut einloggen will kommt

Ihre Anmeldedaten waren nicht korrekt!
Ich vermute das ich bei dem login.php script einen Fehler gemacht habe.
ich poste mal den code.
Habe ja gelernt das man die session einleitung schreiben muss bevor man etwas im browser aus gibt.
im Script wäre aber das Formular als erstes also kann die session ja folglicherweise nicht hinhauen.
habe dann den Session befehl vor das formular gesetzt und kann mich mit den anmeldedaten aus dem tut einloggen aber nicht mit meinen eigenen die ich ja über das register script in der DB gespeichert habe.
vl findest du einen fehler.

PHP-Code:


<? 

session_start();?> 



<form method="post" action="login.php"> 

  <label>Benutzername:</label><input name="username" type="text" /> 

  <br /> 

  <label>Passwort: </label><input name="userpass" type="password" id="userpass" /> 

  <br /> 

  <input name="login" type="submit" id="login" value="Einloggen" /> 

</form> 

<?php 

error_reporting(E_ALL);

include 'sessionhelpers.inc.php'; 

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

{ 

    $userid=check_user($_POST['username'], $_POST['userpass']); 

    if ($userid!=false) 

        login($userid); 

    else 

        echo 'Ihre Anmeldedaten waren nicht korrekt!'; 

} 

if (!logged_in()) 

    echo'<form method="post" action="login.php"> 

        <label>Benutzername:</label><input name="username" type="text"><br> 

        <label>Passwort: </label><input name="userpass" type="password" id="userpass"><br> 

        <input name="login" type="submit" id="login" value="Einloggen"> 

    </form>'; 

else 

    echo '<a href="logout.php">Ausloggen</a>'; 

echo '<p /><a href="logged_in.php">Check</a>'; 

?>

**TobiaZ** · 18.07.2008, 17:40

also ich weis das ich die Passwort variablen mit if vergleichen kann weis aber jetzt nicht wie ich das schreiben muss

Das könnte man als Grundlagen bezeichnen:

PHP-Code:


if($var1 == $var2) echo 'Die Werte stimmen überein.';

und wo im script das reinkommt.

Das Script ist ein logischer Ablauf von verschiedenen Teilprozessen. Bau die Überprüfung an der stelle ein, wo sie auch erfolgen soll. (Manche Fragen sind echt zu einfach um sie zu beantworten...)

Ich vermute das ich bei dem login.php script einen Fehler gemacht habe.

Das könntest du durch Testausgaben ganz einfach überprüfuen.

ich poste mal den code.

Und lässt uns deine Arbeit machen. Wenn du das ganze wesentlich auf die konkrete Stelle beschränken würdest...

Habe ja gelernt das man die session einleitung schreiben muss bevor man etwas im browser aus gibt.

Richtig!

vl findest du einen fehler.

Also, wenn die eingegebenen Zugangsdaten tatsächlich richtig sind und check_user() trotzdem false zurück gibt, dann liegt der Fehler nicht in dem geposteten Code.

Hat es eigentlich einen tieferen Sinn, dass du das Formular zwei mal einbaust, oder ist das nur stupides Copy&Paste?

**candys** · 18.07.2008, 18:13

also am login Script liegt es nicht. Mit dem testuser admin und dem passwort geht es also liegt der fehlerin der Md verschlüsslelung im registerscript.
werde mich jetzt mal auf fehlersuche macchen.
danke schon mal für die hilfe.

**3DMax** · 18.07.2008, 18:37

Original geschrieben von candys
werde mich jetzt mal auf fehlersuche macchen.

du könntest mal dort ansetzen:

PHP-Code:


$Userpass3= md5('UserPass');

**candys** · 18.07.2008, 19:23

das passwort wird falsch verschlüsselt soweit bin ich schon.

Problem mit Register Script