Prüfen ob Email existiert problem

strong97

hey,

ich habe kurz eine function geschrieben, doch ich habe irgendwas falsch gemacht. Weil er zeigt mir auch bei erfolgreicher Registrierung den Fehler 'Email existiert'.

[PHP]
<?php
require_once 'dbconfig.php';
<?php
if(isset($_POST['senden']) && strtolower($_POST['senden'])) {

if(strtoupper($_SERVER['REQUEST_METHOD']) == 'POST') {

$email = htmlentities($_POST['email'], ENT_QUOTES, "UTF-8");
$password = $_POST['password'];
$vorname = htmlentities($_POST['vorname'], ENT_QUOTES, "UTF-8");
$nachname = htmlentities($_POST['nachname'], ENT_QUOTES, "UTF-8");

$error = '';

function check($email) {
include 'sec/verbindung.inc.php';
$sql = 'SELECT email FROM user_daten WHERE email = $email';
$stmt = $DB_con->prepare($sql);
$check_row = $stmt->fetch(PDO::FETCH_ASSOC);

if($stmt->rowCount() > 0) {
return true;
} else {
return false;
}
}

if(check($email) === false) {
$error = 'Email existiert';
}

if(!$error) {

$insert = $DB_con->prepare("INSERT INTO user_daten (email, password, vorname, nachname) VALUES (?, ?, ?, ?");
$insert->BindValue(1, $email);
$insert->BindValue(2, password_hash($password, PASSWORD_BCRYPT, $option));
$insert->BindValue(3, $vorname);
$insert->BindValue(4, $nachname);
if($insert->execute()) {
exit("<article id='erfolg_insert'><section id='content'><p class='susc_reg'>Sie haben erfolgreich einen Account erstellt.</p> <br> <p class='after_reg'><a href='login.php' class='now_log_after_reg'>Jetzt Einloggen</a></p> </section></article><?php");
}
}
}
}
?>
<article id="a1-reg">
<section id="content">
<h2 class="h2-reg"> Registrieren </h2>
<h3 class="h3-reg"> Kostenlos und schnell Registrieren </h3>
<p class="genericon genericon-user"></p>
</section>
</article>

<article id="a2-reg">
<section id="content">
<form action="<?php $_SERVER['REQUEST_METHOD']?>" method="POST">

<?php if(isset($error)) { ?>
<article id='error_info'>
<section id='content'>
<?php echo $error; ?>
</section>
</article>
<?php } ?>

<p><input type="email" name="email" placeholder="E-Mail Adresse"> <input type="email" name="atmail-g" placeholder="E-Mail Adresse Wiederholen"></p>
<p><input type="password" name="password" placeholder="Passwort"> <input type="password" name="psw-g" placeholder="Passwort Wiederholen"></p>
<p><input name="vorname" placeholder="Ihr Vorname"> <input name="nachname" placeholder="Ihr Nachname"></p>
<p class="info-agb"> Beim Senden der Registration stimmen Sie automatisch der <a href="#" class="agb"> AGB</a> zu.</p>
<p><input type="submit" class="send" value="Senden" name="senden"> <input type="reset" class="reset" value="Zurücksetzen"></p>
</section>
</article>

</main>
</body>
</html>
[/PHP]

War jetzt nur ein Beispiel hab oft die Codes verändert etc. nach halber/einer Stunde ging mir die Lust weg..

kanufrosch

Hey,

hast du dir mal
$stmt->rowCount()

ausgeben lassen, um zu gucken, ob deine Abfrage auch wirklich das liefert was du erwartest?

Ist jetzt so auf den ersten Blick das wahrscheinlichste...

Grüße Kanu

strong97

Ausgabe: int(0)

kanufrosch

Und var_dump(check($email) liefert entsprechend Boolean true?

Wenn ja:
Macht zwar aus meiner Sicht grade keinen Sinn, aber mache mal den Vergleich ohne Typ-Prüfung, also nur mit ==

strong97

var_dump(check($email)) = bool(false)

kanufrosch

Ich bin mir nicht sicher ob das deinen Fehler auslöst, aber behebe doch mald as hier:

PHP

<?php
require_once 'dbconfig.php';
            <?php

...

Du doppelst den PHP-Start...

strong97

Hat nichts damit zu tun. Muss die Verbindung in function einfügen sonst zeigt er mir ein Fehler an, dass die Verbindung nicht gefunden wurde.

cottton

Das hast nu davon, wenn Du keine Platzhalter verwendest!
Ok, ich sehe Du verwendest sie ja, aber einmal haste es "vergessen" -- und an der Stelle hast Du den Fehler gemacht.
(steht alles in comments)

PHP

<?php
require_once 'dbconfig.php'; 




if(isset($_POST['senden']) && strtolower($_POST['senden'])) {




    if(strtoupper($_SERVER['REQUEST_METHOD']) == 'POST') { // du guckst in $_POST nach daten, dann is klar, dass es eine POST request is =)

        $email = htmlentities($_POST['email'], ENT_QUOTES, "UTF-8"); // NEIN ... warum denn?
        $password = $_POST['password'];
        $vorname = htmlentities($_POST['vorname'], ENT_QUOTES, "UTF-8");// NEIN ... warum denn?
        $nachname = htmlentities($_POST['nachname'], ENT_QUOTES, "UTF-8");// NEIN ... warum denn?




        $error = '';




        // die function sollte nicht hier definiert werden. die kann doch am anfang oder sogar am ende des scripts stehen
        function check($email) {
	        include 'sec/verbindung.inc.php'; // sicherlich deine db verbindung. das wird bei JEDEM funktions aufruf erneut geladen
	        									// bei jedem aufruf wird (evtl) erneut eine verbindung aufgebaut, anstatt die vorhandene zu nutzen
	        #$sql = 'SELECT email FROM user_daten WHERE email = $email'; // NEIN!!!! "SQL-Injection" -- forum/google suchen und lesen!

	        // hier liegt btw das eigl problem
	        // 	in '' einfachen anführungszeichen wird eine $var nicht platziert. zu hast den >text< $email da rein geschrieben
			// 	du hast also immer abgefragt: gibt mir email von user_daten wo email $email ist =)
			// 	ergo: immer 0 (keine) einträge gefunden




	        $sql = 'SELECT email FROM user_daten WHERE email = ?';
	        $insert->BindValue(1, $email); // NEU 
	        $stmt = $DB_con->prepare($sql);
	        // hier fehlt doch ->execute ? 
	        $stmt->execute() // NEU 
	        $check_row = $stmt->fetch(PDO::FETCH_ASSOC); // weiss grad nicht ob du unbeding fetch´en musst, wenn du nur den rowcount brauchst




	        if($stmt->rowCount() > 0) {
	            return true;
	        } else {
	            return false;
	        }
	    }

    	// schreib doch das, was in der check funktion steht hier direkt hin =)
        if(check($email) === false) {
            $error = 'Email existiert';
        }




        if(!$error) { 
	        $insert = $DB_con->prepare("INSERT INTO user_daten (email, password, vorname, nachname) VALUES (?, ?, ?, ?"); 
	        $insert->BindValue(1, $email);
	        $insert->BindValue(2, password_hash($password, PASSWORD_BCRYPT, $option)); 
	        $insert->BindValue(3, $vorname);
	        $insert->BindValue(4, $nachname);
	        if($insert->execute()) {
	        	exit("<article id='erfolg_insert'><section id='content'><p class='susc_reg'>Sie haben erfolgreich einen Account erstellt.</p> <br> <p class='after_reg'><a href='login.php' class='now_log_after_reg'>Jetzt Einloggen</a></p> </section></article><?php");
	        	// was macht das <?php über mir am ende des strings? ...section></article><?php	     
	        }
   		}




	}
}
?>
    <article id="a1-reg">
        <section id="content">
        <h2 class="h2-reg"> Registrieren </h2>
        <h3 class="h3-reg"> Kostenlos und schnell Registrieren </h3>
        <p class="genericon genericon-user"></p>
        </section>
    </article>

    <article id="a2-reg">
        <section id="content">
        <form action="<?php $_SERVER['REQUEST_METHOD']?>" method="POST">


         <?php if(isset($error)) {   ?>
         <article id='error_info'>
         <section id='content'>
         <?php echo $error; ?>
         </section>
         </article>
         <?php  } ?>

<p><input type="email" name="email" placeholder="E-Mail Adresse"> <input type="email" name="atmail-g" placeholder="E-Mail Adresse Wiederholen"></p>
        <p><input type="password" name="password" placeholder="Passwort"> <input type="password" name="psw-g" placeholder="Passwort Wiederholen"></p>
        <p><input name="vorname" placeholder="Ihr Vorname"> <input name="nachname" placeholder="Ihr Nachname"></p>
        <p class="info-agb"> Beim Senden der Registration stimmen Sie automatisch der <a href="#" class="agb"> AGB</a> zu.</p>
        <p><input type="submit" class="send" value="Senden" name="senden"> <input type="reset" class="reset" value="Zur&uuml;cksetzen"></p>
        </section>
    </article>




</main>
</body>
</html>

Alles anzeigen

BTW:

PHP

<form action="<?php $_SERVER['REQUEST_METHOD']?>" method="POST">

Das macht doch keinen Sinn =)
<?php $_SERVER['REQUEST_METHOD']?> -- OHNE echo schreib dort rein garnix hin.
Was Du sicherlich willst ist, dass die gleiche Seite aufgerufen wird. Dann lass das Feld leer.

PHP

<form action="" method="POST"> <!-- action="" gleiche seite wird aufgerufen -->

strong97

1. Danke für die Mühe und die tolle Erklärung!

2. Wegen der Platzierung der function, ich habe die einfach irgendwo reingequetscht , das sollte alles nur ein Test sein ob es wirklich klappt. Bei Erfolg wollte ich dies dann auch umändern. Deswegen auch keine Platzhalter etc benutzt.

3. Das mit dem PHP war ein fehler den ich vergessen habe weg zu machen sollte nicht stehen

€:

Zitat

("du hast also immer abgefragt: gibt mir email von user_daten wo email $email ist =)
// ergo: immer 0 (keine) einträge gefunden)

Wie sollte ich es dann schreiben?

Habe es so gemacht, (mit dem gleichen Fehler aber dieses mal mit platzhalter)

[PHP]
function check($email) {
$stmt = $DB_con->prepare("SELECT email FROM user_daten WHERE email = ?");
$stmt->BindValue(1, $email);
$stmt->execute();

if($stmt->rowCount() > 0) {
return true;
} else {
return false;
}
}

if(check($email) === false) {
$error = 'Email existiert';
}
[/PHP]

cottton

Nix reinquetschen. Wenn Du da mal was vergisst, dann können "komische Sachen" passieren.

Das Problem war der Query-String:

PHP

<?php
$email = 'q@q.de';
$sql = 'select ... from ... where email = $email';

Ergibt den String
select ... from ... where email = $email
wegen den einfachen Anführungszeichen.

PHP

<?php
$email = 'q@q.de';
$sql = "select ... from ... where email = $email";

Ergibt den String
select ... from ... where email = q@q.de
=)

strong97

Also müsste ich es so schreiben:

[PHP]
$email = $_POST['email'];
$password = $_POST['password'];
$vorname = $_POST['vorname'];
$nachname = $_POST['nachname'];

$error = '';

function check($email) {
include 'sec/verbindung.inc.php';
$stmt = $DB_con->prepare("SELECT email FROM user_daten WHERE email = ?");
$stmt->BindValue(1, $email);
$stmt->execute();

if($stmt->rowCount() > 0) {
return true;
} else {
return false;
}
}

if(check($email) === false) {
$error = 'Email existiert';
}
[/PHP]

Habe die Strings mit ' ' geschrieben und die vorbereitungen mit " ". Falls es so richtig ist , dann funktoniert ist nicht.
Falls du dich fragst wieso ich in der Function wieder eine DB verbindung habe:
Wenn ich dies wegmache dann steht da undefined Variable stmt und undefined method prepare, d.h er findet keine Verbindung.

cottton

ungetestet (klar)

PHP

$email = $_POST['email'];
$password = $_POST['password'];
$vorname = $_POST['vorname'];
$nachname = $_POST['nachname'];




$error = '';




include 'sec/verbindung.inc.php'; // ich denke mal $DB_con wird hier definiert
// evtl hast du die verbindung schon vorher aufgebaut -
// und $DB_con ist schon definiert -- dann brauchst du das include gar nicht
function check($DB_con, $email) { // $DB_con mit reinschicken -- die 
	$stmt = $DB_con->prepare("SELECT email FROM user_daten WHERE email = ?"); 
	$stmt->bindParam(1, $email);
	$stmt->execute();




	if($stmt->rowCount() > 0) {
    	return true;
	} else {
    	return false;
	}
}




if(check($email) === false) {
    $error = 'Email existiert';
}

Alles anzeigen

Was nutzt Du eigl: mysqli_ oder PDO ?

strong97

Scheint irgendwie keine Wirkung zu haben. Habe es so gemacht wie du es im Script hast.

cottton

Fehlermeldung?
Raten bringt nix.
Fang mal eventuelle Fehler bei ->prepare und ->execute ab

Und: Was nutzt Du eigl: mysqli_ oder PDO ?

strong97

Nutze Pdo. Habe es geschafft nur es ist etwas komisch.

Er soll ja den Error zeigen wenn es falsch ist und das Programm abbrechen. Er tut es aber nicht sondern bei true tut er es.

PHP

Klappt alles.




if(check($DB_con, $email) === true) {
$error = ' Email exisitert';
}




Klappt nicht
if(check($DB_con, $email) === false) {
$error = 'Email existiert';
}

Alles anzeigen

Wieso bei true?

cottton

Ha! Hab ich gar nicht gesehen
Deine Funktion gibt do true zurück, wenn $stmt->rowCount() > 0,
also die eMail gefunden wurde.

Tipp: nenne Funktionen am besten immer danach, was sie tun.
In dem Fall hier könnte die Funktion heißen: ifEmailExists($email)
So liest man schon, was da passiert/erwartet wird: if email exists, ...

strong97

haha passiert mal
kk werde ich machen danke nochmal für die Hilfe

Jetzt mitmachen!