Skip to main content
Mejora en la expresión
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Estás cometiendo un error muy común: usar formato de fechas localizado y hacer comparaciones de texto entre ellas.

Deberías acostumbrarte a usar fechas en formato ISO 8601 o equivalentes (como AAAA-MM-DD HH:MM:SS), aunque siempre es recomendable ISO y, a ser posible, en UTC.

De esa forma la comparación de cadenas de caracteres sería equivalente a la comparación de fechas, ya que se estánestarían comparando los elementoslas magnitudes de tiempo de mayor a menor valor (año, mes, día, hora, minutos y segundos).

Tu código quedaría:

<?php
/* Representación de fecha completa en formato extendido ISO 8601 */
$fecha_inicio = date('Y-m-d');

if ($fecha_inicio >= $fechaActual) {
    //retorna la cantidad de segundos entre las fechas 
    $diff = abs(strtotime($fecha_inicio) - strtotime($fechaActual));
    // para pasarlo a dias 60 segundos tiene un minuto y una hora 60
    // minutos y un dia 24 horas
    $total = $diff / (60 * 60 * 24); 

    if($total <= 14 && $total > 7) {
?><img class="alerta" src="img/alerta_amarilla.png"><?php
    } else if ($total <= 7) {   
?><img class="alerta" src="img/alerta_roja.png"><?php
    } else if ($total >= 15 && $total < 30) {   
?><img class="alerta" src="img/alerta_verde.png"><?php
    }
}

Estás cometiendo un error muy común: usar formato de fechas localizado y hacer comparaciones de texto entre ellas.

Deberías acostumbrarte a usar fechas en formato ISO 8601 o equivalentes (como AAAA-MM-DD HH:MM:SS), aunque siempre es recomendable ISO y, a ser posible, en UTC.

De esa forma la comparación de cadenas de caracteres sería equivalente a la comparación de fechas, ya que se están comparando los elementos de mayor a menor valor (año, mes, día, hora, minutos y segundos).

Tu código quedaría:

<?php
/* Representación de fecha completa en formato extendido ISO 8601 */
$fecha_inicio = date('Y-m-d');

if ($fecha_inicio >= $fechaActual) {
    //retorna la cantidad de segundos entre las fechas 
    $diff = abs(strtotime($fecha_inicio) - strtotime($fechaActual));
    // para pasarlo a dias 60 segundos tiene un minuto y una hora 60
    // minutos y un dia 24 horas
    $total = $diff / (60 * 60 * 24); 

    if($total <= 14 && $total > 7) {
?><img class="alerta" src="img/alerta_amarilla.png"><?php
    } else if ($total <= 7) {   
?><img class="alerta" src="img/alerta_roja.png"><?php
    } else if ($total >= 15 && $total < 30) {   
?><img class="alerta" src="img/alerta_verde.png"><?php
    }
}

Estás cometiendo un error muy común: usar formato de fechas localizado y hacer comparaciones de texto entre ellas.

Deberías acostumbrarte a usar fechas en formato ISO 8601 o equivalentes (como AAAA-MM-DD HH:MM:SS), aunque siempre es recomendable ISO y, a ser posible, en UTC.

De esa forma la comparación de cadenas de caracteres sería equivalente a la comparación de fechas, ya que se estarían comparando las magnitudes de tiempo de mayor a menor (año, mes, día, hora, minutos y segundos).

Tu código quedaría:

<?php
/* Representación de fecha completa en formato extendido ISO 8601 */
$fecha_inicio = date('Y-m-d');

if ($fecha_inicio >= $fechaActual) {
    //retorna la cantidad de segundos entre las fechas 
    $diff = abs(strtotime($fecha_inicio) - strtotime($fechaActual));
    // para pasarlo a dias 60 segundos tiene un minuto y una hora 60
    // minutos y un dia 24 horas
    $total = $diff / (60 * 60 * 24); 

    if($total <= 14 && $total > 7) {
?><img class="alerta" src="img/alerta_amarilla.png"><?php
    } else if ($total <= 7) {   
?><img class="alerta" src="img/alerta_roja.png"><?php
    } else if ($total >= 15 && $total < 30) {   
?><img class="alerta" src="img/alerta_verde.png"><?php
    }
}
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Estás cometiendo un error muy común: usar formato de fechas localizado y hacer comparaciones de texto entre ellas.

Deberías acostumbrarte a usar fechas en formato ISO 8601 o equivalentes (como AAAA-MM-DD HH:MM:SS), aunque siempre es recomendable ISO y, a ser posible, en UTC.

De esa forma la comparación de cadenas de caracteres sería equivalente a la comparación de fechas, ya que se están comparando los elementos de mayor a menor valor (año, mes, día, hora, minutos y segundos).

Tu código quedaría:

<?php
/* Representación de fecha completa en formato extendido ISO 8601 */
$fecha_inicio = date('Y-m-d');

if ($fecha_inicio >= $fechaActual) {
    //retorna la cantidad de segundos entre las fechas 
    $diff = abs(strtotime($fecha_inicio) - strtotime($fechaActual));
    // para pasarlo a dias 60 segundos tiene un minuto y una hora 60
    // minutos y un dia 24 horas
    $total = $diff / (60 * 60 * 24); 

    if($total <= 14 && $total > 7) {
?><img class="alerta" src="img/alerta_amarilla.png"><?php
    } else if ($total <= 7) {   
?><img class="alerta" src="img/alerta_roja.png"><?php
    } else if ($total >= 15 && $total < 30) {   
?><img class="alerta" src="img/alerta_verde.png"><?php
    }
}