Hashtableは資産を格納するのに適していますか？

Question

私はActionscript3の背景から来て、これは初めての私の人生でJavaを書くことです。ハッシュテーブルはFlashの辞書と似ているようですが、正しく使用していますか？私は、Hashtableは文字列をキーとして受け入れ、Typefaceはオブジェクトとして受け入れると考えています。これは正しいです？このようなものに適している別のCollectionサブクラスはありますか？是非、n00b Javaを裂けてください。私はこれを学ぶ必要があります。

package com.typeoneerror.apps.app_name.utils; 

import android.content.Context; 
import android.graphics.Typeface; 

import java.util.Hashtable; 

public class FontRegistry 
{ 
    private static FontRegistry _instance; 

    private Context       _context; 
    private Hashtable<String, Typeface>  _fonts; 

    private FontRegistry() 
    { 
     _fonts = new Hashtable<String, Typeface>(); 
    } 

    public static FontRegistry getInstance() 
    { 
     if (_instance == null) 
     { 
      _instance = new FontRegistry(); 
     } 
     return _instance; 
    } 

    public void init(Context context) 
    { 
     _context = context; 

    } 

    public Typeface getTypeface(int resourceId) 
    { 
     String fontName = _context.getResources().getString(resourceId); 
     if (!_fonts.containsKey(fontName)) 
     { 
      String fontPath = "fonts/" + fontName; 
      Typeface typeface = Typeface.createFromAsset(_context.getAssets(), fontPath); 
      _fonts.put(fontName, typeface); 
     } 
     return (Typeface)_fonts.get(fontName); 
    } 
} 

Answer 1

2つの提案があります。

まず、変数型はMapのインターフェイスを参照する必要があります。これにより、将来の柔軟性が増し、他のほとんどのJava開発者とのメッシュ・メッシュ化が進んでいきます。

第2に、実装はHashTableではなく、HashMapでなければなりません。 HashTableはすべてを同期しますが、HashMapではそうではありません。

マルチスレッドアクセスが必要な場合は、HashTableの代わりにConcurrentHashMapを使用することをお勧めします。 ConcurrentHashMapは、アクセス中にマップ全体をロックしないため、パフォーマンスが向上します。

ので、

private Map<String, Typeface>  _fonts; 

と

_fonts = new HashMap<String, Typeface>(); 

最後に、多くのJava開発者は、あなたがアンダースコアでメンバ変数を開始しないことを好むだろう。これは議論の余地があります。

EDIT：最後のニックピック。レジストリにシングルトンパターンを使用しているようです。これは後であなたを噛むことができるので、シングルトンを避けることを検討してくださいhttp://accu.org/index.php/journals/337。しかし、それを無視して、宣言でシングルトンの静的インスタンスをインスタンス化する方が良いかもしれません。これは、初めてフェッチするときに起こりうる競合を回避できます。だから、

：

private static FontRegistry _instance = new FontRegistry; 

Answer 2

私のコメントに展開するには、getInstance()の実装はスレッドセーフではありません。

public class Singleton { 

    // Private constructor prevents instantiation from other classes 
    private Singleton() { 
    } 

    /** 
    * SingletonHolder is loaded on the first execution of Singleton.getInstance() 
    * or the first access to SingletonHolder.INSTANCE, not before. 
    */ 
    private static class SingletonHolder { 
     public static final Singleton INSTANCE = new Singleton(); 
    } 

    public static Singleton getInstance() { 
     return SingletonHolder.INSTANCE; 
    } 
} 

。また、あなたがAndroidのないように、「漏れのために開発するときに注意する必要があります：あなたは本当にシングルトンパターンを使用する必要がある場合、あなたは（私は露骨にwikipedia articleからコピーした）「ビル・ピュー」バージョンを使用することができます"a Context。なぜそれが悪いのか、それを避ける方法については、good articleです。要するに、Context（またはContextを参照するオブジェクト）へのstaticの参照は、Activityインスタンスがガベージコレクションできないことを意味します。