Diskuze: Login a registr
V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.
Martin Konečný (pavelco1998):9.7.2017 21:36
Strčit tu další podmínku do toho bloku 
Pokud neexistuje uživatel s
daným jménem, není důvod kontrolovat heslo (vlastně ani není s čím jej
srovnávat). Tudíž něco takového
if ($rows["0"] == 1) {
if ($password === $heslo) {
// ok
}
}Martin Bubník:10.7.2017 9:31
Děkuji.. Už to nehlásí chybu. Ale stále se mi nedaří přihlásit přes zaregistrovaného uživatele:
<?php
include "db.php";
session_start();
$username = htmlspecialchars($_POST["username"]);
$password = htmlspecialchars($_POST["password"]);
$password = hash("sha256", $password);
if (empty($username) || empty($password)) {
exit("Musíte vyplnit přihlašovací údaje");
//inputy
}
// počet záznamů
$sqlDotaz = "SELECT count(*) FROM users WHERE username_user = :username"; //počet záznamů
$sqlProvedeni = $db->prepare($sqlDotaz);
$sqlProvedeni->execute(array(":username" => $username));
$rows = $sqlProvedeni->fetch();
//kontrola zda li je záznam
if ($rows["0"] == 1) {
$sqlDotaz = "SELECT id_user, password_user1 FROM users WHERE username_user = :username"; //počet záznamů
$sqlProvedeni = $db->prepare($sqlDotaz);
$sqlProvedeni->execute(array(":username" => $username));
$pom = $sqlProvedeni->fetchALL();
//uložení záznamů
foreach ($pom as $value) {
$id = $value["id_user"];
$heslo = $value["password_user1"];
}
if ($password === $heslo ) {
echo "seš přihlášen";
$_SESSION["id_user"] = $id;
} else {
exit("Zadal jste špatně heslo nebo uživatelské jméno");
}
}
?>Screen shot tabulky v databázi je zde přiložen.. Nevím kde je chyba, už nad tím nějakou chvíli dumám..
Michal Haňáček:10.7.2017 10:47
Co má dělat tohle 
?
if ($rows["0"] == 1) { ... }Není to trochy zbytečný? Jednak posíláš 2 dotazy do DB a jednak ti to zbytečně komplikuje ten kód ... myslím.
Miskyns:10.7.2017 10:54
I když při takovém malém kódu, který získává pár údaju je ten
pokles výkonu zanedbatelný, ale při větších aplikacích to může být
horší Ale lepší to
optimalizovat teď než potom 
Ahoj,
v prvom rade by som si ten na tvojom mieste značne osekal. čím viac kódu, tým vačšia šanca, že dôjde niekde k chybe a tiež sa ti to celú aplikáciu spomaluje.
časť:
// počet záznamů
$sqlDotaz = "SELECT count(*) FROM users WHERE username_user = :username"; //počet záznamů
$sqlProvedeni = $db->prepare($sqlDotaz);
$sqlProvedeni->execute(array(":username" => $username));
$rows = $sqlProvedeni->fetch();je zbytočná, pretože username by malo byt v databaze s vlastnostou unique (nemôžeš mať viac užívateľov s rovnakým username). Sql funkcia count(*) ti môže preto vátit buď 0 - užívateľ nie je v DB-tabulke alebo 1 - užívateľ v DB už je. Ak máš veľa záznamov a užívateľ už mysql našiel script bude pokračovať ďalej až kým neprejde celú tabuľku (on nevie, že tam je maximálne len jeden užívateľ).
preto ten kód úplne vynechaj a uprav to takto:
$sqlDotaz = "SELECT id_user, password_user1 FROM users WHERE username_user = :username LIMIT 1"; //počet záznamů
$sqlProvedeni = $db->prepare($sqlDotaz);
$sqlProvedeni->execute(array(":username" => $username));
$result = $sqlProvedeni->fetch(PDO::FETCH_ASSOC);
// ak sme nasli zaznam a sedi meno, heslo
if ($result>rowCount() == 1) {
//uložení záznamů
$id = $record["id_user"];
$heslo = $record["password_user1"];
if ($password === $heslo ) {
echo "seš přihlášen";
$_SESSION["id_user"] = $id;
} else {
exit("Zadal jste špatně heslo nebo uživatelské jméno");
}
}dôvod prečo ťa neprihlási môže byť:
máš napríklad krátky rozsah stĺpca v DTB
prvé dva záznamy nemajú zahashované heslo
skús si vypísať
echo $record["password_user1"];
echo <br/>;
echo $password;
+20 Zkušeností
+2,50 Kč
Zobrazeno 25 zpráv z 25.