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

this w JavaScript

0 głosów
845 wizyt
pytanie zadane 26 października 2018 w JavaScript przez turtelian Obywatel (1,760 p.)

Witam, mam taki problem, piszę kółko i krzyżyk i chciałbym to zrobić "profesjonalnie", tak aby było jak najmnij zbędnego kodu.
Zamysł był taki w for each, nastawiam add evnlistener dla kazdego diva. Po kliknieciu wykonuje się funkcja która przez this.innerhtml zmienia jego zawartość. Lecz widocznie zle rozumiem this. Jak to ugryźć aby funkcja była wykonywana na klikniętym divie?(oczywiscie wiem ze moge dac np id kazdemu elementowi i zrobic to lopatologicznie ale nie o to w tym chodzi :) )
Oto co póki co napisałem(prawie nic, ale pokaże to mój tok myślenia):
 

const divs = document.querySelectorAll('div');

function move() {
    console.log('dziala');
    this.innerHtml = 'X';// tu probowalem kilku opcji żadna nie dziala poprawnie

}
divs.forEach((div) => {
    div.addEventListener('click', move)
})

 

3 odpowiedzi

0 głosów
odpowiedź 26 października 2018 przez thryndl Nałogowiec (30,470 p.)

Prawdopodobnie Event delegation jest odpowiedzią na Twój problem.

komentarz 26 października 2018 przez turtelian Obywatel (1,760 p.)

nie po prostu nie przekazalem obiektu do funkcji move ;)
 

function move(divs) {
    if (counter % 2 == 0) {
        this.innerText = "0";
        this.removeEventListener('click', move);
    } else {
        this.innerText = "X";
        this.removeEventListener('click', move);

    }
    counter++;
    //sprawdzanie warunków
    if (counter > 4) {


    }
    if (counter == 9) {
        alert('remis!');
    }


}
divs.forEach((div) => {
    div.addEventListener('click', move);

})

 

komentarz 26 października 2018 przez elwood Gaduła (4,180 p.)
edycja 26 października 2018 przez elwood

przekazałeś divs a w ogóle z tego nie korzystasz. Swoja drogą to divs to nie są twoje divy tylko event zdarzenia. Aby przekazac divy do funkcji move musiał byś zrobic cos takiego:

const divs = document.querySelectorAll('div');
 
function move(divs, event) {
    console.log('dziala');
    console.log(this)
    console.log(divs)
    this.innerHTML = 'X';// tu probowalem kilku opcji żadna nie dziala poprawnie
 
}
divs.forEach((div) => {
    div.addEventListener('click', move.bind(div, div))
})

teraz zarówno pod this jak i pod argumentem divs masz to samo.

0 głosów
odpowiedź 26 października 2018 przez elwood Gaduła (4,180 p.)
Nie ma czegoś takiego jak innerHtml, jest natomiast innerHTML.
komentarz 26 października 2018 przez elwood Gaduła (4,180 p.)
Niby czemu minus :o
komentarz 26 października 2018 przez X3h Dyskutant (9,540 p.)
Uraziłeś czyjeś uczucia.
–1 głos
odpowiedź 26 października 2018 przez X3h Dyskutant (9,540 p.)
Nie patrzysz do czego się odnosisz. Zobacz co siedzi pod this, jak działa addEventListener. console.log('dziala'); brzmi jak oszukiwanie samego siebie.

Podobne pytania

0 głosów
0 odpowiedzi 1,096 wizyt
0 głosów
0 odpowiedzi 657 wizyt
pytanie zadane 1 listopada 2017 w JavaScript przez Marchiew Dyskutant (7,730 p.)
0 głosów
0 odpowiedzi 291 wizyt
pytanie zadane 13 marca 2016 w JavaScript przez Wirus Stary wyjadacz (14,000 p.)

93,733 zapytań

142,669 odpowiedzi

323,287 komentarzy

63,293 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

Twierdza Linux. Bezpieczeństwo dla dociekliwych

Aby uzyskać rabat -10%, użyjcie kodu pasja-linux, wpisując go w specjalne pole w koszyku.

...