在 AsyncTask 中将对象作为参数传递

Question

我想使用 AsyncTask 查询数据库。现在我可以选择为每种类型的查询创建不同的 AsyncTasks。我倾向于的另一种选择是为每种类型的查询制作单个 AsyncTask。

如果我将参数作为对象传递然后将它们强制转换回去，这是一种不好的做法吗？我可以运行惹上麻烦吗？我应该在构造函数中传递所有内容吗？

public void insert (Item item){
    new queryAsyncTask(mItemDao).execute(INSERT_QUERY, item);
}

public void delete (int uid){
    new queryAsyncTask(mItemDao).execute(DELETE_QUERY, uid);
}

private static class queryAsyncTask extends AsyncTask<Object, Void, Void>{

    private ItemDao mAsyncDao;

    queryAsyncTask(ItemDao dao){
        mAsyncDao = dao;
    }

    @Override
    protected Void doInBackground(Object... objects) {
        switch ((int) objects[0]){
            case DELETE_QUERY:
                mAsyncDao.deleteItem((int)objects[1]);
                break;
            case INSERT_QUERY:
                mAsyncDao.insert((Item)objects[1]);
                break;
        }
        return null;
    }
}

Answer 1

If I am passing params as objects and then casting them back, is this a bad practice? Can I run into some trouble? Should I be passing everything in constructor?

是的，这是一种非常糟糕的做法。考虑您的以下代码：

private static class queryAsyncTask extends AsyncTask<Object, Void, Void>{

    private ItemDao mAsyncDao;

    queryAsyncTask(ItemDao dao){
        mAsyncDao = dao;
    }

    @Override
    protected Void doInBackground(Object... objects) {
        ...
    }
}

然后您可以调用它：

new queryAsyncTask(mItemDao).execute(DELETE_QUERY, uid);

或

new queryAsyncTask(mItemDao).execute(INSERT_QUERY, item);

但是，您也可以使用以下两种方式调用它：

new queryAsyncTask(mItemDao).execute(new Object(), item);
new queryAsyncTask(mItemDao).execute(new ArrayList<String>(), item);

没有给出任何错误。这是因为你的代码没有严格的限制，没有对它的作用给出足够的解释。

您最好为 CRUD 制作每个单独的任务，并通过构造函数传递值（引用）。例如，您可以为 INSERT 创建类似这样的内容：

private static class InsertQueryTask extends AsyncTask<Void, Void, Void> {

    private ItemDao mItemDao;
    private Item mItem;

    InsertQueryTask(ItemDao dao, Item item) {
        mItemDao = dao;
        mItem = item;
    }

    @Override
    protected Void doInBackground(Void... voids) {
       mItemDao.insert(mItem);
    }
}

那么你可以调用它：

new InsertQueryTask(mItemDao, item).execute();

上面的代码行更具可读性和可维护性，因为您仅通过阅读代码的名称就可以知道代码在做什么。

您可以进一步修改您的代码，使其成为 Fluent Interface。像这样：

private static class InsertQueryTask extends AsyncTask<Void, Void, Void> {

    private ItemDao mItemDao;
    private Item mItem;

    InsertQueryTask(ItemDao dao) {
        mItemDao = dao;
    }

    InsertQueryTask with(Item item) {
      mItem = item;
      return this;
    }

    @Override
    protected Void doInBackground(Void... voids) {
       mItemDao.insert(mItem);
    }
}

现在，您可以调用它：

new InsertQueryTask(mItemDao).with(item).execute();

比之前的代码更具可读性。

注意：所有代码尚未测试。

在 AsyncTask 中将对象作为参数传递

Passing Object as Params in AsyncTask

android

object

android-asynctask

android-room