Question

我有此代码：

<?php $search =$_GET['search']?>

<p>Displaying search results for : <font style="font-weight:bolder;font-style:italic"><?php echo $search?></font></p>

<?php
$mysql_host='localhost';
$mysql_user='root';
$mysql_password='can't see my password..sorry';

mysql_connect($mysql_host,$mysql_user,$mysql_password);
@mysql_select_db('galaxall');
?>

<!--the results-->

<p style="background-color:rgb(250,250,250)">
<div style="background-color:rgb(250,250,250);padding-bottom:3%;margin-right:20%">
<?php
$hello='hello everybody!!!';

$query="SELECT * FROM `galaxall_uploads` WHERE Title LIKE '%$search%' ";
if($is_query_run=mysql_query($query) )
{  
    while($query_execute=mysql_fetch_assoc($is_query_run) )
    {
        echo $query_execute ['Title'].'<br>'.''.'<p style="background-color:blue;padding-bottom:5%">'.'<br>' ;
    }
} else {
    echo"Sorry, something went wrong...";
}
?> 
</p>

基本上，这是一个SQL代码，用于按顺序从数据库中检索数据。检索到的每个记录都应进行样式化（使用CSS进行美化）。除了第一个和最后一个结果的格式不正确外，它的效果都很好。例如显示的第一个结果没有任何CSS样式（而第二个，第三个等可以），最后一个显示了CSS样式，但结果本身不存在。

屏幕截图：

如您所见，第一个“ ht”没有蓝色样式，最后一个没有“ ht”（几乎彼此分开）具有蓝色样式

Answer 1

mysql_函数are deprecated from php5.5。
在升级到mysqli时，我建议您学习面向对象的语法，因为它比过程语法更冗长。
在连接时指定目标数据库很简单，因此将它们全部合并为一行。
您正在使用用户提供的数据构建数据库查询，因此需要准备好的语句。当没有提交的值时，您可以避免使用准备好的语句，但是为了降低代码复杂性，我正在编写准备好的语句来处理所有情况。
在设计SELECT子句时，如果不需要执行任务的所有列，则仅请求将要使用的列。
您应该尝试通过在<head>标记中甚至在单独的文件中编写样式声明来避免内联样式。这样可以提高代码的整体可读性和可维护性。
当您需要调试工作时，将错误检查写入代码中将非常有帮助-因此，请养成始终做的习惯。出于安全考虑，切勿向用户显示确切的错误详细信息。
$query_execute对于包含结果集行中数据的变量不是一个很好的选择。它不会“执行”任何东西，所以请不要使自己和将来的代码阅读者（人类）对这种术语感到困惑。
对于您特定的html问题，请尝试消除输出中所有不必要的DOM元素（标记）。似乎您的div元素只需要包含一个p的元素。

未经测试的代码：

echo "<div style=\"background-color:rgb(250,250,250); padding-bottom:3%; margin-right:20%\">";
if (!$conn = new mysqli("localhost", "root", "", "galaxall")) {
    echo "<font style=\"color:red;\">Database Connection Error</font>"; // don't show this to the public--> $conn->connect_error;
} else {
    if (!isset($_GET['search'])) {
        $search = "";
        echo "<p>No search keyword received.  Showing all rows<p>";
    } else {
        $search = strip_tags($_GET['search']);  // perform whatever sanitizing/filtering/preparation techniques in this line
        echo "<p>Displaying search results for: <font style=\"font-weight:bolder; font-style:italic\">$search</font></p>";
    }
    if (!$stmt = $conn->prepare("SELECT Title FROM galaxall_uploads WHERE Title LIKE ? ORDER BY Title")) {
        echo "<font style=\"color:red;\">Prepare Syntax Error</font>"; // don't show this to the public--> $conn->error;
    } else {
        if (!$stmt->bind_param("s", "%$search%") || !$stmt->execute() || !$result = $stmt->get_result()) {
            echo "<font style=\"color:red;\">Statement Error</font>"; // don't show this to the public--> $stmt->error;
        } elseif (!$result->num_rows){
            echo "<p>No matches in the database for \"$search\"</p>";
        } else {
            while ($row = $result->fetch_assoc()) {
                echo "<p style=\"background-color:blue; padding-bottom:5%\">{$row['Title']}</p>";
            }
        }
        $stmt->close();
    }
    $conn->close();
}
echo "</div>";

Answer 2

您不能在div元素内使用p元素，而且您的用于打印数据库结果的代码有误，因此我建议这样做：-

<!--the results-->

<div style="background-color:rgb(250,250,250);padding-bottom:3%;margin-right:20%">
<?php
$hello='hello everybody!!!';

$query="SELECT * FROM `galaxall_uploads` WHERE Title LIKE '%$search%' ";
if($is_query_run=mysql_query($query) )
{


    while($query_execute=mysql_fetch_assoc($is_query_run) )
    {

        echo '<p style="background-color:blue;padding-bottom:5%">'.$query_execute ['Title'].'</p><br/>' ;


    }

}

else
{
    echo"Sorry, something went wrong...";
}


?> 
</div>

Answer 3

这是因为您做错了。尝试以下

while($query_execute=mysql_fetch_assoc($is_query_run) )
    {
        echo '<p style="background-color:blue;padding-bottom:5%">'.$query_execute ['Title'].'</p><br>' ;
    }

嵌套的html元素显示意外样式

3 个答案: