Niedziałający onclick w przycisku

Question

Witam,
próbuję stworzyć prosty quiz oparty na html/javascript, lecz napotykam na pewien błąd. Otóż, skrypt w ogóle się nie wykonuje i mam wrażenie, że z jakiegoś powodu nie działa onclick, którego użyłem. Jestem początkujący, jestem w trakcie oglądania kursu HTML i JS M. Zelenta. Siedzę już nad tym problemem kilka godzin i głowa mi niedługo pęknie. Proszę o poradę, jak się z tym uporać.

 

 

Fragment HTML'a:

<div id="content">
		<u><h3>QUIZ</h3></u>
		<p>Przed Tobą jest do rozwiązania krótki test sprawdzający wiedzę. Aby otrzymać pozytywną ocenę, musisz odpowiedzieć prawidłowo na przynajmniej 2 pytania.</p><br/><br/>
		
		<form>
			<p>1. Czy komputer Altair 8800 został opracowany wcześniej niż Atari 400?<br />
			<input type="radio" id="1" value="1">Tak
			<input type="radio" id="2" value="0">Nie
			<p>2. Czy komputer IBM 5150 posiadał dysk twardy?<br />
			<input type="radio" id="3" value="0">Tak
			<input type="radio" id="4" value="1">Nie
			<p>3. Czy IBM 5150 posiadął większą pamięć ROM niż Atari 400/800?<br />
			<input type="radio" id="5" value="1">Tak
			<input type="radio" id="6" value="0">Nie
			<br /><br />
			<button type="button" onclick="check()">Sprawdź odpowiedzi</button>
		</form>
</div>

 

skrypt JS:

function check(){
	var sum=0;
	for(int i=1; i<=6; i++){
		if(document.GetElementById("1").checked){
			var x = document.GetElementById("1").value;
			sum=+x;
		}
		if(document.GetElementById("2").checked){
			var x = document.GetElementById("2").value;
			sum=+x;
		}
		if(document.GetElementById("3").checked){
			var x = document.GetElementById("3").value;
			sum=+x;
		}
		if(document.GetElementById("4").checked){
			var x = document.GetElementById("4").value;
			sum=+x1
		}
		if(document.GetElementById("5").checked){
			var x = document.GetElementById("5").value;
			sum=+x;
		}
		if(document.GetElementById("6").checked){
			var x = document.GetElementById("6").value;
			sum=+x;
		}

}
if(sum>=2)
		alert("Udało Ci się. Twój wynik to "+sum+" poprawne odpowiedzi");
		else
		alert("Nie udało Ci się.");
}

 

Answer 1

document.getElementById

a nie

document.GetElementById

Oprócz tego jeszcze

sum=+x1

Co to ma niby robić? Pewnie zamiast tej 1 miał być ; ale to i tak nie zmienia faktu, że nie rozumiem co taki zapis ma zrobić. Przypisać do zmiennej sum wartość x na plusie? Może pomyliłeś z +=?

Comments

Poprawiłem "gety" i tę jedynkę, ale dalej nic.

EDIT: Tak, chodziło o +=... Tyle razy już modyfikowałem ten kod w nerwach, że nawet tego nie zauważyłem... Dzięki, ale i tak dalej nie działa. Wg przeglądarki "check is undefined"

---

W pętli masz:

for (int i...

a powinno być for (var i...

---

Jednak działa:-) Przynajmniej częściowo, bo wyszło na to, że pobieram po prostu "stringa" z wartości radio (1, 0) zamiast liczby, więc zamiast dostać sumę punktów, otrzymuję zlepiony ciąg znaków :P

---

parseInt(string) - zamiana na liczby całkowite

parseFloat(string) - zamiana na liczy rzeczywiste

---

parseInt załatwił sprawę, ale w trakcie wyszedł jeszcze jeden (i chyba ostatni) problem: otóż, mogę zaznaczyć wszystkie input radia

 

Jakieś pomysły, Panowie Dobrodzieje?

---

Radioboxy spośród których ma być zaznaczona tylko jedna odpowiedź muszą mieć taki sam name - u Ciebie wcale nie mają atrybutu name, musisz dodać.

---

Wyskakiwał mi jeszcze zły wynik sumy i okazało się, że to przez zupełnie zbędną pętlę for, która została w kodzie po poprzedniej wersji... W każdym razie, teraz wszystko działa. Wielkie dzięki za pomoc!