Ocena aplikacji Java, swing.

0

Witam,
Chciałbym pokazać mój projekt Pogody w java swing. Moim celem było stworzenie aplikacji pobierającej XML z danymi pogodowymi dostarczonych przez yahoo. Parsowanie danych by wyciągnąć z nich informacje (data, temperatura, wschody i zachody słońca) następnie wyświetlanie ich w przystępnej formie. Ponadto aplikacja musi wyświetlać interpolację temperatur.
Chciałbym zebrać wasze informację co mógłbym poprawić w niej od strony kodowej, co zrobiłem źle, a co zrobiłem dobrze. Pomoże mi to w nauce oraz ulepszy moją aplikację.
https://github.com/Artekat/Pogoda

0

Witaj, na pierwszy rzut oka:

  1. Brakuje obsługi wyjątków (try,catch,finally)
  2. Zamiast ciąć stringi przy pomocy String.substring lepiej użyć wyrażeń regularnych
  3. Użycie ikony w ,,O Autorze" będzie działać tylko z Twoim komputerem - poczytaj o wczytywaniu obrazków z zasobów
1
  1. nazewnictwo (np. nazwa klasy GRU - nie wiadomo co ona ma robić? )
  2. za długie metody, mało czytelne
  3. poczytaj o IOC
  4. naucz się korzystać z kolekcji i bibliotek np. Guava, JodaTime i coś do parsowania XMLi, bo te metody PobieraneDane::dataa i PobieraneDane::dzien,
    to wtf.
0

Bardzo przepraszam ze nie udzielałem się w poście aczkolwiek pochłonęły mnie studia.
Ciesze się ze dostałem dobry feedback dotyczący mojej aplikacji, dziękuję za poświęcony czas. Już zaczynam uaktualniać moją aplikację pogodową. Oczywiście zaprezentuję ją tutaj gdy już skończę. Postaram się również przedstawić co poprawiłem by łatwiej pokazać zmiany. Jeszcze raz dziękuję i proszę o uzbrojenie się w cierpliwość ;)

0

Na wstępie chciałbym powiedzieć ze nie wdrożyłem wszystkich rzeczy o których napisaliście.
Rzeczy które wdrożyłem to:

  1. Obsługa wyjątków - Wyrzucam MessageDialog i zamykam aplikację. Wiem ze jest wiele lepszych zastosowań takie jak np zastąpienie pobieranych danych przykładowymi. Zdecydowałem się na ten krok ponieważ jest to dość prosty i zarazem skuteczny sposób.
  2. Wczytywanie obrazu z zasobów - Teraz logo pobierane jest z zasobów, dołączyłem obraz do plików gita.
  3. Nazewnictwo klas i metod - Zdecydowałem się przejść na język angielski, nie wiem czy zrobiłem dobrze ponieważ ciężej jest mi operować innym językiem niż ojczysty.
  4. Skrócenie oraz poprawienie czytelności metod - Podczas skracania miałem wiele problemów z czytelnością własnego kodu, co potwierdza tylko wasze wątpliwości w tej sprawię. Teraz według mnie kod zyskał na czytelności. Podczas tego zabiegu również skróciłem/podzieliłem niektóre metody.

Niestety nie wdrożyłem tych według mnie ważniejszych rzeczy czyli między innymi parsowanie xml. Czas który poświęcił bym na tą "technologię" wolę wykorzystać do napisania aplikacji pod Androida.

Poproszę o informacje zwrotną co do wprowadzonych zmian oraz całej aplikacji. Ponadto mam dodatkowe pytanie: Czy taką aplikacje mogę pokazać przy rekrutacji na praktyki lub (pierwszej) pracy ? Wiem ze nie jest to jakiś szczyt technologi ale jak to mówią "done is better than perfect".

https://github.com/Artekat/Weather

Dziękuję również za przeniesienie dyskusji do odpowiedniego działu.

1 użytkowników online, w tym zalogowanych: 0, gości: 1