Question

我在解决问题时遇到了一些困难。对不起，如果他们的代码太多了。我尝试改进上面代码的架构，因为我讨厌警告和演员，我觉得有些不对劲。

现在，代码。我有一个带有这两个参数化方法的util类（与OpenJPA和CriteriaBuilder相同的签名......）

public class MyUtil {
    public void equal(List<?> l, Object value) {
        // do something (see CriteriaBuilder.equal method)
    }

    public <Y extends Comparable<? super Y>> void greaterThan(List<? extends Y> l, Y value) {
        // do something (see CriteriaBuilder.greaterThan method)
    }
}

然后，我希望能够抽象它们通过接口调用它。

public interface IOperation<T> {
    // maybe make this method generic ? but how ?
    public abstract void doOp(List<T> l, T value); 
}

public abstract class AbstractOperation<T> implements IOperation<T> {
    protected MyUtil myUtil;
}

public class EqualOp extends AbstractOperation<Object> {
    @Override
    public void doOp(List<Object> path, Object value) {
        myUtil.equal(path, value);
    }
}

public class GreaterThanOp<T extends Comparable<? super T>> extends AbstractOperation<T> {
    @Override
    public void doOp(List<T> path, T value) {
        myUtil.greaterThan(path, value);
    }
}

我创建了一个工厂

public class OperationFactory {
    private static OperationFactory instance;
    public static OperationFactory getInstance() {...}

    public IOperation<?> get(String op) {
        if ("=".equals(op)) {
            return new EqualOp();
        } else if (">".equals(op)) {
            return new GreaterThanOp<Comparable<? super Object>>();
        }
        throw new InvalidParameterException();
    }
}

然后我用它：

public class Client {
    public void needOp(String op) {
        IOperation<String> operation = (IOperation<String>) OperationFactory.getInstance().get(op); // How to avoid this cast ?
        List<String> l = null;
        operation.doOp(l, "a string");
    }
}

我的问题是：是否可以在Client类中避免这种强制转换？怎么样？有没有办法建立更好的架构？

感谢您阅读

Answer 1

我假设您可以要求您的类型为Comparable。

参数化EqualOp，如GreaterThanOp：

public class EqualOp<T extends Comparable<T>> extends AbstractOperation<T> {
    @Override public void doOp(List<T> path, T value) ...

并像这样定义get()：

public <T extends Comparable<T>> IOperation<T> get(String op) {
    if ("=".equals(op)) {
        return new EqualOp<T>();
    } else if (">".equals(op)) {
        return new GreaterThanOp<T>();
    }
    ...

避免在泛型层次结构中强制转换

1 个答案: