Como te dijeron, es común el uso de moment.js para abstraer esa capa de la lógica. Sin embargo, también se puede resolver adaptando tus fechas a un formato que cumpla con ISO-8601
var fecha1='01/12/2018',
fecha2='2018-12-01';
console.log('fecha 1 se evalúa como', new Date(fecha1).toISOString().split('T')[0]);
console.log('fecha 2 se evalúa como', new Date(fecha2).toISOString().split('T')[0]);
¿Por qué en el primer caso transpone el mes con el día? Bueno, ese es el formato en inglés. Y justamente por esa ambivalencia es mala idea usarlo en tu código.
Si tienes certeza de que todas tus fechas tienen la forma dd/MM/YYYY
puedes transformarlas haciendo una función de normalización:
function normalizaFecha(fecha_str) {
if(/\d{2}\/\d{2}\/\d{4}/.test(fecha_str)) {
return fecha_str.split('/').reverse().join('-');
}
return fecha_str;
}
Y con eso puedes hacer una comparación del tipo:
if(normalizaFecha(Objeto2.inicio) > normalizaFecha(Objeto1.fin)) {
throw new Error('Fechas se superponen');
}
Edit: Si estás usando un plugin de calendario para que el usuario elija su fecha o la ingrese en un campo de texto y, repito, te consta que viene en el formato DD/MM/YYYY puedes usar la función que te mostré. Usando moment.js podrías usar:
moment('01/06/2016', 'DD/MM/YYYY')
Para indicarle que interprete la fecha con el formato latino.
En tu caso tienes
param[1]['params'].array.forEach(element => {
if(userStart > element.startdatevalidity && userFinish > element.enddatevalidity
|| userStart < element.startdatevalidity && userFinish < element.enddatevalidity
&& userStart > element.enddatevalidity
){
console.log('good');
}else{
console.log('=(');
}
});
En ese código pienso que faltan paréntesis para aislar las condiciones (es mejor usar paŕentesis defensivos, dijo Eric S. Raymond hace casi 30 años)
Creo que es más fácil enfocarlo a la inversa:
- Si la fecha de inicio está contenida en el intervalo -> error
- Si la fecha de fin está contenida en el intervalo -> error
- Si el intervalo contiene al rango ingresado -> error
- Si el rango ingresado contiene al intervalo -> error
La tercera condición no es necesario comprobarla porque cae dentro de la primera.
Abstrayendo esto a una fución:
function validaRango(nuevoRango, rangoExistente) {
let iniNuevo = nuevoRango.startdatevalidity,
finNuevo = nuevoRango.enddatevalidity,
iniExistente = rangoExistente.startdatevalidity,
finExistente = rangoExistente.enddatevalidity;
if (iniNuevo > iniExistente && iniNuevo < finExistente ) {
return false; // fecha inicio se superpone
}
if (finNuevo > iniExistente && finNuevo < finExistente ) {
return false; // fecha fin se superpone
}
if (iniNuevo < iniExistente && finNuevo > finExistente) {
return false; // nuevo rango contiene al rango ya ocupado
}
return true;
}
Si tus valores para startdatevalidity
y enddatevalidity
te consta que vienen en formato DD/MM/YY
entonces usa la función auxiliar que te pasé:
function normalizaFecha(fecha_str) {
if(/\d{2}\/\d{2}\/\d{4}/.test(fecha_str)) {
return fecha_str.split('/').reverse().join('-');
}
return fecha_str;
}
function validaRango(nuevoRango, rangoExistente) {
let iniNuevo = normalizaFecha(nuevoRango.startdatevalidity),
finNuevo = normalizaFecha(nuevoRango.enddatevalidity),
iniExistente = normalizaFecha(rangoExistente.startdatevalidity),
finExistente = normalizaFecha(rangoExistente.enddatevalidity);
if (iniNuevo > iniExistente && iniNuevo < finExistente ) {
return false; // fecha inicio se superpone
}
if (finNuevo > iniExistente && finNuevo < finExistente ) {
return false; // fecha fin se superpone
}
if (iniNuevo < iniExistente && finNuevo > finExistente) {
return false; // nuevo rango contiene al rango ya ocupado
}
return true;
}
Y en el bucle usas:
param[1]['params'].array.forEach(element => {
if(validaRango(newConfig, element)) {
console.log('good');
}else{
console.log('=(');
}
});
Bonus track para usar moment.js con Angular/Typescript
las versiones recientes de moment.js vienen con definición de tipos por si quieres usar typescript con todas sus validaciones:
# instalas la librería
npm i moment
Y en tu componente de AngularJS haces:
import * as moment from 'moment';
Luego puedes por ejemplo crear una función que convierta un string con formato DD/MM/YYYY
a un objeto de tipo moment.Moment
dateToMoment(date_str: string): moment.Moment {
return moment(date_str, 'DD/MM/YYYY');
}
O bien una que reciba un objeto de tipo moment.Moment
y lo devuelva como string:
momentToString(momentObj: moment.Moment): string {
return momentObj.format();
}
En realidad si quieres pasar de una cadena DD/MM/YYYY
a una YYYY-MM-DD
no necesitas usar el tipo moment.Moment
puesto que la función normalizaFecha
sería simplemente:
normalizaFecha(date_str: string): string {
return moment(date_str, 'DD/MM/YYYY').format();
}
Pero sirve para ilustrar el tipo de dato que Typescript esperaría de un objeto moment.
YYYYMMdd
sí se cumple la comparación. – Pablo Lozano el 27 jun. 18 a las 10:26YYYYMMdd
para comparar si la fecha es mayor o menor que otra. De esta forma siempre que una fecha sea mayor que otra significará que es posterior. – Ferni el 27 jun. 18 a las 10:44