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

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

0 0

Líbí se: zatím nikdo

Nelíbí se: zatím nikdo

https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083558

Jiří

(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

0 0

Líbí se: zatím nikdo

Nelíbí se: zatím nikdo

https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083557

Fido123

(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

0 0

Líbí se: zatím nikdo

Nelíbí se: zatím nikdo

https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083556

Raketka

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

0 0

Líbí se: zatím nikdo

Nelíbí se: zatím nikdo

https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083555

Smazany ucet 253

(3 hodnocení)

25. 1. 2015 04:37:11

@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

0 0

Líbí se: zatím nikdo

Nelíbí se: zatím nikdo

https://webtrh.cz/diskuse/ruzne-druhy-vypisu-kategorii-nacpat-do-jedne-funkce/#reply1083554

Pro odpověď se přihlašte.

Přihlásit