Diskuze: Uložení hodnoty do databáze
Martin Konečný (pavelco1998):19.5.2018 13:25
Například první dotaz vůbec nemusí být fetchAll(), jelikož hledáš jenom jednoho uživatele podle $cardId. Tedy ten cyklus je tam zbytečně. Nevím, jak přesně vypadá ta knihovna Db, ale stačilo by něco jako
$query = Db::query('SELECT id,CardId FROM uzivatele WHERE CardId=?', $CardId);
$user = $query->fetch();Další věc je, proč si prvně žádáš data o uživateli podle $cardId (navíc CardId nepotřebuješ vyhledávat, když to už znáš) a teprve potom zjišťuješ dalším dotazem, zda vůbec nějaký takový záznam existuje? Ten dotaz s proměnnou $existuje je zbytečný, protože pokud v prvním dotazu nic nenajdeš, tak v $user bude FALSE nebo něco takového.
Funkce htmlSpecialChars() se používá na výstupu, ne na vstupu. Proti SQL injection se používají jiné funkce, ale hlavně jelikož používáš parametrizované dotazy, tak si ta knihovna tu proměnnou ošetří sama. Tedy tam vůbec být nemusí a správně by ani neměla.
V podmínce if ($vlozit) se znovu připojuješ k databázi, proč?
Dotaz pod tím je stejný případ jako na začátku - hledáš jednoho
uživatele, nepotřebuješ tam nějaký cyklus. Navíc ta data o uživateli si
můžeš získat už v tom prvním dotazu, nepotřebuješ další 
Takovéhle řešení by bylo v pohodě (píšu z hlavy, nemusí být 100% funkční):
Db::connect();
if (isset($_POST["CardId"])) {
$cardId = $_POST["CardId"];
$query = Db::query("SELECT * FROM uzivatele WHERE CardId = ?", $cardId);
$user = $query->fetch(PDO::FETCH_ASSOC);
if ($user !== FALSE) {
Db::query("INSERT INTO dochazka (date,id_user,CardId) VALUES (NOW(),?,?)", $user["id"], $cardId);
echo "Účast hasiče " . $user['jmeno'] . " " . $user['prijmeni'] . " bylo uloženo";
} else {
echo "K tomuto číslu karty není přiřazen žádný uživatel, zaregistrujte kartu";
}
}Martin Konečný (pavelco1998):20.5.2018 13:29
V tom případě místo konstrukce
$query = Db::query("dotaz");
$user = $query->fetch();napiš
$user = Db::querySingle("dotaz");Ty tam děláš zbytečně dva dotazy - jeden na kontrolu, zda existuje
záznam, a druhý na výběr dat o uživateli. Jenže můžeš v klidu udělat
jeden dotaz na data o uživateli a pokud ti nepřijdou data o uživateli
(protože se nic nenašlo), tak to znamená, že takový záznam neexistuje 
michal Čepelák:20.5.2018 13:58
Ale jak zobrazím dole v echo účast hasiče ... tak je tam jméno a
příjmení a to vytahuji z tabulky uzivatele, kterou použivám i pro user.
tak nahoře ten foreach musí zůstat nebo se pletu?
úprava kódu
if (isset($_POST['CardId'])){
$CardId = htmlspecialchars($_POST['CardId'],ENT_QUOTES);
$user = Db::querySingle('SELECT * FROM uzivatele WHERE CardId=?', $CardId);
foreach ($user as $zobuser){
$zobuser['id'];
}
$existuje = Db::querySingle('SELECT COUNT(*) FROM uzivatele WHERE CardId=?', $CardId);
if ($existuje){
$vlozit = Db::query('INSERT INTO dochazka (date,id_user,CardId) VALUES (NOW(),?,?)',$zobuser["id"], $CardId );
if ($vlozit){
echo 'Účast hasiče '.$zobuser['jmeno'].' '.$zobuser['prijmeni'].' bylo uloženo';
}
}else{
echo 'Chyba komunikace s databazí, kontaktujte administrátora';
}
}takle to funguje jak má
vic foto v příloze
Zobrazeno 15 zpráv z 15.