Návrh na zlepšení kódu

Tomina10

(11 hodnocení)

4. 4. 2020 01:38:10

Ahoj,

mám takový problém..

Potřeboval bych použít následující kód proto, aby se mi zobrazil text přes php...

$jednoduchyCMS = 1; include '/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php';

$jednoduchyCMS = 2; include '/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php';

s tímto kódem mi to funguje bezvadně, ale rád bych to zlehčil a zautomatizoval.... jelikož musím pokaždé přidat ručně číslo na další řádek k zobrazení dalšího textu (z příkladu například bych teď přidal $jednoduchyCMS = 3; include '/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php';)

je důležitý čísla kódu udržet....

napadlo mi logické řešení a to -

$jednoduchyCMS = 1-99999; include '/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php';

jenže to samozřejmě fungovat nebude, nemá někdo nápad jak tohle prosím vyřešit?

Moc děkuju :)

4. 4. 2020 01:38:10

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446398

node

(5 hodnocení)

4. 4. 2020 01:50:55

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446397

David Musil

(69 hodnocení)

4. 4. 2020 08:35:15

A to budeš 10krat pod sebou includovat stejný soubor?

4. 4. 2020 08:35:15

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446396

Pavel Mareš

(19 hodnocení)

4. 4. 2020 09:43:29

Nebylo by lepší vysvětlit pointu kodu? Páč někdo může přijít a navrhnout lepší řešení místo jen vytvoření hromady proměnných se stejným číslem.

Protože já fakt nechápu, proč na jednom místě musíš mít XXX řádků na kterých se mění číslo a znova načítáš soubor.

4. 4. 2020 09:43:29

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446395

Tomina10

(11 hodnocení)

4. 4. 2020 11:10:40

Napsal musil.david;1577510
A to budeš 10krat pod sebou includovat stejný soubor?

To je právě to, co nechci.. v tom souboru se nachází texty které mají ID, s kterým text skrze původní kód načtu.

Hledám řešení, kterém by to nemuselo být založeno do budoucna na mém ručním přidáváním řádku ke každému dalšímu ID.

Tyto kódy přidávám do patičky stránky.

A následující kódy mají za úkol vkládat ten konkrétní text ze souboru kamkoliv do HTML kódu {CMS_OBSAH_1}, {CMS_OBSAH_2}, {CMS_OBSAH_1} atd...

jeden bez druhého nefungují

4. 4. 2020 11:10:40

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446394

chladek

(4 hodnocení)

4. 4. 2020 11:21:05

Includovat x krát stejný soubor je zbytečné. Co kdybys sem hodil obsah toho Load.php?

4. 4. 2020 11:21:05

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446393

Tomina10

(11 hodnocení)

4. 4. 2020 11:36:46

Napsal eqeq;1577525
Includovat x krát stejný soubor je zbytečné. Co kdybys sem hodil obsah toho Load.php?

---------- Příspěvek doplněn 04.04.2020 v 13:12 ----------

Napsal node;1577500
for ($jednoduchyCMS = 1; $jednoduchyCMS <= 10; $jednoduchyCMS++) { include "/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php";}

tyjo..... asi jo hele :)))))) Moc děkuju...

4. 4. 2020 11:36:46

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446392

chladek

(4 hodnocení)

4. 4. 2020 13:36:37

Tam to zas nahrává další soubor, kdoví co je tam.. V tom posledním bloku se nahrazuje ten {CMS_OBSAH_X} obsahem, který to načte z index.php, který se zde načítá. Tak nejlíp tohle obalit do funkce s tím co je v index.php, ale nakonec asi bude nejlepší co radí node, jak to tak vidím..

btw. napadá někoho proč /index.php psát jako '/ind'.'ex.php' ? :-)))

4. 4. 2020 13:36:37

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446391

Steeta

(3 hodnocení)

4. 4. 2020 13:39:03

No celkově mi uniká smysl toho celého kódu. Vypadá to na divočinu celé. Ale proč includuješ celou cestu "/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php"; Když už tak jen "/cms/admin/app/views/pjLayouts/Load.php"; Ta cesta je očividně k adresáři na tvém localhostu přímo z Céčka (nebo nějakého disku). Kdekoli na hostingu ti ta adresa nebude fungovat, jen na tvém localhostu. Za předpokladu že výchozí složka localhostu je právě až "htdocs"

a použil bych spíš include_once() místo include() pokud tam ten for() chceš mít..

4. 4. 2020 13:39:03

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446390

Tomina10

(11 hodnocení)

4. 4. 2020 14:25:54

Napsal Steeta;1577548
No celkově mi uniká smysl toho celého kódu. Vypadá to na divočinu celé. Ale proč includuješ celou cestu "/opt/lampp/htdocs/cms/admin/app/views/pjLayouts/Load.php"; Když už tak jen "/cms/admin/app/views/pjLayouts/Load.php"; Ta cesta je očividně k adresáři na tvém localhostu přímo z Céčka (nebo nějakého disku). Kdekoli na hostingu ti ta adresa nebude fungovat, jen na tvém localhostu. Za předpokladu že výchozí složka localhostu je právě až "htdocs"
a použil bych spíš include_once() místo include() pokud tam ten for() chceš mít..

cestu bych v tomhle případě vůbec neřešil, úpravy provádím výhradně na localu...

Každopádně odpověď od Node mi pomohla problém vyřešit :)

Tedy děkuji všem, co si udělali čas a snažili se pomoc :)

4. 4. 2020 14:25:54

https://webtrh.cz/diskuse/navrh-na-zlepseni-kodu/#reply1446389

Pro odpověď se přihlašte.

Přihlásit