JS - pomocy nie wiem co robic

Question

https://jsfiddle.net/e6uv3d4s/1/

Witam, powie ktoś, dlaczego js nie działa tak jak powinien? Po kilku minutach okazało się, że jeśli funkcje start wezmę w komentarze i onload zmienie na writePass to całość działa i rzeczywiście wypisuje mi zakreskowane hasło; dziwne zjawisko.

Comments

Tak z na przysłość polecam używać w pętlach (i ogólnie) let zamiast var

Answer 1

Masz w pętli w start wpisane int (to chyba przywyczajenie z cpp/javy czy czegoś tego typu)

Podmień to int na let.

tak wygląda twój aktualny kod

for(int i = 0; i < 26; i++){
		divCreator = '<div class="letter">A</div>';
		if((i + 1) % 7 == 0) divCreator += '<div style="clear: both;"></div>';
		document.getElementById("alphabet").innerHTML = divCreator;
	}

A tak powinno to wyglądać

for(let i = 0; i < 26; i++){
		divCreator = '<div class="letter">A</div>';
		if((i + 1) % 7 == 0) divCreator += '<div style="clear: both;"></div>';
		document.getElementById("alphabet").innerHTML = divCreator;
	}

 

Comments

Przy okazji polecam korzystać z querySelector zamiast getElementById

---

@wizarddos, kurcze, tak mi c++ zotał we krwi, ze az tak sie myle, dzieki wielkie kolego :D

---

W js-ie możesz zainicjować zmienną w momencie jej pierwszego użycia

function start(){
	for(let i = 0; i < 26; i++) {
		let divCreator = '<div class="letter">A</div>';
		if((i + 1) % 7 == 0) divCreator += '<div style="clear: both;"></div>';
		document.getElementById("alphabet").innerHTML = divCreator;
	}
	writePass();
}
window.onload = start;

 

no i tu linia 2 jest zbędna

var pass = 'East or west, home is the best';
//pass = pass.toUpperCase();

var pass1 = '';

for(let i = 0; i < pass.length; i++){
	if(pass.charAt(i).toUpperCase() != pass.charAt(i).toLowerCase())
		pass1 += '_';
	else pass1 += pass.charAt(i);
}

 

w js-ie możesz użyć taki zapis 

if(pass[i].toUpperCase() != pass[i].toLowerCase()) ...

zamiast

if(pass.charAt(i).toUpperCase() != pass.charAt(i).toLowerCase()) ...