4

我正在尝试将 PHP 与 HTML 连接起来，我想知道它的最佳实践是什么。到目前为止，我是这样处理的：--EDITED TO DISPLAY FULL SCRIPT--

<?php include("header.php"); ?>


<div id="main">
    <table id="mainTable">
        <tr>
            <td id="leftPane">
                <div class="pgbody">

                    <div class="subblock">
                        <form action="<?php echo htmlentities($_SERVER['PHP_SELF']); ?>" method="post">
                        <table width="100%" cellpadding="10" cellspacing="0">
                                <tr>
                                    <td colspan="3" style="width:99%">
                                    <label for="isVersion">InstallShield Version</label><br />
                                    <select name="isVersion" onchange="javascript:setTimeout('__doPostBack(\'ddlVersion\',\'\')', 0)" id="isVersion" class="box">
                                        <option value="2012Spring">2012 Spring</option>
                                        <option value="2012">2012</option>
                                        <option value="2011">2011</option>
                                        <option value="2010">2010</option>
                                        <option value="2009">2009</option>
                                        <option value="2009 Express">2009 Express</option>
                                        <option value="IS2008">2008</option>
                                        <option value="IS2008 Express">2008 Express</option>
                                    </select>

                                    <tr>
                                        <td colspan="3" style="width:99%">
                                            <br />
                                            <input type="checkbox" name="no_internet" value="no_internet"> no_internet
                                        </td>
                                    </tr>

                                    <tr>
                                        <td colspan="3" style="width:99%">
                                        <br />
                                        <input type="submit" name="submit" value="Submit">
                                        </td>
                                    </tr>
                                </tr></table>
                        <?php

                        if(isset($_POST['submit']))
                        {
                            echo "<h2>Response</h2><br />";
                            $isVersion = $_POST["isVersion"];
                            $output_script = "
                                    <p>Hello,  <br />
                                    To activate InstallShield, please follow the steps below:<br /><br />

                                    1. Launch a Command Prompt window and browse to the directory - 'C:\Program Files\InstallShield\\$isVersion\System' (or 'Program Files (x86)' on a 64 bit machine)<br />
                                    2. You will need to pass the parameter '/return' to the executable 'TSconfig' as below<br />
                                    'C:\Program Files\InstallShield\\$isVersion\System\TSconfig.exe /return'<br />
                                    3. Providing the machine has a valid internet connection the license will deactivate and the message in the dialog will reflect this<br />
                                    4. Re-launch InstallShield.exe and you will be presented with the same activation dialog as before<br />
                                    5. Proceed with the activation normally<br />
                                    6. The dialog should show a successful activation message and the product should remain activated at this stage.<br />
                                     </p>";
                        ?>

                        <div class="ResponseBox" style="background-position: 0 0;">
                            <div class="ResponseText">
                                <?php echo $output_script;
                        }
                                 ?>
                        <?php
                        elseif(isset($_POST['submit']) && $_POST['no_internet'])
                        {
                            echo "<h2>Response</h2><br />";
                            $isVersion = $_POST["isVersion"];
                            $no_internet = $_POST["no_internet"];
                            $output_script = "
                                    <p>Hello,  <br />
                                    To activate InstallShield, please follow the steps below:<br /><br />

                                    1. Launch a Command Prompt window and browse to the directory - 'C:\Program Files\InstallShield\\$isVersion\System' (or 'Program Files (x86)' on a 64 bit machine)<br />
                                    2. You will need to pass the parameter '/return' to the executable 'TSconfig' as below<br />
                                    'C:\Program Files\InstallShield\\$isVersion\System\TSconfig.exe /return /$no_internet'<br />
                                    3. Providing the machine has a valid internet connection the license will deactivate and the message in the dialog will reflect this<br />
                                    4. Re-launch InstallShield.exe and you will be presented with the same activation dialog as before<br />
                                    5. Proceed with the activation normally<br />
                                    6. The dialog should show a successful activation message and the product should remain activated at this stage.<br />
                                     </p>";
                        ?>
                        <div class="ResponseBox" style="background-position: 0 0;">
                            <div class="ResponseText">
                                <?php echo $output_script;
                        }
                                 ?>
                            </div>
                        </div>    
                    </div>
            </td>

            <td id="rightPane">
                <div class="PromoBox" style="background-position: 0 0;">
                    <div class="PromoText">
                            <?php
                            echo "<h2>Related KB Article: </h2><br />";
                            echo "<h3>Deactivation of IS - Q201081</h3>";
                            ?>
                    </div>
                </div>
            </td>
                                </tr>
        </tr>
    </table>
</div>

如您所见，PHP 和 HTML 在需要时相互连接。我想知道，但是，如果这实际上是最好的方法，我应该使用 PHP 来回显所有内容而不是像上面那样做吗？
我还试图在下面写一个 elseif 块，这给了我一个错误，指出 elseif 是意外的，这就是让我认为我正在接近这整个事情错误的原因。

当我查看人们制作的大型项目时，我注意到每个文件中几乎没有任何内容，而且肯定没有很多 HTML 是可见的。我假设大部分内容都在对象或类中是否正确？任何建议或参考将不胜感激。提前致谢。

4

2 回答 2

5

您可能会从不同的人那里得到对“最佳实践”的不同描述。我会使用PSR-1代码格式标准，其中包括重点：

  • 文件应该声明符号（类、函数、常量等）或引起副作用（例如生成输出、更改 .ini 设置等），但不应该两者都做。

我还建议您的 php 代码根本不要直接打印出任何 HTML，而是使用模板。我是模板引擎的忠实粉丝，那里有很多。人们还会说“php 本身就是一个模板引擎”，如果您愿意，可以使用它。尽可能多地保留模板（显示）之外的逻辑。

提前与您的团队确定您的标准并坚持下去。关于是否更好echo或?>更好的意见可能存在差异，但一旦您做出决定，请保持一致。保持缩进一致也很重要。如果你这样做，你可能会找到使你elseif不正确的丢失的大括号。

于 2013-05-09T22:18:49.037 回答
2

就像 batmegakapa 已经解释的那样，执行逻辑/内容生成和查看是两个不同的任务。现代 Web 应用程序的复杂性通常要求采用这种方法来保持代码的可读性和可理解性，以及可重用性。

这就是（大致）为什么MVC 模式（模型/视图/控制器）和模板系统是当今大多数框架的组件的原因。

当然，不可能给出一个解决方案，因为这取决于个人喜好和需求——这使得这个问题存在争议，过于本地化，不适合问答风格。

但是，当遵循不混合逻辑和内容的规则时，您很可能会这样做（基本方法）：

no_internet_form.php：

<?php
if(isset($_POST['submit']) && $_POST['no_internet'])
{
    //echo "<h2>Response</h2><br />";
    $Version = $_POST["isVersion"];
    $no_internet = $_POST["no_internet"];
    $script_result = execute_the_incredibly long_script($Version, $no_internet);
}

// more business logic ...

include 'views/no_internet_form.html.php';
?>

views/no_internet_form.html.php：

<?php if(isset($script_result)): ?>
<div class="ResponseBox" style="background-position: 0 0;">
    <div class="ResponseText">
        <?php echo $script_result; ?>
    </div>
</div>
<?php endif; //isset($script_result) ?>

顺便说一句：我怀疑您提供的代码的哪一部分实际上属于该if块，因为这对我来说并不明显。您的示例的 HTML 代码不知何故没有多大意义。

这可能只是为了演示，但我花了几秒钟才真正理解你的例子的含义。这应该更具可读性——尤其是在大型项目中。

于 2013-05-09T22:38:45.697 回答