java - Java で += ではなく =+ の使用を自動的にキャッチする

Question

私は、以下を入力して、テストに入るまでそれをキャッチしないというこの恐ろしい習慣を持っています:

int i = 1;
int j = 2;
i =+ j;  //i equals 2, not 3 as intended; assign only, + is unary and works on the j

もちろん、正しいバージョンは

int i = 1;
int j = 2;
i += j;  //i equals 3, as intended with additive & assignment compound operator

私はこの過ちを無数に犯してきました。テストケースを回避したバグのあるコードがいくつか出回っていても、私は驚かないでしょう。これを体系的に防ぐ方法が必要です。何か案は？

score 10 · Accepted Answer

PMDやCheckstyleなどのツールを定期的に使用します。理想的には、ビルドプロセス/継続的インテグレーションの一部として。

これらのツールのいずれかがデフォルトでこれを問題として認識しているかどうかわからないため、カスタムルールを定義する必要があるかもしれません。

これは、この問題をキャッチするだけでなく、他の多くの潜在的な問題を示唆するものでもあります。

score 10 · Accepted Answer

使用している IDE に応じて、構文の強調表示を行う場合は、強調表示を変更して、パターン=+を探して見にくい色にします。

score 1 · Accepted Answer

次のような標準のテキストユーティリティを使用します。

find . -name \*.java -exec grep -H "=+" {} \;

score 0 · Accepted Answer

単体テストは厳密に行う必要があると思います。現時点では、特定の問題が 1 つ気になっていますが、適切な単体テストを行うと、ほとんどの問題 (スレッド化や読み込みを無視するなど) をキャプチャできます。

クラス/メソッドを分解して、機能の各チャンクをテストし、完全な (またはできるだけ近い) カバレッジを確保できるようにする必要があります。

Clover (商用) またはEmma (オープンソース) がコードカバレッジを管理します。

score 0 · Accepted Answer

balpha がコメントしたように、これを見つける簡単な方法は、コード内の「=+」を grep することです。決して意図的なものではないでしょう。

5 に答える 5