我有一个项目,我正在使用大量的仿制药,但也需要类型安全。 由于堆栈跟踪大约有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
?
为什么是当前抛出的异常,当我尝试访问“错误的类”而不是当我正在进行演员表时?
答案 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