Jak duży blok try catch Opcje

[hugo_amv]

Na pewnej mądrej stronie ( nie pamiętam jakiej, może na java.sun.com, ale na pewno źródło było dość pewne) spotkałem się z opinią, że nie powino się umieszczać duży bloków kodu pomiędzy try catch, a jedynie krótkie fragmetny kodu i odrazu przechwytywać błędy. Ja wolę wszystkie wyjątki przechwytywać w silniku strony, a dopiero potem zajmować się ich obsługą.

[PHP] pobierz, plaintext 
<?php
try{
	/**
	 * Tutaj różne moduły, uruchamiaja różne klasy,
	 * które zwracają różne wyjątki :)
	 */
}catch (SQLException $e){
	// Obługa wyjątków związanych z bazą danych
	echo $e;
}catch (FileException $e){
	// Inny typ wyjątków
	echo $e;
}catch (Exception $e){
	// Pozostałe rodzaje błędów
}
?>
[PHP] pobierz, plaintext 

Jednak na tamtej stronie było wyraźnie napisane, że jest to złą praktyką programistyczną. Jak to w końcu jest?

[Termit_]

A ja pytam po co do żądania o wyświetlenie newsa wraz z komentarzami wysyłać do parsera php kod odpowiedzialny za obsługę wyjątków LoginErrorException czy GalleryImageUploadException? Bo skoro WSZYSTKIE typy wyjątków chcecie mieć w jednym miejscu... przy dużym projekcie to może być spory kawał zbędnego kodu, który można by przenieść do /login.phpm czy /admin/gallery.phpm.

[hugo_amv]

A ja pytam po co do żądania o wyświetlenie newsa wraz z komentarzami wysyłać do parsera php kod odpowiedzialny za obsługę wyjątków LoginErrorException czy GalleryImageUploadException? Bo skoro WSZYSTKIE typy wyjątków chcecie mieć w jednym miejscu... przy dużym projekcie to może być spory kawał zbędnego kodu, który można by przenieść do /login.phpm czy /admin/gallery.phpm.

Hm.. tu akurat masz racje jeśli by wszystkie wyjątki przychwytywać w jedym miejscu. Ale ja nie napisałem, że wszystkie wyjątki mają być łapane w jednym miejscu. Po pierwsze zakładam, że nie każda klasa ma obsługę błędów poprzez wyjątki. Czasem wystarczy sprawdzić czy np. $user->isLogin() zwraca true. Na końcu kodu przechwycam tylko ważne wyjątki (informujące o braku możliwości wykonania jakieś ważnej operacji: na bazie danych, systemie plików) Zamiast GalleryImageUploadException byłby tylko jeden wyjątek FileException z klasy odpowiedzialne za zarządzanie plikami.

Nikt nie zabrania używania krótkich bloków try catch wewnątrz modułów czy klas. Wyjątek klasy SQLException i tak znajdzie swój blok catch, który go obsłuży.

Ten post edytował hugo_amv 3.06.2006, 21:48:46