コードレビュー

私たちのチームは、構造が非常に合理的で、全体の書き込みコード準拠した上で、コードMagiccodeグループを読んでいます。自分自身のことをやっ内の各クラスの各メソッドを。変数がローカル近いで定義されている、何も口に合わ非常に混乱コードが表示されません。しかし、まだここに軟膏で飛ぶそのうちのいくつかは、私の個人的な意見の一部を以下に示します。

1.何のコメントはありません。コードに、私の理解にコメントせずにテキスト全体が、障害物の多くを引き起こしたが、彼らはまた、いくつかのトラブルを引き起こした後、私はそのメンテナンスを信じています。

2.コードが仕様に沿って書かれていますが、スタイルは非常に均一ではありませんが、私は個人的な習慣だけの問題を疑うようになりました。しかし、その後、彼らは前に同じことを発見し、文は同じではありませんした後、どのような意図を知りません。】第1及び第2の比較以下の図、及び図第4及び図5を参照。

以下は、アカウント未開拓を考慮されていないなどのいくつかの場所で3.書き込みは、十分に正確ではありません。

4.あなたの意見を聞くためにここに提示いくつかの場所、純粋に個人的な意見は、あります。たとえば、次の例では、個人が感じている「最初のリストがないかを決定するために、虐待は、少しスローに感じる」が、一部の人は、以下の文言より良いと感じています。

5.まだ言葉が== nullの事態が発生し、どうなる理由は、次の例のように、理解していない、いくつかの質問を持っている、それはスペースの無駄ではありません。

もちろん、コードを見てすることなど、やりがいがあります。

次の例は、単結合パターンは、唯一のグローバルSearchManagerインスタンスが存在するであろうように、コンストラクタが閉鎖され、外部でGetInstanceための唯一の方法は、固有のインスタンス変数を取得しました。以前ここで使用される非常に良いですが、他の場所で見られます。

ます。https://www.cnblogs.com/buaashine/archive/2012/12/13/2815630.htmlで再現

おすすめ

転載: blog.csdn.net/weixin_34219944/article/details/94548467