Habe Problem mit mein Script!

Manuel · 27. März 2009

Hallo,
ich habe probleme mit meinem Skript. Ich habe heute selbst eines geschrieben da ich gerade mich an PHp wende und habe nirgendswo etwas kopiert (sonst lern ich ja nichts :D ). Also mein eigentliche Problem ist, dass ich bei activate.html einen falschen Code eingebe und er dann mir sagt:

Der Code ist korrekt und dein Benutzerkonto wurde aktiviert!

Wenn ich nichts eingebe, dan bringt er mir erst:

Der Code ist falsch!

So, ich poste mal das alles in einen Spoiler wie ich das mache (wäre sonst zu lang) ;)

Mein Quellcode

register.html

HTML

Um vollzugriff auf unsere Homepage zu haben.<br /><br /><form action="register.php" method="post"> Dein Nickname: <input type="text" name="nickname"><br /> Deine Email: <input type="text" name="email"><br /> Dein Passwort: <input type="text" name="pw"><br /> Dein Passwort wiederholen: <input type="text" name="pw2"><br /><br />  <input type="submit" value="Registrieren"></form>

register.php

PHP

<?php $nickname = $_POST['nickname']; $email = $_POST['email']; $pw = $_POST['pw']; $pw2 = $_POST['pw2']; $day = date("d.m.Y"); $day2 = date("H:i:s"); $day3 = date("d.m.Y H:i:s"); $code = rand(100000000,999999999); $mailnachricht = "Hallo $nickname,du hast dich heute am $day um $day2 registriert. Um deine Registrationabzuschließen, musst du auf folgenden Link klicken und dein Konto aktivieren.Dazu brauchst du folgende Daten und den Link:---------------------------------------------------------------Link: http://www.meinedomain.de/sc/activate.htmlDein Passwort: $pwDein Zugangscode: $code---------------------------------------------------------------** Was mache ich wenn der Link nicht funktioniert? **Wenn der oben eingetragene Link nicht funktioniert, danngebe einfach den Link der oben steht, in die Brwoser URL ein.** Der Link geht immer noch nicht! **Schreibe den Webmaster unter meine@domain.de an und sage dein Problem.Du kannst aber auch in unser Forum gehen und uns dort schreiben:http://www.meinedomain.de/forum/index.php"; if(empty($_POST['nickname']) || empty($_POST['email']) || empty($_POST['pw'])) { echo "Geb bitte einen Nicknamen, eine Email und ein Passwort ein!"; } else { mail("$email", "Abschließen der Registrierung", "$mailnachricht", "From: meine@domain.de"); echo "<b>$nickname</b>,<br />   du hast dich erfolgreich registriert. Wir werden dir einen Zugangscode schicken, damit du dein Account aktivieren kannst. Dein Zugangscode, haben wir bereits an <b>$email</b> geschickt.<br /><br />   <a href=\"activate.html\">Benutzer Konto aktivieren!</a>"; $eintrag="$nickname|$email|$pw|$code|$day3"; $datei = fopen("register.txt", "a"); fwrite($datei, "\n".$eintrag); fclose($datei); }?>

activate.html

HTML

<form action="activate.php" method="post"> Dein Code: <input type="text" name="code"><br />Dein Passwort: <input type="text" name="pw"><br /> <input type="submit" value="Absenden"></form>

activate.php

PHP

<?php $code = $_POST['code']; $pw = $_POST['pw'];  if (empty($_POST['code']) || empty($_POST['pw'])) { echo "Du musst deinen Zugangscode eingeben das wir dir per Mail geschickt haben und dein Passwort eingeben!"; } else { echo "Benutzerkonto wurde erfolgreich aktiviert!"; }?>

register.txt (lasst ihr lehr oder gebt da Vesprit|meine@domain.de|hallo|240233267|28.03.2009 20:19:31 ein um euch nicht registrieren zu müssen )

Code

So, ich freue mich auf antwort. Wer mir kommt mit "Boah mit Textdatei ist es total Hackerfutter", muss ich sagen, dass ich es selbst weis aber imoment PHP lern und noch kein MySQL :D

Eddit: Danke an dev/null/, der das schöne "User per Mail aktivieren" Tutorial geschrieben hat. Ohne ihn, würde ich nicht wissen wie man eine Zufalszahl erstellen lassen kann. Wer das Tutorial lesen will, klickt bitte hier (Mods, Link bitte nicht entfernen, da es ja auf einen Beitrag in das BisaBoard leitet ;) )

**TimWolla** · 27. März 2009

Erstmal legst du als Aktivierungsdatei mehrere an sosnt wird die eine immer wieder überschrieben.

Dann guck dir mal die Reihenfolge der Funktionen / Variablenzuweisungen in der activate.php an.

Extra keine genaue Lösung damit man was lernt :-)

Manuel · 27. März 2009

Zitat von /dev/null

Erstmal legst du als Aktivierungsdatei mehrere an sosnt wird die eine immer wieder überschrieben.

Wie? Das hab ich jetzt nicht ganz verstanden. Mehrere Datein?

activate.php

PHP

<?php
 $aktiv = $_POST['code'];
$datei = fopen("register.txt","r");
 fgets($datei,9999999);
 fclose($datei);
 
 if ($aktiv == $datei)
 {
 echo "Der Code ist falsch!";
 }
 else
 {
 echo "Der Code ist korrekt und dein Benutzerkonto wurde aktiviert!";
 }
?>

Alles anzeigen

Meinst du so? Warte ich teste schnell

Eddit: Jetzt kann ich einfach absenden und er bringt mir eine OK-Meldung :S

**TimWolla** · 27. März 2009

Mehrere Dateien mit den Aktivierungscodes sonst wird die eine immer überschrieben.

z.B. active-username.txt

Dein Script msus so aussehen:

PHP

<?php
 $aktiv = $_POST['code'];
$datei = fopen("register.txt","r");
 $inhalt = fgets($datei,9);
 fclose($datei);
 
 if ($aktiv != $inhalt)
 {
 echo "Der Code ist falsch!";
 }
 else
 {
 echo "Der Code ist korrekt und dein Benutzerkonto wurde aktiviert!";
 }
?>

Alles anzeigen

Nur der Aktivierungsmechanismus fehlt.

Manuel · 27. März 2009

OK, danke ich werde es mal testen und mit notieren das ich weiterarbeiten kann

Eddit: Es geht nicht. Egal was ich mache, ich habe einen falschen und den richtigen Code mehrmals eingegeben. Er birngt aber immer:
Der Code ist falsch!

Hm, ich habe auch keine Lösung gefunden. Ich such solange weiter bis du postest ;)

**TimWolla** · 28. März 2009

Erstmal solltest du bei der Register.php prüfen ob die Daten überhaupt eingegeben wurden, bevor du die Email versendest. Sonst wird immer, bei jedem Seitenaufruf eine Versendet.

An dem anderen Code sehe ich keinen Fehler. Teste evtl. mal ob ein gültiger Code in der register.txt ist. Und wie lang der ist wenn länger als 9 Zeichen lese ich nicht genug aus. Aber es wird nur eine Zufallszahl bis 9 Zeichen ausgesucht.

Manuel · 28. März 2009

Genau 9 Ziffern und der Code ist:
538381769

Ich habe ihn sogar aus der Textdati kopiert.

Eddit: Zur Register.php, habe ich probleme. Ich kann sowas noch nicht. Ich kann zwar Eingaben überprüfen aber nicht das er erst sendet, wenn alles ausgefüllt ist. Ich glaub da muss man irgendwas mit TRUE anstellen. Dann zu Textdatei, kann ich nicht mehr als eine Sache eintragen. Dazu brauch ich nämlich noch irgendwas wer ett, wenn du sowas amchen kanns das ein | zur Abtrennung benutzt wird.

Eddit: Ah, habe was gefunden bei einem Tutorial einer Webseite. Das Skript sieht so aus (ich probiers gleich mal ):

PHP

$eintrag="$nickname|$email|$pw|$code";


   $datei = fopen("register.txt", "a");
   fwrite($datei, "\n".$eintrag);
   fclose($datei);

Chris! · 28. März 2009

Wie wärs damit?
Wenn du PHP 5 hast, kannst du zum lesen und speichern von Dateien einfach das nutzen:
$dateiinhalt = file_get_contents($dateiname);
file_put_contents($dateiname, $dateiinhalt);

Und du kanns aus einem array einen string machen und umgekehrt:
$string = serialize($array);
$array = unserialize($string);

so, wenn du beides verbindest, kannst du ein array in einer textdatei speichern..
damit kannst du das ganze etwas einfacher gestalten..

Manuel · 28. März 2009

Hilft mir net wirklich weiter. Wie gesagt ich weis nicht was ein Array ist und bräuchte einen fertigen Quelltext ansonsten gehts net. Ich will ja nur an der activate.php rumbasteln. Wenn du das amchst das du mir einen fertigen Code schickst, wär ich dir dankbar. Sonst kann ich nichts damit anfangen, sorry... :|

Chris! · 28. März 2009

Ein Array ist in Prinzip eine Liste aus vielen Variablen..
Lern lieber erstmal PHP..

Manuel · 28. März 2009

Ja, wär jetzt schön wenn du den Codeposten könntest...

Chris! · 28. März 2009

dazu müsste man aber wissen, was ein array ist und auch dein ganzes script umschreiben.. aber es wird dadurch auch einfacher..

Manuel · 28. März 2009

Dann wage ich mich jetzt erstmal an MySQL. Mods, bitte offen lassen da ich auf das Thema zurückgreife wenn cih Grundlagen von MySQL kann.

Fred · 28. März 2009

wenn du immer noch nicht weiß was ein array ist , poste ich hier mal ...
(hoffe das wird jetzt nicht, wie immer ,als spam angesehen)

also ein "normales array" geht so:

PHP

$array = array ("teil1", "teil2", "teil3");echo $array[1]; //PHP fänngt immer bei 0 an zu zählen, also wird hier "teil2" ausgegeben

oder

PHP

$array[0] = "Teil1";$array[1] = "Teil2";$array[2] = "Teil3"; // wenn du nach der reihenfolge arbeitest , kannst du auch die zahlen weglassen echo $array[1];

dann gibt es noch assoziative arrays:

PHP

$person = array ( "Vorname" => "Daniel" , "Name" => "Meier" );echo $person["Vorname"]; //gibt dann Daniel aus, wenn du alles ausgeben möchtest, würde ich das mit einer for schleife , wenn du auch den schluessel ausgegeben haben möchtest nimm die foreach schleife

oder

PHP

$personen["Vorname"] = "Daniel";$personen["Name"] = "Meier";echo $person["Vorname"]; //ebenso wie oben

und natürlich auch die mehrdimensionalen arrays:

PHP

$person1 = array ( "Vorname" => "Daniel" , "Name" => "Meier" );
$person2 = array ( "Vorname" => "Max" , "Name" => "Bäcker" );
$person3 = array ( "Vorname" => "Markus" , "Name" => "Müller" );


$personenliste = array ( $person1, $person2, $person3 );
echo $personenliste[1]["Name"]; // gibt dann Bäcker aus

war ne menge tipparbeit , hoffe es wird nicht wieder gelöscht

mfg

Chris! · 28. März 2009

Also ich würd dir jetzt erstmal lieber das mit den Arrays statt MySQL ;)

Manuel · 29. März 2009

Also das Tutorial hilft mir schon. Jetzt weis ich wenigstens was ein Array ist. Hm... ich habe Probleme das ich die Daten in die MySQL datenbank eintrage.

Chris! · 29. März 2009

ich kann dir nur davon abraten, da du nicht sicher genug programmierst.. also ich mein bei mysql muss man auf vieles achten und dazu muss man mehr wie die basics verstehen..

Manuel · 29. März 2009

OK, ich frag dich halt wegen Sicherheitsdingens da. Also ich habe noch ne Frage zu meinem Skript. Wenn ein User sich registriert muss er auf die activate.html zugreifen und sein Passwort und seinen Aktivierunsgcode den er per Mail bekommen hat, eingeben. Dann wird abgesendet und dann soll MySQL bzw. PHP sagen ob der Code und das Passwort aus der Datenbank stimmen. Wenn ja, trägt, kommt eine Meldung die sagt dein Benutzerkonto wurde jetzt aktiviert. Dann soll in der Tabelle aktiv eine 1 stehen das es soeben aktiviert wurde. Wenn noch nicht aktiviert wurde steht da eine 0. So weit so gut, man sollte beim Login dann überprüfen, ob eine 0 in der Tabelle aktiv steht. Wenn ja, Meldung ausgeben das der User erst das Konto aktivieren soll. Wenn eine 1 drinn steht, wird erfolgreich eingeloggt wenn die Daten aus der Datenbank überein stimmen. Hat da jemand eine Idee wie ich vorgehe und irgendwie ein Tut oder ein Codeschnipsel? Ich will nämlich gerne weiter programmieren hab auch schon rumprobiert... ;)

Eddit: Ach und Chris, eine Sicherheitsmasnahme hab ich schon. Das Passwort in md5 umgewandelt und in die Datenbank eingetragen. Sonst frag ich dich einfach

**TimWolla** · 30. März 2009

Erstmal die Daten für die Datenbank mit den entsprechenden Funktionen: intval() mysql_real_escape_string() absichern, ist das mindeste.

Dann solltest du bitte mein Aktivierungstutorial genau befolgen. Sonst benötigst du 2 Spalten statt einer.

Manuel · 30. März 2009

Ich werds mir durchlesen. Wenn ich Probleme habe, poste ich hier ;)

Eddit: Und was sollte mir das helfen? ich sehe da keine PHP codes auser dem rand und so. Ich werde es mir dann nochmal durchlesen. Wömöglich nur überflogen