Battleships - ocena aplikacji

Odpowiedz Nowy wątek
2019-11-03 23:10
0

Witam,
zaczynam naukę javy i chciałbym, żeby ktoś ocenił mój kod.
Do zrobienia jeszcze HardComputerPlayer, ale to powolutku będę rozwijał
Prosta konsolowa aplikacja do gry w statki
Link https://github.com/arturdziocha/battleship
Proszę o porady odnośnie poprawek
Pozdrawiam

edytowany 3x, ostatnio: furious programming, 2019-11-03 23:34
możesz wrzucić do repo gradle wrappera i wtedy nie trzeba samemmu instalować ;) - danek 2019-11-04 07:50
Czy wystarczy gradlew i gradle.bat? Czy jeszcze coś dodać? - adziocha 2019-11-04 08:34
cały folder gradle jeszcze, bo tam to siedzi całe ;) - danek 2019-11-04 08:35
jeśli uda Ci się zbudować przez ./gradlew build to znaczy ze masz wszystko - danek 2019-11-04 08:35

Pozostało 580 znaków

2019-11-04 08:47
0

Posypało się przy /.gradlew build
Mam zainstalowanego gradla i jak uruchamiam gradle run/gradle clean test wszystko działa.

$ ./gradlew build
Could not unzip C:\Users\P\.gradle\wrapper\dists\gradle-4.10.3-bin\31t79e2qsceia4mkbojplrgx\gradle-4.10.3-bin.zip to C:\Users\P\.gradle\wrapper\dists\gradle-4.10.3-bin\31t79e2qsceia4mkbojplrgx.
Reason: error in opening zip file
Exception in thread "main" java.util.zip.ZipException: error in opening zip file
        at java.util.zip.ZipFile.open(Native Method)
        at java.util.zip.ZipFile.<init>(Unknown Source)
        at java.util.zip.ZipFile.<init>(Unknown Source)
        at java.util.zip.ZipFile.<init>(Unknown Source)
        at org.gradle.wrapper.Install.unzip(Install.java:218)
        at org.gradle.wrapper.Install.access$600(Install.java:27)
        at org.gradle.wrapper.Install$1.call(Install.java:75)
        at org.gradle.wrapper.Install$1.call(Install.java:48)
        at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:69)
        at org.gradle.wrapper.Install.createDist(Install.java:48)
        at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:107)
        at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
edytowany 1x, ostatnio: adziocha, 2019-11-04 08:49

Pozostało 580 znaków

2019-11-04 10:50
0

Już działa.
Trzeba było upgradować wrapera na wyższą wersję

$ gradle wrapper --gradle-version 5.6.1 --distribution-type all
$ ./gradlew build
$ ./gradlew run

Teraz bardzo bym prosił o ocenę i ewentualne jechanie co jest źle i do poprawy.
Pozdrawiam

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0