为什么我写的程序看起来很散呢？

Question

有时候看到别人的程序很紧凑，但是感觉自己写的很零散，这是为什么呢？
请大家教教我。

Answer 1

建议看下mvc框架结构，真心很不错，我也是刚接触

Answer 2

public function uList($role)
{
$sql = "SELECT * FROM qsj_user_profile
RIGHT JOIN qsj_user ON qsj_user.uid = qsj_user_profile.uid
WHERE qsj_user.role = $role
ORDER BY qsj_user.regdate DESC
";
$query = $this->db->query($sql);
return $query->result_array();
}给你们看看看吧

Answer 3

return $this->db->query($sql)->result_array();

Answer 4

return  $this->db->query("SELECT * FROM qsj_user_profile
        RIGHT JOIN qsj_user ON qsj_user.uid = qsj_user_profile.uid
        WHERE qsj_user.role = $role
        ORDER BY qsj_user.regdate DESC")->result_array();

Answer 5

public function find()
{
$sql = "SELECT * FROM qsj_user_profile
RIGHT JOIN qsj_user ON qsj_user.uid = qsj_user_profile.uid";
if(!empty($_GET['email']))
{
$sql.="AND qsj_user.email=".$_GET['email'];
}
if(!empty($_GET['nickname']))
{
$sql.="AND qsj_user_profile.nickname = ".$_GET['nickname'];
}
if(!empty($_GET['province']))
{
$sql.="AND qsj_user_profile.province =".$_GET['province'];
}
if(!empty($_GET['city']))
{
$sql.="AND qsj_user_profile.city =".$_GET['city'];
}
if(!empty($_GET['dist']))
{
$sql.="AND qsj_user_profile.dist = ".$_GET['dist'];
}
if(!empty($_GET['style']))
{
$sql.="AND qsj_user_profile.style =".$_GET['style'];
}
$query = $this->db->query($sql);
return $query->result_array();

}这是我写的一个多条件查询的语句，请问大家，有没有别的写法或思路去解决，让程序看起来比较优雅。

Answer 6

$sql = 'select ..........';
$fields = array('email','nickname',.....);
foreach($fields as $field) {
  if(!empty($_GET[$field])) {
    $value = mysql_real_escape_string($_GET[$field]);
    $sql .= " AND qsj_user_profile.{$field} = '{$value}' ";
  }
}

Answer 7

没有好的设计思想，可以看看Design Pattern那本书

Answer 8

这事说来话长。
简而言之。
一　没写够。
二　写过不改善。
三　没有欲望。
四　来csdn 太少~

Answer 9

public function uList($role)
    {
        $query = $this->db->query("SELECT * FROM qsj_user_profile
        RIGHT JOIN qsj_user ON qsj_user.uid = qsj_user_profile.uid
        WHERE qsj_user.role = $role
        ORDER BY qsj_user.regdate DESC
        ");
        
        return $query->result_array();
    }