使用echo输出HTML在PHP中被认为是不好的做法？

在PHP中,我使用if语句来识别用户是否登录,并根据结果显示主菜单(如果已登录)或“您需要登录”消息(如果没有).我是这样做的：

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
  "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
    <Meta http-equiv="Content-Type" content="text/html;charset=utf-8" />
    <link rel="stylesheet" href="style.css" type="text/css" />
    <title>Home</title>
</head>
<body>
    <div id="header">
       <a href="index.PHP"><img src="wtcdblogo.png" alt="WTC DB logo" /></a>
    </div>
    <?PHP 
       if($_SESSION['loggedIn'] == 1) {
          echo "<div id='main'>MAIN MENU stuff goes here</div>";
       } else {
          echo "<div id='main'>Please login...</div>";
       } 
    ?>
</body>
</html>

如您所见,显示主菜单或“请登录”消息的代码由echo产生.这是不好的做法,也许有更好的方法？

顺便说一下,我在上面的代码片段中删除了回声中的大部分HTML.主菜单由一个列表组成,但我没有打扰,因为它与问题无关,我想.

解决方法:

我认为这是不好的做法.不确定其他人的想法.首先,在语法高亮显示的文本编辑器中看起来很糟糕,然后你必须担心转义字符串等.

我是这样做的：

   <div>
      <? if ($_SESSION['loggedIn'] === 1): ?>
         <div id="main">Main Menu stuff goes here</div>
      <? else: ?>
         <div id="main">Please log in...</div>
      <? endif ?>
   </div>

您可以跳出PHP 标记并使用直接HTML.这样做有利有弊.我喜欢它比回声的方式更好.其他选项是根据if语句的结果将新视图呈现到这些区域.很多可能性,但上面只是一种方法,使一点点更清洁,(我认为)更好.

使用echo输出HTML在PHP中被认为是不好的做法？

相关文章