Problem bei eienr IF Abfrage (isset($a->name) ? if($a->name==1)'

[Gunnar Bittersmann]

@@Jörg Reinholz

So macht man es sich selbst schwer.

$foo='<td>';
if ( ! empty($a['name']) && $a['name'] == 1 ) {
   $foo .= '<img src="yes.png" alt="Ja">'
} else {
   $foo .= '<img src="no.png" alt="Nein">' 
}
$foo .= '</td>';

… ist viel besser lesbar und ergo weniger fehlerträchtig.

Besser ja, aber nicht gut. So macht man es sich selbst schwer.

<td>
  <?php if ( ! empty($a['name']) && $a['name'] == 1 ): ?>
    <img src="yes.png" alt="Ja">
  <?php else: ?>
    <img src="no.png" alt="Nein">
  <?php endif; ?>
</td>

… ist viel besser lesbar und ergo weniger fehlerträchtig.

Mir scheint zudem, dass im vorgestellten Fall die Information im alt-Attribut angezeigt werden sollte, z.B. falls jemand Grafiken nicht anzeigen lässt.

Oder nicht sehen kann.

Gemäß RFC 2119 muss es hier muss heißen, nicht sollte.

LLAP 🖖