problem z funkcją JS

Question

Witam napotkałem problem przy tworzeniu dynamicznego formularza w JS. Chce aby JS prawdzal ktory radio jest zaznacony i dodawał do zmiennej odpowiednia wartosc 

<script type="text/javascript">
function zatwierdz()
{
	if(document.getElementById("checkbox1").checked == false &&  document.getElementById("checkbox2").checked == false && document.getElementById("checkbox3").checked == false)
	{
		alert("Musisz wybrać ofertę!");
	}
	else if (document.getElementById("checkbox3").checked)
	{
		var cenajsprzatanie = 70;
		var cenajmycie = 25;
		wybor = document.getElementById('imgstart');
		wybor.innerHTML = '<div class="wybor"><div class="opcja">Sprzątanie</div><div class="rodzaj"><input type="radio" name="rodzaj" checked id="radio1"oncilck="sprawdz()">Grób Pojedynczy<input type="radio" name="rodzaj"id="radio2" onclick="sprawdz()">Grób podwójny<input type="radio" name="rodzaj" id="radio3"oncilck="sprawdz()">Grobowiec</div><div id="wynik">'+cenajsprzatanie+'zł';'</div></div>'
	}
	else if (document.getElementById("checkbox3").checked)
	{
		var cenaswiateczna = 150;
		
	}
	else
	{
		var cenaroczna = 350;
	
	}
}
</script>
<script type="text/javascript">
function sprawdz()
{
	if(document.getElementById("radio2").checked ==true)
	{
		cenajsprzatanie = cenajsprzatanie + 25; //to ma sie pojawiac wyzej jezeli zaznaczony jest 2 radio
		
		
	}
}

 

Comments

Taka drobna wskazówka: CZYTELNOŚĆ KODU. Nazwy zmennych czy funkcji nie pisz cenaswiateczna, tylko cenaSwiateczna, ew. (jeśli w cenajsprzatenie te 'j' musi być w środku z jakiegoś powodu) to 'cenaJSprzątanie'.

---

Niestety to nie rozwiazuje mojego problemu

---

im gorsza jakość kodu tym mniej osób chętnych do pomocy ;)

Answer 1

https://codepen.io/anon/pen/rmaMma

Tak wygląda Js. Oczywiście nie na 6+, w żadnym wypadku. Ale jak zmienisz styl i nawyki z obecnych na te, które są w tym przykładzie to będzie progres. 

Answer 2

Źle do tego podchodzisz jeśli sprawdzasz to później najlepiej pobrać wartość jaka obecnie jest poprzez document.getElementById("wynik").innerText, dodać to niego odpowiednią kwoteę a następnie wpisać ponownie juz obliczona cene.

Comments

Mógłbym prosić o kod?

---

Nic chciałem robić za ciebie no ale skoro nie ogarniasz to proszę. Tylko pytaj jak czegoś nie wiesz ! ;)

 

var staryWynik = document.getElementById("wynik").innerText;
document.getElementById("wynik").innerText = parseInt(staryWynik) + 25;

 

---

Niestety nic to nie dało, po wybraniu radio cana dalej sie nie zmienia

---

Może źle zapisuje warunek albo radio nie ma funkcji checked?

Answer 3

Posprawdzaj sobie to console.log i zobacz co dostajesz w odpowiedzi w konsoli. Tak najszybciej znajdziesz przyczynę.