Diskuze – Lekce 6 - Jednoduchý redakční systém v Nette - Výpis článku
ZpětUpozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.
)
Jindřich Máca:31.10.2016 18:45
Ahoj, tak to vezmu zase postupně, protože je opět vidět, že děláš
něco, čemu úplně nerozumíš. 
- Máš chybu hned v názvu metody. Zase nějaká nepozornost? Funkci máš pojmenovanou getArticleId() a voláš metodu getReferenceId().
- Úplně špatně jsi pochopil chování metody get(). Dokumentace říká
Returns row specified by primary key.
To znamená, že vrátí celý záznam (všechny sloupečky) podle konkrétního zadaného primárního klíče (nebo false, pokud záznam s takovým primárním klíčem neexistuje).
Každopádně tu metodu jako takovou nemáš úplně špatně. Vypadat by měla nějak takto:
/* Měl by se vykonat přesně ten databázový dotaz, který jsi poslal, který ale může vrátit pole
záznamů v případě, že sloupeček s URL není unikátní, takže pro jistotu vytáhneme první shodu. */
// Načte první záznam článku s danou URL z databáze, pokud existuje.
if (!($article = $this->database->table('article')->select('article_id')->where('url', $url)->fetch())) {
// Co se má udělat pokud článek se zadanou URL neexistuje.
} else {
/* V proměnné $article jsou teď uloženy všechny sloupečky daného záznamu vyžádané
v SELECT, takže zde je to pouze sloupeček 'article_id'. Z principu je to ale pole, takže samotné
ID z něj musíme ještě vytáhnout. */
return $article['article_id']; // Vrací ID článku.
}Pozn.: Nezkoušel jsem to.
- Ta druhá funkce, na kterou jsi se ptal a měla by vracet ID vloženého článku, se upraví asi nějak takto:
if (!$article[self::COLUMN_ID]) {
$row = $this->database->table(self::TABLE_NAME)->insert($article);
return $row[self::COLUMN_ID];
} else {
$this->database->table(self::TABLE_NAME)->where(self::COLUMN_ID, $article[self::COLUMN_ID])->update($article);
return $article[self::COLUMN_ID];
}Pozn.1: Opět jsem to nezkoušel.
Pozn.2: Nedělal jsem tady kontrolu toho, jestli se článek skutečně
podaří vložit, či editovat.
- To, že to máš rozpracované by nemělo vadit, pokud používáš např.
verzovací systém jako je Git , či jenom verzování v rámci
IDE, nebo si na to jednoduše napiš Unit test.
Doufám, že už je to všechno jasnější. 
P.S.: Neber si to špatně, ale čím víc dotazů pokládáš, tím mám větší pocit, že ještě úplně neovládáš programování v PHP a principy OOP jako takové. Možná bych nejdříve doporučil zaměřit se na ně, místo toho, aby ses rovnou vhrnul na celý e-shop ve frameworku. Navíc, pokud to neděláš jenom jako cvičení a plánuješ komerční použití, tak je potřeba myslet ještě na spoustu dalšího zabezpečení a neudělat v kódu amatérské chyby, jinak hrozí potenciální katastrofa v podobě např. úniku uživatelských dat apod.
Jen taková poznámka k čistčímu kódu, kdyby jsi nahradil
$this->database->table(self::TABLE_NAME) novou funkcí např. getAll(),
nemusel by se ten dlouhý kus kódu pokaždé opisovat. Pokud se mýlím, vyveď
mě z omylu
Jindřich Máca:20.2.2017 18:08
Určitě souhlasím, já jsem dokonce šel ještě trochu dál a mám
vlastní knihovnu, kde pokud podědíš z dané třídy, automaticky tam tu
metodu getAll() i nějaké další máš.
Zobrazeno 10 zpráv z 105.