Variable undefined

[erxaca]

Hola compañeros,

haber si me podeis echar un cable y decirme porque me dice que la variable imgbebida no esta definida:

JS

Código HTML:

	var j = 0;
	var bebidas = new Array(10);
	var altbebidas = new Array(10);
	var namebebidas = new Array(10);
    function cambiarValores(obj, prod) { //Recibimos el objeto y el nombre de la bebida
      var img = obj.src, k;
	  img = img.split('/').pop();
      img = img == 'boton-bebidas-' + obj.alt + '-off.png' ? 'images/boton-bebidas-' + obj.alt + '-on.png' : 'images/boton-bebidas-' + obj.alt + '-off.png';
      obj.src = img;
	  for(i=0;i<10;i++) {
	    if(bebidas[i] == prod) {
		  var bebida = bebidas[i];
		  var altbebida = altbebidas[i];
          var imgbebida = document.getElementsByName(bebida)[0];
		  imgbebida.src = 'images/boton-bebidas-' + altbebida + '-off.png';
	    }
		var conta = 0;
		for(k=0;k<10;k++){if(i!=k){conta++;}}
		if(conta==10){i++;}
	  }
	  bebidas[j] = prod;
	  altbebidas[j] = obj.alt
	  namebebidas[j] = obj.name;
}

HTML

Código HTML:

 <td>
	              <img src="images/boton-bebidas-1-off.png" name="c1" alt="1" onclick="cambiarValores(this,'CocaCola')" />
	              <input type="hidden" name="coca-cola-1" value="bebida" />
	            </td>
	            <td>
	              <img src="images/boton-bebidas-2-off.png" name="c2" alt="2" onclick="cambiarValores(this,'CocaCola')" />
	              <input type="hidden" name="coca-cola-2" value="bebida" />
	            </td>
	          </tr> 

Saludos ;)

[caricatos]

Hola:

Según parece quieres referenciar un elemento por su nombre (name), pero ese valor se lo has asignado al value...

Saludos

[IsaBelM]

le has asignado el value el nombre del elemento

edito: lo siento, creo que aún no habías publicado

[erxaca]

Os refereis a estas lineas no?

Código HTML:

var bebida = bebidas[i];

Código HTML:

 <input type="hidden" name="coca-cola-2" value="bebida" /> 

Mismo nombre de variable y mismo valor de value. Si es eso ya lo probé cambiando el nombre de la variable javascript por bbida por ejemplo y sigue dando el mismo fallo, imgbebida undefined.

Saludos ;)

[laratik]

No entiendo muy bien la función del script, pero el error que comentas sucede por que estas enviando como valor 'CocaCola' y ningun elemento dentro del html tiene ese nombre, si en cambio utilizas por ejemplo:

Código HTML:

<img src="images/boton-bebidas-1-off.png" name="c1" alt="1" onclick="cambiarValores(this,'c1')" /> 

El error desaparecera, como consejo trata de organizar mejor tu codigo, asignaciones como:

Código Javascript:

var bebida = bebidas[i];
 

se me hacen innecesarias, si ya sabes que bebidas[i] es igual a prod, sigue utilizando prod.

[erxaca]

Funciona como dices laratik pero no me vale


Por que no puedo enviar una cadena, guardarla y comprobar si la nueva cadena pasada no es la misma que la anterior? Me parece un fallo por parte de javascript a no ser que tenga una explicación logica y util.

laratik debo pasar en cada serie un mismo valor de alguna manera. La idea es la siguiente, tenemos varias series de numeros:

1 2 3 4 5
1 2 3 4 5

Al seleccionar un numero(cada numero es una imagen) de la primera serie, este cambiara su imagen. Al seleccionar sobre otro numero de la misma serie, la anterior numero volvera a tener la imagen por defecto y el nuevo numero seleccionado cambiara su imagen. Así sucesivamente en cada serie sin importar la seleccion entre distintas series.

[IsaBelM]

el error está aquí

var imgbebida = document.getElementsByName(bebida)[0];

según tu lógica, bebidas[i] es igual a cocacola pero no existe ningún control con tal nombre

[erxaca]

Iniciado por IsaBelM

el error está aquí

según tu lógica, bebidas[i] es igual a cocacola pero no existe ningún control con tal nombre

Hola de nuevo IsaBelM,

intentas decirme que el fallo es porque bebidas[i] esta vacio? utilizo mal el array?

Siento si la pregunta tiene poco fundamento pero estoy empezando con javascript

[IsaBelM]

prueba de asignandole a los controles el name CocaCola

<input type="hidden" name="CocaCola" value="bebida" />

suponiendo que esta condicional se cumple if(bebidas[i] == prod)

[laratik]

Si entiendo lo que necesitas, creo que has planteado mal la lógica de la función, cambia los valores de imágenes y prueba la siguiente función:

Código HTML:

<html>
<head>
<script type="text/javascript">
var anterior = new Array();
var anteriorsrc = new Array();

function cambiarValores(ele,serie) {
	if(typeof anteriorsrc[serie] != "undefined") {
		anterior[serie].src = anteriorsrc[serie];
	}
	anterior[serie] = ele;
	anteriorsrc[serie] = ele.src;
	ele.src = "images/flecha_azul.gif";
}
</script>

</head>
<body>
<table>
<tr>
<td>
	              <img src="images/flecha_hover.png" name="c1" alt="1" onClick="cambiarValores(this,1)" />
	            </td>
	            <td>
	              <img src="images/errorIcon.gif" name="c2" alt="2" onClick="cambiarValores(this,1)" />
	            </td>
	          </tr>
<tr>
<td>
	              <img src="images/flecha_hover.png" name="c1" alt="1" onClick="cambiarValores(this,2)" />
	            </td>
	            <td>
	              <img src="images/errorIcon.gif" name="c2" alt="2" onClick="cambiarValores(this,2)" />
	            </td>
	          </tr>
</table>
</body>
</html> 

El código anterior crea dos lineas de imágenes dentro de la tabla, como comentas:
1 2
1 2
si pulsas el 2 de la primera linea se carga una nueva imagen:
1 3
1 2
y si pulsas el 1, el 2 vuelve a la normalidad y ahora la nueva imagen se colocara en 1:
3 2
1 2
Lo mismo sucede en la segunda linea, por lo menos eso fue lo que entendi que necesitas. Como te ha comentado IsaBelM y mi persona, el error que presentas se debe a que bebidas[i] CONTIENE UN VALOR (no esta vació) que lo tratas de asignar como si fuera el name de un elemento, pero resulta que ningún elemento tiene este name. SALUDOS.

[erxaca]

laratik gracias ;), es mas o menos esa la idea.

No entiendo esta parte de tu código:

Código HTML:

	if(typeof anteriorsrc[serie] != "undefined") {
		anterior[serie].src = anteriorsrc[serie];
	}

que quiere decir?


De todas formas no entiendo por qué en tu código le pasas para realizar el control un número y funciona, mientras yo le pasaba una cadena

Código HTML:

onclick="cambiarValores(this,'CocaCola')"

y no funcionaba.

[paskuini]

se la deben haber tomado :P

(lo siento, no he podido resistirme...)

[laratik]

Iniciado por erxaca

No entiendo esta parte de tu código:

Código HTML:

	if(typeof anteriorsrc[serie] != "undefined") {
		anterior[serie].src = anteriorsrc[serie];
	}

que quiere decir?

Quiere decir que si el elemento en anteriorsrc[serie] no ha sido definido no ejecute la siguiente linea, esto para evitar posibles errores en la asignación del array. Sucedería al principio cuando NO haz creado ningún elemento dentro de esa posición.

Iniciado por erxaca

no entiendo por qué en tu código le pasas para realizar el control un número y funciona, mientras yo le pasaba una cadena

Código HTML:

onclick="cambiarValores(this,'CocaCola')"

y no funcionaba.

No tiene nada que ver lo que le pases, lo que interesa en como lo utilices dentro de la función, ne mi caso el numero que le envió es el indice del array (series, que por cierto debe utilizar 0 y 1 no 1 y 2):

function cambiarValores(ele,serie) {
if(typeof anteriorsrc[serie] != "undefined") {
anterior[serie].src = anteriorsrc[serie];
}
anterior[serie] = ele;
anteriorsrc[serie] = ele.src;
ele.src = "images/flecha_azul.gif";
}

En cambio tu lo que hacías al enviar la cadena era utilizarla como name de un elemento:

function cambiarValores(obj, prod)

bebidas[i] == prod

var bebida = bebidas[i]; //asignación innecesaria bebida [i] es igual a prod

var imgbebida = document.getElementsByName(bebida)[0]; //exactamente lo mismo que hacer: var imgbebida = document.getElementsByName(prod)[0];

si prod = "CocaCola" y no existe ningún elemento con este name, pues te va a saltar el mensaje de que imgbebida es indefinido.

Espero que halla sido lo suficientemente claro. SALUDOS.

[erxaca]

Espero que halla sido lo suficientemente claro. SALUDOS.

Mejor explicado imposible, muchas gracias ;)

Saludos