Forum PHP.pl

Siema pisze program w js który będzie dodawać wartości po wybraniu ich.
Może teraz jaśniej mam kilka inputów typu checkbox (są one wczytywane z bazy danych ale teraz to ominąłem dla ułatwienia):

[HTML] pobierz, plaintext 
<table>
<tr><td style="width: 400px;">Zanacz  kolorem  5zł</td><td>TAK <input type="checkbox" class="price" name="prce[]" value="5" onchange="price(this);"/></td></tr>
<tr><td style="width: 400px;">Zawsze na początku strony  10zł</td><td>TAK <input type="checkbox" class="price" name="prce[]" value="10" onchange="price(this);"/></td></tr>
</table>
<input type="hidden" id="price" name="price" value="0">
[HTML] pobierz, plaintext 

Po zaznaczeniu opcji w formularzu uruchamaiany jest skrypt js:

[PHP] pobierz, plaintext 
function price(){
 var pri = document.getElementById('price').value;
if(document.getElementByClassName('price').checked){
 var ad = document.getElementByClassName('price').value;
 var suma = pri + ad;
 document.getElementById('price').value = suma;
 }
}
 
[PHP] pobierz, plaintext 

ale funkcja nie chce zmienić wartości co robię nie tak. Użyłem this aby odwołąć się konkretnie do tego input

Masz dwa elementy o klasie price wiec to:

[HTML] pobierz, plaintext 
document.getElementByClassName('price')
[HTML] pobierz, plaintext 

pewnie będzie tablicą.
Poza tym powinieneś wg mnie przekazywać obiekt this jako parametr funkcji np:

[HTML] pobierz, plaintext 
function price( oElement ){
[HTML] pobierz, plaintext 

Wtedy oElement będzie konkretnym przekazanym checkboxem.

Dla Twojego HTML-a funkcja price() powinna wyglądać tak:

[JAVASCRIPT] pobierz, plaintext 
function price(element) {
 if(element.checked) {
  document.getElementById("price").value = document.getElementById("price") + element.value;
 } else {
  document.getElementById("price").value = document.getElementById("price") - element.value;
 }
}
[JAVASCRIPT] pobierz, plaintext 

nadal niedział tak jak by nie była wywoływana ;/

Nie działa, bo zgubiłem value po drodze. Poza tym trzeba rzutować łańcuch znaków na liczbę, bo + działa jak konkatenacja.

[JAVASCRIPT] pobierz, plaintext 
function price(element) {
 if(element.checked) {
  document.getElementById("price").value = Number(document.getElementById("price").value) + Number(element.value);
 } else {
  document.getElementById("price").value = Number(document.getElementById("price").value) - Number(element.value);
 }
}
[JAVASCRIPT] pobierz, plaintext