Diskuze: Změna hesla v PHP
V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.
Josef Kuchař - Pepa489:2.9.2016 23:17
U řádku na kterém je
Db::query('SELECT id FROM uzivatele WHERE jmeno=?')nedosazuješ za otazník hodnotu. 
Podle mě je tam chyb hned několik:
1:
Db::query('SELECT id FROM uzivatele WHERE jmeno=?') // Chybí parametr, který se dosadí za otazník
Db::query('SELECT id FROM uzivatele WHERE jmeno=?', [$jmeno]) // Takto by mohla vypadat oprava2:
$_POST['nove_heslo'] = $_POST['nove_heslo_znovu'] // Pokud se ptáš na shodu, je nutné použít dvě nebo tři rovnítka
$_POST['nove_heslo'] === $_POST['nove_heslo_znovu'] // Takto by mohla vypadat oprava3:
Db::query('
UPDATE SHA1(heslo)
FROM uzivatele
WHERE id=?
', $_POST['nove_heslo'] . "123456789"); // Nejsem si jist, ale myslím, že ve wrapperu z této sítě je nutno dávat parametry do pole
Db::query('
UPDATE SHA1(heslo)
FROM uzivatele
WHERE id=?
', [$_POST['nove_heslo'] . "123456789"]); // Takto by mohla vypadat oprava+20 Zkušeností
+2,50 Kč
Martin Štěpánek :3.9.2016 11:33
if ($_POST['stare_heslo'] = Db::query('SELECT id FROM uzivatele WHERE jmeno=?')ještě kontroluješ jestli se staré heslo rovná id uživatele
Martin Konečný (pavelco1998):3.9.2016 12:09
jo, ale takhle to tam máš ty 
když s tím v podstatě začínáš, tak to dělej pomalu a sleduj, jaká
metoda má jaké návratové hodnoty. Sice neznám místní DB wrapper, ale je
mi jasný, že metoda Db::query() ti nebude vracet jeden konkrétní výsledek
jednoho sloupce.
Ty potřebuješ zjistit, jestli se staré heslo shoduje s tím v databázi. K
tomu máš dvě možnosti:
- vytáhnout si heslo z DB podle uživatele. To pak porovnáš v PHP s heslem zadaným ve formuláři To hledej podle jeho ID, ne podle jména - máš zaručený, že díky PK (primární klíč) to ID bude vždycky unikátní a navíc se podle něj hledá rychleji.
- poslat dotaz do DB, kde do WHERE zadáš ID uživatele a heslo. Pokud se vrátí jeden záznam, tak se heslo shoduje. Pokud se vrátí nula záznamů, heslo bylo zadaný špatně. Tenhle způsob ale v podstatě nebudeš moct použít, pokud hashuješ způsobem, který DB neumí. Být tebou bych proto zvolil možnost a).
A mimo to, rovnítko v PHP můžeš použít na tři způsoby
$a = 10; // přiřazení hodnoty, tady bude výsledek vždycky TRUE! tohle když použiješ v podmínce, tak ti vždycky projde
$a == 10; // porovná proměnnou s číslem 10, při tom nezáleží na datovém typu ($a může být 10 jako číslo i 10 jako řetězec)
$a === 10; // porovná proměnnou s číslem deset a zároven udělá shodu datových typů. Pokud $a bude řetězec ($a = "10"), tak výsledek bude FALSEMartin Konečný (pavelco1998):3.9.2016 20:16
A to jsem myslel, že to vrací výsledek "operace". Nicméně se to do
podmínky přetypuje na bool, což ve většině případů znamená TRUE a
začátečník na tom hezky narazí, pokud nebude přesně vědět, co dělá
Každopádně díky za upozornění
Martin Konečný (pavelco1998):5.9.2016 17:41
díky za informaci Pokud
používáš PDO, je to OK. Já používám primárně Doctrine 2, případně
Nette\Database (pokud fakt musim a neni jiná možnost), s ničím jiným
zkušenosti nemám
Jan Lupčík:9.9.2016 6:40
Jenom takové dvě menší poznámky. 
- Z hlediska pohodlnosti pro uživatele bych doporučoval mít dvě podmínky. Nejdřív ověřit staré heslo a dát mu zprávu o tom, že je špatně a pak ověřovat, zda opakování nového hesla je správně a dát mu o tom zprávu.
- V místní třídě s databází se musí ty parametry pro dotaz předávat v poli.
Zobrazeno 28 zpráv z 28.