Code Smell Of “Exception Handling”

Photo by mostafa meraji on Unsplash

Biz developerların en sevdiği şey sanırım doğru loglamadır. Bir hata bildirisi geldiği zaman temiz bir loglama yapılmışsa fixlemek sadece 10 dakikamızı bile alabilir.

Hata sarmallama için kullandığımız try catch bloklarının kullanımı çok büyük önem taşımaktadır. Ben bu sarmallamada en çok yaptığımız bir yanlıştan bahsetmek istiyorum.

public void islem() throws Exception{
   int toplam = 0;    try {
        toplam = 5+4;
    }catch (Exception e){
        throw new Exception("hata mesajı", e);
    }
}

yukarıdaki kod bloğunda islem() adında bir methodumu var. bu islem() methodunda herhangi bir hata ile karşılaşırsak catch bloğunda hatayı yakalayıp sarmallamak istiyoruz. method imzasını yanında throw Exception vererek diyoruz ki; islem() methodunun çağrıldığı yerde sarmallamayı zorunlu tut. Bu kodda yakaladığımız istisnayı loga basmak ya da herhangi başka bir işlem yaptırmak dışında kodun akışını kesip özel olarak kendi istisnamızı fırlatmak istiyoruz. Böyle bir durumda standart Exception fırlatmak yerine bu kod bloğundan hata aldığımı belirten özel bir exception fırlatmam daha doğru olacaktır. Sistemin fırlattığı istisna ile benim özel olarak fırlattığım istisnanın ayrı olması daha doğru bir yaklaşımdır. Uzun bir method içerisinde bu durumu daha iyi gözlemleyebilirsiniz. Sonuç olarak kodun yeni hali aşağıdaki şekilde olmalıdır.

public void islem() throws GenericException{
    try {
        int toplam = 5+4;
    }catch (Exception e){
        throw new GenericException("işlem methodu bir hata ile karşılaştı.", e);
    }
}

Kodu bu şekilde yazdığımız zaman code smell’in önüne geçmiş oluyoruz. Eğer Sonar gibi sistemlere entegre çalışıyorsanız zaten size bu uyarıyı verecektir. Sonar çıktısında der ki:

Using such generic exceptions as Error, RuntimeException, Throwable, and Exception prevents calling methods from handling true, system-generated exceptions differently than application-generated errors.
Noncompliant Code Example
public void foo(String bar) throws Throwable { // Noncompliant throw new RuntimeException(“My Message”); // Noncompliant }

public void foo(String bar) throws Exception { // Noncompliant throw new RuntimeException(“My Message”); // Noncompliant }

Aynı zamanda farkettiyseniz throw new GenericException(“işlem methodu bir hata ile karşılaştı.”, e); içerisinde kendime ait özel mesajımı da yazdım. Bu hata aldığınız kodun loglarınızı taratırken çok daha kolaylık sağlayacaktır.