良好的编码实践

Question

我今年六月毕业,所以在此之前我想做好自己在行业工作的准备:)

My Question:

例如,我想使用列表框列出表中的所有歌曲,这就是我实现它的方式:

playlist.php

<tr id='' >
<td  width="" class="">&nbsp;&nbsp;<font color=#00000 />Select Songs</td>
<td width="" colspan="">
<select size='25'multiple='multiple' id="select_songs" name="playlist_songs[]">
<?php
display_songs_list();
?>
</select></td>
</tr>

我有一个单独的php_functions.php文件,我已经实现了所有的PHP功能

function display_songs_list(){
    $query = "select * from songs order by ID asc";

    $result = mysql_query($query);
    if(!$result)
    echo "<script language = 'javascript'>alert('$result Sorry couldn't connect to the database...');</script>";
    else{
        $num_rows = mysql_num_rows($result);
        if($num_rows > 0){
            while($row = mysql_fetch_array($result)){
                $ID = $row['ID'];
                $title = $row['title'];
                $value = $ID.'_'.$title;
                echo "<option id=\"$ID\" value=\"$value\">$ID : $title</option>";
            }
        }
    }
}

是否推荐这种实施方式.我还能做些什么来增加scalability/maintainability/re-usability.请指导我这个.是否建议遵循任何行业编码标准,如果是这样,您所遵循的是什么.谢谢.

Answer 1

• color=#00000不推荐使用内联样式属性()和kill kittens.了解CSS.
• 除了表格数据(想想电子表格)之外,不建议使用表格,而是使用CSS作为布局.
• 硬编码空白 也不可取,如果它没有添加任何含义,学习使用CSS添加间距.
• echo从同一个函数中输入HTML 或 Javascript是不好的.Javascript警报将在您调用该函数时导致无效的HTML语法.
• 如此紧密地混合数据库调用和HTML并不好.研究MVC分离.
• 在半成品页面中间向用户输出随机的,无意义的Javascript警报是不好的.您应该显示专用的错误页面.请参阅MVC,它可以帮助您实现此目的.
• 混合单引号和双引号的属性是不一致的,使代码更难以阅读.坚持使用一种类型的报价.
• 显示的标记中没有任何表单可提交所选的选项