Que veis mal en este codigo

chari · #1 (**permalink**) 06/05/2004, 05:26

<HTML>
<HEAD>
<TITLE>Ejemplo de un cuadro prompt</TITLE>
</HEAD>
<BODY>
<SCRIPT LANGUAGE ="Javascript">
var producto;
var tipoproducto;
var precio;
var unidades;

unidades = prompt("Introduzca el numero de unidades")
while (unidades != 0 && unidades != null){
function tipo_de_producto(tipo_producto){
switch (tipoproducto) {
case "1":
precio = 400;
break;
case "2":
precio = 300;
break;
case "3":
precio = 200;
break;
case "4":
precio = 100;
break;
}
}
function IVA(){
if (unidades < 10){
var IVA ="16%";
}
else
{
var IVA ="4%";
}

function precio_con_IVA(){
if (unidades < 10){
var precioconIva= (precio * unidades) * 1.16;
}
else
{
var precioconIva= (precio * unidades) * 1.04;
}
}
tipo_de_producto();
IVA();
precio_con_IVA();
producto = prompt("Introduzca nombre del producto");
tipoproducto = prompt("Introduzca tipo del producto (Del 1 al 4):");

alert("Producto: " + producto + " Tipo de Producto: " + tipoproducto + " Unidades: " + unidades + " Precio Total: " + precioconIva + " IVA Aplicado: " + IVA);

unidades = prompt("Introduzca el numero de unidades")
}
}
</SCRIPT>
</BODY>
</HTML>

epplestun · #2 (**permalink**) 06/05/2004, 06:32

ala aqui tienes:

Código:

<HTML>
<HEAD>
<TITLE>Ejemplo de un cuadro prompt</TITLE>
</HEAD>
<BODY>
<SCRIPT LANGUAGE ="Javascript">
var producto;
var tipoproducto;
var precio;
var unidades;
var precioconIva;

unidades = prompt("Introduzca el numero de unidades");
while (unidades != 0 && unidades != null){

	function tipo_de_producto(tipo_producto){
		switch (tipoproducto) {
			case "1":
				precio = 400;
				break;
			case "2":
				precio = 300;
				break;
			case "3":
				precio = 200;
				break;
			case "4":
				precio = 100;
				break;
		}
	}
	
	function IVA() {
		if (unidades < 10) {
			var IVA ="16%";
		} else {
			var IVA ="4%";
		}
	}

	function precio_con_IVA() {
		if (unidades < 10) {
			var precioconIva= (precio * unidades) * 1.16;
		} else {
			var precioconIva= (precio * unidades) * 1.04;
		}
	}

	tipo_de_producto();
	IVA();
	precio_con_IVA();

	producto = prompt("Introduzca nombre del producto");
	tipoproducto = prompt("Introduzca tipo del producto (Del 1 al 4):");
	unidades = prompt("Introduzca el numero de unidades")

	alert("Producto: " + producto + "\n Tipo de Producto: " + tipoproducto + "\n Unidades: " + unidades + "\n Precio Total: " + precioconIva + "\n IVA Aplicado: " + IVA);

}
</SCRIPT>
</BODY>
</HTML>

chari · #3 (**permalink**) 07/05/2004, 01:51

Hola he retocado un poco el codigo pq habia algunos apartados que no funcionaban. Ahora solo me falla el apartado que deberia de mostrarme el precio con IVA del producto, me muestra undefinided. Creo que elproblema esta que el switch no me devuelve precio.

Pq puede ser.

Este es el codigo:

<HTML>
<HEAD>
<TITLE>Ejemplo de un cuadro prompt</TITLE>
</HEAD>
<BODY>
<SCRIPT LANGUAGE ="Javascript">
var producto;
var tipoproducto;
var precio;
var unidades;
var precioconIva;
var IVA_producto;

unidades = prompt("Introduzca el numero de unidades");
while (unidades != 0 && unidades != null){

function tipo_de_producto(tipoproducto){
switch (tipoproducto) {
case "1":
precio = 400;
return precio;
break;
case "2":
precio = 300;
return precio;
break;
case "3":
precio = 200;
return precio;
break;
case "4":
precio = 100;
return precio;
break;
}
}

function IVA(unidades) {
if (unidades < 10) {
IVA_producto ="16%";
} else {
IVA_producto ="4%";
}
}

function precio_con_IVA(precio,unidades) {
if (unidades < 10) {
var precioconIva= (precio * unidades) * 1.16;
} else {
var precioconIva= (precio * unidades) * 1.04;
}
}

tipo_de_producto();
IVA(unidades);
precio_con_IVA(precio,unidades);

producto = prompt("Introduzca nombre del producto");
tipoproducto = prompt("Introduzca tipo del producto (Del 1 al 4):");

alert("Producto: " + producto + "\n Tipo de Producto: " + tipoproducto + "\n Unidades: " + unidades + "\n Precio Total: " + precioconIva + "\n IVA Aplicado: " + IVA_producto);

unidades = prompt("Introduzca el numero de unidades");
}
</SCRIPT>
</BODY>
</HTML>

epplestun · #4 (**permalink**) 07/05/2004, 02:30

Oye, por que no te molestas en leer el codigo que te puse? te da undefined por que la variable la redefines dentro de una funcion!!!!!!!

Código:

precioconIva = ....

no le pongas var por delante ya que redefines la variable y queda englobada solo en el ambito de la funcion ,es decir no puede ser accedida desde fuera de esa funcion .... por eso te da error.

chari · #5 (**permalink**) 07/05/2004, 02:40

Lo siento pero el codigo que tu me diste no funcionaba, ojala hubiese funcionado, si no te lo diría.

Y lo que dices de que hay declarar var precioIVA, no se si te has dado cuenta de que esta declarado var precioconIVA que es lo que utilizo y no me sale undefinided sino Nan que al escribirlo me habia confundido.

Y eso de que leas, suelo leer las cosas antes de utilizarlas, y las compruebo pero si no funcionan mejor decirlo, por si acaso quiere utilizar el codigo alguien.

Saludos.

epplestun · #6 (**permalink**) 07/05/2004, 02:58

Disuclpa, he pagado los platos rotos contigo ... estoy mirandome tu code ahora a ve si te lo soluciono

epplestun · #7 (**permalink**) 07/05/2004, 03:02

ale ya funciona jeje

Código:

<HTML>
<HEAD>
<TITLE>Ejemplo de un cuadro prompt</TITLE>
</HEAD>
<BODY>
<SCRIPT LANGUAGE ="Javascript">
var producto;
var tipoproducto;
var precio;
var unidades;
var precioconIva;
var IVA_producto;

unidades = prompt("Introduzca el numero de unidades");

producto     = prompt("Introduzca nombre del producto");
tipoproducto = prompt("Introduzca tipo del producto (Del 1 al 4):");
//while (unidades != 0 && unidades != null){

function tipo_de_producto(tipoproducto){
	switch (tipoproducto) {
		case "1":
			precio = 400;
			break;
		case "2":
			precio = 300;
			break;
		case "3":
			precio = 200;
			break;
		case "4":
			precio = 100;;
			break;
	}
	return precio;
}

function IVA(unidades) {
	if (unidades < 10) {
		IVA_producto = "16%";
	} else {
		IVA_producto = "4%";
	}
}

function precio_con_IVA(precio,unidades) {
	if (unidades < 10) {
		precioconIva = (precio * unidades) * 1.16;
	} else {
		precioconIva = (precio * unidades) * 1.04;
	}
}

tipo_de_producto(tipoproducto);
IVA(unidades);
precio_con_IVA(precio,unidades);

alert((precio * unidades));
alert("Producto: " + producto + "\n Tipo de Producto: " + tipoproducto + "\n Unidades: " + unidades + "\n Precio Total: " + precioconIva + "\n IVA Aplicado: " + IVA_producto);

//unidades = prompt("Introduzca el numero de unidades");
//}
</SCRIPT>
</BODY>
</HTML>

tenias que pasarle como paremetro a la funcion el valor

chari · #8 (**permalink**) 07/05/2004, 03:50

Ahora si funciona. Muchas gracias.

epplestun · #9 (**permalink**) 07/05/2004, 03:56

para eso estamos