return false + opis, Jak zrobic return false + opis bledu? Opcje

[shimano]

Witam.

Zastanawiam sie, jak zrobic, aby funkcja w przypadku bledu zwracala wartosc NULL lub FALSE i jednoczesnie komunikat bledu?

Cos na styl:

[PHP] pobierz, plaintext 
function funkcja() {
  // kod (..)
  if(!$a) {
    return (FALSE, "Wystapil blad!");
  }
  return TRUE;
}
 
$i = funkcja();
if(!$i) {
  echo $i;
}
else {
  echo "Wszystko w porzadku, dzialam dalej...";
}
[PHP] pobierz, plaintext 

Oczywiscie cos takiego nie dziala, ale mysle, ze dobrze obrazuje to co, chce osiagnac.

[#luq]

W funkcjach zastosuj sobie jeden parametr który będzie referencją na zmienną z teksem komunikatu

[PHP] pobierz, plaintext 
    function a( $param, &$err ){
        if( !$param ){
            $err = 'Jakieś info';
            return false;
        }
        else{
            return true;
        }
    }
 
    if( !a( false, $errorText ) ){
        echo $errorText;
    }
[PHP] pobierz, plaintext 

Natomiast pisząc obiektowo masz wyjątki od tego.

[krowal]

I tak bóg stworzył wyjątki (Exceprions)

[PHP] pobierz, plaintext 
<?php 
function mojaFunkcja($param){
  if ($param == 'cos_jest_nie_tak'){
    throw new Exception('Wiadomosc bledu');
  }
}
 
try{
  mojaFunkcja('cos_jest_nie_tak');
}catch(Exception $e){
  echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

@#luq byłeś pierwszy

Ten post edytował krowal 26.09.2009, 21:33:34

[shimano]

Hm, no to ciagle nie to, o co mi chodzi... Moze jakies inne sugestie?

Edit:
Jedyne co mi przychodzi do glowy to:

[PHP] pobierz, plaintext 
function funkcja() {
  global $errstr;
  // kod (..)
  if(!$a) {
    $errstr = "Wystapil blad!";
    return false;
  }
  return TRUE;
}
 
$i = funkcja();
if(!$i) {
  echo $errstr;
}
else {
  echo "Wszystko w porzadku, dzialam dalej...";
}
[PHP] pobierz, plaintext 

Niestety te rozwiazanie nie jest ani zbyt ambitne, ani zbyt dobre, ani nawet najwygodniejsze... Wiec moze jakies pomysly?


Ten post edytował shimano 27.09.2009, 19:54:11

[krowal]

A czemu nie możesz użyć do tego wyjątków ?

[PHP] pobierz, plaintext 
<?php 
$globalErrorsArray = array();
 
function mojaFunkcja($param){
  if ($param == 'cos_jest_nie_tak'){
    throw new Exception('Wiadomosc bledu');
  }
}
 
try{
  mojaFunkcja('cos_jest_nie_tak');
}catch(Exception $e){
  $globalErrorsArray[] = $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

ps. deklarowanie globali w funkcji ? bez sensu... btw. deklarowanie globali w ogóle jest bez sensu.

[#luq]

@krowal dobrze mówi. Chodź użycie wyjątków w kodzie proceduralnym to takie lekkie mieszanie, nie ma w tym nic złego no ale mieszanie to mieszanie
Jeśli takie rozwiązanie Ci się nie podoba to wrzucaj kolejny parametr który będzie referencją na string (ew. tablicę) błędu (błędów) tak jak to pokazałem. Z globalem nie ma co kombinować bo tak jak wyżej napisano nie ma sensu, jest nie eleganckie i takie prostackie.

[shimano]

No ja powiedzialbym nawet, ze mieszanie to bardzo lekko powiedziane. Polecacie wyjatki, a czy sami ich uzywacie w skomplikowanych kodach, gdzie jest wiele warunkow? Poza tym chyba ta funkcja nie zostala stworzona konkretnie do tego, co potrzebuje wiec smialbym nawet stwierdzic, ze na dluzszy czas stosowanie wyjatkow do tego, co chce osiagnac moze byc niebezpieczne.

Tak wiec wyjatki odpadaja. Bynajmniej w takiej formie, w jakiej obecnie PHP je zawiera.

A co do zmiennych globalnych. Pisalem, ze nie jest to dobre wyjscie, ani ladne, ani madre, itp. Ale tez nie jest bardzo bardzo zle, czy smiertelnie niebezpieczne. Zwlaszcza, ze w tym przypadku taka globalna zmienna zawiera tylko string do wyswietlania. Nie ma na niej zadnych innych operacji.

Swoja droga to az dziwne, ze chlopaki z PHP nie wprowadzili do tej pory mozliwosci zwrotu "wartosci" wraz z typem true albo false.

P.S.
Uprzedzajac ewentualna kolejna propozycje...
Poza zmiennymi globalnymi jeszcze moznaby zastosowac zwrot tablicy. Owszem, rozwiazanie lepsze od globali. Ale tak samo srednio wygodne.


Ten post edytował shimano 28.09.2009, 18:44:39

[230005]

Poza tym chyba ta funkcja nie zostala stworzona konkretnie do tego, co potrzebuje

To moim zdaniem źle potrzebujesz .

wiec smialbym nawet stwierdzic, ze na dluzszy czas stosowanie wyjatkow do tego, co chce osiagnac moze byc niebezpieczne.

Jakim cudem miałoby to być niebezpieczne? Rzucasz wyjątek - blok kodu zostaje przerwany - skrypt idzie dalej.

Swoja droga to az dziwne, ze chlopaki z PHP nie wprowadzili do tej pory mozliwosci zwrotu "wartosci" wraz z typem true albo false.

A znasz jakiś język, w którym coś takiego jest? W sumie to nawet ciężko mi byłoby sobie coś takiego wyobrazić. Tzn. jak według ciebie miałoby to wyglądać? Z resztą kurde - po to są właśnie wyjątki - jak jakiś zostanie wypluty, to albo drukujesz komunikat błędu i przechodzisz dalej, albo w bloku catch drukujesz komunikat błędu i dajesz kod obsługujący taką sytuację...

[krowal]

Swoja droga to az dziwne, ze chlopaki z PHP nie wprowadzili do tej pory mozliwosci zwrotu "wartosci" wraz z typem true albo false.

W ostateczności możesz zrobić tak że funkcja zwraca true albo jakąś wiadomość:

[PHP] pobierz, plaintext 
<?php 
function test($param){
  if ($param){
    return true;
  }else{
    return 'niestety wyskoczyl blad';
  }
}
 
$valid = test(false);
if ($valid !== true){ //WAZNE! zwróć uwagę na operator porównania ze sprawdzaniem typów "!== , ==="
  echo $valid;
}
?>
[PHP] pobierz, plaintext 

W Twoim przypadku zupełnie wystarczy

Ja jednak nadal upieram się przy wyjątkach, i twoje pytanie czy korzysta się z nich w skomplikowanym kodzie = tak korzysta się, chyba w każdym frameworku masz błędy obsługiwane przez wyjątki.
Jeśli będziesz już wiedział jak one działają i jak można je przechwytywać to zobaczysz o co chodzi