2010-02-11 5 views
-4

Dans mon éditeur de code, il affiche normalement le code PHP en caractères gras. Je constate que certains de mes codes n'apparaissent pas en caractères gras. Quelqu'un peut-il s'il vous plaît regarder le code ci-dessous et me dire si je fais quelque chose de mal? Merci d'avance.Pourquoi mon éditeur ne met-il pas en évidence tout mon code PHP?

<td class="hr"><?php echo htmlspecialchars("payroll number")."&nbsp;" ?></td> 
    <td class="dr"><input type="text" name="GevityNo" maxlength="10" value="<?php echo str_replace('"', '&quot;', trim($row["GevityNo"])) ?>"></td> 
</tr> 
<tr> 
    <td class="hr"><?php echo htmlspecialchars("employee name")."&nbsp;" ?></td> 
    <td class="dr"><textarea cols="25" rows="1" name="employee_name" maxlength="75"><?php echo str_replace('"', '&quot;', trim($row["employee_name"])) ?></textarea></td> 
</tr> 
<tr> 
    <td class="hr"><?php echo htmlspecialchars("Sex")."&nbsp;" ?></td> 
    <td class="dr"><select name="Sex"> 
    <option value=""></option> 
    <?php 
     $lookupvalues = array("male","female"); 
     reset($lookupvalues); 
     foreach($lookupvalues as $val){ 
      $caption = $val; 
      if ($row["Sex"] == $val) {$selstr = " selected"; } else {$selstr = ""; } 
      ?> 
      <option value="<?php echo $val ?>"<?php echo $selstr ?>><?php echo $caption ?></option> 
    <?php } ?> 
    </select> 
</td> 
+1

Ce code fonctionne-t-il? – SilentGhost

+0

Je serais confus si j'étais aussi votre éditeur. Utilisez le formatage et l'indentation de code appropriés et vous n'aurez pas besoin de vous demander si c'est correct. – soulmerge

+2

@KAH: s'il vous plaît, ne touchez plus le code. Cela a été corrigé au moins 3 fois maintenant – SilentGhost

Répondre

5
$ cat > /home/tkn/tmp/foo.php 
    ... snip ... 
^C 
$ php -l /home/tkn/tmp/foo.php 
No syntax errors detected in /home/tkn/tmp/foo.php 

Alors, oui, il est un code valide php. Obtenez un meilleur éditeur.

1

La syntaxe fait partie de l'éditeur de code que vous utilisez. Si votre code fonctionne correctement, il n'y a pas de problème. .

0

En général, ces problèmes sont liés à tort échappé \ » Sans voir tout le code est difficile de dire

Quoi qu'il en soit, vérifiez que cette partie de votre code est ok.

<?php echo $selstr ?>><?php echo $caption ?>< 

Vérifier la >> < partie

+1

C'est valide, même si ce n'est pas joli ... –

0

Pour rendre plus lisible, pensez votre code sur l'utilisation du alternative syntax for control structures:

<?php foreach(array("male","female") as $val): ?> 
    <option value="<?php echo $val ?>" <?php echo ($row["Sex"] == $val)?'selected="selected"' : '' ?> > <?php echo $val ?> </option> 
<?php endforeach; ?> 

À mon avis, il est plus facile de ee la structure du code.

+1

Cela ne change pas vraiment grand chose: ': ... end' n'est vraiment pas différent de' {...} '. La partie importante à prendre de cette réponse, de quelque manière que vous choisissiez, est de mettre en retrait PHP et HTML de manière cohérente. – bobince

+0

@bobince: Je ne pense pas que ça change beaucoup, mais quand vous, par ex. structures de contrôle imbriquées, je suppose que l'on peut être vraiment confus par tous les '' parce qu'il n'est pas nécessairement clair où il appartient. Je voulais juste souligner que vous vérifiez son code plus facile pour les erreurs si elle est plus lisible. –

Questions connexes