Mały projekt do oceny

0

Cześć. Napisałem swój pierwszy jako taki projekt i chętnie dowiem się co zrobiłem dobrze, a co źle oraz jakie inne funkcjonalności można jeszcze do niego dodać.
Projekt

0

W zasadzie nie ma co oceniać... kilka getterów i jeden kontroler.

2

Jw. ten projekt jest za mały, żeby dać jakikolwiek feedback, ale... szukałem testów i: https://github.com/Tobiasz126/currecncy-exchange-application/blob/master/src/test/java/com/example/cea/CurrecncyExchangeApplicationTests.java

1

Tak na szybko :

  1. Wstrzykuj przez konstruktor
  2. Wywal logikę z kontrolerów
  3. Po co ci settery skoro z nich nawet nie korzystasz ? Zmieniłbym na obiekt immutable z samymi getterami.
  4. Dodaj obsługę błędów. Co jeśli api, do którego strzelasz jest np. niedostępne ?
  5. Nie dzieliłbym "projektu" na warstwy typu kontrolery. Po to w Javie domyślnie masz package scope by ukrywać klasy, a dopisywać public tylko jak jest taka potrzeba.
    6*. @RequestMapping( method = RequestMethod.GET) zamieniłbym na @GetMapping
0

Witam

Chciałbym się podpiąć tematycznie pod wątek, co prawda w nieco innym języku, ale liczy się idea:

Jakiś czas temu udało mi się skończyć mały projekt skryptu,który symuluje scrollowanie strony w javascript. Pomysł w sumie zaczerpnięty z oferii, ale udało sie chyba całkiem nieźle wykonać. W sumie spędziłem nad tym ok 20h.

Skrypt zawiera 3 różne animacje , obrazki pobrane z banków. Skrypt , style oraz szkielet strony jest osobno.
Postanowiłem wystawić go na sprzedaż za niewielką sume, ale nie jestem pewien czy zbyt wiele sobie za niego nie żądam.

Dokonałem tez niewielkiej analizy tego typu skryptów poprzez codecanyon.
Co na razie wystawiłem na allegro olx, ale planuje może jeszcze na dwóch. Cena wywoławcza 100 zł choć teraz obniżone do 80.
Ofert kupna na razie nie ma więc nie wiem co ew źle zrobiłem.

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