Gra papier-kamień-nożyczki

Question

Robię grę papier-kamień-nożyczki w JavaScript i natknąłem się na problem, ponieważ chcę zrobić zabezpieczenie (gdy nie wybierzemy kamienia, papieru, nożyczek to wyskakuje alert, abyśmy wybrali) za pomocą flagi, jednak nie chce mi działać. Według mnie jest dobrze zrobione pod względem logicznym, ale nie chce tak jak wspomniałem działać. Proszę o jakieś wskazówki, nie chcę gotowca ;)

const options = ['papier', 'kamień', 'nożyce'];
const paper = document.querySelector('img[data-option]');
const stone = document.querySelector('img[data-option="kamień"]');
const scissors = document.querySelector('img[data-option="nożyczki"]');
const play = document.querySelector(".start");
const aiChoice = document.querySelector('span[data-summary="ai-choice"');
let protectionChoice = false;

if (paper.classList.contains("active") || stone.classList.contains("active") || scissors.classList.contains("active")) {
    protectionChoice = true;
}

paper.addEventListener("click", () => {
    paper.classList.add('active');
    stone.classList.remove("active");
    scissors.classList.remove("active");
});
stone.addEventListener("click", () => {
    stone.classList.add("active");
    paper.classList.remove("active");
    scissors.classList.remove("active");

});
scissors.addEventListener("click", () => {
    scissors.classList.add("active");
    paper.classList.remove("active");
    stone.classList.remove("active");

})


play.addEventListener("click", () => {
    if (protectionChoice) {
        alert("wybierz dłoń");
    } else {
        const option = Math.round(Math.random() * (options.length - 1));

        aiChoice.textContent = options[option];

    }


})

 

Answer 1

Zastanów się, kiedy sprawdzany jest warunek z linijki 9

Comments

Popraw spójność nazw (nożyce, nożyczki) i zmień drugi if na else if, bo w takiej formie (if if) to jest błąd, ponieważ ten drugi warunek się wykonuje niezależnie od pierwszego i z racji tego,że przy remisie się nie spełnia wyrzuca else, czyli WYGRAŁEŚ

Brak zabezpieczenia w przypadku gdy user nic nie wybrał i uruchomił grę.

Dobrze było by żeby po zaznaczeniu wyboru przez usera od razu pojawiała się nazwa w

Twój wybór:  ...

Jeżeli kod działa (po poprawkach powinien) daj go do oceny bo ja nie jestem specjalistą.

Moim zdaniem obiekty nie są potrzebne, a przypinanie nasłuchu do wszystkich obrazków również.

Powodzenia

---

Ok.

Jeśli chodzi o flagi, to mogę takie zabezpieczenie zrobić właśnie nią? i właśnie czemu służą flagi, bo do zabezpieczeń to na pewno.

A czemu nie obiektami? chyba kod wydaje się czystszy

---

Bardziej bym powiedział, że do sprawdzenia/ustawienia stanu obiektu.

A właściwości obiektów nie wykorzystujesz, to po co one są?

Ale tak jak napisałem daj kod do oceny.

---

co to znaczy sprawdzenia/ustawienia stanu obiektu?

Tak, nie używałem, bo jeden obiekt źle napisałem tzn miałem przekazać do właściwości obiektu number a nie stringa, ale już w finalnej wersji jest wykorzystany obiekt.

---

Kiedy portrzebujemy odróżnić od siebie dwie możliwosci np. tak i nie , istnieje i nie istnieje, właczony i wyłączny to w js mamy typ logiczny, który ma tylko dwie wartości true i false. Więc sprawdzamy ten stan lub ustawiamy ten stan.