2016-03-20 13 views
0

最近私は私の会社を切り替えました。コーディング標準に従って、私の新しい会社では必須です。テクニカルリードのレビューでは、私は「プライベートメソッド内でプライベートメソッドを呼び出すべきではない」というレビューコメントを得ました。プライベートメソッドのコーディング標準の明確さ

これはどこにでも続いている一般的な練習ですか?プライベートメソッドのコードサイズが増えている場合(50行以上60行以上)、そのような場合にメソッドのサイズと複雑さをどのように減らすことができますか?

異なるメソッドで異なる機能を定義すると、わかりやすくなりますが、上記のプライベートメソッドの標準に従うと、この場合のメソッドの複雑さを処理する方法があります。

+4

私があなたのところにいたら、私は別の仕事を探し始めるでしょう。ルールを作るためのルールのように思えますが、それは良い実践では根拠がないからです。あなたの疑問は十分に確立されています。 –

+0

@ジムガリソンええ、私はどのような種類の[tin god](http://www.dictionary.com/browse/tin-god)がそのようなルールを思いついたのか、そしてもしあれば、どんな正当化が発明されたのだろうと思います。 – Andreas

+0

私はいつもこのルールに疑問を持っていました。私は今新しい仕事を探し始めなければなりません:):D – Raghav

答えて

6

いいえ、これは一般的ではありません。私はそのようなルールを聞いたことがないし、それについて何の理由も考えることはできない。プライベートメソッドを別のプライベートメソッドから呼び出すことには何の問題もありません。

+0

完全に同意します。共通のコードを持つときはいつでも、同じロジックを全面的に複製するのではなく、再利用のための別の方法に入れなければなりません。 DRY([自分を繰り返さないでください](https://en.wikipedia.org/wiki/Don't_repeat_yourself))。また、大きすぎるメソッドを、より小さな管理可能な明確な責任に分割する必要があります。明快さとコード概要を助けます。 – Andreas

関連する問題