[krasnoludkolo]

Hej
Prośba o codereview. Piszę na zaliczenie grę w statki.

Opis ogólny co gdzie i jak:

Pakiety:
ai-boty
game - publiczne api modułu gry
game.logic - implementacja interface gry
view - nieśmieszny żart, czyli moja próba podpięcia javyFX do projektu (nie używam za bardzo FX dlatego wyszło jak wyszło...)

Najbardziej zależy mi na ocenie części "serwerowej" czyli samej logiki gry. Czy ma sens i czy trzyma się jakoś kupy.

Link do gita https://github.com/krasnoludkolo/Battleship

#codereview #java #programowanie #naukaprogramowania

[sorhu]

@krasnoludkolo
Lombok. Meh.

[krasnoludkolo]

@sorhu: tylko w jakiś klasach bez logiki

[sorhu]

@krasnoludkolo
forEach(elements::add)

Zrobiłbyś collecta.

[A.....t]

@sorhu co jest złego w lomboku?

[sorhu]

@krasnoludkolo
W Ship zrobiłbyś set elementów, żebyśmy nie musiał iterować po wszystkich za każdym razem jak szukasz współrzędnych.

[sorhu]

@AwizisieAkat
To że używa nieoficjalnych haków i to, że w nowszych wersjach Javy nie będzie działał.

[krasnoludkolo]

@sorhu:

Zrobiłbyś collecta.

faktycznie niedopatrzenie

W Ship zrobiłbyś set elementów, żebyśmy nie musiał iterować po wszystkich za każdym razem jak szukasz współrzędnych.

Nie pomyślałem, faktycznie (btw czy zmienia to coś poza czytelnością?)

[krasnoludkolo]

Do niektórych rzeczy można sie przyczepić, ale nie ma większego wpływu.

@BosmanPociagowy: tzn? ;)

[krasnoludkolo]

W Ship zrobiłbyś set elementów, żebyśmy nie musiał iterować po wszystkich za każdym razem jak szukasz współrzędnych.

@sorhu: jednak zmieniam, ważne jest który element jest pierwszy, który ostatni. Łatwiej chyba raz zapamiętać tę informację niż przy każdym wyświetlaniu sprawdzać które są krawędziami (docelowo chodzi o wyświetlanie innych grafik na końce statku)

[sorhu]

@krasnoludkolo

Nie pomyślałem, faktycznie (btw czy zmienia to coś poza czytelnością?)

Szybkość działania, choć musiałbyś ogromną planszę mieć, żebyś zauważył różnicę.

A nawet jeśli nie chcesz seta, to lista na metodę contains. Nie trzeba z ręki pętli pisać.

[Andrew7642]

@krasnoludkolo: Ale wyświetlanie innych grafik to jest aspekt wyświetlania, nie logiki. Nie mieszaj tego.

[sorhu]

@krasnoludkolo
Jak kolejność robi Ci różnicę, to użyj LinkedHashSet.

[krasnoludkolo]

@BosmanPociagowy:
@Andrew7642:

Tylko co w zamian? każdorazowo klient powinien obliczać które pola są krawędziami statku? Czy zmienić zupełnie reprezentację danych w odpowiedz?

[Andrew7642]

@krasnoludkolo: Obliczaj, to jest na tyle statyczna i prosta gra że nie ma to żadnego wpływu na wydajność. Wyobraź sobie ile kodu musisz wypatroszyć jeśli zmienisz sposób wyświetlania (np będziesz chciał komin dać na środku statków długich na 5 pól).

[krasnoludkolo]

Modułowość troche słabo rozbita, a przynajmniej nazewnictwo ich.

@BosmanPociagowy: co przez to rozumiesz?

[krasnoludkolo]

@BosmanPociagowy: Wydzieliłem pakiet game gdzie jest publiczne api od gry (i tylko to widzą klienci) a sama wewnętrzna implementacja jest w game.logic. O to chodzi czy coś innego?