Nawet w Plusie można napotkać się na exception:
Ktokolwiek jest odpowiedzialny za strone serwerową serwisów Plusa, to pewnie zapomniał o paru flagach na serwerze. A na pewno o obłudze błędów!! Zostawianie e.printStackTrace() jest niezbyt pro… Z drugiej strony catch all na Exception e też nie jest fajnie…
Najgorsze jest, że takie małe błędy na stronach Plusa to normalka. Najlepszy jest e-bok. Full of zasadzkas. Ktoś musiał nieźle dostać w łapę że przyklepał taką aplikację na testach… Eh.. chyba trzebabyło skorzytać z Tomcata 🙂
„Zostawianie e.printStackTrace() jest niezbyt pro…” – a potem to nie wiadomo gdzie co się sypie, zastosowanie rozwiązania jest jak najbardziej profesjonalne, z tym że powinni wyświetlić jedynie stronę o tym, iż w systemie wystąpił błąd, zaś zrzut stosu odpisałby się im tylko w logach serwera, ale widać z jakiś przyczyna tego nie zrobili. Wyjątki zawsze powinny być przechwytywane i odpisywane, jest to tak powszechna praktyka w projektach jak papier toaletowy. Opisuje się ją przy okazji każdej książki związanej z refaktoryzacją i dobrym stylem kodowania. Sam w praktyce poprawiałem już kod z pustym try{ }catch{ } bardzo fajnie szuka się wtedy przyczyny błędu, pakowanie komunikatów logerra w ten blok, też jest chybionym pomysłem – robimy niepotrzebnie coś co jest już gotowe na wejściu.