Java project - Code review

pytanie zadane 25 stycznia 2021 w Nasze projekty przez Bakkit Dyskutant (7,600 p.)
zamknięte 17 marca 2021 przez Bakkit

Witam. Proszę o ocenę kodu mojego projektu napisanego w języku Java.

To mój pierwszy projekt w tym języku, jest to plugin na drop na serwer gier Minecraft, jeśli ktoś nie zna biblioteki bukkit, a zna dość dobrze Javę to myślę, że i tak jest w stanie wyłapać jakieś błędy i coś doradzić.

Plugin przetestowałem i działa tak jak powinien.

Z rzeczy do poprawki, które sam wyłapałem to utworzenie config.yml ręcznie, a nie tak jak jest to zrobione teraz czyli w głównej klasie, zmiany będą dodawane do kolejnych wersji projektu.

Projekt: Github

komentarz zamknięcia: Rozwiązanie problemu.

2 odpowiedzi

odpowiedź 26 stycznia 2021 przez Subtelny Początkujący (480 p.)
wybrane 17 marca 2021 przez Bakkit

Najlepsza

Poczytaj o pryncypiach SOLID'u, code smellach oraz levelach abstrakcji. Niestety ale projekt ubolewa niemiłosiernie pod tym względem.

komentarz 26 stycznia 2021 przez Wiciorny Ekspert (270,110 p.)

też nie chciałem tego wprost pisać, myślałem " że podstawowe rzeczy sa zachowane" ale trudno mówić o code-review jak projekt pisany :D "jak na studiach" ... gdzie semantyka kodu podstawowa już kuleje a co dopiero mowa o jakichkolwiek wzorcach, coś niby gdzieś tam dzwoni- ale nie w tym kościele

komentarz 27 stycznia 2021 przez Bakkit Dyskutant (7,600 p.)

@Subtelny, dzięki za opinie, mógłbym prosić o linki do takowych źródeł wiedzy?

odpowiedź 26 stycznia 2021 przez Wiciorny Ekspert (270,110 p.)

Najpierw zadbaj o podstawy kodu: semantyke, potem proś kogoś o code-review, bo żeby tracić czas na CR - stosować "wzorce" czy zastosowania, najpierw ... wypadałoby trzymać się chociaż paradygmatów podstawowych programowania obiektowego, czy programowania w javie z zachowaniem np - bardziej deklaratywnego sposobu pisania niż imperatywnego, co bardziej pójdzie w strone funkcyjnego pisania w Javie, ale z kolei - pozwoli na przejrzystosć kodu, czy też stosowanie pragmatycznego i "języka prozy" aby łatwiej było zrozumieć.
Dodatkowo jak się na coś decydujesz, to korzystaj z bibliotek już "dla konkretnej " wersji Javy, a nie tak, że w jednych klasach masz Utils z jdk 8 np, a w innych zastosowania z JDK 11- ja wiem, że akurat to jest :D spowodowane brakiem wiedzy na tej płaszczyźnie.

komentarz 27 stycznia 2021 przez Bakkit Dyskutant (7,600 p.)

Dziękuję za opinie. Mógłbyś wskazać chociaż jeden kawałek kodu gdzie coś byś dodał / zmienił albo całkowicie usunął?

komentarz 27 stycznia 2021 przez Bakkit Dyskutant (7,600 p.)

@Wiciorny, kurczę, liczyłem się z tym, że kod jest słaby, bo wiadomo to początki, ale liczyłem również, że ktoś mi wskaże bezpośrednio miejsca gdzie popełniłem błąd: "Tutaj zamień to na to, w tym miejscu możesz to zrobić inaczej, to usuń jest niepotrzebne". Jeśli masz czas i chęci to prosiłbym o to :D

Nie mówię oczywiście o całym projekcie ale chociaż 2-3 miejscach występowania błędu.