[PHP] Problem z wyjątkami Opcje

[Capellini]

[PHP] pobierz, plaintext 
<?php
function wikipedia($sl)
  {
    try
    {
      file_get_contents("http://www.pl.wikipedia.org/wiki/". $sl);
      return true;
    }
 
    return false;
  }
?>
[PHP] pobierz, plaintext 

Funkcja ma zwrócić wartość true, jeżeli z funkcją file_get_contents nie ma żadnych kłopotów i wartość false, jeżeli jest jakiś błąd.

Nie jestem jeszcze zbyt obeznany w wyjątkach w PHP więc pomóżcie.

[blooregard]

Wyjątki nie zwracają wartości.
Jeśli wystąpi wyjątek, to w bloku catch() {} łapiesz wyjątek wyrzucony metodą throw() z bloku try {}.

Czyli Twój kod powinien wyglądać tak:

[PHP] pobierz, plaintext 
<?php
function wikipedia($sl)
 {
   try
   {
     if (! ($pobrane =file_get_contents("http://www.pl.wikipedia.org/wiki/". $sl)) {
          throw new Exception("Błąd. Nie udało sie pobrac czegos tam");
     
   }
    
   return $pobrane;
 } catch (Exception $e) {
     echo $e->getMessage();    <- wyswietli komunikat bledu z wyrzuconego wyjatku, czyli to, co dales w wywolaniu throw()
}
?>
[PHP] pobierz, plaintext 

[Capellini]

Funkcja ma zwracać wartość true, jeżeli można otworzyć stronę, a jeżeli nie to ma zwracać false.

Ta funkcja nie ma zwracać tego z file_get_contents tylko ma powiedzieć, czy można to coś pobrać.

Jeżeli nie można tego zrobić przy użyciu wyjątków, to w jaki inny sposób można to zrobić?

EDIT:

Próbowałem po prostu zwrócić zamiast $pobrane to true i zamiast wyświetlać $e->getMessage() to zwrócić false, ale jeżeli file_get_contents nie działa jak należy, to i tak wypisuje błąd.

Ten post edytował Capellini 31.01.2009, 16:16:11

[230005]

Ale użycie tego throw new Exception() samo w sobie sprawi, że w wymaganej sytuacji funkcja zwróci false. Właśnie po to istnieje taka konstrukcja - aplikacja przerywa w tym miejscu działanie (czyli jeśli po throw... dałeś return false, to się to return false nie wykona) i zwróci false.

[Capellini]

Ok.

Ale kod blooregard'a niestety nie działa bo (pomijając już małe błędy typu brak zakończenia nawiasu) wyrzuca mi błąd składni: unexpected T_RETURN, expecting T_CATCH więc czy ten catch nie powinien być zaraz po try? Zrobiłem tak, ale wtedy jest co prawda komunikat "Błąd. Nie udało sie pobrac czegos tam" i funkcja co prawda zwraca false, ale wypisuje także ten błąd, więc wygląda to tak:

Warning: file_get_contents(): HTTP request failed! HTTP/1.0 403 Forbidden in E:\Moje\Najnowsze\funkcje.php on line 37 Warning: file_get_contents(http://www.pl.wikipedia.org/wiki/da): failed to open stream: No error in E:\Moje\Najnowsze\funkcje.php on line 37 Błąd. Nie udało sie pobrac czegos tam

Ja chcę żeby po prostu, jak wystąpi jakiś błąd, funkcja tylko zwróciła false i żeby nie wypisywało Warning: file_get_con... chcę żeby po prostu nie traktowało tego jako błąd. Czy da się tak zrobić?

Chodzi o to, że jak to zobaczy, że file_get_contents nie działa jak trzeba, to nie ma panikować, bo wszystko jest ok i ma tylko zwrócić false, a cały skrypt ma działać dalej.

Ten post edytował Capellini 31.01.2009, 20:06:03

[230005]

No więc:

Kod

<?php

function wikipedia()
{
  if(file_get_contents("http://www.pl.wikipedia.org/wiki/"))
      return true;
  else
      return false;  
}

if(@wikipedia())
    echo "ok";
else
    echo "nie ok";
  
?>

Wyjaśnienie: widzisz to wywołanie funkcji wikipedia w ifie poza funkcją? Ten znaczek - @ to wygłuszanie błędów, czyli jeśli strony nie znajdzie to tylko wypisze ten komunikat (nie ok u mnie )

[Capellini]

Teraz działa, dokładnie o to chodziło. W dodatku bardzo proste rozwiązanie.

Ten post edytował Capellini 31.01.2009, 22:03:34