所以我只是想知道打开和关闭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>
BTW我正在谈论的php变量是来自DB的特定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;
?>
答案 0 :(得分:1)
我假设您的示例中重复的<html>
标记只是占位符,实际上是<div>
和构成页面实际内容的其他元素。
你的例子就是通常所说的“意大利面条代码”,因为它很快就会变成一个难以维护的混乱,因为你无法清楚地看到HTML的概述,也不能在一个地方看到所有的PHP代码。
要记住的主要事情是将应用程序逻辑(例如数据库查询)与表示分离(HTML和表示逻辑,如循环遍历数组以将其显示为HTML列表)。
至少你想把主要的PHP代码放在文件的顶部,就像你说的那样,但如果它在一个单独的文件中会更好。
P.S。任何关于PHP的初学者书都会详细讨论这个问题。