contar clicks

[zapt142]

Hola a todos los foreros

Necesito contar las veces que es pulsado un boton en un formulario y mostrarlo en un campo de texto, hasta el momento tengo lo siguiente

Código HTML:

function suma_completa_uno(){
	a= 0;
	document.ordenes.completa_uno.onclick = a++;
	document.ordenes.cantidad_completa_uno.value = a++;
}

pero con este codigo al presionarlo la primera vez cambia de cero a 1 en el value del campo de texto pero no suma los demás clicks que le aga

y mi formulario es este:

Código PHP:

 <form name="ordenes" method="post">
<tr>
<?php
$query = "SELECT guisos .  * , menus.id_guiso1, menus.dia
FROM menus
INNER JOIN guisos
WHERE menus.id_guiso1 = guisos.id_guiso
AND menus.dia = '$hoy' ";
$result = mysql_query($query,$conn);
$fila = mysql_fetch_array($result);
?>
<td><input type="button" name="completa_uno" value="<?php echo $fila['desc_guiso']; ?>" onclick="suma_completa_uno()"/></td>
<td><input type="text" name="cantidad_completa_uno" value=0 readonly="readonly" /></td>
<td><input type="submit" name="borra_completa_uno" id="borrar_completa_uno" value="Borrar" /></td>  

gracias de antemano

[Adler]

Hola

La variable a ha de ser global

Suerte

[maycolalvarez]

primero le asignas la función suma_completa_uno() al evento onclick del input name="completa_uno", luego en la misma función estableces a=0; y pretendes reasignarle al evento onclick del mismo input un a++ (que no es una función) ¿te parece lógico?

además tienes un input name="cantidad_completa_uno" que supongo que debe de ser acumulador que necesitas, ¿entonces por qué no lo usas directamente?

otro punto que creo suponer es que vas a generar varias filas, por lo que necesitas un contador en php para que javascript las diferencie ¿no es así:?
<td><input name="cantidad_completa_1" />...</td>
<td><input name="cantidad_completa_2" />...</td>
<td><input name="cantidad_completa_3" />...</td>

entonces el php seria así:

Código PHP:

  <?php
$result = mysql_query($query,$conn);
$cont=0;
while($fila = mysql_fetch_array($result)){
$cont++;
?>
<td><input type="button" name="completa_<?php echo $cont; ?>" value="<?php echo $fila['desc_guiso']; ?>" onclick="suma_completa(<?php echo $cont; ?>)"/></td>
<td><input type="text" name="cantidad_completa_<?php echo $cont; ?>" value=0 readonly="readonly" /></td>
<td><input type="submit" name="borra_completa_<?php echo $cont; ?>" id="borrar_completa_uno" value="Borrar" /></td>
<?php
}
?>   

y la parte del javascript quedaría completamente funcional así:

Código Javascript:

function suma_completa(i_cont){
    //forma correcta de obtener los inputs:
    var contador = document.forms['ordenes'].elements['cantidad_completa_'+i_cont];
    contador.value = parseInt(contador.value)+1;
}
 

[zapt142]

gracias por sus respuestas

pero no no voy a generar varias filas solo necesito saber la cantidad de clicks que se hacen en el botón.

y me podrías explicar un poco mejor tu respuesta porque no la entendi del todo

[Adler]

Iniciado por zapt142

y me podrías explicar un poco mejor tu respuesta porque no la entendi del todo

La respuesta de ¿qui´´en?

[zapt142]

Pues de los dos si se pudiera no esta de más...

pero concentrandome en la tuya Adler tendria que poner la a global no?

seria asi:

Código HTML:

var a = 0;
function suma_completa_uno(){
	document.ordenes.completa_uno.onclick = a++;
	document.ordenes.cantidad_completa_uno.value = a++;
}

pues me sigue asiendo lo mismo solo la primera vez que hago click cambia el value.

[Adler]

Hola

El problema lo tienes en esta linea

document.ordenes.completa_uno.onclick = a++;

Est´´as tratando a la variable a como una función ya que le est´´as asignando al elemento un evento. Cambia el nombre del campo por cualquier otro que no sea el de un evento. Mira que hay nombres sobre los que escoger

Suerte

[zapt142]

Ya funciona gracias por la explicación.
además de investigar por mi parte..
bueno pos aqui les dejo el codigo por si alguien algún día
necesita algo similar

Código HTML:

var a = 0;
function suma_completa_uno(){
	document.ordenes.cantidad_completa_uno.value = a++;
}