帮助我解决我的草率HTML/PHP

帮助我解决我的草率HTML/PHP,php,Php,这听起来可能是一个愚蠢的问题,但我该如何使这段代码看起来…更整洁 echo "<h3><font face='helvetica'><font size='4'><b><font color='B80000'>$title</font></font></font></b> <font color='A0A0A0'>$category &nbsp;</fon

这听起来可能是一个愚蠢的问题,但我该如何使这段代码看起来…更整洁

    echo "<h3><font face='helvetica'><font size='4'><b><font color='B80000'>$title</font></font></font></b> <font color='A0A0A0'>$category &nbsp;</font><font color='A0A0A0'><a href='profile.php?id=$userid'>$user</a></font>
    <font face='helvetica'><font size='3'><br>&nbsp;$desc</font></font><br>

       <h3><font color='101010'> &nbsp;$city,$state&nbsp;$zip&nbsp;<font color='A0A0A0'>$date</font>  </font></h3>";
?>
代码没有什么问题,但看起来太草率了——我想知道是否有人能帮我把它弄得整洁

不要使用-将这些规则移动到CSS,您可以在那里设置元素的样式。而且并不是所有的计算机都有Helvetica,所以您可能需要将其更改为Helvetica,Arial;因此,设计在一定程度上与您的意图保持一致。 不要使用两次-将第二个h3放入或类似的东西中,并使用CSS来设置样式。 $city,$state$zip是脏的-在PHP中用空格连接字符串,然后回显字符串。 与使用echo输出PHP不同,只需在HTML内部对某些元素使用echo,这将使其更具可读性。 在将数据回显到页面之前,对从数据库中出来的每一段数据使用htmlspecialchars。这可以防止无效的HTML和XSS漏洞。 那应该会更紧些

不要使用-将这些规则移动到CSS,您可以在那里设置元素的样式。而且并不是所有的计算机都有Helvetica,所以您可能需要将其更改为Helvetica,Arial;因此,设计在一定程度上与您的意图保持一致。 不要使用两次-将第二个h3放入或类似的东西中,并使用CSS来设置样式。 $city,$state$zip是脏的-在PHP中用空格连接字符串,然后回显字符串。 与使用echo输出PHP不同,只需在HTML内部对某些元素使用echo,这将使其更具可读性。 在将数据回显到页面之前,对从数据库中出来的每一段数据使用htmlspecialchars。这可以防止无效的HTML和XSS漏洞。 那应该会更紧些

将所有代码移动到style=属性

还考虑从PHP ECHO中移动HTML,即

<h3>
<a href="<?php echo ... ?>"><?php echo ... ?></a>
</h3>
将所有代码移动到style=属性

还考虑从PHP ECHO中移动HTML,即

<h3>
<a href="<?php echo ... ?>"><?php echo ... ?></a>
</h3>
了解

了解。开始使用一个

或者从将CSS与其他数据分离开始。使用PHP输出动态数据,使用HTML/CSS输出静态数据。避免两者混用

来自谷歌的链接

了解

了解。开始使用一个

或者从将CSS与其他数据分离开始。使用PHP输出动态数据,使用HTML/CSS输出静态数据。避免两者混用


来自谷歌的链接

我已经为您将其放入了Herdeoc,但大部分更改都需要您自己完成

    // use heredoc's: http://www.php.net/manual/en/language.types.string.php#language.types.string.syntax.heredoc
    $out = <<HER 
    <h3><!-- you should use CSS to do this. You should get rid of font tags -->
        <font face='helvetica'> <!-- you don't need three either, you can put them all in one -->
            <font size='4'>
                <b>
                    <font color='B80000'>
                        $title
                    </font>
                </font>
            </font>
        </b><!-- move this so that it matches above -->

        <font color='A0A0A0'>
            $category &nbsp;
        </font>
        <font color='A0A0A0'>
            <a href='profile.php?id=$userid'>
                $user
            </a>
        </font>
        <font face='helvetica'>
            <font size='3'>
                <br> <!-- make this self closing -->
                &nbsp;$desc
            </font>
        </font>
        <br>
        <h3><!-- nested??!? You're already in H3-->
            <font color='101010'>
                &nbsp;$city,$state&nbsp;$zip&nbsp;
                <font color='A0A0A0'>
                    $date
                </font>  
            </font>
        </h3>
HER;

    ?>

我已经为您将其放入了heredoc,但大多数更改都需要您自己完成

    // use heredoc's: http://www.php.net/manual/en/language.types.string.php#language.types.string.syntax.heredoc
    $out = <<HER 
    <h3><!-- you should use CSS to do this. You should get rid of font tags -->
        <font face='helvetica'> <!-- you don't need three either, you can put them all in one -->
            <font size='4'>
                <b>
                    <font color='B80000'>
                        $title
                    </font>
                </font>
            </font>
        </b><!-- move this so that it matches above -->

        <font color='A0A0A0'>
            $category &nbsp;
        </font>
        <font color='A0A0A0'>
            <a href='profile.php?id=$userid'>
                $user
            </a>
        </font>
        <font face='helvetica'>
            <font size='3'>
                <br> <!-- make this self closing -->
                &nbsp;$desc
            </font>
        </font>
        <br>
        <h3><!-- nested??!? You're already in H3-->
            <font color='101010'>
                &nbsp;$city,$state&nbsp;$zip&nbsp;
                <font color='A0A0A0'>
                    $date
                </font>  
            </font>
        </h3>
HER;

    ?>

您可以做的第一件事是关闭PHP标记,写出HTML,同时在打开标记时留下空格、换行符和缩进,并在需要时重新打开PHP:

?>
<h3>
    <font face='helvetica'>
        <font size='4'>
            <b>
               <font color='B80000'><?php echo $title; ?></font>
        (...)
            </b>
        </font>
    </font>
</h3>
<?php 
同时也要研究一下这个问题

从非语法的角度来看,如果使用CSS而不是所有这些标签,从长远来看,您会得到更好的服务:


您可以做的第一件事是关闭PHP标记,写出HTML,同时在打开标记时留下空格、换行符和缩进,并在需要时重新打开PHP:

?>
<h3>
    <font face='helvetica'>
        <font size='4'>
            <b>
               <font color='B80000'><?php echo $title; ?></font>
        (...)
            </b>
        </font>
    </font>
</h3>
<?php 
同时也要研究一下这个问题

从非语法的角度来看,如果使用CSS而不是所有这些标签,从长远来看,您会得到更好的服务:


谢谢我现在就接受答案,但它告诉我要等几分钟:P@melee:我冒昧地添加了5—95%的PHP开发人员都会忘记这一点。@Marie McCullin您应该看看一些开源PHP项目,看看它们是如何完成的—这应该让您了解开发人员倾向于使用的风格。下次你在设计东西的时候,它会从一开始就给你一个更好的视角。谢谢你们,我真的很欣赏它,汉克斯!我现在就接受答案,但它告诉我要等几分钟:P@melee:我冒昧地添加了5—95%的PHP开发人员都会忘记这一点。@Marie McCullin您应该看看一些开源PHP项目,看看它们是如何完成的—这应该让您了解开发人员倾向于使用的风格。下次你在设计东西的时候,它会从一开始就给你一个更好的视角。谢谢你们,我真的很感激