Získej svůj iPhone v nové soutěži! Získej svůj iPhone v nové soutěži!
Nová překladatelská soutěž ITnetwork.cz o telefon iPhone, sluchátka Beats a další věcné ceny za 4 hodiny práce.
Přidej si svou IT školu do profilu a najdi spolužáky zde na síti :)
Avatar
Ondřej Pech
Člen
Avatar
Ondřej Pech:10. října 19:27

Ahoj, napsal jsme funkce kde čas od času zkontroluji jestli se mi na nějakém místě vrátilo false a když jo tak chci aby ta funkce hodila FALSE. Použil jsme return ale funkce pokračuje dál, to už nechci, protože další věci nedostanou naplněné potřebné proměnné. Proč mi to nefunguje? Nechci používat exit a ukončovat celý script, jen potřebuji aby se další kód dané funkce neprovedel a funkce vrátila FALSE. Díky za rady

 
Odpovědět 10. října 19:27
Avatar
Marian Benčat
Redaktor
Avatar
Odpovídá na Ondřej Pech
Marian Benčat:10. října 19:37

Použil jsme return ale funkce pokračuje dál,

hoď kód, return ukončuje VŽDY současnou funkci, takže pravděpodobně děláš něco špatně, nebo něco špatně chápeš.

Editováno 10. října 19:37
Nahoru Odpovědět  +2 10. října 19:37
In Smalltalk, everything is an object, In Clojure, everything is a list, In Javascript, everything is fucking mistake
Avatar
Ondřej Pech
Člen
Avatar
Ondřej Pech:10. října 19:57

Konkrétní kód nemůžu ale zkusím to trochu obecně jak to mám:

<?php
$vysledek = priklad($parametry);

if(!$vysledek){
        $log->error('Bla Bla Bla');
}

// FUNKCE ----------------

function priklad($parametry)
{
        $kontakty = $spravceKontaktu->vratIdkontaktuPodleLicencnihoKlice('AAAAA');
        if(!$kontakty) {
                return false;
        }

        $deaktivace = $spravceKontaktu->deaktivujKontakt($kontakty['id']);
        if(!$deaktivace) {
                return false;
        }

        return true;
}

když první funkce ve funkci priklad vrati false tak se mi stejně spusti druha funkce a ta pochopitelně háže chybu, protože nezná parametr $kontakt['id']. TA by se podle mě ale už vůbec neměla provést a funkce priklad by měla vrátit false. Ale neděje se tak. Ještě může být chyba někde v těch knihovnách že nevrací false když ten kontakt nenajde ale na první podle to vypadá že by to mělo fungovat.

 
Nahoru Odpovědět 10. října 19:57
Avatar
Honza Bittner
Šupák
Avatar
Odpovídá na Ondřej Pech
Honza Bittner:10. října 20:21

A co vrací "$spravceKontaktu->vratIdkontak­tuPodleLicencni­hoKlice('AAAA­A');" ?

Nahoru Odpovědět 10. října 20:21
Milovník Dartu. Student FIT ČVUT. Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/...
Avatar
Marian Benčat
Redaktor
Avatar
Odpovídá na Honza Bittner
Marian Benčat:10. října 20:44

To vrátí IDKontaktuPodle­LicencnihoKli­ce NeumisCist? :D

Editováno 10. října 20:44
Nahoru Odpovědět  +2 10. října 20:44
In Smalltalk, everything is an object, In Clojure, everything is a list, In Javascript, everything is fucking mistake
Avatar
Marian Benčat
Redaktor
Avatar
Odpovídá na Ondřej Pech
Marian Benčat:10. října 20:46

Jinak ano... já samozřejmě vím, že PHP je dynamický SHIT a tudíž můžeš dělat různé věci... ale vratIdkontaktu­PodleLicencni­hoKlice by mi taky evokovalo, ze to treba bude vracet nejaky GUID, nebo Integer? ;-)

tudíž mi přijde poměrně nebezpečný dávat toto do podmínky a využít jakéhokoliv implicitního přetypování na boolean.. ale já nejsem PHPkář...

Editováno 10. října 20:47
Nahoru Odpovědět  +1 10. října 20:46
In Smalltalk, everything is an object, In Clojure, everything is a list, In Javascript, everything is fucking mistake
Avatar
Ondřej Pech
Člen
Avatar
Ondřej Pech:11. října 8:51

Vrací buď id kontaktu nebo FALSE v případě že nenajde. A to false právě odchytávám a když nenajde potřebuju aby to skočilo ven

 
Nahoru Odpovědět 11. října 8:51
Avatar
Petr D
Člen
Avatar
Odpovídá na Ondřej Pech
Petr D:11. října 9:21

Udělej si var_dump() vrácené hodnoty z $kontakty. Podle tvého modelového příkladu nedostáváš ID, ale pole...
V podmínce použij ===, čili porovnání hodnoty a datového typu Comparison Operators

Editováno 11. října 9:22
 
Nahoru Odpovědět  +1 11. října 9:21
Avatar
Jirka Jr
Člen
Avatar
Odpovídá na Ondřej Pech
Jirka Jr:11. října 10:54

souhlasim s tim co napsal Findthee

pridej si tam var_dump:

$kontakty = $spravceKontaktu->vratIdkontaktuPodleLicencnihoKlice('AAAAA');
var_dump($kontakty);
if(!$kontakty) {
        return false;
}

a pokud ti to skutecne vypise "bool(false)" a pritom funkce neskonci hned v tomto prvnim IFu, tak jsi objevil bug v PHP

pokud ti to vypise neco jineho, s cim nebudes vedet co a jak, tak to sem hod a popereme se s tim

 
Nahoru Odpovědět 11. října 10:54
Avatar
Odpovídá na Ondřej Pech
Vladislav Ladicky:11. října 11:43

To je predovsetkym zly pristup. Funkcia ma mat jediny vystupny bod, okrem ineho aj preto, aby si sa nestratil, ako sa ti to prave stalo. Prinajmensom to vsak bude prehladnejsie.

function priklad ($parametry) {
  $vysledok = false
  $kontakty = $spravceKontaktu->vratIdkontaktuPodleLicencnihoKlice('AAAAA');
  $deaktivace = $spravceKontaktu->deaktivujKontakt($kontakty['id']);

  if ($kontakty || $deaktivace) { //Ak to ide, testuj na true, nie false. A OR je vhodnejsie, lebo ma skratene vyhodnocovanie
    $vysledok = true;
  }

  return $vysledok;
}
Editováno 11. října 11:46
 
Nahoru Odpovědět 11. října 11:43
Avatar
Petr D
Člen
Avatar
Odpovídá na Vladislav Ladicky
Petr D:12. října 9:17

Tak to určitě ne. V rozumné míře můžeme použít kolik returnů chceme. V PHP platí pouze obecné pravidlo, nepoužívat break, else, elseif pokud předtím použiji return v if nebo case.
Např. místo.:

if (true) {
    return 1;
} else {
    return 0;
}

použít:

if (true) {
    return 1;
}
return 0;

Navíc tvůj příklad může skončit chybou hned na 4 řádku (opominu chybějící středník na druhém), pokud $spravceKontaktu->vratIdkontak­tuPodleLicencni­hoKlice('AAAA­A') vrátí false.

 
Nahoru Odpovědět 12. října 9:17
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 11 zpráv z 11.