2011-01-07 9 views
3

複数の作者のコードを扱うときに、中括弧の優先順位(同じ行と新しい行)の問題が頻繁に発生します。それはあなたの好みを使って既存のスタイルをマッチさせることに関して、良い/悪い練習か、あるいは問題ではないでしょうか?前回の著者の中括弧の使用法と一致させる必要がありますか?

新しいコードをクラスに追加して既存のコードを修正する場合は状況が変わりますか?最後に、スタイルを一致させる必要がある場合、マッチがどのくらい伝播する必要がありますか?すなわち、ファイル、クラス、サブクラスなど

例:

if(this) 
{ 
    doThat(); 
} 

対。

if(this){ 
    doThat(); 
} 

答えて

6

一般的に言えば、一貫性のない書式設定されたコードは、自分以外のスタイルで書式設定されたコードよりも読みにくく、理解しにくくなります。

オープンソースプロジェクトで作業する場合は、使用しているスタイルと一致するように丁寧なものと見なされます。コードを共有しない場合は、スタイルに従うか、リフォーマッタを使用して既存のコードをすべて実行します。

+0

Reformatters FTW! –

0

あなたの同僚と話し合い、一貫したスタイルを決定してください。一度それを行うと、そのスタイルで新しいコードをすべて書き、必要に応じて古いコードを変換します。これはプロジェクトとそれに取り組んでいる人々に固有の問題です。

ごくまれに、コンセンサスに来ることはできませんが、私はあまり心配しません。人々が括弧を動かして時間を費やしている世界ではあまりにも多くのバグがあります。 :D

+0

"これはプロジェクト特有の問題です"どのように? 「GUIアプリケーションではなく組み込みシステムを使用しているため、自分たちのラインでブレースを使用することが重要です」 :-) – Oddthinking

+0

@「プロジェクトに取り組んでいる人々に固有の」プロジェクト固有の@しかし、私は空白の代わりにタブを使って、Webアプリ開発のバグを20%削減すると聞いています...:D –

3

多くのコードがバージョン管理システムにチェックインされていれば大変重要です。単純なブレーススタイルの変更のために、歴史には騒がしい変化がたくさんあります。私は自分のためにそれをしません。あなたはローマにいます....

2

これを共有する場合は、一貫性のために彼のスタイルに合ってください。そうでない場合は、快適に感じることをしてください。私は個人的にあなたのスタイルを好みます。

0

中括弧を配置する場所(コード賢さ)に関係なく、それは単なる問題です。経験豊富なコーダーは同じ行に書いている傾向がありますが、次の行に置くのが好きです。これはコードのブロックが簡単に見えるようにするためです。中ぐらいのコーディング規則をコーディングすることを望むなら、あなたのチームに相談して1つを選択してください。しかし、おそらく笑います。

+1

Sigh。彼らは笑いません。彼らは来週は強く主張し、他のチームの生産性を失うだろう。 – Oddthinking

0

コードを熟読し、全体に散らばった数多くのミスマッチなスタイルを見つけること以外には何もありません。明示的または暗黙的ではなく、プログラム上の誰かと共同作業を行う場合は、標準を使用する必要があります。中括弧のような単純なものは、少なくとも問題ではありませんが、あなたのプロジェクトが経営陣によって再検討され、その中で一致した小さなものを得ることさえできない場合、悪いです。あるいは、あなたが友人と一緒に個人的なプロジェクトに取り組んでいて、他の人たちと一緒に参加することに決めたら、プロジェクトは専門家ではない人から始めるでしょう。このようなグループで働くときは、グループの基準よりも個人的な意見は少ないと言います。

関連する問題