Skip to main content
se añadieron 847 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Porque en el if haces una asignación al poner un sólo signo de =, pon dos, o tres, si quieres comparación estricta.

if($datos == "AC") {

O bien:

if($datos === "AC") {

Para más detalles, revisa el apartado Operadores de comparación en el Manual de PHP.

PERO, si quieres, puedes simplificar todo:

                  $datos = $data2['TC_Estado_eq'];

                  if($datos = "AC") {
                    echo '<td width=80 class=center><font color="#FF0000">Activo</font></td>';
                  
                  } else{
                    echo '<td width=80 class=center><Font color="#FF8C00">Inactivo</FONT></td>';
                 }

Usando un operador ternario que asigne las propiedades en un array:

$datos = $data2['TC_Estado_eq'] == 'AC' ? 
             array('txt'=>'Activo', 'color'=>'#FF0000') : 
             array('txt'=>'Inactivo', 'color'=>'#FF8C00') ;

printf('<td width=80 class=center><font color="%s">%s</font></td>',$datos['txt'],$datos['color']);

Así el código quedaría más limpio y sería más portable.

Porque en el if haces una asignación al poner un sólo signo de =, pon dos, o tres, si quieres comparación estricta.

if($datos == "AC") {

O bien:

if($datos === "AC") {

Para más detalles, revisa el apartado Operadores de comparación en el Manual de PHP.

Porque en el if haces una asignación al poner un sólo signo de =, pon dos, o tres, si quieres comparación estricta.

if($datos == "AC") {

O bien:

if($datos === "AC") {

Para más detalles, revisa el apartado Operadores de comparación en el Manual de PHP.

PERO, si quieres, puedes simplificar todo:

                  $datos = $data2['TC_Estado_eq'];

                  if($datos = "AC") {
                    echo '<td width=80 class=center><font color="#FF0000">Activo</font></td>';
                  
                  } else{
                    echo '<td width=80 class=center><Font color="#FF8C00">Inactivo</FONT></td>';
                 }

Usando un operador ternario que asigne las propiedades en un array:

$datos = $data2['TC_Estado_eq'] == 'AC' ? 
             array('txt'=>'Activo', 'color'=>'#FF0000') : 
             array('txt'=>'Inactivo', 'color'=>'#FF8C00') ;

printf('<td width=80 class=center><font color="%s">%s</font></td>',$datos['txt'],$datos['color']);

Así el código quedaría más limpio y sería más portable.

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Porque en el if haces una asignación al poner un sólo signo de =, pon dos, o tres, si quieres comparación estricta.

if($datos == "AC") {

O bien:

if($datos === "AC") {

Para más detalles, revisa el apartado Operadores de comparación en el Manual de PHP.