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 将其存储为字符串。喜欢[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);
    }
}

对于上面提到的示例,其中持有一个 ValueHolerInterval.class并且 DatabaseId id 提供一个String,这显然会失败。在这一点上,我期待一个异常,因为我正在尝试将Object(db.get Delivers Object) 转换为通用类型TInterval在本例中)。然而,数据库条目的真实类型是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 的新类,他可能不会意识到“手动”转换,并遇到了同样的麻烦。

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

-首先进行非泛型转换

- 比较两种类型

- 如果无法进行通用转换,则发出通知。

 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 是 type 时,DatabaseID 的 value 是 type并且 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);
...

编辑:我看到你已经有了类似的逻辑。为了使其更加面向对象工作,我建议:

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) 由于类型擦除而成功。

如果它对您的应用程序有意义,您可以尝试以下方法,其中保留对预期类的引用:

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 回答