Wpis z mikrobloga

Mam pytania do mojego kodu w #java - napisałem sobie prosty monitor domen (czy odpowiadają i jakim statusem). Kod jest dostępny na GH: https://github.com/masterix/domain-monitor
1) https://github.com/masterix/domain-monitor/blob/master/src/MonitorApp.java czy te bloki try/catch nie lepiej byłoby połączyć w całość i rozbić tylko akcję w catch ze względu na wyłapywany wyjątek? Teraz tak bez sensu jest trochę zmienna domainList przed pierwszym blokiem try/catch, żeby była widoczna w nastepnym
2) Chcę stworzyć powiadamianie dla niektórych statusów np 404/503 i leci do mnie SMS, 2xx nic się nie dzieje a pozostałe wywołują wysłanie maila i mam z tym problem koncepcyjny. Stworzyłem NotificationManagera + interface NotificationSenderInterface - coś jak observer. Intuicyjnie to Manager powinien decydować, kto śle wiadomość, ale przy implementacji łatwiej chyba, jeśli Manager powiadomi Senderów i oni decydują na podstawie obiektu Domain i statusu, czy słać powiadomienie czy nie. Komunikat też musi być różny (mailem można posłać dłuższą treść, ale w SMS już tylko domena + status) - osadzić to w osobnych plikach z szablonami? Taki Sender powinien też mieć dostep do statusu domeny, więc w interface wymuszam obecność metody setDomain( https://github.com/masterix/domain-monitor/blob/master/src/mx/NotificationSenderInterface.java ) - jednak to tak nie do końca pasuje, bo gdy będę chciał informować o czymś innym to ten Sender będzie uwiązany do konieczności posiadania domeny. Czy to powinienem jeszcze bardziej rozbić na kolejne klasy? Gdzie Sender jest tylko dodatkowym dekoratorem(?) dla klasy, która faktycznie wysyła SMS/mail?
3) poproszę o #codereview jeśli ma ktoś chwilę
  • 3