Gra ruchome linie (puzzle)

pytanie zadane 29 sierpnia 2019 w Nasze projekty przez kodowiec Początkujący (410 p.)

Witam,

Stworzyłem prostą gierkę w ruchome linie, gra typu puzzle. Opis i rozwiązanie na stronie :)

http://ruchome-linie-game.opx.pl/

Będę wdzięczny za jakiekolwiek uwagi, ocenę i krytykę. Pozdrawiam!

1 odpowiedź

odpowiedź 29 sierpnia 2019 przez cani Bywalec (2,060 p.)

Fajna gierka. Zerknąłem na kod i zobaczyłem, że używałeś dużo else if'ów. Moim zdaniem lepszym rozwiązaniem byłoby użycie switch statement. Zamiast:

        if (i === 1) {
            line2_class[i].style.backgroundColor = "darkblue";
            line2_class[i].innerHTML = tab_ball[random];
            tab_ball.splice(random,1);
        }
        else if (i === 2) {
            line2_class[i].style.backgroundColor = "purple";
            line2_class[i].innerHTML = tab_ball[random];
            tab_ball.splice(random,1);
        }
        else if (i === 3) {
            line2_class[i].style.backgroundColor = "darkcyan";
            line2_class[i].innerHTML = tab_ball[random];
            tab_ball.splice(random,1);
        }
        else if (i === 4) {
            line2_class[i].style.backgroundColor = "yellow";
            line2_class[i].innerHTML = tab_ball[random];
            tab_ball.splice(random,1);
        }

Mógłoby to wyglądać tak:

        switch (i){
            case 1:
              line2_class[i].style.backgroundColor = "darkblue";
              line2_class[i].innerHTML = tab_ball[random];
              tab_ball.splice(random,1);
            break;
            case 2:
              line2_class[i].style.backgroundColor = "purple";
              line2_class[i].innerHTML = tab_ball[random];
              tab_ball.splice(random,1);
            break;
            case 3:
              line2_class[i].style.backgroundColor = "darkcyan";
              line2_class[i].innerHTML = tab_ball[random];
              tab_ball.splice(random,1);
            break;
            case 4:
              line2_class[i].style.backgroundColor = "yellow";
              line2_class[i].innerHTML = tab_ball[random];
              tab_ball.splice(random,1);
            break;
         }