Dotaz: Nefunguje primitivní podmínka v PHP

[Host]

Zdravíčko ve spolek. LámU si hlavu proč mi nefunguje tato podmínka. Pořád se mi vyhodnocuje jako STOP.

if(my_fce() == 0 ){
echo("NEXT");
}else{
echo("STOP");
}

var_dump(my_fce());//array(0) { }

Děkuji za případný help ;)

Odpovědi

[Tarmaq]

protoze ti my_fce() vraci pole obsahujici 0 a ne primo tu 0, vzdyt je to v tom var_dumpu videt

[Host]

a co to značí? jak to můžu opravit?

[Tarmaq]

nemam kristalovou kouli, jaky je smysl te funkce? co vsechno vraci?

[blizzz]

echo empty(my_fce()) ? "NEXT" : "STOP";

[Jan Trávníček]

to pole je prázdné ne že by obsahovalo nulu.

[Tarmaq]

nojo sorry jsem se nejak prekoukl ;] v tom pripade asi autor potrebuje tohle:

if (!count(my_fce())) {
  echo("NEXT");
} else {
  echo("STOP");
}

ale jen hadam, tezko rict o co mu vlastne jde ;]

[Dejv]

Spis

if (count(my_fce()) != 0) {
  echo("NEXT");
} else {
  echo("STOP");
}

Pokud je mi znamo, count() vraci integer, nikoliv boolean

[Kit]

Ono je to vcelku jedno, protože 0 je považována za FALSE, zbytek je TRUE.

Jedno je to také z toho důvodu, že celá konstrukce se mi jeví jako nesmyslná. Myslím tu úplně první od tazatele Host. Nenapadá mě smysl takového bloku.

[OldFrog {Ondra Nemecek}]

Co je špatného na použití fce v podmínce?

[Šangala]

Kdo to říkal, že je to špatné? (Obecná odpověď: „nic“. )

[Kit]

Použití funkce v podmínce je vcelku OK, ale připadá mi podivné použití funkce vracející pole.

Host pojmenoval tuto funkci my_fce(). Předpokládám tedy, že je to jeho vlastní funkce a že si do ní může dát co chce. Co ho vedlo k tomu, že do té funkce nedal i celý rozhodovací blok včetně obou větví?

[oldfrog]

To bych nechal na něm...

[Radek Hladik]

Špatného na to není nic, je to i vcelku běžná praxe. Je ovšem potřeba mít na zřeteli, že taková funkce může měnit stav programu, což je omezení, které se v teroii programovacích jazyků poměrně dost řeší. Napřiklad výsledek takové podmínky nejde cacheovat. Nebo při složitější konstrukci je vůbec otázka, zda se funce provede:

if (my_fce1() && my_fce2())

my_fce2 se vůbec nemusí zavolat (pokud se používá lazy vyhodnocování) a například v tomto případě neplatí komutativita &&, protože význam není shodný s :

if (my_fce2() && my_fce1())

Takže jak říkám, špatně to není, velmi často se to používá, ale je potřeba mít jistotu, že člověk ví, co dělá. Ostatně oblíbené

do_something() or die('error')

to celé posouvá ještě o krok dál

[OldFrog {Ondra Nemecek}]

Máte pravdu, ta funkce musí být pro dané použití navržená. Měl by to být getter. Ten nemění stav a může navíc případně zajišťovat i cachování výsledku.

[Ash]

To je moc omezující, může to být i cokoliv jiného, u čeho nás zajímá pravdivost návratové hodnoty.

[OldFrog {Ondra Nemecek}]

Ano, getter chápu totiž v širším pojetí, kdy nemusí nutně vracet jen instanční proměnnou. Pokud budu mít např. kontejner, může mít metody isEmpty() nebo contains($oObject) a budu je považovat v podstatě za gettery, protože mi vracejí nějakou vlastnost (charakteristiku) kontejneru.

[Ash]

já ovšem mluvil spíš o if(shutdown_internet_now()) relax();, což je něco jiného než ten víš "getter" if(is_internet_down()) relax(), ale přitom je to pořád legitimní užití funkce v podmínce.

[Jirka]

A proč ne?

if (!empty(my_fce())) {
  echo("NEXT");
} else {
  echo("STOP");
}

Pokud vím, tak count() prochází všechny prkvy v poli. Při větším počtu prvků by mohlo vyhodnocení podmínky trvat relativně dlouho.

[Ash]

Tak to víte něco, co není pravda :)

[Mr.S1lent.cz]

PHP Array si IMHO drzi v metadatech datove struktury pocet prvku, cili funkce "count" jen ziska klicovou informaci z metadat.

[Šangala]

A přesně tak jak má, tady je to napsané.

array operator anything −> array is always greater

Šílenec by mohl napsat:

if(intVal(array()) == 0){
  echo('NEXT');
}else{
  echo('STOP');
}

Ale určitě hledáte count()…

[Kit]

Myslím si, že to není dost šílené na to, aby to nikdo nikde nepoužil

Ale stejně by mě zajímalo, k čemu to Host vlastně potřebuje. Velmi často zde někdo napíše kus kódu tak vytržený z kontextu, že vůbec nevystihuje podstatu problému, jehož přímé řešení by určitě nebylo tak kostrbaté.

[Šangala]

Dle specifikace PHP to není s ničím v rozporu, tak proč ne, aspoň se při čtení člověk zamyslí co tím autor vlastně myslel
Až mě to sem tam mrzí, třeba by to bylo i zajímavé…

[Kit]

Napadlo mě například volání funkce, která v poli vrací indexy výskytů podřetězce v řetězci. Pokud nebylo nic nalezeno, udělá jednu akci, jinak udělá druhou. Záleželo by pak na těch akcích, zda by se to celé třeba dalo přepsat na jeden regulární výraz. Pro jiné okrajové podmínky by mohlo být úplně jiné řešení.

To se tak někdy programátor zafixuje na nějakém řešení a nemůže se ho zbavit a lepí ho všude tam, kam přijde. Také by mě zajímalo, co vlastně Host řeší a také mě mrzí, že se o to s námi nechce podělit.