Zadejte hledaný výraz...

Různé druhy výpisu kategorií – nacpat do jedné funkce?

Raketka
verified
rating uzivatele
16. 1. 2015 17:45:59
Ahoj,
jsem v programování dost samouk se spousty špatnejma návykama. Mam na jednom webu několik sekcí, kde se vypisujou produkty, jednou nejnovější, podruhý nehledanější, potřetí z jiný tabulky z databáze a počtvrtý s modifikovanym vzhledem. Výpis pro každou stránku mam php souborech - pro každou stránku samostatně. Dneska jsem si ale uvědomil, že jsem mohl napsat funkci, kterou bych mohl z každý podstránky s parametrama volat a tak ušetřit třeba 20 řádek kódu v každym souboru. Jenže když jsem zkoušel tu funkci napsat, tak jsem zjistil, že tam musim nacpat přes 5 podmínek, aby byla použitelný ve všech případech. A teď se nemůžu rozhodnout, zda je teda lepší napsat jednu universální funkci s mnoha podmínkama, anebo to prostě ty modifikace udělat natvrdo v každym souboru zvlášť. Díky za názory.
16. 1. 2015 17:45:59
https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083558
Jiří
verified
rating uzivatele
(1 hodnocení)
16. 1. 2015 18:22:29
Napsal bych to jako třídu, čímž se zruší duplicity a zjednoduší správa kódu, s tím že odlišnosti řešit podmínkami nebo děděním, podle toho co je pro danou situaci vhodnější.
16. 1. 2015 18:22:29
https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083557
Fido123
verified
rating uzivatele
(13 hodnocení)
16. 1. 2015 18:23:55
Pokud to děláš jenom v PHP bez frameworku, tak bys mohl mít na každé stánce kód, který načte produkty do nějakého třeba pole, zavolá metodu ListProducts a jako parametry jí předá to pole produktů a název metody pro detail produktu (něco jako ShowProductDetail, ShowProductDetailModified). Samotná metoda ListProducts projede pole a pro každý produkt zavolá tu metodu pro zobrazení detailu produktu, které předá produkt. No a ty metody ShowProductDetail a ShowProductDetailModified dostanou jako argument produkt a ten zobrazí.
16. 1. 2015 18:23:55
https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083556
Raketka
verified
rating uzivatele
16. 1. 2015 21:45:59
Díky za tipy, udělal jsem tu třídu. Každopádně jsem při tom předělávání zjistil, že ten svůj kód bych mohl minimalizovat a vylepšovat snad donekonečna. Asi budu muset napsat ještě hodně řádků kódu, než se to naučim psát pořádně :)
16. 1. 2015 21:45:59
https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083555
@Raketka: pokaždé, když dělám code review sám sobě zjistím, že bych to mohl napsat efektivněji a daleko lépe ;) takže si z toho nic nedělej, zkušenosti tě posouvají dál
25. 1. 2015 04:37:11
https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083554
Pro odpověď se přihlašte.
Přihlásit