Wypisywanie w divie liczby pobranej z input.

pytanie zadane 18 grudnia 2018 w JavaScript przez p4tryk Nowicjusz (220 p.)

Witam, stworzyłem sobie poniższy skrypt, który powinien wypisać wszystkie dzielniki, wpisanej w input, liczbie. Program działa natomiast nadpisuje strone przez co pole wpisywanej liczby i przycisk znikają. Moje pytanie jest: co zrobić aby tak się nie działo? Próbowałem z divem i getElementById (potraktowałem to jako komentarz: /* ... */ i o ten fragment kodu mi chodzi, dlaczego nie działa) ale nie działa, bo wyświetla tylko ostatni dzielnik.

Za pomoc z góry dziękuje.

<!doctype html>
<head lang="pl">
<meta charset=utf-8>
<title>JS</title>



</head>

<body>

<p>
<input type="text" id="pole"/>
<input type="button" value="Pokaż dzielniki liczby" onclick="getText()" />




<div id="wynik"></div>
</p>
<script type="text/javascript">

function getText()
{

var pole = document.getElementById("pole");
var z = pole.value;

for (var i=1; i<=z;i++)
{ 
if((z%i) != 0) continue;
else /*document.getElementById("wynik").innerHTML = */document.write(i+" | ");
}
}


</script>
</body>
</html>

1 odpowiedź

odpowiedź 18 grudnia 2018 przez ScriptyChris Mędrzec (190,190 p.)
wybrane 18 grudnia 2018 przez p4tryk

Najlepsza

Nie używaj document.write (który czyści stronę, gdy wywołasz tą metodę po załadowaniu dokumentu). Odkomentuj getElementById i po prostu przypisz do property innerHTML (a w Twoim przypadku bardziej do textContent, bo nie wrzucasz treści HTML tylko tekst) pobranego elementu jakąś wartość.

komentarz 18 grudnia 2018 przez Tomek Sochacki Ekspert (227,510 p.)

A tak na marginesie to ostatnio zauważyłem, że chrome wręcz wali warningi na document.write :) Niestety nie zawsze da się tego uniknąć, ja w jednym projekcie mam skrypty AdOcean i za chiny ludzi od ADO nie porzekonasz do zmiany skryptów... i musiałem użyć wrappera asynchronicznego (https://github.com/krux/postscribe). Choć jak ktoś ma inne, lepsze rozwiązanie to chętnie posłucham :) A to zostawiam, może komuś się przyda...

komentarz 18 grudnia 2018 przez p4tryk Nowicjusz (220 p.)

@JSHolic, zmieniłem linijkę na:

else document.getElementById("wynik").textContent= i+" | ";

Działa tak samo jak z .innerHTML - wyświetla ostatni dzielnik bo nadpisuje poprzednie (a chce aby wypisał wszystkie i zostawił wypisane, co miałem jeszcze zmienić? :)