0

私たちの会社ではコーディング規約に従おうとしていますが、私の先輩の何人かはコーディング規約をいくつか教えてくれました。それらのほとんどは私が知っているものであり、良いものですが、そのうちの1つは同等性の整合であり、コード全体の整合性は少し奇妙です。特に構成値で一連の割り当てを行う場合、通常は等式アライメントを行います。お気に入り:

$params = array(
                        'dbdriver'  => $dns['scheme'],
                        'hostname'  => (isset($dns['host'])) ? rawurldecode($dns['host']) : '',
                        'username'  => (isset($dns['user'])) ? rawurldecode($dns['user']) : '',
                        'password'  => (isset($dns['pass'])) ? rawurldecode($dns['pass']) : '',
                        'database'  => (isset($dns['path'])) ? rawurldecode(substr($dns['path'], 1)) : ''
                    );

または好き:

var $next_link          = '>';
var $prev_link          = '<';
var $last_link          = 'Last ›';
var $uri_segment        = 3;
var $full_tag_open      = '';
var $full_tag_close     = '';
var $first_tag_open     = '';
var $first_tag_close    = ' ';
var $last_tag_open      = ' ';
var $last_tag_close     = '';
var $first_url          = ''; // Alternative URL for the First Page.
var $cur_tag_open       = '&nbsp;<strong>';
var $cur_tag_close      = '</strong>';
var $next_tag_open      = '&nbsp;';
var $next_tag_close     = '&nbsp;';
var $prev_tag_open      = '&nbsp;';
var $prev_tag_close     = '';
var $num_tag_open       = '&nbsp;';
var $num_tag_close      = '';
var $page_query_string  = FALSE;
var $query_string_segment = 'per_page';
var $display_pages      = TRUE;
var $anchor_class       = '';

しかし、通常のコードでライブラリをロードし、値をインスタンス化し、メントードを呼び出し、変数で応答を取得している場合でも、等式アライメントが必要ですか?ec2classのコンストラクターがあるように:

public function __construct(array $options = array())
{
    $this->api_version = '2012-07-20';
    $this->hostname = self::DEFAULT_URL;
    $this->auth_class = 'AuthV2Query';

    return parent::__construct($options);
}

またはこのような場所:

        $this->CI->load->library('awslib');

    $s3          = new AmazonS3();
    $bucket_name = ($is_private) ? 'private_s3_bucket' : 'public_s3_bucket';
    $bucket      = $this->CI->config->item($bucket_name);
    $options     = array(
        'fileUpload' => $file_path,
        'acl'        => AmazonS3::ACL_PUBLIC
    );

    try {
        $response = $s3->create_object($bucket, $file_name, $options);
    } catch (Exception $e) {
        echo "<pre>";
        exit($e);
        echo '</pre>';
    }

したがって、最後の2つの例では、平等の調整が本当に重要であり、そのような標準はありますか?ライブラリのインスタンス化、他のもののロード、条件付きで値を割り当てることの組み合わせである場合、それが私にとってより読みやすいとは思わないので、平等の配置は重要ではないかもしれないと思うので、標準があるかどうか教えてください多くのフレームワークやライブラリなどで、これらのステートメントに混合と割り当てがある場合は常に、そのような場所としての平等の調整を見たことがありません。だから教えてください。

4

1 に答える 1

0

パターンを認識できるように、意味のある方法でコードを揃えることが重要だと思います。ただし、コードの再配置に多くの時間を費やしているようで、最小限の読みやすさが得られます。

この場合、それが私の会社であれば、慣例を変更するか、少なくとももう少し柔軟にしようとします。地元だから特に。等値演算子の配置をベスト プラクティスとして言及しているドキュメントは見つかりませんでした。

于 2013-06-18T13:48:56.420 に答える