Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> Problem z łapaniem wyjątków, exception handler
Gligamesh
post 7.01.2016, 18:42:13
Post #1





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

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


Ten post edytował Gligamesh 7.01.2016, 18:43:10
Go to the top of the page
+Quote Post
nospor
post 7.01.2016, 19:03:30
Post #2





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 7.01.2016, 21:11:17
Post #3





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

czyli że handlera uruchamia samo throw ?

new mError() nie przerwie działania skryptu.



Go to the top of the page
+Quote Post
nospor
post 8.01.2016, 11:06:04
Post #4





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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?


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 8.01.2016, 17:27:10
Post #5





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

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.

Ten post edytował Gligamesh 8.01.2016, 17:37:25
Go to the top of the page
+Quote Post
nospor
post 8.01.2016, 17:43:41
Post #6





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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 :/


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 8.01.2016, 17:51:59
Post #7





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

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

Ten post edytował Gligamesh 8.01.2016, 18:02:45
Go to the top of the page
+Quote Post
nospor
post 8.01.2016, 18:22:33
Post #8





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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.


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 8.01.2016, 18:39:47
Post #9





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

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ć ?



Go to the top of the page
+Quote Post
nospor
post 8.01.2016, 18:51:41
Post #10





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 8.01.2016, 22:00:09
Post #11





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

czyli do handlera ląduje eHandler i eDisplay a jak przekazać dane z mError ?


Ten post edytował Gligamesh 8.01.2016, 22:01:23
Go to the top of the page
+Quote Post
nospor
post 8.01.2016, 22:58:33
Post #12





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
No przeciez w handlerze masz wyjatek, ktory wlasnie zostal zgloszony czyli masz mError


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
Gligamesh
post 8.01.2016, 23:44:43
Post #13





Grupa: Zarejestrowani
Postów: 227
Pomógł: 0
Dołączył: 13.06.2003
Skąd: rykowice

Ostrzeżenie: (0%)
-----

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


Ten post edytował Gligamesh 8.01.2016, 23:46:58
Go to the top of the page
+Quote Post
« Następny starszy · PHP · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 19.07.2025 - 09:36
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn