java - 使用空检查、类型转换值的更好方法？

Question

我正在编写一个类似于 AQuery 的库但具有用于操作元素的精炼语法。

本质上，AQuery 的作用是安全地访问 View 层次结构，并允许您调用 ImageView 和 TextView 等对象的子类方法。

我使用以下代码编写了使用 View 子类的通用方法:

我的Query对象是用于操作 View 层次结构的基础对象。基本格式如下:

public class Query {
    private View mView;
    // ...
}

接下来是通用接口(interface)。这是 Query 对象的内部接口(interface):

private interface Operation<T extends View> {
    public void execute(T view);
}

接下来是Query中的run方法。这会检查此查询代表的当前节点，如果成功，则调用操作对象上的 execute 方法:

private <T extends View> Query run(Class<T> cls, Operation<T> operation) {
    T t = cls.cast(mView);

    if (t == null) {
        Log.e(TAG, "view is not a " + cls.getSimpleName());
    } else {
        operation.execute(t);
    }

    return this;
}

现在模板代码已经写好了，我使用类似这样的方法来实现功能:

public Query text(final CharSequence str) {
    return run(TextView.class, new Operation<TextView>() {

        @Override
        public void execute(TextView view) {
            view.setText(str);
        }
    });
}

对于修改 View 层次结构的每个方法，我都必须编写这段类似样板的代码。

有什么方法可以重构此代码以使 text 等方法更简单吗？

Answer 1

仅供引用，您在这里并没有真正检查 mView 的类型。如果 mView 不可分配给类型 T，Class.cast 将抛出 ClassCastException，因此那里的日志消息实际上并不代表发生的情况。当且仅当 mView 为 null 时，t == null 才为 true。

如果没有 Query 将做什么的一些 stub ，就很难说出您想要实现的目标。如果您的使用允许查询的参数化，那么您可以使该操作成为该查询的函数。这将为您提供与查询类型匹配的 View 的编译时检查。例如

public interface Query<ViewT extends View> {
    void run(ViewT view);
}

public Query<TextView> text(final CharSequence str) {
    return new Query<TextView>() {
        public void run(TextView view) {
            view.setText(str);
        }
    };
}

如果这是不可能的(即 View 类型在编译时永远未知)，那么您仍然可以参数化它的实现，并且当且仅当参数类型与查询类型匹配时简单地执行操作。例如:

public interface Query {
    void run(View view);
}

private abstract class TypedQuery<ViewT extends View> implements Query {
    private final Class<ViewT> viewType;

    private TypedQuery(Class<ViewT> viewType) {
        this.viewType = viewType;
    }

    public final void run(View view) {
        if (viewType.isInstance(view)) {
            runInternal((ViewT) view);
        } else {
            Log.e(TAG, "view " + view + " is not a " + viewType.getSimpleName());
        }
    }

    protected abstract void runInternal(ViewT view);
}

public Query text(final CharSequence str) {
    return new TypedQuery<TextView>(TextView.class) {
        @Override
        protected void runInternal(TextView view) {
            view.setText(str);
        }
    };
}