2009-08-08 9 views
1
if ($row['active'] == 1) echo '<a href="prof?id=$id">'.htmlspecialchars($row['username']).'</a>'; else echo htmlspecialchars($row['username']); 

Pourrais-je écrire ceci plus court et plus propre en quelque sorte?Débutant si instruction aide

Répondre

0

Vous pouvez profiter de l'opérateur ternary.

echo ($row['active'] == 1) 
    ? '<a href="prof?id=$id">'.htmlspecialchars($row['username']).'</a>' 
    : htmlspecialchars($row['username']) 
; 

(I a divisé le code sur des lignes distinctes pour le bien de la mise en forme

1

Je suppose que vous avez fait une erreur en mettant le $ id dans une chaîne de guillemets simple, et signifiait pour php de mettre la valeur de $ id à sa place là-bas .

$name=htmlspecialchars($row['username']); 
if($row['active'] == 1) { 
    echo "<a href='prof?id=$id'>$name</a>"; 
} else { 
    echo $name; 
}