Java - HashMapsのHashMapを初期化する

Question

私はjavaに慣れていないし、単純なNaiveBayesクラシファイアを作成して練習しています。私はまだオブジェクトのインスタンス化には新しく、HashMapのHashMapを初期化するために何をすべきか考えています。新しい観測値をクラシファイアに挿入する際、特定のクラスで見えないフィーチャ名に対して新しいHashMapを作成できますが、初期化する必要はありますか？この質問は、ナイーブベイズ分類器に固有ではない

import java.util.HashMap; 

public class NaiveBayes { 

    private HashMap<String, Integer> class_counts; 
    private HashMap<String, HashMap<String, Integer>> class_feature_counts; 

    public NaiveBayes() { 
     class_counts = new HashMap<String, Integer>(); 
     // do I need to initialize class_feature_counts? 
    } 

    public void insert() { 
     // todo 
     // I think I can create new hashmaps on the fly here for class_feature_counts 
    } 

    public String classify() { 
     // stub 
     return ""; 
    } 

    // Naive Scoring: 
    // p(c | f_1, ... f_n) =~ p(c) * p(f_1|c) ... * p(f_n|c) 
    private double get_score(String category, HashMap features) { 
     // stub 
     return 0.0; 
    } 

    public static void main(String[] args) { 
     NaiveBayes bayes = new NaiveBayes(); 
     // todo 
    } 
} 

注、私はいくつかのコンテキストを提供するだろうと思いました。

Answer 1

はい、初期化する必要があります。あなたはclass_feature_countsに値を追加したい場合は

class_feature_counts = new HashMap<String, HashMap<String, Integer>>(); 

、あなたもそれをインスタンス化する必要があります。

HashMap<String, Integer> val = new HashMap<String, Integer>(); 
// Do what you want to do with val 
class_feature_counts.put("myKey", val); 

Answer 2

参照変数を介してオブジェクトを使用する前にオブジェクトを作成する必要があります。そのオブジェクトがどれほど複雑であるかは関係ありません。最も一般的なケースですが、コンストラクタで初期化する必要はありません。必要に応じて、代わりに「遅延初期化」を使用することができます。

Answer 3

再帰ジェネリックデータ構造を、マップのマップのように、ではないあからさま悪い考えながら、しばしばありますあなたがリファクタリングできる何かを示す - 内側の地図は、単に地図ではなく、最初の注文オブジェクト（地図を保持している）であることがよくあります。これらの内部オブジェクトはまだ初期化する必要がありますが、多くの場合、よりクリーンで明確な開発方法です。

たとえば、Map<A,Map<B,C>>がある場合、実際にはA to Thingのマップが格納されていることがよくありますが、Thingが格納される方法は偶然にマップです。あなたは、多くの場合、クリーナーと実物がマップであるという事実を隠し、代わりのもののように定義されてMap<A,Thing>のマッピング保管しやすく、それを見つけることができます。また

public class Thing { 
    // Map is guaranteed to be initialized if a Thing exists 
    private Map<B,C> data = new Map<B,C>(); 

    // operations on data, like get and put 
    // now can have sanity checks you couldn't enforce when the map was public 
} 

---

を、グアバのMulitmap/Multisetユーティリティに見て、このような場合には非常に便利です。特に内部オブジェクト初期化を自動的に行います。あなたのケースでは、実際にGuava Multisetを実装したい場合はいつでも、Map<E, Integer>を実装することができます。よりきれいで明瞭。

Answer 4

1. HashMapで変数を宣言しないでください。それはあまりにも限定的です。

2. はい、class_feature_countsを初期化する必要があります。エントリを追加するので、有効なマップでなければなりません。実際には、それぞれを開始する方法が1つしかないので、コンストラクタではなく宣言時に両方を初期化します。私は今あなたがJava 7を使用していることを願っています。この方法は簡単です。

   プライベートマップ< String、Integer> classCounts = new HashMap <>（）;

   プライベートマップ<文字列、マップ<文字列、整数>> classFeatureCounts = new HashMap <>（）;

コンパイラは、タイプを<>から推測します。また、変数名を標準的なJavaのラクダのcaseスタイルに変更しました。 classCountsとclassFeatureCountsは接続されていますか？