Výpis galerie – kategorie

Urcite by to slo resit jednim souborem. navic kdyz stejny kod pouzivas 3x, udelej si na to funkci.Co se tyka problemu, vyres to napr. pomoci pole, do ktereho si budes ukladat jiz zobrazene obrazky. Pri vyberu z db pak jen overis jestli uz v poli jsou. Aby se ti to moc nebilo, vypis si nejdrive posledni, pak oblibene a nakonec nahodne kategorie.

Napsal Ajtak;885383

...

if ( isset($_GET) )

if ( $_GET > 0 )

if ( $_GET < 4 )

$whatImgList = $_GET;

else $whatImgList = 1;

else $whatImgList = 1;

else $whatImgList = 1;

Jseš si jistý že chyba není v tomhle, protože jsi nenapsal jesli se jedná o jednotný výpis (všechny 3 věci dohromady) nebo oddělený, kde by mělo význam si kontrolovat GET proměnné...Pokud by to bylo jednotné tak tenhle kód by vykonal pokaždé to samé...

Napsal Ajtak;885383

switch ( $whatImgList ) {

case 1: $fea = $main -> featured();

break;

case 2: $fea = $main -> latest();

break;

case 3: $fea = $main -> random();

break;

}

----------------------------------------Potom mě ještě napadá si dyžtak onu GET proměnnou ošetřit funcí intval();Aby sis byl jistý na 100% že tam je číslo a nevrací ti to textový řetězec, viz:if ( isset($_GET) ) if ( intval($_GET) > 0 ) if ( intval($_GET) < 4 ) $whatImgList = intval($_GET); else $whatImgList = 1; else $whatImgList = 1; else $whatImgList = 1; ----------------------------------------EDIT1:

Napsal winexec;885389

Urcite by to slo resit jednim souborem.

Stím musím souhlasit----------------------------------------EDIT2: Teď mě ještě napadlo elegantnější řešení tvého výpisu, ty if-y se mi zdají šílené$whatImgList = isset($_GET) ? intval($_GET) : 1;switch ( $whatImgList ) { case 2: $fea = $main -> latest(); break; case 3: $fea = $main -> random(); break; default: $fea = $main -> featured(); }

Upřesním. Původně to bylo na stránkách, kde se volala GETem proměnná whatimglist a byla vždy jen jedna volaná kategorie. Já bych to chtěl sloučit do jedné stránky.Tento skript jsme koupili, a jelikož je psaný v kohana frameworku, tak to nechám v těch dvouch souborech.Takže já vlastně chci na tu jednu stránku udělat, aby byly všechny 3 kategorie. Zkoušel jsem to mnohokrát již upravit, ale vždy to buď házelo chybu, nebo se nic nestalo

Napsal Ajtak;885393

Upřesním.

Původně to bylo na stránkách, kde se volala GETem proměnná whatimglist a byla vždy jen jedna volaná kategorie. Já bych to chtěl sloučit do jedné stránky.

No jasně dělá to přesně to co má podle zdrojáku... nastaví všude jedničky a voala vypisuje všude to samé, to by ani to pole nezachránilo... Každopádně na toto zcela určitě by bylo vhodné napsat funkci s parametrem, jak doporučuje kolega výše. Nebo to lze obejít přes for cyklus, ve kterém by byl ten switch s výpisem.