-1

所以我只是想知道打开和关闭 PHP 语句是否被认为是不好的做法，让我解释一下我的意思。我知道我可以在代码的开头创建变量，但我喜欢将东西组合在一起。我不确定用我的所有变量创建一个大的 PHP 语句是否更好/更差/与打开和关闭 PHP 语句类似下面的例子。

<html>
<head></head> <---- HTML STUFF
<?php
(php stuff where connection to mysql db goes and other variables and errors)
?>
<body>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
</body>
<html>

顺便说一句，我正在谈论的 php 变量是来自数据库的特定 select 语句。

实际代码：还是应该选择语句在 beg 或 sep 文件中？

<table>
<tr>
<td align="left" width="200px">
Cover: Original Total
</td>
<td width="200px" align="center">
<?php
$original = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"original\"";
$orig_con = mysqli_query($comic_connect, $original);
$orig_total = mysqli_num_rows($orig_con);
echo $orig_total;
?>
</td>
</tr>
<tr>
<td width="200px" align="left">
Cover: Variants Total
</td>
<td width="200px" align="center">
<?php
$variants = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"variant\"";
$variant_con = mysqli_query($comic_connect, $variants);
$variant_total = mysqli_num_rows($variant_con);
echo $variant_total;
?>
</td>
</tr>
<tr>
<td align="left" width="200px">
Cover: Baby Totals
</td>
<td width="200px" align="center">
<?php
$baby = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"baby\"";
$baby_con = mysqli_query($comic_connect, $baby);
$baby_total = mysqli_num_rows($baby_con);
echo $baby_total;
?>
4

1 回答 1

1

我假设<html>您示例中的重复标签只是占位符，实际上是<div>s 和构成页面实际内容的其他元素。

您的示例通常称为“意大利面条代码”，因为您无法清楚地看到 HTML 的概览，也无法在一个地方看到所有 PHP 代码，因此它很快就会变成无法维护的混乱。

要记住的主要事情是将应用程序逻辑（例如您的数据库查询）与表示（HTML 和表示逻辑，如循环数组以将其显示为 HTML 列表）分开。

至少您希望像您说的那样将主要的 PHP 代码放在文件的顶部，但如果它放在单独的文件中会好得多。

PS任何关于PHP的初学者书籍都会详细讨论这一点。

于 2013-05-11T05:24:43.160 回答