arrays

ignatar20 · #1 (**permalink**) 14/10/2012, 11:41

Muy buenas tarde.
Hago este tema para saber porque no me funciona el siguiente código:

var cArray = new Array(7);
var pArray = new Array(7);
var nArray = new Array(7);
var tArray = new Array(7);
var totalcompra = 0;

cArray[0] = document.getElementById('c').value;
cArray[1] = document.getElementById('c1').value;
cArray[2] = document.getElementById('c2').value;
cArray[3] = document.getElementById('c3').value;
cArray[4] = document.getElementById('c4').value;
cArray[5] = document.getElementById('c5').value;

pArray[0] = document.getElementById('a').value;
pArray[1] = document.getElementById('a1').value;
pArray[2] = document.getElementById('a2').value;
pArray[3] = document.getElementById('a3').value;
pArray[4] = document.getElementById('a4').value;
pArray[5] = document.getElementById('a5').value;

for (var i = 0; i <= 7; i++)
{
totalcompra = totalcompra + (parseInt(cArray[i]) + parseInt(pArray[i])) ;

}

alert(totalcompra);

A la hora de ejecutar me sale que totalcompra es NaN.
Nose donde fallo, si alguien sabe donde esta el error decidmelo por favor.

Gracias

emprear · #2 (**permalink**) 14/10/2012, 13:43

Por que 7? si tu array no tiene esa cantidad de elementos
Supongo, sin ver tu html, que sería algo asi

Código HTML:

Ver original<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>titulo</title> 
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
 
<script type="text/javascript">
//<![CDATA[
function suma(){
var cArray = new Array();
var pArray = new Array();
var nArray = new Array();
var tArray = new Array();
var totalcompra = 0;
 
cArray[0] = document.getElementById('c').value;
cArray[1] = document.getElementById('c1').value;
cArray[2] = document.getElementById('c2').value;
cArray[3] = document.getElementById('c3').value;
cArray[4] = document.getElementById('c4').value;
cArray[5] = document.getElementById('c5').value;
 
pArray[0] = document.getElementById('a').value;
pArray[1] = document.getElementById('a1').value;
pArray[2] = document.getElementById('a2').value;
pArray[3] = document.getElementById('a3').value;
pArray[4] = document.getElementById('a4').value;
pArray[5] = document.getElementById('a5').value;
 
for (i=0;i<cArray.length;i++)
 
{
totalcompra = totalcompra + (parseInt(cArray[i]) + parseInt(pArray[i])) ;
//alert(parseInt(cArray[i]) + parseInt(pArray[i]));
 
}
 
alert(totalcompra);
}
//]]>
</script>
</head>
<body onload="suma();">
<form action="#">
<p>
<input type="text" id="a" value="5" />
<input type="text" id="a1" value="5" />
<input type="text" id="a2" value="5" />
<input type="text" id="a3" value="5" />
<input type="text" id="a4" value="5" />
<input type="text" id="a5" value="5" />
<br />
<input type="text" id="c" value="8" />
<input type="text" id="c1" value="5" />
<input type="text" id="c2" value="5" />
<input type="text" id="c3" value="5" />
<input type="text" id="c4" value="5" />
<input type="text" id="c5" value="5" />
</p>
</form>
</body>
</html>

Saludos

ignatar20 · #3 (**permalink**) 14/10/2012, 13:56

Este es el código que tengo ( ya he quitado el tamaño del array)

Código HTML:

Ver original<html>
    <head>
        <title>Factura</title>
        <script>
            function precio_total ()
            {
                var cArray = new Array ();
                var pArray = new Array ();
                var nArray = new Array ();
                var tArray = new Array ();
                var totalcompra = 0;
                
                cArray[0] = document.getElementById('c').value; 
                cArray[1] = document.getElementById('c1').value;
                cArray[2] = document.getElementById('c2').value;
                cArray[3] = document.getElementById('c3').value;
                cArray[4] = document.getElementById('c4').value;
                cArray[5] = document.getElementById('c5').value;
                cArray[6] = document.getElementById('c6').value;
                
                pArray[0] = document.getElementById('a').value;
                pArray[1] = document.getElementById('a1').value;
                pArray[2] = document.getElementById('a2').value;
                pArray[3] = document.getElementById('a3').value;
                pArray[4] = document.getElementById('a4').value;
                pArray[5] = document.getElementById('a5').value;
                pArray[6] = document.getElementById('a6').value;
                
                for (var i = 0; i <= 6; i++)
                {   
                    totalcompra = totalcompra + (parseInt(cArray[i]) * parseInt(pArray[i])) ;
                    
                
                }
                
                alert(totalcompra);
                
            }
 
        </script>
    </head>
    <body>
        <form>
            <table border="1">
                <tr>
                    <td >Nombre Art&iacute;culo</td>
                    <td >Cantidad</td>
                    <td >Precio(articulo)</td>
                </tr>
                <tr>
                    <td ><input type="text" id="n"></td>
                    <td ><input type="text" id="c"></td>
                    <td ><input type="text" id="a"></td>    
                </tr>
                <tr>
                    <td ><input type="text" id="n1"></td>
                    <td ><input type="text" id="c1"></td>
                    <td ><input type="text" id="a1"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n2"></td>
                    <td ><input type="text" id="c2"></td>
                    <td ><input type="text" id="a2"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n3"></td>
                    <td ><input type="text" id="c3"></td>
                    <td ><input type="text" id="a3"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n4"></td>
                    <td ><input type="text" id="c4"></td>
                    <td ><input type="text" id="a4"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n5"></td>
                    <td ><input type="text" id="c5"></td>
                    <td ><input type="text" id="a5"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n6"></td>
                    <td ><input type="text" id="c6"></td>
                    <td ><input type="text" id="a6"></td>
                    
                </tr>
                <tr>
                    <td ><input type="text" id="n7"></td>
                    <td ><input type="text" id="c7"></td>
                    <td ><input type="text" id="a7"></td>
                    
                </tr>
                <tr>
                    <td colspan="4" align="right"><input type="button" style="width:100px" id="boton" value="total" onClick= "precio_total ()"></td>
                </tr>
                
            </table>
        </form>
    </body>
</html>

Lo que pasa es que si pongo "totalcompra = totalcompra + (parseInt(cArray[0]) * parseInt(pArray[0])) ;" si me sale la multiplicación entre ambos, pero al poner "i" me sale NaN, y no se dónde está el fallo

furoya · #4 (**permalink**) 14/10/2012, 14:42

Es que ya te lo resolvieron.

¿Entendés el código?

Suponiendo que la respuesta sea "sí", ¿miraste la corrección?

Probé el tuyo con los valores 0 por omisión en todos los input y funciona perfectamente. Puede que alguna sintaxis no sea reconocida en algún navegador, pero en principio, anda.

emprear · #5 (**permalink**) 14/10/2012, 15:20

Cuando hacés cosas como estas, lo primero es validar que los campos NO estén vacíos y que contengan, obviamente, un número

Saludos

IsaBelM · #6 (**permalink**) 15/10/2012, 06:46

Cita:

var arr = (cArray[i] == '')? 0:parseInt(cArray[i]);
totalcompra += (arr * pArray[i]);

se valida que el control cantidad no esté vacío. en caso de estar vacío, su valor será 0. el control precio ha de tener un valor por defecto

ignatar20 · #7 (**permalink**) 16/10/2012, 08:00

Ya lo hice, pero de otra forma xD
Gracias por las respuestas

Por cierto, ¿cómo hago que, si una posición del array que está vacío me ponga un 0 en vez de "undefined"?

Ésta es la función que tengo, a ver si me podéis decir donde fallo xD:

Código HTML:

Ver originalfunction precio_total ()
            {
                var cArray = new Array ();
                var pArray = new Array ();
                var nArray = new Array ();
                var tArray = new Array ();
                var totalcompra = 0;
                
                cArray[0] = document.getElementById('c').value; 
                cArray[1] = document.getElementById('c1').value;
                cArray[2] = document.getElementById('c2').value;
                cArray[3] = document.getElementById('c3').value;
                cArray[4] = document.getElementById('c4').value;
                cArray[5] = document.getElementById('c5').value;
                cArray[6] = document.getElementById('c6').value;
                cArray[7] = document.getElementById('c7').value;
                
                pArray[0] = document.getElementById('a').value;
                pArray[1] = document.getElementById('a1').value;
                pArray[2] = document.getElementById('a2').value;
                pArray[3] = document.getElementById('a3').value;
                pArray[4] = document.getElementById('a4').value;
                pArray[5] = document.getElementById('a5').value;
                pArray[6] = document.getElementById('a6').value;
                pArray[7] = document.getElementById('a7').value;
                
                for (var i = 0; i <= 6; i++)
                {   
                    if (cArray[i] == "")
                    {
                        totalcompra=totalcompra;
                    }
                    else
                    {
                        if(pArray[i] == "")
                        {
                            totalcompra=totalcompra;
                        }
                        else
                        {
                            totalcompra = totalcompra + (parseInt(cArray[i]) * parseInt(pArray[i])) ;
                            tArray[i] = parseInt(cArray[i]) * parseInt(pArray[i]);
                        }
                    }
                    
                    if(tArray[i] == "")
                    {
                        tArray[i] = 0;
                    }
                
                }
                
                /*for (var x=0; x <=6; x++)
                {
                    if(tArray[x] == "")
                    {
                        tArray[x] = 27;
                    }
                }*/

Saludos y gracias de nuevo

furoya · #8 (**permalink**) 16/10/2012, 08:51

Lo validás igual que hacés cuando buscás uno vacío (""), pero con undefined.

Una duda ¿los valores en dinero son siempre redondos? ¿No hay centavos?.

ignatar20 · #9 (**permalink**) 16/10/2012, 15:04

Ya está.
La verdad es que lo de los céntimos no me lo pedía, pero me gustaría saber como se haría, por si en algún momento me lo piden.

Gracias por su ayuda

furoya · #10 (**permalink**) 16/10/2012, 18:29

Decinos la verdad : en tu vida abriste un manual de javascript, ¿no?.

Y lo peor es que no te puedo decir nada, porque apenas te explicamos cómo se mira un asunto, lo resolvés.
Supongo que ya tenés experiencia con otros lenguajes.

Para los decimales, validás con parseFloat().
parseInt() te devuelve solamente la parte entera.

Igual, me parece que habría que poner una máscara, para que siempre el valor termine con céntimos, aunque sean 0.00. Pero sería una exquisitez. Esperá a que te lo pidan y después te ocupás.