listenerda yazdığın kod çok karışık ve anlaşılamaz durumda bunun nedenlerini sırayla sayayım:
1 - Enum olarak kontrol edebileceğin bir şeyi string olarak kontrol ettirmişsin.
Eki Görüntüle 139317
2 - Itemini burada bir değişkene atamana rağmen yeniden çekmişsin.
Eki Görüntüle 139318
3 - çoğu yerde "event.getWhoClicked()" kullanmışsın eğer bir şeyi birden fazla kullanacaksan bir değişkene ataman senin için daha iyi olacaktır.
4- kodunda aşırı art arda if - else - else if kullanımı var bu kodunu çok düzensiz yapıyor.
5 - aynı şekilde bu event methodu bir complex method yani çok uzun ve karışık bu kodunu daha performanslı ve daha düzenli bir şekilde çalıştırabilmek için bu methodunun kodlarını parçalara ayırman gerekiyor.
6 - algoritmadaki hatayı sormuşsun bir algoritmayı birden fazla şekilde yapabilirsin önemli olan çalıştırmak değil aralarından en iyi ve en düzenli olan algoritmayı seçmektir.