Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Problem z łapaniem wyjątków
Forum PHP.pl > Forum > PHP
Gligamesh
7.01.2016, 18:42:13
Witam

Zrobiłem prostą obsługę wyjątków i błędów. Teoretycznie wszystko działa ale nie rozumiem jednej rzeczy. Dlaczego gdy rzucam mError działa moja metoda (poprawnie) i jednocześnie set_exception_handler (jako drugi) dublując wykonanie całości questionmark.gif

//wersja robocza. 

[PHP] pobierz, plaintext 
  1. Throw new mError($e, 'mysqli_connection'); 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
  1. namespace libs\core;
  2.  
  3. use Exception;
  4. use ErrorException;
  5.  
  6. error_reporting(E_ALL);
  7.  
  8. Class mError extends Exception
  9. { 
  10.     function __construct($e, string $type = null, string $comments = null) 
  11.     {
  12.         $this->file = $e->getFile();
  13.         $this->line = $e->getLine();
  14.         $this->type = $type;
  15.         $this->comments = $comments;
  16.         parent::__construct($e->getMessage(), $e->getCode());
  17.  
  18.         $this->eHandler();
  19.     }
  20.  
  21.     function eHandler() 
  22.     {
  23.         $message = 'Napotkano nieoczekiwany błąd aplikacji.';
  24.         $notice = 'Przepraszamy, ale nie możemy wyświetlić strony której poszukujesz. Prosimy spróbować ponownie za kilka minut.'; 
  25.  
  26.         switch($this->type) {
  27.             case 'not_found':
  28.                 header("HTTP/1.1 404 Not Found");
  29.                 header("Status: 404 Not Found"); 
  30.                 http_response_code(404);
  31.                 $message = 'Strony nie znaleziono.';
  32.                 $notice = 'Przepraszamy, ale strona którą chcesz zobaczyć nie istnieje.';   
  33.             break;
  34.             case 'mysqli_query':
  35.             case 'mysqli_connection':
  36.                 $message = 'Brak połączenia z serwerem MySQL';
  37.             break;
  38.         }
  39.  
  40.         $this -> eDisplay($message, $notice);
  41.     }
  42.  
  43.     function eDisplay(string $message, string $notice) 
  44.     {
  45.         echo '<!DOCTYPE html>
  46.             <html lang="pl">
  47.             <head>  
  48.                 <meta charset="utf-8">
  49.                 <title>'.$message.'</title>
  50.                 <meta name="robots" content="noindex">
  51.                 <style type="text/css">
  52.                     body {font-family: sans-serif; font-size: 14px; margin: 40px;} 
  53.                     fieldset {border: 1px solid #a4a4a4; padding: 10px; background-color: #f2f2f2; margin: 10px 0}
  54.                     legend {padding: 0 10px; font-weight: bold; }
  55.                     h1, h3 {font-weight: normal;}
  56.                     h3 {border-bottom: 1px solid #a4a4a4;  padding-bottom: 20px;}
  57.                     ul {list-style: none; margin: 0; padding:0;}
  58.                     li {padding-bottom: 4px;}
  59.                 </style>
  60.             </head>
  61.             <body>';
  62.         echo '<h1>'.$message.'</h1><h3>'.$notice.'</h3>';
  63.         if(error_reporting() == E_ALL) {
  64.             echo '<ul>'.
  65.                         '<li>Code: '.$this -> getCode().'</li>'.
  66.                         '<li>Message: '. $this -> getMessage()."</li>".
  67.                         '<li>File: '. $this -> file."</li>".
  68.                         '<li>Line: '. $this -> line."</li>".
  69.                         //'<li>'.$this -> comments.'</li>',
  70.                  '</ul>';
  71.             if(isset(Bootstrap::$stat)) {
  72.                 echo '<fieldset><legend> Bootstrap: </legend><pre>';
  73.                 print_r (Bootstrap::$stat);
  74.                 echo '</pre></fieldset>';   
  75.             }
  76.         } else {
  77.             echo 'Spróbuj <a href="java script: window.location.reload()">odświeżyć stronę</a> lub wróć na <a href="/">stronę główna</a>.';
  78.         }
  79.         echo '</body></html>';  
  80.     }
  81. }
  82.  
  83. set_exception_handler(function($exception) { 
  84. //    echo '<pre>';
  85. //    print_r($exception);
  86. //    echo '</pre>';
  87. // 
  88.  
  89.     New mError($exception);
  90. });
  91.  
  92. set_error_handler(function ($errno, $errstr, $errfile,  $errline) {
  93.     throw new ErrorException($errstr, null, $errno, $errfile, $errline, null);
  94. });
[PHP] pobierz, plaintext
nospor
7.01.2016, 19:03:30
Nie bardzo rozumiem co cie dziwi... skoro rzucasz wyjatkiem
Throw new mError
i skoro ustawiles ser_exception_handler to logiczne ze ci sie wlacza exception handler

edit:
zamiast tak Throw new mError($e, 'mysqli_connection');
to chyba powinienies robic samo new mError($e, 'mysqli_connection');
by dzialalo jak chcesz
Gligamesh
7.01.2016, 21:11:17
czyli że handlera uruchamia samo throw ?

new mError() nie przerwie działania skryptu.



nospor
8.01.2016, 11:06:04
Cytat
czyli że handlera uruchamia samo throw ?
No aniby ty myslales ze co go uruchamia?

Cytat
new mError() nie przerwie działania skryptu.

No to rzucaj normalny wyjatek a wyswietlaniem bledu zajmiesz sie w set_exception_handler a nie w konstruktorze mError... no naprawde nie widzisz tej duplikacji ktora robisz?
Gligamesh
8.01.2016, 17:27:10
Normalnego wyjątku nie rzucę bo nie da się (lub ja nie umiem) zdefiniować w tym przypadku $type i $comments. Zastąpienie $message nie wchodzi w grę.


BTW: wiesz że nie ładnie jest odpowiadać pytaniem na pytanie ? łachy nie robisz.
nospor
8.01.2016, 17:43:41
facepalmxd.gif
A gdzie ja bylem niegrzeczny? Jak masz okres to nie czytaj postow na forum bo widze troche nadwrazliwy jestes wtedy (o, teraz mozesz powiedziec ze jestem niegrzeczny, ale to jest nastepstwo twojej 'prowokacji' biggrin.gif ).

Powodzenia na dalszej drodze zycia.

edit:
Cytat
wiesz że nie ładnie jest odpowiadać pytaniem na pytanie ? łachy nie robisz.

a wiesz, ze nie ladnie jest edytowac wlasne posty, gdy ktos juz na nie odpowiedzial?? Teraz moj post nie ma sensu, haha

poza tym nie odpowiedzialem pytaniem na pytanie. Wpierw udzielilem ci odpowiedzi. Ale widze troche niedowidzisz i czytasz tylko wybiorczo posty :/
Gligamesh
8.01.2016, 17:51:59
Jeśli ktoś pyta, znaczy że nie wie. Oczekuje odpowiedzi, chociaż prostej TAK / NIE, na pytanie nie odpowiada się pytaniem. Jeśli przypomnienie o dobrych zasadach nazywasz prowokacją no to jak sam napiłeś "powiedzenia".

Nie jestem przewrażliwiony tylko nie lubię zbędnego filozofowania. Udzieliłeś odpowiedzi na moje pytanie, dziękuje (kliknąłem nie w ten post). Nie wiem tylko po co te "nie trafione" sugestie, jeśli da się inaczej / lepiej to napisz konkretnie.

Cytat
a wiesz, ze nie ladnie jest edytowac wlasne posty, gdy ktos juz na nie odpowiedzial?? Teraz moj post nie ma sensu, haha

Mój post: 26 minut temu, edycja 16 minut temu , Twój: 10 minut temu
nospor
8.01.2016, 18:22:33
Cytat
Mój post: 26 minut temu, edycja 16 minut temu , Twój: 10 minut temu
Nie wnikam. Wiem ze jak pisalem mojego posta, to twoja tresc wygladala inaczej.
Tak czy siak nie ladnie jest tak edytowac posta, ze tresc jest zupelnie inna smile.gif

Cytat
Oczekuje odpowiedzi, chociaż prostej TAK / NIE, na pytanie nie odpowiada się pytaniem
Udzielilem ci odpowiedzi. A pytanie bylo tylko dodatkiem, bo nie moglem uwierzyc, ze naprawde nie ogarniasz, iz throw rzuca wyjatkiem.

Cytat
eśli przypomnienie o dobrych zasadach nazywasz prowokacją no to jak sam napiłeś "powiedzenia".

1) Prowokacja nazwalem twoj oryginalny post a nie zmodyfikowany. Widzisz, nazmieniasz, naedytujesz a potem sam sie gubisz. Takze juz sam naocznie widzisz jakie sa wady totalnej edycji tresci
2) Twoj zmodyfikowany post tez nie byl przypomnieniem o dobrych zasadach, gdyz mnie nie dotyczyl. Jak juz nie raz powiedzialem, ja ci udzieliem odpowiedzi. Pytanie bylo tylko malym dodatkiem.
Gligamesh
8.01.2016, 18:39:47
dobra dobra wystarczy , dostałeś zapomogę więc się nie czepiaj tongue.gif

Cytat
Normalnego wyjątku nie rzucę bo nie da się (lub ja nie umiem) zdefiniować w tym przypadku $type i $comments.

mógłbyś się do tego ustosunkować ?



nospor
8.01.2016, 18:51:41
mError ma byc samym wyjatkiem. Wowczas mozesz go sobie zdefiniowac jak chcesz i miec parametry jakie chcesz.
A z racji, ze mError bedzie tylko wyjatkiem, to masz z niej wywalic caly mechanizm wyswietlania bledow i inicjalizacji ich wyswietlania. To ma byc w oddzielnej klasie, ktora bedzie odpalana w set_exception_handler
Gligamesh
8.01.2016, 22:00:09
czyli do handlera ląduje eHandler i eDisplay a jak przekazać dane z mError ?
nospor
8.01.2016, 22:58:33
No przeciez w handlerze masz wyjatek, ktory wlasnie zostal zgloszony czyli masz mError
Gligamesh
8.01.2016, 23:44:43
mówisz o czymś takim ? nie rozumiem jak handler ma się dobrać do $type i $comments 

[PHP] pobierz, plaintext 
  1. Class mError extends Exception
  2. { 
  3.     function __construct($e, string $type = null, string $comments = null) 
  4.     {
  5.         $this->file = $e->getFile();
  6.         $this->line = $e->getLine();
  7.         $this->type = $type;
  8.         $this->comments = $comments;
  9.         parent::__construct($e->getMessage(), $e->getCode());
  10.     }
  11.  
  12. }
  13.  
  14. set_exception_handler(function($exception) { 
  15. 	function eHandler() {}
  16. 	function eDisplay() {}
  17. });
  18.  
  19. set_error_handler(function ($errno, $errstr, $errfile,  $errline) {
  20.     throw new ErrorException($errstr, null, $errno, $errfile, $errline, null);
  21. });
[PHP] pobierz, plaintext
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.