Návrh na zlepšení kódu

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 :)

Kód:

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

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

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.

Původně odeslal musil.david

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í

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

Původně odeslal eqeq

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

Kód:

<?php
if (!isset($_GET['iframe']))
{
	$content = ob_get_contents();
	ob_end_clean();
	ob_start();
}
if (!isset($_GET['controller']) || empty($_GET['controller']))
{
	$_GET["controller"] = "pjFront";
}
if (!isset($_GET['action']) || empty($_GET['action']))
{
	$_GET["action"] = "pjActionViewPhp";
}
$_GET["id"] = $jednoduchyCMS;
if(isset($pjHide))
{
	$_GET["hide"] = 1;
}
if(isset($pjLocale))
{
	$_GET["locale"] = $pjLocale;
}
$dirname = str_replace("\\", "/", dirname(__FILE__));
include str_replace("app/views/pjLayouts", "", $dirname) . '/ind'.'ex.php';


if (!isset($_GET['iframe']))
{
	$app = ob_get_contents();
	ob_end_clean();
	ob_start();
	$app = str_replace('$','&#36;',$app);
	echo preg_replace('/\{CMS_OBSAH_'.$jednoduchyCMS.'\}/', $app, $content);
}
?>

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

Původně odeslal node

Kód:

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

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

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' ? :-)))

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..

Původně odeslal Steeta

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 :)