🚀 go-pugleaf

W dniu 2018-11-11 o 21:49, Jivanmukta pisze:

> Pliki .html, .css, .js przechodzą pomyślnie walidację.
> Nie mam innego elementu o id='property_pictures' ani innego o 
> id="main_title", literówki w id nie ma.
> DOM dokument jest załadowany w chwili wywołania.
> Nie usuwam elementów property_pictures ani main_title przy pomocy 
> JavaScript.
> Przejrzałem w Google strony "getElementById returns null" i nie 
> znalazłem przyczyny mojego błędu.
> Prosiłbym o podpowiedź co robię źle.

Taki błąd powstaje raczej tylko w przypadku gdy DOM nie jest w pełni 
załadowany. Nic innego mi do głowy nie przychodzi. A kiedy może się tak 
dziać? Z praktyki webdewelopera  najczęściej to:

1. Jakiś skrypt zmienia / przebudowuje strukturę dokumentu. Również może 
to być refresh.

2. Strona ładuje się i następuje chwilę potem przekierowanie do jej 
samej (3xx). Sprawdź po stronie sieci w narzędziach czy nie ma czegoś 
takiego.

Ustaw też w na początku funkcji animateMainTitle jakiś log do konsoli by 
sprawdzić czy nie następuje podwójne wywołanie w samym JS.

3. Ponadto z nazwy w/w funkcji wnioskuję iż naruszasz strukturę DOM by 
coś zanimować. Nie wiem jak konkretnie animujesz więc tylko zgaduję. 
Jeśli np. używasz innerHTML gdzieś dalej do zapisu, to wtedy takie cuda 
mogą się dziać. Nie należy używać do tego celu innerHTML bo jest 
strasznie wolne gdyż wymusza odświeżanie _CAŁEJ_ struktury DOM. Wtedy 
okresowo możesz tracić dostęp do DOM mimo iż pozornie wydaje Ci się, że 
nic się w niej nie zmienia. To dość częsty błąd początkujących 
programistów JS a i zaawansowani czasem też o tym nie wiedzą.

-- 
Pozdrawiam,
Marek

> 3. Ponadto z nazwy w/w funkcji wnioskuję iż naruszasz strukturę DOM by 
> coś zanimować. Nie wiem jak konkretnie animujesz więc tylko zgaduję. 
> Jeśli np. używasz innerHTML gdzieś dalej do zapisu, to wtedy takie cuda 
> mogą się dziać. Nie należy używać do tego celu innerHTML bo jest 
> strasznie wolne gdyż wymusza odświeżanie _CAŁEJ_ struktury DOM. Wtedy 
> okresowo możesz tracić dostęp do DOM mimo iż pozornie wydaje Ci się, że 
> nic się w niej nie zmienia. To dość częsty błąd początkujących 
> programistów JS a i zaawansowani czasem też o tym nie wiedzą.
> 

// Animuj literkę w napisie tytułowym:

var letterIndex = -1;
var mainTitle;

function animateMainTitle() {
     'use strict';
     if (mainTitle === undefined) {
         mainTitle = document.getElementById("main_title").innerHTML;
     }
     letterIndex = (letterIndex + 1) % mainTitle.length;
     var s = mainTitle.substr(0, letterIndex) +
             '<span class="selected_letter">' +
             mainTitle.substr(letterIndex, 1) +
             '</span>' +
             mainTitle.substr(letterIndex + 1);
     document.getElementById("main_title").innerHTML = s;
     setTimeout(animateMainTitle, 100);
}

Czy gdybym zamiast innerHTML umieścił wszystkie literki w span'ach i 
ustawiał tylko atrybut class odpowiedniej literki to byłoby lepiej?

W dniu 19.11.2018 o 08:54, Jivanmukta pisze:
> 
>> 3. Ponadto z nazwy w/w funkcji wnioskuję iż naruszasz strukturę DOM by 
>> coś zanimować. Nie wiem jak konkretnie animujesz więc tylko zgaduję. 
>> Jeśli np. używasz innerHTML gdzieś dalej do zapisu, to wtedy takie 
>> cuda mogą się dziać. Nie należy używać do tego celu innerHTML bo jest 
>> strasznie wolne gdyż wymusza odświeżanie _CAŁEJ_ struktury DOM. Wtedy 
>> okresowo możesz tracić dostęp do DOM mimo iż pozornie wydaje Ci się, 
>> że nic się w niej nie zmienia. To dość częsty błąd początkujących 
>> programistów JS a i zaawansowani czasem też o tym nie wiedzą.
>>
> 
> // Animuj literkę w napisie tytułowym:
> 
> var letterIndex = -1;
> var mainTitle;
> 
> function animateMainTitle() {
>      'use strict';
>      if (mainTitle === undefined) {
>          mainTitle = document.getElementById("main_title").innerHTML;
>      }
>      letterIndex = (letterIndex + 1) % mainTitle.length;
>      var s = mainTitle.substr(0, letterIndex) +
>              '<span class="selected_letter">' +
>              mainTitle.substr(letterIndex, 1) +
>              '</span>' +
>              mainTitle.substr(letterIndex + 1);
>      document.getElementById("main_title").innerHTML = s;
>      setTimeout(animateMainTitle, 100);
> }
> 
> Czy gdybym zamiast innerHTML umieścił wszystkie literki w span'ach i 
> ustawiał tylko atrybut class odpowiedniej literki to byłoby lepiej?

Zrobiłem tak i jest dobrze:

var letterIndex = 0;
function animateMainTitle() {
     'use strict';
     document.getElementById('letter' + letterIndex).className = '';
     letterIndex = (letterIndex + 1) % 24;
     document.getElementById('letter' + letterIndex).className = 
'selected_letter';
     setTimeout(animateMainTitle, 100);
}

Dzięki za pomoc.

W dniu 2018-11-19 o 08:54, Jivanmukta pisze:

>      document.getElementById("main_title").innerHTML = s;

No więc widzę, że trafiłem w sedno :-) Standardowy błąd.

> Czy gdybym zamiast innerHTML umieścił wszystkie literki w span'ach i 
> ustawiał tylko atrybut class odpowiedniej literki to byłoby lepiej?

Zdecydowanie lepiej. A jeśli z jakiegoś powodu musisz zmieniać HTML, to 
rób to tym:

https://developer.mozilla.org/en-US/docs/Web/API/Element/insertAdjacentHTML

Po to powstała ta funkcja by nie używać innerHTML.

-- 
Pozdrawiam,
Marek

W dniu 2018-11-19 o 22:57, Marek S pisze:
> W dniu 2018-11-19 o 08:54, Jivanmukta pisze:
>
>>      document.getElementById("main_title").innerHTML = s;
>
> No więc widzę, że trafiłem w sedno :-) Standardowy błąd.
>
>> Czy gdybym zamiast innerHTML umieścił wszystkie literki w span'ach i
>> ustawiał tylko atrybut class odpowiedniej literki to byłoby lepiej?
>
> Zdecydowanie lepiej. A jeśli z jakiegoś powodu musisz zmieniać HTML, to
> rób to tym:
>
> https://developer.mozilla.org/en-US/docs/Web/API/Element/insertAdjacentHTML
>
> Po to powstała ta funkcja by nie używać innerHTML.
>

Programowania z wykorzystaniem innerHTML nauczyłem się z książek...
Dzięki za pomoc.

---
Ta wiadomość została sprawdzona na obecność wirusów przez oprogramowanie antywirusowe Avast.
https://www.avast.com/antivirus