将Object转换为泛型类型时未引发无效的ClassCast异常

时间:2013-02-20 23:09:20

标签: java android generics casting

我有一个项目,我正在使用大量的仿制药,但也需要类型安全。 由于堆栈跟踪大约有200个调用,当出现当前错误时,我担心我无法提供所涉及的完整源代码。我会试着把它解决到目前为止我发现的问题并提供一个简短的例子。

首先,有一些类,持有通用值。每个通用实现Comparable。这可能是一个Long,一个布尔 - 甚至是实现这个接口的自定义类。

每个类都有一个DatabaseID,它是一个包含特定类型默认值的枚举条目。 大部分时间此DatabaseId的默认值与Holdingclass的Generic-Class属于同一类型。

这样的事情:

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

Interval是一个实现Comparable接口的自定义类。但Interval也是上述规则的一个例外。 “interval”的Generic-Holding Class具有通用类型T“Interval.class”,而持久化Interval的DatabaseValue将其存储为字符串。对于离散值,请使用[5,250][5]

现在,在某些时候,我需要“加载”存储在Database id后面的值,并将其注入到holding类的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传递Object)转换为泛型类型T({{1}在这个例子中)。但是,数据库条目的实际类型是Interval

但是:没有抛出异常,因此永远不会执行String。应用程序一致,直到一个点,其中setValue(null)被调用并用于将它与我期望ValueHolder.getCurrentvalue()的相同类型的另一个值进行比较。 (在这种情况下为currentValue

Interval

此时,JAVA注意到“CurrentValue”是一个String,并抛出异常:

  

java.lang.ClassCastException:java.lang.String无法强制转换为   my.namespace.Interval

实际问题

我可以很容易地抓住这个szenario,就像这样:

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

但是如果另一个开发人员出现并实现了一个新的类扩展Comparable,他可能不会意识到“手动”转换,并遇到了同样的麻烦。

在写这篇文章时,我找到了一个通知开发人员的解决方案,如下所示:

- 首先进行非泛型演员

- 比较两种类型

- 如果无法进行通用演员,则会收到通知。

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);
    }
}

所以唯一仍然没有答案的问题是:

为什么 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为this.setValue((T) db.get(this.getDatabaseId()));类型时,DatabaseID的值为Interval类型并且db.get返回String

为什么是当前抛出的异常,当我尝试访问“错误的类”而不是当我正在进行演员表时?

2 个答案:

答案 0 :(得分:4)

因为Java泛型是在编译器级别实现的,它将所有内容都转换为Object(或其他绑定),并在使用对象的代码中插入适当的强制转换,而不是简单传递它的代码。

编辑:为了解决您的问题,您可能希望在持有者中有一个抽象方法,它返回它可以存储的正确类型,并将其与从数据库返回的值的类型进行比较(==比较或其他类型的比较),如果比较失败,则抛出异常(或做其他事情)。

修改:例如:

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

编辑:我发现您已经有类似的逻辑了。为了让它更有效,我建议:

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);
    }
    ...
}

理想情况下,为了最大限度地减少代码,每个存储类型都可能有一个实用程序类,它将负责转换。

答案 1 :(得分:1)

正如前面的回答所指出的那样,setValue(Object)因type 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)