ベストプラクティス：型の安全警告を処理するにはどうすればよいですか？

Question

次の文は、Eclipseでの警告の原因：

Map<String, String> options = DefaultCodeFormatterConstants.getEclipseDefaultSettings(); 

警告：

Type safety: The expression of type Map needs unchecked conversion to conform to Map<String,String> 

この警告を処理するためのベストプラクティスは何ですか？ SupressWarning（ "未チェック"）アノテーションを追加する必要がありますか？全体のクラスのために？それとも方法だけ？それとも私は警告を無視するべきですか？または、この警告をどうやって取り除くことができますか？

Answer 1

DefaultCodeFormatterConstants.getEclipseDefaultSettingsまでは適切に生成されていますが、内部ラッパーを生成し、その内部の内部変数宣言の注釈を持つことを検討したいと思います。注釈が「安全」である理由を説明するコメントも含めます。

通常、SuppressWarningsアノテーションのスコープを可能な限り小さくし、可能な限り少ないものにしたいとします。また、毎回あなたの推論を説明するコメントが必要です。

Answer 2

永久に「取り除く」ための唯一の方法は、パラメータ化されたマップを返すJava 5以降のバージョンのAPIにアップグレードすることです。

'クリーンアップ'の並べ替えを行う最も簡単な方法は、自分のクラスでRAWタイプを返すメソッドをラップし、そこでキャストを実行し、チェックされていない警告を抑制する方法です。この方法では、少なくとも1つの場所で行われ、誤って「本物の」警告が表示されない場所には@SuppressWarningsを使用していません。

Answer 3

宣言のこれらの警告は、行が非常に迷惑であることがわかりました。無意味なことに、内部が深く、タイプ消去があることを知っています。コンパイラは何も言わずに不平を言っています。だから私はそれらを無視する。しかし、これはベストプラクティスではないかもしれません。 Java 7では、IIRCのいくつかの改良が行われました。

List<String> = new ArrayList<>(); 

「弦」の冗長な入力を省き、警告を回避します。

注 - 私は、宣言行にではなく、でない警告に注意します。

Answer 4

メソッドが生のマップを返す場合は、マップが実際にMap<String, String>であることを（ドキュメントまたはコードから）確認し、警告を無視することだけができます。コンパイラは、返されたMapがMap<String, String>であることを確認する方法を持っていません。

最小の可能な範囲で警告を無視する必要があります。そうでないと、コンパイラからの警告なしに他の安全でないコードを追加することができます。