13

这可能已经被问过一百万次了，但是我在用有界类型参数的抽象类上编写复制构造函数时遇到了麻烦。我有一些看起来像这样的代码：

public abstract class Superclass<T> {
    Set<? extends Variable<T>> vars;

    public abstract Superclass<? extends T> copy();

    class Variable<T> {
        T value;
    }
}

class Foo extends Superclass<Integer> {
    public Foo copy() {
        Foo _newFoo = Foo();
        Set<FooVariable> _newVars = new HashSet<FooVariable>();
        _newVars.addAll(this.vars);
        _newFoo.vars = _newVars;
    }

    class FooVariable extends Variable<Integer> { /* ... */ }
}

class Bar extends Superclass<String> {
    public Bar copy() {
        Bar _newBar = Bar();
        Set<BarVariable> _newVars = new HashSet<BarVariable>();
        _newVars.addAll(this.vars);
        _newBar.vars = _newVars;
    }

    class BarVariable extends Variable<String> { /* ... */ }
}

由于除了变量类型之外，两者的方法copy都是相同的，我希望能够将该代码移动到超类中的具体方法中。但是我无法弄清楚（a）如果在 a 上调​​用具体方法如何返回一个实例，如果在 a 上调​​用一个实例，并且（b）根据需要用s 或s 填充集合。FooBarpublic Superclass<? extends T> copyFooFooBarBarvarsFooVariableBarVariable

有人可以帮忙告诉我我错过了什么吗？谢谢。

4

3 回答 3

2

这种Superclass呢？

public abstract class Superclass<T> {

    Set<? extends Variable<T>> vars;

    public Superclass<? extends T> copy() {
        Superclass<T> _newSuperclass = this.getNewInstance();
        Set<Variable<T>> _newVars = new HashSet<Variable<T>>();
        _newVars.addAll(this.vars);
        _newSuperclass.vars = _newVars;
        return _newSuperclass;
    }

    public abstract Superclass<T> getNewInstance();

    class Variable<T> {

        T value;
    }
}

关键是您只需要getNewInstance()在子类而不是构造函数中实现。

所以Foo看起来就像：

class Foo extends Superclass<Integer> {

    @Override
    public Superclass<Integer> getNewInstance() {
        return new Foo();
    }

    class FooVariable extends Variable<Integer> { /* ... */ }
}
于 2013-06-25T18:25:26.093 回答
0

引入第二个泛型类型参数来表示Variable<T>, U。

然后，满足Foo.FooVariable<T>和Bar.BarVariable<T>的边界，U可以从该copy方法返回。

编辑

我已经更改了代码以将实现移动copy到超类中。它依赖于一种newInstance方法（@OndrejBozek 已经介绍过）。

public abstract class Superclass<T, U extends Variable<T>> {
    Set<U> vars;

    class Variable<T> {
        T value;
    }

    public Superclass<T, U> copy() {
        Superclass<T, U> _newSuperclass = newInstance();
        Set<U> _newVars = new HashSet<U>();
        _newVars.addAll(vars);
        _newSuperclass.vars = _newVars;
       return _newSuperclass;
    }

    public abstract Superclass<T, U> newInstance();
}

class Foo extends Superclass<Integer, Foo.FooVariable> {
    public Foo newInstance() { return new Foo(); }

    class FooVariable extends Variable<Integer> { /* ... */ }
}

class Bar extends Superclass<String, Bar.BarVariable> {
    public Bar newInstance() { return new Bar(); }

    class BarVariable extends Variable<String> { /* ... */ }
}
于 2013-06-25T18:19:24.207 回答
0

测试了这段代码，它有几个警告，但它做了你想要的：

public <C extends Superclass<? extends T>> C copy() throws InstantiationException, IllegalAccessException {
    C result= (C) this.getClass().newInstance();
    HashSet newVars= new HashSet();
    newVars.addAll(this.vars);
    result.vars= newVars;
    return result;
}

一句话：这不是复制构造函数。这只是一种复制方法。构造函数没有返回类型，其名称等于类名。

于 2013-06-25T18:36:57.650 回答