Prośba o sugestie

Prośba o sugestie
R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0

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ę

edytowany 1x, ostatnio: rafal20-1988
Phestek
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 7 lat
  • Postów:89
0

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

Zobacz pozostały 1 komentarz
Phestek
@scibi92 co to za Linux, jak to zakamuflowane Ubuntu :D
S9
A no tak przecież jedyny trve linux to Arch PS. To prawda
Phestek
@karolinaa w końcu znalazła kogoś, kto ją zrozumie :D
bogdans
@Phestek, jak kompilujesz w terminalu i nie zgadza się kodowanie, to javac -encoding utf-8 Board.java.
Phestek
@bogdansa karamba, jaka ta Java skomplikowana. Dzięki za radę :D
R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0

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

LukeJL
  • Rejestracja:około 11 lat
  • Ostatnio:mniej niż minuta
  • Postów:8409
0
  • 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...


edytowany 4x, ostatnio: LukeJL
S9
  • Rejestracja:ponad 10 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Warszawa
  • Postów:3573
0

A w jakim IDE piszesz w ogóle?


"w haśle <młody dynamiczny zespół> nie chodzi o to ile masz lat tylko jak często zmienia się skład"
R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0

Eclipse

z nazwami to wiem że mam bałagan.

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

edytowany 1x, ostatnio: rafal20-1988
S9
  • Rejestracja:ponad 10 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Warszawa
  • Postów:3573
0

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


"w haśle <młody dynamiczny zespół> nie chodzi o to ile masz lat tylko jak często zmienia się skład"
Burdzi0
  • Rejestracja:prawie 9 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Futurama
  • Postów:887
0

Wrzuć Runnable Jara jak możesz


Bite my shiny metal ass!
Life throws you an error code like that, you don't have the luxury of a ZnVja2luZw== pop-up explanation *Robię projekty studenckie, pisz priv ;) *
edytowany 1x, ostatnio: Burdzi0
bogdans
Moderator
  • Rejestracja:prawie 17 lat
  • Ostatnio:prawie 5 lat
0

@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?


To smutne, że głupcy są tak pewni siebie, a ludzie mądrzy - tak pełni wątpliwości. Bertrand Russell
R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0

ok po za Stringami.

R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0
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

S9
  • Rejestracja:ponad 10 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Warszawa
  • Postów:3573
0

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ąć ;)


"w haśle <młody dynamiczny zespół> nie chodzi o to ile masz lat tylko jak często zmienia się skład"
LukeJL
  • Rejestracja:około 11 lat
  • Ostatnio:mniej niż minuta
  • Postów:8409
0

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 :)


R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0
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.

R1
  • Rejestracja:prawie 10 lat
  • Ostatnio:prawie 5 lat
  • Postów:208
0

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.

Phestek
Zmienne powinny mieć jak najmniejszy zasięg.
R1
ale nie pogarsza to czytelności kodu w sytuacji kiedy deklaracje są w różnych miejscach? nie jest czytelniej gdy wszystkie zmienne są w jednym miejscu?
Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)