Prośba o sugestie

Witam,

wiem że jakiś czas temu był podobny wątek, ale uznałem że założę swój wątek aby nie wprowadzać zamieszania z tamtym.

Od jakiegoś czasu pisałem sobie gierkę w celach edukacyjnych i utrwalających wiedzę. Chciałbym o ile to możliwe aby osoby bardziej doświadczone rzuciły na to okiem i podpowiedziały mi co trzeba ewentualnie poprawić, gdzie są ewentualne błędy, błędne nazewnictwo zmiennych, metod klas itp (według konwencji jakich powinno się używać w javie. Kolejna rzecz, która mnie zastanawia to fakt czy napisana przeze mnie gra została napisana zgodnie z obiektywistycznym podejściem, czytelność kodu itp.

Jeżeli jakiegoś potrzebnego pliku nie dodałem proszę o informację.

Za wszelkiego rodzaju sugestie dziękuje.

edit.

spacja - strzelanie
strzałki poruszanie się

Faktem jest, że nie umiem w Javę, ale raczej nie powinieneś używać polskich znaków w komentarzach.

Teoretycznie nie powinno się w ogóle używać języka polskiego w kodzie. Będę musiał to poprzerabiać w przyszłości.

• Masz klasę Spirte zamiast Sprite
• Nie dość, że masz polskie nazwy zmiennych/klas etc. to w dodatku pisane z dysleksją (delta_losowani?? move_przeciwniko??)
• pomieszane nazwy angielskie i polskie w jednej zmiennej: getTyp
• błąd w nazwie angielskiej: masz vin i drawVin zamiast win i drawWin
• getImageDimensions sugeruje, że jest to coś w rodzaju gettera, czyli że powinno zwracać wymiary obrazka (tak jak masz getX i getY). Natomiast w kodzie widzę, że funkcja ta... ustawia wymiary obrazka (czyli pewnie setImageDimensions powinno być)
• kryptyczne nazwy zmiennych (vis? getVis - co to jest? visible? dla mnie to niepotrzebny skrót).
• pomieszanie nazewnictwa under_score z camelCase, czyli masz start_najechany ale już stopNajechany

Czyli masz totalny bałagan w nazwach zmiennych. Radziłbym wyłączyć (tak, wyłączyć) autocomplete w IDE, bo mam wrażenie, że IDE ci przeszkadza, bo pozwala tolerować błędy (np. zrobiłeś literówkę w klasie Spirte, a potem IDE zapewne ci podpowiedziało w innym pliku również Spirte... i wszystko jest okej, kod się uruchamia, mimo że wszędzie jest jeden wielki burdel...).

Jakbyś pisał bez autouzupełniania i byś musiał z palca wpisywać, to byś się szybko nauczył tworzyć spójne i ładne nazwy zmiennych...

A w jakim IDE piszesz w ogóle?

Eclipse

z nazwami to wiem że mam bałagan.

a co poza tym? jeżeli chodzi o składnie?

Do Javy SE masz za friko IntelliJ Community więc sugeruje się przesiąść
M.in. podpowiada nazwy referencji ładnie ;)

Wrzuć Runnable Jara jak możesz

@rafal20-1988 napisał:

Teoretycznie nie powinno się w ogóle używać języka polskiego w kodzie. Będę musiał to poprzerabiać w przyszłości.

Co Ty opowiadasz? A wewnątrz Stringów?

ok po za Stringami.

scibi92 napisał(a):

Do Javy SE masz za friko IntelliJ Community więc sugeruje się przesiąść
M.in. podpowiada nazwy referencji ładnie ;)

od dłuższego czasu piszę pod eclipse, czytałem już w paru miejscach że nie jest polecany eclipse ale jakoś na IntelliJ nie mogę się przestawić.

Burdzi0 napisał(a):

Wrzuć Runnable Jara jak możesz

gotowe

Jakie masz problemy z Intellij?
Ja też nie mogłem się przestawić ale uwierz mi że naprawde warto :)
Poza tym w firmach z tzw. RIGCzem stosują IntelliJ więc jeśli chcesz programować komercyjnie to wypada ogarnąć ;)

IntelliJ ma inspekcję literówek w kodzie (tj. czy nazwy zmiennych, klas itp. są poprawnymi słowami w języku angielskim), więc by ci podkreśliło falowaną linią to nieszczęsne spirte czy vin :)

scibi92 napisał(a):

Jakie masz problemy z Intellij?
Ja też nie mogłem się przestawić ale uwierz mi że naprawde warto :)
Poza tym w firmach z tzw. RIGCzem stosują IntelliJ więc jeśli chcesz programować komercyjnie to wypada ogarnąć ;)

Fajnie by było zmienić pracę na programistę ale do tego to mi bardzo wiele brakuje co widać na załączonym obrazku:) Obecnie trzeba się trzymać pracy jaką mam :)

LukeJL napisał(a):

IntelliJ ma inspekcję literówek w kodzie (tj. czy nazwy zmiennych, klas itp. są poprawnymi słowami w języku angielskim), więc by ci podkreśliło falowaną linią to nieszczęsne spirte czy vin :)

w eclipse też jest słownik który można teoretycznie odpalić:) chyba to nie jest jedyny "powód" dla jakiego warto przesiadać się na IntelliJ.

zmienne prywatne w klasach powinny być deklarowane jak najbliżej miejsca wykorzystania czy lepiej deklarować je wszystkie np. na początku aby były w jednym miejscu?

Pytam bo zainstalowałem IntelliJ i tam twardo chce mi poprzenosić zmienne prywatne w miejsca gdzie to zmieniam.