java-kod do oceny

Question

Witam, uczę się programować. Mam kilka pytań :)

jakie błędy popełniam i co mogę poprawić w swoim kodzie? czego powinienem się dalej uczyć?

 

link do aplikacji:

https://github.com/Szymon98-code/game/tree/master/gra/saper

Answer 1

1. Nazwy po angielsku! Sory ale pisanie kodu po polsku nie ma wiekszego sensu... nawet jak nie umiesz dobrze nagielskiego to czas sie go zaczac uczyc a przy dzisiejszych narzedziach do tlumaczenia pisanie kodu po angielsku nie powinno byc problemem

 

2. Nazwewnictwo - Klasy powinieneś pisać z wielkiej litery 

  public PrzyciskiSterowanie(plansza plansza){
        this.plansza=plansza;
        odkryte=0;
    }

zmienne ktore mowia czego dotycza 

 Object z=e.getSource();

 

3. Unikaj takich zagniezdzonych ifow + forow... podziel ten fragment na klasy :

 for(int i=0;i<plansza.a.m;i++){
            for(int j=0;j<plansza.a.n;j++){
                if(z==plansza.przyciski[i][j]){
                    plansza.przyciski[i][j].setIcon(null);

                    if(plansza.a.tab[i][j].wartosc==9){
                        ImageIcon ikona=new ImageIcon("mina.JPG");
                        plansza.przyciski[i][j].setIcon(ikona);
                        plansza.koniecGry();
                    }
                    else if(plansza.a.tab[i][j].wartosc==0)plansza.ods_zer(i,j);
                    else{
                        plansza.przyciski[i][j].setText(Integer.toString(plansza.a.tab[i][j].wartosc));
                        plansza.a.tab[i][j].czyOdsloniety=true;
                        plansza.przyciski[i][j].setEnabled(false);
                        odkryte++;}
                    if(odkryte==(plansza.a.m*plansza.a.n)-plansza.a.bomby)System.out.println("suckes");
                }
            }
        }

 

Comments

1. Nazwy po angielsku! Sory ale pisanie kodu po polsku nie ma wiekszego sensu...

Wg. mnie może mieć sens, jak stawiamy pierwsze kroki w programowaniu i nie znamy składni. Pisanie po polsku zmiennych daje nam przewagę w tym aby szybciej orientować się co jest nasze a co należy do biblioteki/języka. Co nie zmienia faktu, że po tym wstępnym etapie musimy przejść na angielski. 

if(x-1>=0&&y-1>=0&&tab[x-1][y-1].wartosc!=9)tab[x-1][y-1].wartosc++;
Od siebie dodam że lepiej dłuższe warunki w ifach wydzielić do oddzielnym zmiennych i nadać im coś mówiącą nazwę.

---

Dzięki za odpowidzi:) na pewno poprawie błędy. Mam jeszcze jedno pytanie. Czego powinienem sie dalej uczyć?

---

@miro, zakładam, że osoba która zapoznała się z  gitem jest na wyższym poziomie i można od niej wymagać kodu po angielsku. Natomiast masz racje nie jest to konieczne przy pierwszych kilku programach :)

 

@uwegi, a w jakim kierunku chciałbyś iść? WebService, Android...? 

---

Raczej web..

---

Hibernate, JPA a potem Java EE.

Answer 2

Ten projekt nie posiada żadnej struktury - wszystko jest w jednym "folderze", + są w repo znajdują się pliki które już są skompilowane (.class, a powinny być same źródła czyli .java) - jak poprawisz to możesz zawołać :)

Comments

poprawione