コードの品質は、より大きな能力を強化するために、より高い基準のためのアクセス基準の個人や企業のコードを反映した
あなたの参照交換用のコードを書くときに、自分の心の中の問題を紹介するこの記事を
- コードが最終的に見直しの他のメンバーを与えるので、コミットするとき、同じコミット下に置くの変化の同じクラスに属するください、変更は、見直し、あなたのロジックを理解するために他人を容易コミット別の下に置かれ、関係ありません。
2 .commitメッセージは、通常の3つのセクション、明確に記述する必要があります
簡単な紹介、カテゴリ、偉業、スタイル、ドキュメント、構築のこの変更：最初の段落 ...、 との紹介
第二段落：はじめに、なぜこれを行います変更
第三段落：このようJIRAリンクとして掲載あなたの単一のリンクのニーズを、入れてください
。目的はその正確性を確保するために、ロールバックコードを促進したり、コードをレビューするときに、後に明確に記述し、詳細にする
3.push前に、マスターをリベースしてくださいそして、このようにマスターが明確であることを保証し、-f押し
これらは、開発プロセスの話されている、以下のコードと
、家の装飾の多くのコードを書くためには、連結されている
1.特定する責任を
各変数について
a）は、可視範囲 - あなたの変数がやってするために使用されている、あなたは、公共、民間、または唯一のサブクラスを使用する必要がありますか？
b）の責任 - それは問題では責任それを使用して、変数の名前は？それが合理的であるという名前の、対応されてはい、この変数は、関数と関連していないだけ？
C）多重化は - あなたの変数はクラスレベル、またはインスタンスレベルである必要があり、それから共有すべき、他のライブラリがすでに同様の変数が提供されていない、この変数を定義する必要があるのでしょうか？
コードのすべての行
a）は、コードのあなたの行をやってするために使用され、それが唯一のものではありません
B）あなたは、異なるコードにこのコード行を共有したいと思いますか
C）は、符号が同様の機能を実現されていないが存在します
各機能
a）の責任 - ひとつだけ
b）の多重化 - あなたが共有する必要があり、他人にそれを達成することができますか？
c）の開閉を - ？行く開口部を変更することは容易ではないと閉じハードコードコードは設定ファイルに移動することはできません？
d）のテスト - ？テストすることができない、ユニットテストを書くのは簡単です
E）例外は - あなたがオンまたはオフ投げ、例外をスローする必要があり、例外はそれに対処する必要がある、ログのシンプルなノート、または自分自身のワークアウトが発生した、またはしたいと、パッケージの下に例外をスローし、またはその結果に関係なく、親を聞かせています私たちを助けるために対処？なぜこれを行いますか？
f）のログ - デバッグログを再生するために必要な場所フォーマットをログ、エラーログレベルをプレイしないといけない場所があり、ログの内容は意味がありませんあなたは、人々の維持を確保する必要があるが、ログが知って見あなたが意味するかです？
各クラス
a）はシングル責任か
B）私たちは多重他人かどうか、再利用できるかどうか
C）シャッターか
D）試験を容易に試験することができるかどうか
e）の例外処理は合理的です
f）の戦いログが合理的です
各パッケージ
各プロジェクト