4

私は多くのジェネリックを使用しているが、タイプセーフも必要とするプロジェクトを持っています。スタック トレースは約 200 回の呼び出しであるため、現在のエラーが表示された場合、関連する完全なソース コードを提供できません。これまでに発見した問題を掘り下げて、簡単な例を示します。

まず、一般的な値を保持するクラスがあります。各ジェネリックは Comparable を実装します。それは、Long、Boolean、またはこのインターフェースを実装するカスタム クラスである可能性があります。

このような各クラスには、特定のタイプのデフォルト値を含む列挙型エントリである DatabaseID があります。ほとんどの場合、この DatabaseId のデフォルト値は、Holdingclass の Generic-Class と同じタイプです。

このようなもの：

public enum DatabaseId{
   A_LONG_VALUE(0L),
   A_BOOLEAN_VALUE(false),
   A_INTERVAL("");
}

Interval は、 Comparable インターフェースを実装するカスタム クラスです。しかし、インターバルも上記のルールの例外です。「intervals」の Generic-Holding Class には「Interval.class」のジェネリック タイプ T がありますが、Interval を永続化する DatabaseValue はそれを文字列として格納します。like[5,250]または[5]離散値の場合。

ここで、ある時点で、データベース ID の背後に格納されている値を「ロード」し、それをホールディングのクラスの CurrentValue 属性に挿入する必要があります。

これは階層全体の「非常に」親クラスで行われるため、これも一般的な方法で行われます。

    public void initialize() {
    Database db = Database.getInstance();
    try {
        this.setValue((T) db.get(this.getDatabaseId()));
    } catch (Exception ex) {
        Log.e(LOG_TAG, "Invalid cast while initializing Value Holder.", ex);
        this.setValue(null);
    }
}

上記の例では、ValueHoler が保持Interval.classされており、DatabaseId id が を配信しているString場合、これは明らかに失敗します。この時点で、例外が発生することが予想されます。これは、 Object(db.get delivery Object) をジェネリック型T(Intervalこの例にある) にキャストしようとしているからです。ただし、データベース エントリの実際の型は ですString。

ただし: 例外はスローされないため、setValue(null)実行されることはありません。ValueHolder.getCurrentvalue()アプリケーションは、が呼び出され、期待される同じタイプの別の値と比較するために使用されるポイントまで続きcurrentValueます。(Intervalこの場合)

this.getCurrentValue().containsInterval(this.getRequiredValue());

この時点で、JAVA は「CurrentValue」が文字列であることを認識し、例外をスローします。

java.lang.ClassCastException: java.lang.String は my.namespace.Interval にキャストできません

実際の問題

このシナリオは、次のように簡単に理解できます。

public void initialize() {
    Database db = Database.getInstance();
    try {
        if (this.getValueHolderValueType() == Interval.class){
          //Interval has an constructor for the string represenatation of it                
          this.setValue((T) new Interval((String)db.get(getDatabaseId())); 
        }else{
            this.setValue((T) db.get(this.getDatabaseId()));
        }
    } catch (Exception ex) {
        Log.e(LOG_TAG, "Invalid cast while initializing valueHolder",ex);
        this.setValue(null);
    }
}

しかし、別の開発者がやって来て Comparable を拡張する新しい Class を実装した場合、彼は「手動の」キャストに気付かず、同じ問題に遭遇する可能性があります。

これを書いているときに、次のように開発者に通知する解決策を見つけました。

-非ジェネリックキャストを最初に行う

- 両方のタイプを比較

-汎用キャストが不可能な場合に通知を発生させます。

 if (this.getValueHolderValueType() == Interval.class){
            this.setValue((T) new Interval((String)db.get(getDatabaseId())));
        }else if (this.getValueHolderValueType() != this.getDatabaseId().getType()){
            Log.e(LOG_TAG, "Invalid cast while initializing ValueHolder. DatabaseId delivers '"+this.getDatabaseId().getType()+"', but valueHolder expects '"+this.getValueHolderValueType()+"'. Maybe you are missing a non-generic-cast right here?");    
        }else{
            this.setValue((T) db.get(this.getDatabaseId()));
        }

したがって、答えられていない唯一の質問は次のとおりです。

T がタイプで、DatabaseID の値がタイプで、 db.getが?を返す場合、例外が発生しthis.setValue((T) db.get(this.getDatabaseId())); ないのはなぜですか?IntervalStringObject

キャストを行っているときではなく、「間違ったクラス」にアクセスしようとしたときに例外がスローされるのはなぜですか?

4

2 に答える 2

4

Javaジェネリックはコンパイラレベルで実装されているため、オブジェクトを単純に渡すコードではなく、オブジェクトを使用Objectするコードにすべてを(または他のバインドに)変換し、適切なキャストを挿入します。

編集:問題に対処するには、ホルダーに格納できる正しい型を返す抽象メソッドを保持し、それをデータベースから返される値の型と比較することをお勧めします(==比較または他の種類のいずれか) 。 、比較が失敗した場合は例外をスローします（または何か他のことを行います）。

編集：例えば：

Object candidate = db.get(this.getDatabaseId());
boolean canBeStored = this.getStorageType().isInstance(candidate);
...

編集：すでに同様のロジックが整っているようです。OO をより機能させるために、次のことをお勧めします。

Object original = db.get(this.getDatabaseId());
T modified = this.adaptObjectToContainer(original);
...

protected T adaptObjectToContainer(Object original) {
    if(this.getValueHolderValueType().isInstance(original)) {
        return (T)original;
    }
    throw new ClassCastException(...);
}

// subclasses may do relevant conversions, throw an exception, etc
protected Interval adaptObjectToContainer(Object original) {
    ...
    if(original instanceof String) {
        return new Interval(original);
    }
    ...
}

理想的には、コードを最小限に抑えるために、変換を担当するストレージ タイプごとにユーティリティ クラスが存在する可能性があります。

于 2013-02-20T23:26:20.443 に答える
1

前の回答で指摘されたように、 setValue(Object) はタイプ erasureにより成功します。

アプリケーションにとって意味がある場合は、期待されるクラスへの参照が保持されている次のアプローチを試すことができます。

class Foo<T> {

 private final Class<T> klass;
 private T value;

 Foo(Class<T> klass) {
  this.klass=klass;
 }

 void setValue(Object obj) {
   value=klass.cast(obj); 
 }
}

それで

Foo<Integer> foo = new Foo<Integer>(Integer.class);
foo.setValue("bar");//throws exception

Class.cast(Object)を参照

于 2013-02-20T23:30:07.403 に答える