[Bruno_]

#java #naukaprogramowania

Proszę o jakiś code review (｡◕‿‿◕｡)
link do repo

Chyba mój ostatni lub przedostatni projekt w javafx (w planach mam jeszcze game of life ale to może już w javascripcie). Teraz lecę już w Springa ᕦ(òóˇ)ᕤ

[63274682374]

@Bruno_:
- dużo magic numbers
- niefinalne, niestatyczne pole w klasie ColumnPair.X
- "private static int points = 0;" niepotrzebnie static
- Birbo.animationState przyjmuje tylko dwie wartości, zmień na boolean (albo enum jeśli będzie więcej stanów)

[doubt]

@Bruno_:
- kod w pakiecie domyślnym to antypattern: https://stackoverflow.com/questions/7849421/is-the-use-of-javas-default-package-a-bad-practice
- Columns jest wątkiem, a nigdzie z tego nie korzystasz(?)
- w forze w Columns (24 linia) masz jakąś logikę, że "jeżeli cośtam na i-tym elemencie to usuń i-ty element" a zaraz potem znowu getujesz i-ty element z kolekcji (czyli możliwe że inny). Potencjalnie może tu polecieć IndexOutOfBoundsException

[Bruno_]

@doubt: @63274682374: troche popoprawialem
Mam jeszcze pytanie co do pic rel, ktorej wersji z tych dwoch uzywac?
nie da rady po prostu return condition bo musze ziterowac po wszystkich elementach, czyli albo robie lokalna zmienna albo mam dwa razy return
jeszcze chyba z interfejsem mozna to zrobic ale to nie chcialo mi sie bawic w to xD

[radoslaw_rumian]

@Bruno_: ani pierwszy, ani drugi:

public boolean isPointScored() {
 return columnPairs.stream()
 .filter( cp -> cp.getX() == bitbPosX)
 .anyMatch()
}

[Bruno_]

@radoslaw_rumian no to to jest to co miałem na myśli mówiąc o interfejsie (lambda) XD jutro jeszcze rzucę okiem