[PHP]Wysyłanie pustego pliku do przeglądarki przy obsłudze wyjątków Opcje

[albrzykowski]

Witam,

Mam następujący problem, mam własną klasę do obsługi bazy danych i drugą raportującą błędy na zasadzie zapisania informacji o błędzie w bazie.

Oto fragment kodu gdzie skrypt zaczyna się dziwnie zachowywać (opiszę poniżej):

[PHP] pobierz, plaintext 
<?php
public function execute($query){
                $this->result = mysql_query($query);
                if($this->result!=FALSE){
                        return $this->result;
                }
                else{
                        throw new ExceptionHandler(_(__CLASS__.': query error: '.$query),3);
                        return FALSE;
                }
  }
?>
[PHP] pobierz, plaintext 

Jeśli w zapytaniu pojawi się błąd, np. zwykła literówka na zasadzie brak ', wtedy zamiast uzyskania błędu w przeglądarce, lub zapisania go przez ExceptionHandler, serwer wysyła mi pusty plik PHP do sićgnięcia przez przeglądarkę (plik typu: application/x-httpd-php).

Jeśli zarzucę obsługę błedów własną klasą i klasycznie zrobie tak:

[PHP] pobierz, plaintext 
<?php
public function execute($query){
                $this->result = mysql_query($query) or die(mysql_error);
?>
[PHP] pobierz, plaintext 

Skrypt zachowuje się już w bardziej oczekiwany sposób, tj. wyrzuca błąd i zatrzymuje się.

Proszę o sugestie i z góry dziękuję.

Ten post edytował albrzykowski 18.03.2009, 19:20:48

[bazyliszek83]

Wklej proszę troche wiecej kodu. M.in. kod klasy "ExceptionHandler" i jakiś prosty przykład użycia funkcji execute()...

[albrzykowski]

Hej,

No teraz zapytania nie wkleje ale przykładowo np w memencie gdy wyglądałoby tak (z błędnem)

[SQL] pobierz, plaintext 
SELECT * FROM tabela WHERE id=3'
[SQL] pobierz, plaintext 

(błąd z ' )

Kod klasy do obsługi wyjatków jest taki:

[PHP] pobierz, plaintext 
<?php 
/**
 * Class to handle the exceptions
 *
 */
class ExceptionHandler extends Exception{
        /**
         * @var string
         */
        protected $message;
        /**
         * Constructor
         * @param $message
         * @param $code
         * @access public
         */
        public function __construct($message,$code=NULL){
                parent::__construct($message,$code);
                $this->log();
        }
        /**
         * Method to return message
         * @access public
         * @return string
         */
        public function __toString(){
                return $this->message;
        }
        /**
         * Method to log the exceptions
         * @access public();
         */
        public function log(){
                $db = new Model();
                $sql = 'INSERT into exception_logs (message,code,file,line) VALUES(';
                $sql.= '''.$this->message.'',';
                $sql.= $this->code.',';
                $sql.='''.$this->file.'',';
                $sql.=$this->line.')';
                $db->execute($sql);
        }
}
?>
[PHP] pobierz, plaintext 

Dziękuję i pozdrawiam!

[bazyliszek83]

Spróbuj może to "zdebagować" i zamiast wyrzucać wyjątek, zrobić echo "Błąd!!!"; . Będziemy wtedy wiedzieli czy w ogóle dochodzi do tego momentu.

[albrzykowski]

Hej,

Przygotowałem specjalnie błąd:

w kontrolerze dodaje newsa:

[PHP] pobierz, plaintext 
<?php
$this->content->update('newses','title,excerpt,content,published','''.$newsTitle.'''::''.$newsExcerpt.''::''.$newsContent.''::''.$newsPublished.''',array('id'=>'='.$newsId));
?>
[PHP] pobierz, plaintext 

To :: rozgranicza pola dla metody update modelu (tylko dodaje żeby było wiadomo)
I błąd jest zaraz po $newsTitle o jeden pojedynczy cudzysłów za wiele.

metoda update wywołuje wewnetrzną metodę execute którą już wcześniej opisałem:

[PHP] pobierz, plaintext 
<?php
public function execute($query){
        $this->result = mysql_query($query);    
        if($this->result!=FALSE){
            return $this->result;
        }
        else{
            throw new ExceptionHandler(_(__CLASS__.': query error: '.$query),3);
            return FALSE;
        }
    }
?>
[PHP] pobierz, plaintext 

I w momencie próby zapisania newsa skrypt nie zgłasza nic tylko dostaje do pobrania plik PHP... pusty.
Może powinienem obsłużyć lepiej raportpowanie i zamknąć odpowiednie partie kodu w try{} i catch{} tylko nie zabardzo wiem jak to ugryźć.

Dzięki i pozdrawiam!!

[Noddi]

Napisz w jaki sposób wyłapujesz ten wyjątek, bo narazie pokazałeś, że go tylko wyrzucasz.

Pozdrawiam.

[albrzykowski]

Hej,

Wyłapuje wyjątek w ten sposób:

[PHP] pobierz, plaintext 
<?php
public function select($table,$columns,$conditions=NULL)
    {
        $columns = $this->makeColumns($columns);
        $sql = 'SELECT '.$columns;
        $sql.= ' FROM '.$table;
        if(!empty($conditions))
        {
            $sql.= $this->makeConditions($conditions);
        }
        $sql.=' ORDER BY 'id'';
        if(isset($this->limit)){
            $this->makePagination($table,$conditions);
            $sql.= ' LIMIT '.$this->pagination['startFrom'].','.$this->pagination['limit'];
        }
        try{
            $this->execute($sql);
        }
        catch(ExceptionHandler $e){continue;}
    }
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[Noddi]

Pusta strona pojawia się dlatego, że nie masz włączonego raportowania błędów. Wyjątki wyłapujesz nieprawidłowo, continue możesz wykorzystywać tylko w pętlach. Zamiast continue albo wyświetl treść wyjątku, albo np. zapisz do pliku.

[albrzykowski]

Hej,

Napisze po kolei:

- 1 w configu, w wersji dev mam:

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
ini_set('display_errors',1);
?>
[PHP] pobierz, plaintext 

- 2 Jesli chodzi o continue, to rzeczywiście głupi błąd, z mojej strony, nie mniej nie pomogło mi w niczym jego usunięcie.
- 3 Wolałbym obsługę zapisywać w bazie a nie w pliku choć przyznam, że pod niektórymi względami plik może bardziej nadawałby się do tego...
- 4 Wyjątków nie chcę wyrzucać na ekran. Raportowane są tylko błędy "poważniejsze" np właśnie związane z DB i nie chce powiadamiać o nich, "zwykłych" userów.
Natomiast napisałeś ~Noddi, że nieprawidłowo wyłapuję wyjątki, byłbym wdzięczny jeśli byś to rozwiną i zasugerował inne rozwiązanie.

Z góry dzięki!!

[Noddi]

Błędu dopatrywałem właśnie w nie wyświetlaniu errorów, bo przy continue w bloku catch powinno wyświetlić error o nieprawidłowym zastosowaniu continue. Co do możliwości wyświetlania, zapisu do pliku/bazy danych to tylko i wyłącznie Twój wybór. Napisałem, że nieprawidłowo wyłapujesz wyjątki ze względu na umieszaczenie continue w catch.

PHP + Apache masz prawidłowo skonfigurowane, inne skrypty działają?

Pozdrawiam.