帮助我解决我的草率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 </fon
echo "<h3><font face='helvetica'><font size='4'><b><font color='B80000'>$title</font></font></font></b> <font color='A0A0A0'>$category </font><font color='A0A0A0'><a href='profile.php?id=$userid'>$user</a></font>
<font face='helvetica'><font size='3'><br> $desc</font></font><br>
<h3><font color='101010'> $city,$state $zip <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
</font>
<font color='A0A0A0'>
<a href='profile.php?id=$userid'>
$user
</a>
</font>
<font face='helvetica'>
<font size='3'>
<br> <!-- make this self closing -->
$desc
</font>
</font>
<br>
<h3><!-- nested??!? You're already in H3-->
<font color='101010'>
$city,$state $zip
<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
</font>
<font color='A0A0A0'>
<a href='profile.php?id=$userid'>
$user
</a>
</font>
<font face='helvetica'>
<font size='3'>
<br> <!-- make this self closing -->
$desc
</font>
</font>
<br>
<h3><!-- nested??!? You're already in H3-->
<font color='101010'>
$city,$state $zip
<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项目,看看它们是如何完成的—这应该让您了解开发人员倾向于使用的风格。下次你在设计东西的时候,它会从一开始就给你一个更好的视角。谢谢你们,我真的很感激