javascript, czy to jest zgodne z dobrymi praktykami ?

Question

witam wszystkich bardzo serdecznie. jako, że jestem niezbyt dobry w JS muszę zadać takie pytanie:

HTML:

<div id="divadd"></div>
<div class="acenter">
<button type="button" id="sa">Sprawdź adres</button>
</div>

JAVASCRIPT

<script>
document.getElementById("sa").addEventListener("click", function(){
var newdiv = document.createElement("div");
    newdiv.appendChild(document.createTextNode("ppp123"));
var divadd = document.getElementById("divadd");
divadd.appendChild(newdiv);
});
</script>

Czy coś takiego jest zgodne z dobrymi praktykami ?

Zaznaczę, że wszytko działa.

Comments

newdiv.appendChild(document.createTextNode("ppp123"));

Zamiast tego wystarczy

newdiv.textContent = "ppp123";

(ogólnie text node to coś z czym w praktyce nie trzeba mieć wielkiej styczności)

---

no fakt... słuszna uwaga

Answer 1

Nie.

1. Nie używaj var - teraz są już const oraz let

2. Nazwy zmiennych są złe. Co to jest "sa", newdiv -> newDiv (ale nazwa też nic nie mówi)

Comments

a możesz to objaśnić, ponieważ nie znalazłem żadnej informacji dlaczego nie powinno się używać var ?

---

https://softwareengineering.stackexchange.com/questions/274342/is-there-any-reason-to-use-the-var-keyword-in-es6

---

No ok ale jeśli chciałbym przerobić ten mój skrypt tak, że jakaś zmienna ma być widoczna także w innych funkcjach to jak to zrobić? czy jest sens używać wtedy let a nie prościej użyć var ?

Może jest jakiś inny sposób, nie jestem biegły w JS dlatego pytam

---

Stosowanie globalnych zmiennych w takim wypadku nie jest dobrą praktyka - lepiej przekazać tą zmienną do funkcji poprzez parametry.