Comparación de fechas

occiso · #1 (**permalink**) 17/12/2010, 13:23

Saludos compañeros,

Tengo que hacer una clasificación de facturas. Si la fecha de factura es menor a una determinada fecha la inserto en una variable y si no la inserto en otra. El problema es curioso. Ahora mismo funciona correctamente si indico la fecha con el formato dd/mm/yyyy; pero si le aplico otro formato funciona al revés de como debería.

Ejemplo en el código:
si fecha factura <= 31/11/2010 --> versión antigua
si fecha factura > 31/11/2010 --> versión nueva.

Código PHP:

  <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> 
<html> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"> 
<title>Insert title here</title> 
<script> 
 
/*Divide la factura en dos tipos: los que se visualizan en la versión antigua y en la nueva. 
 *  @param {p_facturas} id facturas separadas por comas (en caso de array su índice debe corresponderse con p_fechaFactura) 
 *  @param {p_fechaFactura} fecha en la que se realizan la facturas separadas por comas (formato dd/mm/aaaa) 
 *  @return devuelve las facturas separadas en dos partes por los caracteres ##. 
 */ 
 
function clasificarFacturasSegunFecha(p_facturas, p_fechaFactura) { 
     
    //Fecha a partir de la que  empezamos a aplicar la nueva versión de crear facturas. 
    var FECHA_LIMITE = new Date('30/11/2010'); 
             
    var facturaVersionAntigua = new String(''); 
    var facturaVersionNueva   = new String(''); 
         
    var listaFacturas = new String(p_facturas); 
    listaFacturas = listaFacturas.split(',');  
     
    var listaFechaFactura = new String(p_fechaFactura); 
    listaFechaFactura = listaFechaFactura.split(','); 
             
    debugger; 
    var index; 
    for (index=0; index< listaFacturas.length; index++){ 
        if (new Date(listaFechaFactura[index]).getDate() <= FECHA_LIMITE.getDate()){ //solo funciona correctamente con formato dd/mm/yyyy 
        //if (Date.parse(listaFechaFactura[index]) <= FECHA_LIMITE){  //no funciona 
            if (!facturaVersionAntigua.length ) facturaVersionAntigua = listaFacturas[index]; 
            else facturaVersionAntigua += ',' + listaFacturas[index];                         
        }         
        else { 
            if (!facturaVersionNueva.length)  
                facturaVersionNueva = listaFacturas[index]; 
            else  
                facturaVersionNueva += ',' + listaFacturas[index]; 
        } 
                 
    } 
    return facturaVersionNueva +"##"+ facturaVersionAntigua; 
} 
 
</script> 
</head> 
<body> 
<button onclick="clasificarFacturasSegunFecha('11,22', '25/11/2010, 10/12/2010')">formato dd/mm/yyyy</button> 
<button onclick="clasificarFacturasSegunFecha('11,22', 'Thu Nov 25 2010 00:00:00 GMT+0100, Fri Dec 10 2010 00:00:00 GMT+0100')">otro formato</button> 
</body> 
</html>

Supongo que se trata de algún problema de conversión, pero no logro averiguarlo.

snahor_ · #2 (**permalink**) 17/12/2010, 19:50

No se en que navegador te funciona esto

Código:

new Date('30/11/2010')

es incorrecto o no es valido, dependiendo del navegador.

La manera correcta de crear un nuevo objeto Date (para tu caso) es

Código:

new Date(2010, 10, 30)

Donde el primer parametro es el a#o, el segundo es el numero de mes pero en base a 0 (es decir se empieza a contar desde 0, por ejemplo enero es el mes 0), y el ultimo, es el dia.

Unas sugerencias, no es necesario que uses new String, basta con que uses ''.

occiso · #3 (**permalink**) 18/12/2010, 05:03

Cita:

Iniciado por snahor_

No se en que navegador te funciona esto

Código:

new Date('30/11/2010')

es incorrecto o no es valido, dependiendo del navegador.

La manera correcta de crear un nuevo objeto Date (para tu caso) es

Código:

new Date(2010, 10, 30)

Donde el primer parametro es el a#o, el segundo es el numero de mes pero en base a 0 (es decir se empieza a contar desde 0, por ejemplo enero es el mes 0), y el ultimo, es el dia.

Unas sugerencias, no es necesario que uses new String, basta con que uses ''.

Bueno, si miras en la web de sun de referencia sobre el objeto Date verás

Cita:

new Date(dateString)

dateString: String value representing a date. The string should be in a format recognized by the Date.parse method. (en donde se incluye el dd/mm/yyyy)

De todas maneras lo que me has dicho ya lo había probado y tampoco funciona; es fácilmente comprobable.

Para testear utilizo firefox y firebug.

EDITO: He estado haciendo pruebas y en parte tenías razón. Estabas en lo correcto porque la forma correcta de llamar al constructor es como me has dicho:

Código:

new Date(2010, 10, 30)

El error es que la fecha generada no es la del 30 de Nov de 2010, sino la del 30 de Dec de 2010; es decir, que los meses empieza a contar desde [0-11] siendo el cero el mes de enero y el once el mes de diciembre, por eso me seguía funcionando de una manera inesperada. si le indicas la fecha como:

Código:

new Date(2010, 12, 30)

Te la indica como 30 de enero del 2011. Me parece una buena cagada por parte de los que desarrollaron ese contructor, es anti-intuitivo total.

Tengo la posibilidad de utilizar los formatos de conversion del Extjs. no sé si merece la pena utilizarlos o hay una solución en javascript que sea más elegante. Seguro que no soy el primero que me topo con este dilema.

occiso · #4 (**permalink**) 18/12/2010, 10:48

Código PHP:

  <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> 
<html> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"> 
<title>Insert title here</title> 
<script type="text/javascript"> 
 
/*Divide la factura en dos tipos: los que se visualizan en la versión antigua y en la nueva. 
 *  @param {p_facturas} id facturas separadas por comas (en caso de array su índice debe corresponderse con p_fechaFactura) 
 *  @param {p_fechaFactura} fecha en la que se realizan la facturas separadas por comas (formato dd/mm/aaaa) 
 *  @return devuelve las facturas separadas en dos partes por los caracteres ##. 
 */ 
 
function clasificarFacturasSegunFecha(p_facturas, p_fechaFactura) { 
     
    //Fecha a partir de la que  empezamos a aplicar la nueva versión de crear facturas. 
    var FECHA_LIMITE = new Date(2010,10,30); 
 
    var facturaVersionAntigua = new String(''); 
    var facturaVersionNueva   = new String(''); 
         
    var listaFacturas = new String(p_facturas); 
    listaFacturas = listaFacturas.split(',');  
     
    var listaFechaFactura = new String(p_fechaFactura); 
    listaFechaFactura = listaFechaFactura.split(','); 
 
    debugger; 
    var index; 
    for (index=0; index< listaFacturas.length; index++){ 
        if (new Date(listaFechaFactura[index]).getTime() <= FECHA_LIMITE.getTime()){ 
            if (!facturaVersionAntigua.length ) facturaVersionAntigua = listaFacturas[index]; 
            else facturaVersionAntigua += ',' + listaFacturas[index]; 
        }         
        else { 
            if (!facturaVersionNueva.length)  
                facturaVersionNueva = listaFacturas[index]; 
            else  
                facturaVersionNueva += ',' + listaFacturas[index]; 
        } 
                 
    } 
    return facturaVersionNueva +"##"+ facturaVersionAntigua; 
} 
 
</script> 
</head> 
<body> 
<button onclick="clasificarFacturasSegunFecha('11,22', '25/11/2010, 10/12/2010')">formato dd/mm/aaaa</button> 
<button onclick="clasificarFacturasSegunFecha('11,22', 'Thu Nov 25 2010 00:00:00 GMT+0100, Fri Dec 10 2010 00:00:00 GMT+0100')">otro formato</button> 
</body> 
</html>

Bueno, ahora mismo funciona bien utilizando la opción "otro formato", pero no cuando la fecha de formato es dd/mm/yyyy. Me gustaría que independientemente del formato en el que le llegan las fechas en el parámetro p_fechaFactura realizara bien la comparación. ¿Alguna idea?