Salut,
j'aurais fais un truc similaire à ça :
<?php foreach($users as $k=>$v): $v = current($v); ?>
<tr class="row-<?php echo v'role'];?>">
<td><?php echo $v'id']; ?></td>
<td><?php echo $v'username']; ?></td>
<td><?php echo $v'role']; ?></td>
<td>
<?php echo $this->Html->link("Editer", array('action'=>'edit',$v'id']),array('style'=>'color:#3fbcef;text-decoration:underline')); ?> /
<?php echo $this->Html->link("Supprimer", array('action'=>'delete',$v'id']),array('style'=>'color:#df0024;text-decoration:underline'),'Voulez vous vraiment supprimer cet utilisateur ?'); ?>
</td>
</tr>
<?php endforeach; ?>
.row-user td {
background-color: #aabbcc;
}
.row-modo td {
background-color: #bbccdd;
}
.row-admin td {
background-color: #ccddee;
}
Cela permet de découper le code html qui permet la structure du site et le CSS qui permet le design. Et ça permet de gagner en simplicité en enlevant les if/else.