• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Gdy klikam na przycisk, wykonuje się coś innego niż powinno

VPS Starter Arubacloud
+1 głos
288 wizyt
pytanie zadane 23 maja 2021 w JavaScript przez Fonteinbloem Użytkownik (660 p.)

Hej, zrobiłem prosty licznik, gdy klikam przycisk dodaje mi tę wartość do głównej sumy i problem polega na tym, że gdy będę szybko klikał przyciski, to zamiast wykonać się to co powinno, po prostu ustawia wartość na zero i to bez względu jaki przycisk kliknę. Dodaję kod.

const counterDiv = document.querySelector('.counter');
const btns = document.querySelector('.button-container');


btns.addEventListener('click', count);

function count({target}) {
    let counterValue = Number(counterDiv.textContent);
    if (target.classList.contains('decrease')) {
        let number = Number(target.textContent);
        counterValue+=number;
    } else if (target.classList.contains('increase')) {
        let number = Number(target.textContent);
        counterValue+=number;
    } else {
        counterValue=0;
    }
    counterDiv.innerText = counterValue;

    if (counterValue > 0) {
        counterDiv.style.color = "green";
    } else if (counterValue < 0) {
        counterDiv.style.color = "red";
    } else {
        counterDiv.style.color = "white";
    }

}
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Counter</title>
    <link rel="stylesheet" href="style.css">
</head>
<body>
    <main>
        <div class="container">
            <h2 class="logo">COUNTER</h2>
            <div class="counter">0</div>
            <div class="button-container">
                <button class="btn decrease">-10</button> 
                <button class="btn decrease">-5</button> 
                <button class="btn decrease">-1</button>
                <button class="btn reset">Reset</button>
                <button class="btn increase">+1</button>
                <button class="btn increase">+5</button>
                <button class="btn increase">+10</button>
            </div>
        </div>
    </main>

    <script src="app.js"></script>
</body>
</html>

 

3 odpowiedzi

+3 głosów
odpowiedź 23 maja 2021 przez ScriptyChris Mędrzec (190,190 p.)
wybrane 23 maja 2021 przez Fonteinbloem
 
Najlepsza

Pewnie dlatego, że jeśli szybko klikasz, to czasem nie trafisz w przycisk, tylko delikatnie obok i - z powodu stosowania event delegation - targetem eventu jest kontener o klasie "button-container", co ustawia zerową wartość wyniku i (w dalszym warunku) biały kolor tekstu (więc wynik jest niewidoczny na stronie). Dodanie warunku wykluczającego działanie funkcji w takim przypadku powinno pomóc.

1
komentarz 23 maja 2021 przez Fonteinbloem Użytkownik (660 p.)
Dzięki!
+1 głos
odpowiedź 23 maja 2021 przez ITshnyk Obywatel (1,800 p.)

Nie wiem dlaczego dokładnie jest błąd, gdyż jestem początkujący w JS ale zmieniłem if na poniższą postać:

if (target.classList.contains('decrease')) {
        let number = Number(target.textContent);
        counterValue+=number;
    } else if (target.classList.contains('increase')) {
        let number = Number(target.textContent);
        counterValue+=number;
    } else if (target.classList.contains('reset')){
        counterValue = 0;
    }

I działa poprawnie. Podejrzewam, że gdy szybko klikasz klawisze, to jest to rejestrowane jako inne kliknięcie i jest wykonywany ten kod:

 

else {
        counterValue=0;
    }
komentarz 23 maja 2021 przez Fonteinbloem Użytkownik (660 p.)
Tak, już wiem co było źle, dzięki!
–1 głos
odpowiedź 23 maja 2021 przez VBService Ekspert (251,210 p.)
edycja 24 maja 2021 przez VBService

Może odwołaj się bezpośrednio do button-ów, to pozwoli uniknąć "problemu" (w cudzysłowie, bo event delegation sam w sobie nim nie jest, tylko trzeba z niego "świadomie" korzystać wink ) opisanego przez @ScriptyChris-a.

Propozycja zmiany kodu

const counterDiv = document.querySelector('.counter');
const buttons = document.querySelectorAll('.button-container button');

buttons.forEach(button => {
  button.addEventListener('click', count);
})

function count({target}) {
  let cssColor = 'white';
  let counterValue = 0;
  
  if (!target.classList.contains('reset')) {
    counterValue = Number(counterDiv.textContent) + Number(target.textContent);   
    cssColor = (counterValue < 0) ? 'red':'green';
  }
  
  counterDiv.innerText = counterValue;
  counterDiv.style.color = cssColor;
}

 

[ edit ]

Propozycja, wersja bez odwoływania się bezpośrednio do button-ów.

const counterDiv = document.querySelector('.counter');
const btns = document.querySelector('.button-container');

btns.addEventListener('click', count);

function count({target}) {
  if (target.classList.contains('btn')) {
    let cssColor = 'white';
    let counterValue = 0;

    if (!target.classList.contains('reset')) {
      counterValue = Number(counterDiv.textContent) + Number(target.textContent);   
      cssColor = (counterValue < 0) ? 'red':'green';
    }

    counterDiv.innerText = counterValue;
    counterDiv.style.color = cssColor;
  }
}

 

komentarz 24 maja 2021 przez Comandeer Guru (599,730 p.)
Tylko po co? Wykrycie, co było kliknięte, rozwiązuje problem.

Podobne pytania

0 głosów
1 odpowiedź 1,062 wizyt
pytanie zadane 12 listopada 2016 w HTML i CSS przez evilblade Nowicjusz (120 p.)
+1 głos
1 odpowiedź 260 wizyt
0 głosów
0 odpowiedzi 100 wizyt

92,453 zapytań

141,262 odpowiedzi

319,086 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...