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

Krótki kod JS który powinien działać

+2 głosów
100 wizyt
pytanie zadane 4 października w JavaScript przez StaramSieJakMoge Początkujący (270 p.)

Hej stworzyłem w JS krótkie funkcje które miały kolorować wiersze w tabeli ale z jakiegoś powodu nie chcą zadziałać. Docelowo ma się kolorować co drugi wiersz ale póki co nie działa nawet w tej formie. Z góry bardzo dziękuję.

function load() {
  text_color = document.getElementByID('color-picker');
  text_color.addEventListener('change', ColorChange, false);



}

function ColorChange() {
  var color = text_color.value;
  const table = document.getElementById('tabelka');
  const rows = table.getElementsByTagName('td');


  for(var i = 0; i < rows.lenght; i++) {
    rows[i].style.background= color;
  }
}

 

3 odpowiedzi

+1 głos
odpowiedź 4 października przez VBService VIP (137,950 p.)
edycja 4 października przez VBService

Masz literówkę, wink 

rows.lenght  --}  rows.length 

Docelowo ma się kolorować co drugi wiersz

jeżeli chcesz wiersze kolorować, to lepiej użyć tr zamiast td

const rows = table.getElementsByTagName('tr');

 

nie jest to wymagane, ale możesz "doprecyzować" jaki styl chcesz "ustawiać"

rows[i].style.backgroundColor = color;

 

Nie widać całego kodu js-a, jeżeli ta zmienna text_color, która "przechowuje" element DOM nie jest globalna to w funkcji ColorChange() użycie jej zwróci błąd

Uncaught ReferenceError: text_color is not defined

ponieważ przypisanie do niej elementu DOM

text_color = document.getElementById('color-picker');

następuje w funkcji load() i w ten sposób jest tylko "widziana" w tej funkcji (zmienna lokalna).

 

ponieważ funkcja ColorChange() w Twoim kodzie jest wywoływana ze "zdarzenia"

text_color.addEventListener('change', ColorChange, false);

to możesz zastosować "e"

what does (e) mean?

 

e is the short var reference for event object which will be passed to event handlers.

The event object essentially has lot of interesting methods and properties that can be used in the event handlers.

wtedy funkcja może "przybrać wygląd"

function ColorChange(e) {
  const color = e.target.value;
  const table = document.getElementById('tabelka');
  const rows = table.getElementsByTagName('tr');
  
  for (let i = 0; i < rows.length; i++) {
    rows[i].style.backgroundColor = color;
  }
}

 

1
komentarz 4 października przez StaramSieJakMoge Początkujący (270 p.)
Bardzo dziękuję, śpieszę się z projektem i jestem już po prostu zmęczony dlatego pomijam oczywiste literówki,
+1 głos
odpowiedź 4 października przez krissto7 Gaduła (3,030 p.)
Jeśli chcesz co drugi wiersz to w warunku for możesz i++ zamienić na i = i +2 lub i+= 2.

Tu masz literówke document.getElementByID('color-picker'); zamień na document.getElementById('color-picker');

oraz w rows.lenght na rows.length.

Dodatkowo brak deklaracji zmiennej const/let/var w funkcji load, no chyba , że to jest deklaracja i nadpisujesz(brak całości kodu więc można się jedynie domyślać).

text_color.value w funkcji ColorChange jest nie zdefiniowane ponieważ zadeklarowałeś zmienną w innym scopie , więc z poziomu tej funkcji nie da się odczytać zawartości , propozycja to przenieś text_color = document.getElementById('color-picker') do scopu globalnego , ewentualnie przy wywołaniu funkcji ColorChange możesz wywołać tę funkcje z argumentem w postaci tej zmiennej , czyli () => ColorChange(text_color);
+1 głos
odpowiedź 4 października przez qax Mądrala (6,510 p.)

Można też tak:

<input type="color" id="color-picker" />
<table id="table">
<tr><td>1x1</td><td>1x2</td></tr>
<tr><td>2x1</td><td>2x2</td></tr>
<tr><td>3x1</td><td>3x2</td></tr>
<tr><td>4x1</td><td>4x2</td></tr>
<tr><td>5x1</td><td>5x2</td></tr>
<tr><td>6x1</td><td>6x2</td></tr>
<tr><td>7x1</td><td>7x2</td></tr>
</table>
const colorPicker = document.getElementById('color-picker');
const table = document.getElementById('table');
const rows = table.querySelectorAll('tr');
colorPicker.addEventListener('change', colorChange, false);

function colorChange() {
	let color = colorPicker.value;
  for(let i = 0; i < rows.length; i++) {
  	if (i % 2 === 0) {
    	rows[i].style.background= color;
    }
  }
}

 

1
komentarz 4 października przez VBService VIP (137,950 p.)

Skoro mamy w użyciu querySelectorAll to można się pokusić o taki zapis wink

const colorPicker = document.getElementById('color-picker');
const table_rows = document.querySelectorAll('#table tr:nth-child(even)');
colorPicker.addEventListener('change', colorChange, false);
 
function colorChange() {
  const color = colorPicker.value
  for (const row of table_rows)
    row.style.backgroundColor = color;
}

 

1
komentarz 4 października przez qax Mądrala (6,510 p.)

Czyli możliwości od groma wink

Podobne pytania

0 głosów
1 odpowiedź 76 wizyt
0 głosów
1 odpowiedź 117 wizyt
pytanie zadane 15 grudnia 2015 w C i C++ przez martix3 Użytkownik (680 p.)
0 głosów
1 odpowiedź 99 wizyt
pytanie zadane 21 lipca w C i C++ przez Daaa22 Mądrala (7,300 p.)

85,875 zapytań

134,647 odpowiedzi

298,918 komentarzy

56,743 pasjonatów

Advent of Code 2021

Top 15 użytkowników

  1. 1048p. - rucin93
  2. 1042p. - Whistleroosh
  3. 989p. - adrian17
  4. 984p. - Mikbac
  5. 982p. - Mateusz Bogdan
  6. 912p. - nidomika
  7. 867p. - Michal Drewniak
  8. 859p. - CC PL
  9. 854p. - Argeento
  10. 704p. - ScriptyChris
  11. 692p. - s. Dorota Kowalewska
  12. 683p. - tokox
  13. 660p. - Vinox
  14. 645p. - TheLukaszNs
  15. 628p. - WhiskeyTaster
Szczegóły i pełne wyniki

Motyw:

Akcja Pajacyk

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

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...