[T.....i]

Connect Four to moja gra napisana w #java , proszę o #codereview bo nie ma kodu idealnego - a jak jest to na pewno nie jest on napisany przeze mnie ( ͡° ͜ʖ ͡°)/

#programowanie

[Endriu_]

@Tylko_Japonki:

nie ma kodu idealnego

Pewnie, że nie - w końcu to Java (✌ ﾟ ∀ ﾟ)☞
Zaraz zerknę ;)

[GotoFinal]

@Tylko_Japonki:

2017

brak mavena lub odpowiednika ;/

No ale nie ma dependów, to przynajmniej można uznać że zbędne tutaj.

No i nie robi się klas w default package, czyli bez package :<
(dalej nie przeglądałem jak na razie)

[fegwegw]

@Tylko_Japonki:

Nie stosuj wildcardów (*) przy importach.
Używaj camelCase do nazywania zmiennych.
Używaj klamer {} przy ifach i forach, nawet, jak instrukcje są jednolinijkowe.
Nie duplikuj kodu - jeśli kilka metod używa wspólnego kodu, postaraj się go wyabstrahować do osobnej metody.
Statyczne importy wrzuć na 'górę'.

[GotoFinal]

@Tylko_Japonki: w nazwach zmiennych nie używa się _, i nazwy wewnętrznych klas też powinny być z wielkiej litery w CamelCase

[T.....i]

@GotoFinal: O tym że tej "startowej klasy" nie można zostawić w default package to nie wiedziałem - dzięki.
@fegwegw: Teraz jak popatrzyłem na konwencję to nie wiem dlaczego ubzdurałem sobie, że nie mogę używać camelCase do zmiennych. Nad tą duplikacją staram się jeszcze pracować. Dzięki.

[GotoFinal]

@Tylko_Japonki:
https://github.com/Open96/ConnectFour/blob/master/gui/Game.java#L25
nie lepiej nextBoolean?

[T.....i]

@GotoFinal: Co do zmiennych to już wiem :) Co do klas wewnętrznych to w tym kodzie nie ma ani jeden wewnętrznej klasy, a przynajmniej tak mi się wydaje.

[GotoFinal]

@Tylko_Japonki: https://github.com/Open96/ConnectFour/blob/master/game/Field.java#L7 enum też klasa, nie dyskryminuj ( ͡º ͜ʖ͡º)

[T.....i]

@GotoFinal: O nextBoolean nie wiedziałem. A co do enuma to proszę nie bij (╯︵╰,)

[w.....j]

@Tylko_Japonki: przy duplikacji bardzo łatwo jest się kontrolować, jak coś kopiujesz i wjlejasz to masz pewność że możesz to zrobić bardziej genetycznie ale nie zawsze musisz.

[panczekolady]

Nie stosuj wildcardów (*) przy importach.

@fegwegw: W czym to przeszkadza przy prostych klasach?

[fegwegw]

@panczekolady: W robieniu sobie dobrych nawyków.