Skip to main content
2 de 3
se añadieron 670 caracteres en el cuerpo
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

No se aconseja mezclar en un mismo flujo código PHP / Javascript. Porque PHP trabaja del lado del servidor y Javascript del lado del cliente.

Esto lo puedes resolver de una forma más limpia, poniendo una clase .css en el elemento, según el valor. Eso lo haremos en esta línea:

$color=$usuario['reposicion'] == 0 ? 'red' : 'black'; 

Luego, crearemos un <span class="$color"> que tomará red o black según el caso y la regla .css se encargará de aplicar el color. En el .css aprovechamos también para poner una regla llamada row-center para centrar las filas, y quitamos el style que aplicas directamente sobre los elementos, lo cual es otra mala práctica.

En caso de que los valores sean cambiados por el usuario (en el cliente), y se requiera una actualización dinámica, también se puede hacer. En este caso ya desde Javascript, escuchando ese cambio y asignado al elemento una clase diferente. No parece ser el caso, pero de serlo, usar una clase .css te servirá también para ese caso de uso.

Veamos un ejemplo.


.css

En primer lugar, tendrías que crear un archivo de estilo con las siguientes reglas:

.red {
  color: #ee2500
}

.black {
  color: #333
}

.row-center {
  text-align: center
}

PHP

En PHP introduciremos algunas mejoras que den claridad y simplicidad al código usando sprintf, es algo parecido a implementar plantillas. De paso evitaremos ids repetidos en los elementos, lo cual es otra mala práctica que puede llevar a errores si se usará en un futuro getElementById del lado del cliente.

DEMO

<table>
    <thead>
       <th>id</th>
       <th>No. Devolucion</th>
       <th>Reposicion</th>
    </thead>
    <tbody>
    <?php

    #AQUÍ EL CÓDIGO QUE DEFINE $rol_user Y DEMÁS ...

    if($rol_user==='admin' || $rol_user==='administrativo'){
        $td='<td>%s</td>';
        $anchor='<a id="enlace%s" href="repo.php?repos=%s"><span class="%s">%s</span></a>';
        $rows='';
          foreach($usuarios as $usuario) {
              $rows.='<tr class="row-center">';
              $rows.=sprintf($td,$usuario['id']);
              $rows.=sprintf($td,$usuario['Devolucion']);
              $color=$usuario['reposicion'] == 0 ? 'red' : 'black';
              $a=sprintf($anchor,$usuario['id'],$usuario['reposicion'],$color,$usuario['reposicion']);
              $rows.=sprintf($td,$a);
              $rows.='</tr>';
          }
          echo $rows;
    }

    #AQUÍ LO QUE SIGUE DE TU CÓDIGO

?>        
   </tbody>
</table>    

El código creará las filas como se puede ver en el snippet siguiente. Si observas el valor de la fila 2 está en rojo, porque es 0.

.red {
  color: #ee2500
}

.black {
  color: #333
}

.row-center {
  text-align: center
}
<table>
  <thead>
    <th>id</th>
    <th>No. Devolucion</th>
    <th>Reposicion</th>
  </thead>
  <tbody>
    <tr class="row-center">
      <td>1</td>
      <td>Devolución 1</td>
      <td><a id="enlace1" href="repo.php?repos=1"><span class="black">1</span></a></td>
    </tr>
    <tr class="row-center">
      <td>2</td>
      <td>Devolución 2</td>
      <td><a id="enlace2" href="repo.php?repos=0"><span class="red">0</span></a></td>
    </tr>
    <tr class="row-center">
      <td>3</td>
      <td>Devolución 3</td>
      <td><a id="enlace3" href="repo.php?repos=3"><span class="black">3</span></a></td>
    </tr>
  </tbody>
</table>

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