2

=私の同僚の何人かは、代入ステートメントの各演算子の前後にスペース/タブを追加するこのインデント スタイルに従います。ブロック内に割り当てステートメントが 1 つしかない場合でも、これを行います。私は、読みやすく美しいコードを書くことに全力を注いでいます。しかし、ファイル全体でこの種のアライメントが本当に必要なのでしょうか?

この種のインデントを示すサンプル コードを以下に示します。

    public function getCMSSectionData($args="1")
    {
        $sql                    =   "SELECT * FROM     med_cms_section WHERE $args";            
        $data                   =   $this->getdbcontents_sql($sql); 
        return $data;
    }

    public function insertCMSSection($dataArray)
    {

        $sendCount  =   100;

        $sql        =   "select * from med_email_pending order by priority desc limit ".$sendCount;
        $res        =   $this->db_query($sql);

        while($row = mysql_fetch_array($res))
        {
            $id         =   $row["id"];
            $to         =   $row["to"];
            $from       =   $row["from"];
            $subject    =   $row["subject"];
            $message    =   $row["message"];
            $priority   =   $row["priority"];
            $this->sendmail($to,$from,$subject,$message);
            $this->sendmail("user@example.com",$from,$subject,$message);
            $sql        =   "delete from med_email_pending where id=".$id;
            $this->db_query($sql);
        }
    }

これじゃないですか:-

$sql = "SELECT * FROM med_cms_section WHERE $args";         
$data = $this->getdbcontents_sql($sql);

これより良い：-

$sql                    =   "SELECT * FROM med_cms_section WHERE $args";            
$data                   =   $this->getdbcontents_sql($sql);
4

2 に答える 2

2

いいえ、少なくとも 1 つの理由から、これは恐ろしい考えです。行を追加する必要がある場合は、他のいくつかの行を再フォーマットする必要がある場合があります。これは、ソース管理のリビジョン間で行われた変更を確認したい場合に、差分ビューアーを混乱させます。

于 2012-10-09T20:45:46.117 に答える
2

いいえ。コードの変更がより困難になります (IDE がこれを行わない限り、手動で調整する必要があります)。また、コードが読みやすくなるとは思いません。多分それは主観的ですが。

于 2012-09-17T09:14:18.103 に答える