8

私はソナーで冒険を始めました;）

デフォルト構成のソナーには、PMD UnnecessaryLocalBeforeReturn エラーがメジャー レベルに設定されています。

    List<Todo> filtered = em.createQuery(query).getResultList();
    return filtered;

これは、上記のコードを 1 行で変更する必要があることを意味します。

帰国前にこの「不要な」ローカル化を同僚に勧めているので、私にとっては非常に興味深いです。

デバッグしやすいと思います。戻り行にブレークポイントを設定すると、そこに到達したときにこの値の準備が整い、ステートメントを選択したり、「式をステップオーバー」したりする必要がなくなると確信しています。

さらに、メソッドのリターンポイントを減らすことにプラスの影響があると信じています。

私の質問は: Checkstyle、PMD、FindBugs などのプロジェクトからのエラーがエラーとして認識された理由について何らかの説明/議論はありますか?

4

1 に答える 1

1

ポイントがリストのコンテンツのみを表示している場合は、メソッドの呼び出し元にブレーク ポイントを配置することもできます。もう 1 つのオプションは、ブレーク ポイントを設定して値を評価することです (Eclipse と IntelliJ はうまく機能します)。

なぜそれが悪い習慣と見なされるのですか?

変数への参照は不要ですが、変数への参照を追加するだけです。

これは、ガベージ コレクターのワークロードを増やすだけです。

于 2014-11-10T15:47:20.910 に答える