duda con codigo no me funciona como quiero

[privatefta]

hola y saludos a ver como me echan una mano es que tengo este codigo trabajando

Código PHP:

<?php
    if ($row["baniado"] == "1") {
      $color = "<tr class=\"bann\" align=\"left\">";
    }
    if ($row["baniado"] == "0") {
      $color = "<tr class=\"odd\" align=\"left\">";
    }
     if($row["fecha_vencimiento"] < date('Y-m-d')) {
      $color = "<tr class=\"disabled\" align=\"left\">";
    }
    print("".$color."");
?>
 

me trabaja todo bien pero cuando llega al ultimo if que es el de la fecha si quiero que funcione la de if ($row["baniado"] == "1") { no funciona ya que se queda siempre if($row["fecha_vencimiento"] < date('Y-m-d')) { y quiero que muestre la que este en el uso si cambio muestre los cambios

[jmabreu]

explicate bien

[privatefta]

Código PHP:

<?php
    if ($row["baniado"] == "1") {
      $color = "<tr class=\"bann\" align=\"left\">";
    }
    if ($row["baniado"] == "0") {
      $color = "<tr class=\"odd\" align=\"left\">";
    }
     if($row["fecha_vencimiento"] < date('Y-m-d')) {
      $color = "<tr class=\"disabled\" align=\"left\">";
    }
    print("".$color."");
?>
 

es que el codigo me funciona bien pero hay un problema que si se activa el ultimo if que el utimo es este
if($row["fecha_vencimiento"] < date('Y-m-d')) {
$color = "<tr class=\"disabled\" align=\"left\">";
}

pues deja de funcionar los demas ya que llego la fecha pero si la coloco con fecha vencida y quiero colocarlo como baniado pues sige como vencido

[andresdzphp]

¿Qué tipo de campo es fecha_vencimiento? en caso de que sea Timestamp comparas con la función time()

Código PHP:

if($row["fecha_vencimiento"] < date('Y-m-d')) { //No puedes hacer esto
 

Si esta en formato de fecha no puedes comparar fechas de esta forma, necesitas funciones como strtotime.

Un ejemplo básico:

Código PHP:

<?php
$fecha_act = time();
$fecha_venc = '02-01-2012 13:00:00';
 
if (strtotime($fecha_venc) < $fecha_act) {
    echo 'Esta vencido';
} else {
    echo 'No esta vencido';
}
 

Saludos.

[jmabreu]

te dire lo que entendi y tu me dices si es eso vale

quieres que compruebe si esta baneado o no si esta baneado por ejemplo ponga rojo si no que ponga azul y si la fecha del baneo a expirado que quieres que haga hay es donde me pierdo

[privatefta]

majo meno es como dices jmabreu es que tengo ese codigo para mostrar un color

<?php
if ($row["baniado"] == "1") {
$color = "<tr class=\"Color Amarillo\" align=\"left\">";
}
if ($row["baniado"] == "0") {
$color = "<tr class=\"Color blanco\" align=\"left\">";
}
if($row["fecha_vencimiento"] < date('Y-m-d')) {
$color = "<tr class=\"Color Rojo\" align=\"left\">";
}
print("".$color."");
?>

ok pues cuando la $row["baniado"] == "0" me muestra el color blanco si la coloco $row["baniado"] == "1" me muestra amarillo y si esta $row["fecha_vencimiento"] < date('Y-m-d') me muestra rojo pero el problema que tengo es que si ya llego la fecha $row["fecha_vencimiento"] < date('Y-m-d')) la marca en rojo pero si quiero ponerla en $row["baniado"] == "1" para que salga amarillo sigue saliendo rojo a ver si pude esplicarme mejor y gracias por las ayuda.

[jmabreu]

estoy pesimo pero bueno hay te explico lo que dices y te doy una mejor solucion
esto lo que hace la primera linea comprueba si esta baneado y si la fecha de vencimiento es menos a la actual y el segundo lo mismo pero si ya paso la fecha y el ultimo si no esta baneado

if($row['baneado']=='1' && $row["fecha_vencimiento"] < date('Y-m-d')){
//aqui pones el color que quieras

}
elseif($row['baneado']=='1' && $row["fecha_vencimiento"] > date('Y-m-d')){
//aqui pones el color que quieras

}elseif($row['baneado']=='0'){
//color blanco
}



claro esta comprueba la fecha como te han indicado mas arriba la pongo asi pero no la compruebes de esa manera

[andresdzphp]

Cuando aprendan a comparar fechas les va a funcionar el código...

[jmabreu]

jajaja no te piques a mi me da flojera poner la comparacion claro porque ya la pusiste tu pero bueno no te pongas gruñon ya le pongo el codigo tuyo aunque es escribirlo dos veces

[jmabreu]

andres tu codigo esta mal :D ciento corregirte


//comprobamos que no haya expirado conste el formato de la fecha es 31-12-2000


$fecha_act = strtotime(date('d-m-Y')); $fecha_venc = strtotime($row['fechavencimiento']);
if($fecha_venc < $fecha_act){$estado = 'expiro';}else{$estado = 'activo';}




//comprobamos que este baneado y que haya pasado la fecha de baneo o no

if($row['baneado']=='1' && $estado=='activo'){
//aqui pones el color que quieras sea el rojo o amarillo

}
elseif($row['baneado']=='1' && $estado=='expiro'){
//aqui pones el color que quieras esta baneado pero el baneo expiro

}elseif($row['baneado']=='0'){
//color blanco
}

[andresdzphp]

Iniciado por jmabreu

andres tu codigo esta mal :D ciento corregirte

si! ¿donde está mal? más bien vos estás mal:

Código PHP:

$fecha_act = strtotime(date('d-m-Y'));
 

Esto es totalmente innecesario... Para eso existe time()

[jmabreu]

jejeje pruebalo y veras el que posteaste

[jmabreu]

perdon andres estaba discutiendo con otro pana un tema de fechas disculpa lo tuyo esta bien lo que esta mal es el otro sigo discutiendo con el ::DDDDDD

[andresdzphp]

Iniciado por jmabreu

jejeje pruebalo y veras el que posteaste

¿Sabes que hace la función time()? mejor te invito a leer el manual:

http://php.net/manual/es/function.time.php

Estamos comparando fechas en formato unix actual, por lo tanto no es necesaria la función strtotime ni la función date con su formato d-m-Y. Ese trabajo lo hace la función time().

[jmabreu]

jajaja que me equivoque de post viejo que te lo cojas con calma me equivoque de gente al corregir no te hiba a corregir a ti

[privatefta]

Iniciado por jmabreu

estoy pesimo pero bueno hay te explico lo que dices y te doy una mejor solucion
esto lo que hace la primera linea comprueba si esta baneado y si la fecha de vencimiento es menos a la actual y el segundo lo mismo pero si ya paso la fecha y el ultimo si no esta baneado

if($row['baneado']=='1' && $row["fecha_vencimiento"] < date('Y-m-d')){
//aqui pones el color que quieras

}
elseif($row['baneado']=='1' && $row["fecha_vencimiento"] > date('Y-m-d')){
//aqui pones el color que quieras

}elseif($row['baneado']=='0'){
//color blanco
}



claro esta comprueba la fecha como te han indicado mas arriba la pongo asi pero no la compruebes de esa manera

Gracias amigo me funciono la el elseif

[jmabreu]

pero usa la ultima o comprueba la fecha como la envio andres que es al manera correcta de comprobar la fecha de vencimiento

[privatefta]

como esta aqui

Código PHP:

<?php
    if ($row["baniado"] == "1") {
      $color = "<tr class=\"bann\" align=\"left\">";
    }
     elseif($row["fecha_vencimiento"] < date('Y-m-d')) {
      $color = "<tr class=\"disabled\" align=\"left\">";
    }
    elseif ($row["baniado"] == "0") {
      $color = "<tr class=\"odd\" align=\"left\">";
    }
    print("".$color."");
?>
 

y me trabaja sin problema gracias a todos por la ayuda

[jmabreu]

solo una recomendacion ahora que estas comenzando utiliza palabras en vez de numeros pero eso queda a tu gusto

$row['baniado']=='baneado'

pero bueno eso no cambia nada pero considero que es mas comodo que estar con eso de 0 - 1

bueno ahora si voy a comer algo :D

[privatefta]

amigo es que eso es consulta a la sql y 0 = no y 1 = si