cagdasgerede / JavaCheckers

Checkers game written in Java language.
https://sloenthran.pl
GNU General Public License v3.0
0 stars 0 forks source link

I just added the menu #8

Closed ozanmujde closed 3 years ago

ozanmujde commented 3 years ago

I added the Menu feature with using java awt

cagdasgerede commented 3 years ago

Bu kadar mı yapacağın değişiklikler?

Eğer öyleyse bir demo videosu da ekle buraya.

ozanmujde commented 3 years ago

Hocam 3 buton istenmişti onları ekledim ama eğer başka bir şey eklememi isterseniz ekleyebilirim demo videosundan önce.

cagdasgerede commented 3 years ago

Daha bir kaç kez daha git gel olacak burada. O yüzden şu ana kadar yaptıklarını demo et. Sonra yeniden video çekmen gerekecek.

ozanmujde commented 3 years ago

https://youtu.be/7EfboHxYHdU son commiti çok küçük bir değişiklik diye buraya yüklemedim, başka değişikliklerle eklerim diye düşündüm.

ozanmujde commented 3 years ago

Hocam merge edeyim mi kodu ?

ozanmujde commented 3 years ago

Hocam bir kaç eklenti daha yapmak istiyorum ama onları finaller 17 sinde bitiyor ondan sonra yapsam sıkıntı olur mu?

ozanmujde commented 3 years ago

Hocam pazartesi perşembe arası final haftasıda perşembe dediğiniz değişiklilere başlasam sıkıntı yaratır mı?

cagdasgerede commented 3 years ago

Ozan Perşembe'ye kadar bekleyemeyiz.

ozanmujde commented 3 years ago

Hocam söylediğiniz şeyleri değiştirdim videoyuda atıyorum şimdi kod açıklamasıyla

ozanmujde commented 3 years ago

https://youtu.be/q-RcErX6Clg yeni videom yayında.

ozanmujde commented 3 years ago

Kod içerisinde bıraktığım notlara göre kodunu güncelle.

Ayrıca kodunda herhangi bir test yok. Yeni eklediğin davranışlar ile ilgili testler ekle.

Değişiklileri yaptım 1 2 yerde ekstra space errora sebep oldum ama onun için commit atmak istemedim. Test caseleri ekleyeceğim hocam. bu arada yeni eklememi istediğiniz bir özellik ya da değişmesini istediğiniz bir şey var mı? Saygılarımla.

ozanmujde commented 3 years ago

Hocam test bizim dosyada otomatik olarak yok.Diğer arkadaşlarla çelişmemesi için belli bir formatta mı olmalı?

ozanmujde commented 3 years ago

Hocam bir de benim eklediğim özellik ana kodun private datasını kullanan bir method private değişkenler üzerinde test case i tam olarak nasıl yazmam gerekiyor? Bunların dışınca tek bir classın bir methodunu dependencies hatası yüzünden test edemiyorun çok test yazmak gerekiyorda test işini sonra yapıp önce eklentileri eklesem süre dolayısıyla

ozanmujde commented 3 years ago

`(base) [ozan@ozan-20hns03t00 JavaCheckers]$ gradle test

Configure project : Project : => no module-info.java found

Task :test java.lang.instrument.IllegalClassFormatException: Error while instrumenting sun/util/resources/cldr/provider/CLDRLocaleDataMetaInfo. at org.jacoco.agent.rt.internal_43f5073.CoverageTransformer.transform(CoverageTransformer.java:94) at java.instrument/java.lang.instrument.ClassFileTransformer.transform(ClassFileTransformer.java:246) at java.instrument/sun.instrument.TransformerManager.transform(TransformerManager.java:188) at java.instrument/sun.instrument.InstrumentationImpl.transform(InstrumentationImpl.java:563) at java.base/java.lang.ClassLoader.defineClass2(Native Method) at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1108) at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:183) at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:784) at java.base/jdk.internal.loader.BuiltinClassLoader.findClassInModuleOrNull(BuiltinClassLoader.java:705) at java.base/jdk.internal.loader.BuiltinClassLoader.findClass(BuiltinClassLoader.java:586) at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:634) at java.base/java.lang.Class.forName(Class.java:546) at java.base/java.util.ServiceLoader.loadProvider(ServiceLoader.java:854) at java.base/java.util.ServiceLoader$ModuleServicesLookupIterator.hasNext(ServiceLoader.java:1078) at java.base/java.util.ServiceLoader$2.hasNext(ServiceLoader.java:1301) at java.base/java.util.ServiceLoader$3.hasNext(ServiceLoader.java:1386) at java.base/sun.util.cldr.CLDRLocaleProviderAdapter$1.run(CLDRLocaleProviderAdapter.java:89) at java.base/sun.util.cldr.CLDRLocaleProviderAdapter$1.run(CLDRLocaleProviderAdapter.java:86) at java.base/java.security.AccessController.doPrivileged(AccessController.java:554) at java.base/sun.util.cldr.CLDRLocaleProviderAdapter.(CLDRLocaleProviderAdapter.java:86) at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:64) at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500) at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:481) at java.base/sun.util.locale.provider.LocaleProviderAdapter.forType(LocaleProviderAdapter.java:188) at java.base/sun.util.locale.provider.LocaleProviderAdapter.findAdapter(LocaleProviderAdapter.java:287) at java.base/sun.util.locale.provider.LocaleProviderAdapter.getAdapter(LocaleProviderAdapter.java:258) at java.base/java.util.Calendar.createCalendar(Calendar.java:1693) at java.base/java.util.Calendar.getInstance(Calendar.java:1661) at java.base/java.text.SimpleDateFormat.initializeCalendar(SimpleDateFormat.java:677) at java.base/java.text.SimpleDateFormat.(SimpleDateFormat.java:621) at java.base/java.text.SimpleDateFormat.(SimpleDateFormat.java:600) at org.gradle.process.internal.worker.child.SystemApplicationClassLoaderWorker.getLastResortErrorLogFile(SystemApplicationClassLoaderWorker.java:162) at org.gradle.process.internal.worker.child.SystemApplicationClassLoaderWorker.call(SystemApplicationClassLoaderWorker.java:111) at org.gradle.process.internal.worker.child.SystemApplicationClassLoaderWorker.call(SystemApplicationClassLoaderWorker.java:71) at worker.org.gradle.process.internal.worker.GradleWorkerMain.run(GradleWorkerMain.java:69) at worker.org.gradle.process.internal.worker.GradleWorkerMain.main(GradleWorkerMain.java:74) Caused by: java.io.IOException: Error while instrumenting sun/util/resources/cldr/provider/CLDRLocaleDataMetaInfo. at org.jacoco.agent.rt.internal_43f5073.core.instr.Instrumenter.instrumentError(Instrumenter.java:159) at org.jacoco.agent.rt.internal_43f5073.core.instr.Instrumenter.instrument(Instrumenter.java:109) at org.jacoco.agent.rt.internal_43f5073.CoverageTransformer.transform(CoverageTransformer.java:92) ... 37 more Caused by: java.lang.IllegalArgumentException: Unsupported class file major version 59 at org.jacoco.agent.rt.internal_43f5073.asm.ClassReader.(ClassReader.java:195) at org.jacoco.agent.rt.internal_43f5073.asm.ClassReader.(ClassReader.java:176) at org.jacoco.agent.rt.internal_43f5073.asm.ClassReader.(ClassReader.java:162) at org.jacoco.agent.rt.internal_43f5073.core.internal.instr.InstrSupport.classReaderFor(InstrSupport.java:280) at org.jacoco.agent.rt.internal_43f5073.core.instr.Instrumenter.instrument(Instrumenter.java:75) at org.jacoco.agent.rt.internal_43f5073.core.instr.Instrumenter.instrument(Instrumenter.java:107) ... 38 more

BoardTest > trying() FAILED java.lang.ExceptionInInitializerError at BoardTest.java:51 Caused by: java.lang.RuntimeException at BoardTest.java:51

1 test completed, 1 failed

Task :test FAILED

FAILURE: Build failed with an exception.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0. Use '--warning-mode all' to show the individual deprecation warnings. See https://docs.gradle.org/6.7/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 2s 4 actionable tasks: 1 executed, 3 up-to-date`

ozanmujde commented 3 years ago

Son attığım kod bu hatayı veriyor hocam gerçekten uğraşıp çözemediğim için soruyorum daha fazla uğraşırsam yetişemyecek gibi.Her yerde static kullanılmış staticlerde test çalışmıyor hocam internette bir cevap bulamadım.En son kendim restart için ayrı class açıp deneyeyim dedim ama static şeylerde direkt hata atıyor methodlar daha teste çağırılırken hata atıyor hatta Design'i mocklamaya çalışıtığımda bile hata alıyorum

ozanmujde commented 3 years ago

Hocam javafx kullanan design,board gibi methodlara test yazılmıyordu.O yüzden restart methoda ayrı class açtım ona test yazdım.Yazılan EndGame classı pek iyi çalışmıyordu zamanla ona da evrilebilir.Ama test etmek için yazılan bir class aslında.Test için de ayrı bir method yazdım içinde design olmayan ama aynı method design testte toolbox exception atıyordu dependencies den dolayı çözmek hem sıkıntıydı.removePawn zaten adamın yazdığı bir method olduğundan onları test etmek gerek yoktur diye düşündüm.

ozanmujde commented 3 years ago

Screenshot_2020-12-21 Restart

ozanmujde commented 3 years ago

Hocam yıldızları ne intellij idea ne de visual studio code işlevini yapmadığı için elle yapıyorum.O yüzden o biraz sürebilir document okuyarak vs.Hemen hızlıca bunları atmak istedim Desing classını test ederken mock() da dahi hata atıyordu o yüzden kalan yerlerin testini yazmıştım.Eğer Design'ı da isterseniz biraz mock'u okuyarak vs deneyebilirim

ozanmujde commented 3 years ago

Başka bir şey eklememi denememi istiyorsanız yapabilirim hocam.