User mit Vorname und Nachname ansprechen

strong97 · 24. Dezember 2015

Hallo ein zehntes mal ,

Wollte mal Fragen wie ihr es machen würdet den User mit Nachnamen und Vornamen anzusprechen nach erfolgreichem Login.

Habe es zurzeit so gemacht ist aber bissl bescheurt da man sich nur mit der email einloggen soll und nicht auch mit dem Vornamen. (Habe in dem Beispiel nur den Vornamen benutzt)

[PHP]
public function login($email,$password $vorname)
{
try
{
$stmt = $this->db->prepare("SELECT email,password,vorname FROM user_daten WHERE email = :email OR :vorname");
$stmt->execute(array(':email'=>$email, ':vorname'=>$vorname));
$userRow=$stmt->fetch(PDO::FETCH_ASSOC);
if($stmt->rowCount() > 0)
{
if(password_verify($password, $userRow['password']))
{
$_SESSION['email'] = $userRow['email'];
$_SESSION['vorname'] = $userRow['vorname'];
return true;
}
else
{
return false;
}
}
}
catch(PDOException $e)
{
echo $e->getMessage();
}
}
[/PHP]

Ich wusste nicht wie ich vorname mit einem Platzhalter verwenden sollte.

Nach dem Login erfolgreich war könnte ich easy den User so ansprechen: Willkommen $_SESSION['vorname']. Nur das muss ich hier irgendwie anders machen.

cottton · 24. Dezember 2015

Warum sollte man sich mit Vornamen einloggen können?
Ich finde es schon schlimm, wenn man sich mit der eMail einloggen kann/muss. (ich raff es nicht, warum das bei "den Großen" wie FB, google und co so gemacht wird)

Ich würde es nicht empfehlen, den Vornamen für den Login zu verwenden.

PHP

public function login($email,$password, $vorname) // , (komma) vergessen
    {
        try
        {
            $stmt = $this->db->prepare("SELECT email,password,vorname FROM user_daten WHERE email = :email OR :vorname");
            /* fatal error: die query hier ergibt folgendes:
            	SELECT email,password,vorname FROM user_daten 
            	WHERE email = 'bsp@bsp.de' OR 'benutzername'




            	diese query gibt immer den nutzer zurück, der die email besitzt
            	ODER den ersten in der reihe!




            	warum: OR 'ein nicht leerer string' 
            		wird als WAHR (es ist, also es ist nicht "nicht") gedeutet.
            		Ähnliches bsp: OR 1




            	richtig wäre: 
	            	SELECT email,password,vorname FROM user_daten 
            		WHERE email = :email OR vorname = :vorname -- NEU: vorname = 
            */
            $stmt->execute(array(':email'=>$email, ':vorname'=>$vorname));
            #$userRow=$stmt->fetch(PDO::FETCH_ASSOC); unnötig
            $count = $stmt->columnCount()// NEU
            #if($stmt->rowCount() > 0)
            if($count > 0) // NEU
            {
                if(password_verify($password, $userRow['password']))
                {
                    $_SESSION['email'] = $userRow['email'];
                    $_SESSION['vorname'] = $userRow['vorname'];
                    return true;
                }
                else
                {
                    return false;
                }
            }
        }
        catch(PDOException $e)
        {
            echo $e->getMessage();
            return false; // NEU -- hier sollte auch false zurückgegeben werden
        }
    }

Alles anzeigen

wolf · 24. Dezember 2015

@cotton, wieso denn nicht mit der Email?

cottton · 24. Dezember 2015

Einfach wegen der Mehr-Information - mit der eMail weiß "man" gleich 2 Sachen: eMail und Login-Name.
Es ist auch wahrscheinlicher, dass jemand die gleiche eMail auf anderen Seiten nutzt, als einen Nickname.

(Und Kleinigkeiten wie zB Smartphone-login -- email eingeben ... nervt.)

strong97 · 24. Dezember 2015

Ja aber man soll sich ja nicht mit dem Vornamen einloggen können, das meinte ich ja. Das sollte ne Probe sein damit der User sich später ein Nicknamen erstellt. Nur muss ich erstmal probieren ob ich den Sinn verstehe wie ich es mache und es funktoniert. Im Script von dir kann man sich ja mit vorname und email einloggen, man sollte sich ja aber nur mit der mail einloggen.

cottton · 24. Dezember 2015

Wieso hast Du das mit dem :vorname dann überhaupt erst reingenommen?
Verstehe jetzt nicht ganz, wo das Problem ist.

wolf · 24. Dezember 2015

@cotton mit input[type=email] ist das auch easy.. Aber das selbe nickname Problem ist nur Sicherheitstechnisch eines..
Denn der Nutzer tut sich leichter seine Email zu merken als 1000 nicknames.. wobei er dann sowieso nur 2-3 standart-nicknames haben wird (schließen wir mal den kleinen Prozentsatz der Sicherhritsbewussten Nutzer aus) und somit das Problem des gleichen 1.Faktors der Authentifizierung besteht, nur komplizierter für den Nutzer, richtig?

@strong ... was, wenn du zwei Nutzer mit selbem Vornamen hast?

strong97 · 24. Dezember 2015

Sry das ich mich nicht deutlicher ausgedruckt habe. Nickname meine ich eigenen Benutzername, (falls ich die Email weglasse).

@cotton , ich habs reingenommen weil ich nicht wusste wie ich es machen sollte, und dies hat als einzigstes funktoniert. Nur Problem war das man sich auch halt mit dem Vornamen einloggen kann.

Was ich möchte:

z.B ich melde mich irgendwo an gebe meinen vornamen und nachnamen an. Dann wenn alles geklappt hat soll z.B in der nächsten Seite stehen. Willkommen "Vorname & Nachname" mit einer SESSION.

cottton · 24. Dezember 2015

Na dann brauchst Du doch aber den :vorname nicht in der WHERE-Bedingung.
Login per eMail und PW. Die Query änderst Du in der Hinsicht, dass Du mehr auswählst (SELECT)
Vorschlag:

PHP

<?php
    public function login($email, $password)
    {
    	if(!strlen($email)) { 
    		// error ausgeben: feld kann nicht leer sein 
    		return false;
    	} 
    	if(!strlen($password)) { 
    		// error ausgeben: feld kann nicht leer sein 
    		return false;
    	}
    	$user = $this->getUserByEmail($email);
		if($user and is_array($user) and !empty($user)) { 
			// if würde hier noch die felder prüfen wie zB if($this->isValidUser($user)) ... nur so am rande
            if(password_verify($password, $user['password'])) {
            	// würde das session setzen auslagern like: $this->setSession($user); ... auch nur so am rande
              	$_SESSION['email'] = $userRow['email'];
              	$_SESSION['vorname'] = $userRow['vorname'];            	
              	//$_SESSION['password'] = $userRow['password']; das passwort sollte nicht mehr gebraucht werden
                return true;
            } 
		} 
		// error ausgeben: email oder passwort falsch
		return false;
    } 
    public function getUserByEmail($email) 
    {
    	/* INFO
    	function gibt einfach nur den gefundenen nutzer zurück, oder false (wenn nicht email existiert)
    	*/ 
        try  {
        	$sql = "SELECT `email`, `password`, `vorname` FROM `user_daten` WHERE `email` = :email LIMIT 1;";
        	/* INFO
        	- LIMIT 1, weil wir ja nur einen nutzer erwarten. (gäbe es 2 mit der gleichen email, dann wäre etwas von grund auf falsch)
			- abschließendes semikolon ; immer. warum sollte man die query "offen" lassen (es sei denn du hängst per PHP noch was an ...).
        	*/
        	// prepare
        	$stmt = $this->db->prepare($sql);
        	if(!$stmt) {
        	    // fehler auslesen mit: $this->db->errorInfo() 
        	    // könnte hier $this->errorLast = $this->db->errorInfo(); dir überlassen
        	    return false;
        	} 




        	// execute
        	$para = array(':email'=>$email);
        	if(!$stmt->execute($para)) {
        	    // fehler auslesen mit: $this->db->errorInfo() 
        	    // könnte hier $this->errorLast = $this->db->errorInfo(); dir überlassen
        	    return false;
        	}




        	// fetch
	        /* INFO
			bei PDO gibt es kein ->fetchFirst(). du könntest fetchAll() nutzen, oder eben so:
	        */
	        $row = $stmt->fetch(PDO::FETCH_ASSOC); // eine row lesen (erwarten ja nur eine)
	        /* INFO
	        $row sollte in etwa so aussehen:
	        $row = array(
				'email' => '...daten...',
				'password' => '...daten...',
				'vorname' => '...daten...',
	        )
	        */
	        return $row;  
        } catch(PDOException $e) {
            //echo $e->getMessage(); // exception ausgeben kann fatal sein, wenn benutzer daten in der exception enthalten sind (pw!)
            return false; // NEU -- hier sollte auch false zurückgegeben werden
        }
    }

Alles anzeigen

EDIT: Fehlerbeseitigung

strong97 · 25. Dezember 2015

Tolle erklärung, verstehe es auch jetzt, aber muss es nicht $row['password'] etc heißen?

2. Der findet die Daten nicht

login.php

PHP

if(isset($_POST['btn-login'])) {
		$email = $_POST['txt_mail'];
		$password = $_POST['txt_password'];

		if($user->login($email,$password)) {
			$user->redirect('home.php');
		} else {
		$error = "Falsche Daten";
		}
	}

3. Habe im Script das mit ':vorname'=>$vorname weggemacht weil es sonst eine Fehlermeldung erzeugt. Falls ich es einfüge:

PHP

Notice: Undefined variable: vorname in C:\xampp\htdocs\class.user.php on line 45




Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in C:\xampp\htdocs\class.user.php on line 46




Notice: Undefined variable: row in C:\xampp\htdocs\class.user.php on line 15

Alles anzeigen

cottton · 25. Dezember 2015

Ja, das stimmt. Das hab ich übersehen (ich übersehe viel in den letzten Tagen :D)
Ich hab da paar wirklich blöde Fehler drin -- ich hab meinen letzten Post mal editiert.

strong97 · 25. Dezember 2015

Passiert^^.

So jetzt loggt der sich ein aber i-was spinnt immer noch. In meiner home.php steht drin:

PHP

if(!isset($_SESSION['email'])) {
header ("Location: page.php");
}

Einloggen klappt aber werde direkt zu page.php weitergeleitet. Das heißt so wie, session['email'] gibt es nicht aber daten waren richtig.

€: Wieso heißt das $user['password'] ? und nicht $row['password'] ? Falls ich $row['password'] mache steht da undefined variable row

cottton · 25. Dezember 2015

Weil der Rückgabewert von $this->getUserByEmail($email) in $user gepackt wird.
$user = $this->getUserByEmail($email);
Du kannst die Variable natürlich auch umbenennen.

Was ist/passiert in page.php? Soll man dort hin, wenn man sich eingeloggt hat?
(Thema Benennung - page.php sagt nix aus =) )

Zitat

Das heißt so wie, session['email'] gibt es nicht aber daten waren richtig

Hä Stell den Glühwein weg

Zitat

Einloggen klappt aber werde direkt zu page.php weitergeleite

Ja, muss ja. Denn Wenn Login klappt, wird doch $_SESSION['email'] gesetzt.

Btw: würde nicht nur auf isset prüfen, sondern auch, ob die session email nicht leer ist.
Warum: wenn $_SESSION['email'] mal FALSE ist, dann ist es auch gesetzt.

strong97 · 25. Dezember 2015

€: alles ok danke euch

User mit Vorname und Nachname ansprechen

Jetzt mitmachen!

Teilen