Mam problem ze zdarzeniem i odczytywaniem danych z input[type="file"]. Chcę zrobić tak że po wybraniu pliku, div .upload-button zmienia kolor. Mam taki kod:

check_upload(i) - to powoduje natychmiastowe wykonanie funkcji check_upload a nie przypisanie jej jako callback
Jak koniecznie chcesz przekazywac to "i" to powinno byc tak:

document.getElementsByClassName("upload")[i].addEventListener("change", function() {
check_upload(i);
});

Choc i to nie jestem pewien czy to i pojdzie poprawnie. No i generalnie poczytaj o callbackach

Lepiej by było:


i później do grupy elementów (NodeList) normalnie przypinać. Mogliby swoją drogą ujednolicić co poszczególne funkcje zwracają (querySelectorAll)

Pomogło (prawie) wrzucenie check_upload(i) do funkcji EventListenera, ale nadal nie działa poprawnie. Konkretnie, przeglądarka pluje się że nie może odczytać VALUE z danego inputa. A to dlatego, że zmienna i wypluwa liczbę swoich iteracji a nie właściwą. Zawsze jest z tym problem w petli for...

No bo ty w ogole tego i nie potrzebujesz...
Przeciez masz tam this ktore jest twoim elementem wiec:

Generalnie tak, ale muszę mieć potem i tak i żeby móc pokolorować odpowiedni DIV innym kolorem.

No ale majac swoj glowny obj to jaki potem problem pobrac sasiadow ktorych musisz pokolorowac?

Wydaje mi się, że optymalniej było by pobrać numer i - mniej kodu i w przypadku rozbudowy skryptu stwarza większe możliwości.

No wlasnie operacje na i nie stwarza wiecej mozliwosci.

Twoj kod html dla danego input powinien wygladac tak:

[HTML] pobierz, plaintext 
<div class="input-container">
   <input twoj input >
   <div class="jakas-klasa-dla-jednego-div"></div>
   <div class="jakas-klasa-dla-drugiego-div"></div>
</div>
[HTML] pobierz, plaintext 

Wowczas majac obj twojego inputa, dobierasz sie do rodzica, czuli do .input-container i potem w nim sobie szukasz div o klasach jakie chcesz. To dopiero daje mozliwosci rozwoju

Bawienie sie w jakies "i" predzej czy poznij przysporzy ci klopotow. A wiem bo sam tak zaczynalem wlasnie w zabawy w durne "i". Teraz jade poprostu na kontenerach i klasach i nie ma zadnych problemow. Masz obiekt bazowy i tylko na tej podstawie jestes w stanie dobrac sie do wszystkich innych elementow majacych cos wspolnego z tym obiektem

@viking istnieje złota zasada, żeby nie tykać prototypów obiektów hosta – zwłaszcza, że jak sam zauważyłeś, musiałbyś zmienić kilka klas kolekcji. A przecież wystarczy


Co do problemu: a czemu nie po prostu przypiąć jeden handler dla całego kontenera i sprawdzać, które pole zostało zmienione przez event.target? https://jsfiddle.net/Comandeer/vuveay15/

Od FF 50 forEach działa też bezpośrednio na NodeList (w końcu).

W idealnym świecie NodeList byloby subklasą Array, która zwracałaby niemutowalne tablice (Object.freeze w konstruktorze choćby). No ale nie żyjemy w takim