活动类上应该有任何逻辑吗？

Question

我最近在阅读有关设计模式的文章，尤其是有关低耦合和委托的文章。

我想知道，Activity 类是否应该有任何逻辑，或者它是否只服务于视图。

例如我有一个名为BattleActivity 的活动，它应该作为两个玩家之间的某种会话。很多Push Notifications 发生在那里，而且这个类也作为Observer 工作，所以那里有很多交流。

现在我正试图弄清楚我可以将什么逻辑移动到一个单独的对象（以及我是否应该），然后只使用该活动。

我的活动方法之一示例：

private void postCastedSpell(final int spellId) {
        Call call = StaticGlobalContainer.api.postSpellToBattle(Integer.parseInt(battleId), Integer.parseInt(MainActivity.CURRENT_USER_ID), spellId, 100);
        call.enqueue(new Callback<User>() {
            @Override
            public void onResponse(Response<User> response, Retrofit retrofit) {
                User user = response.body();
                if (response.code() == 202) {
                    // 200
                    Log.i("Posting spell to battle", "Success");
                    Boolean affectedUserIsOpponent = isUserOpponent(user);
                    if (affectedUserIsOpponent && user.currentHp<1){
                        StaticGlobalContainer.battleOnResult(Constants.WON, getApplicationContext());
                    }else {
                        updateBattleLog(affectedUserIsOpponent, user, spellId);
                    }
                    // TODO: do something here
                } else {
                    // 404 or the response cannot be converted to User.
                    Log.e("Posting spell to battle", "Error:" + response.errorBody());
                }
            }

            @Override
            public void onFailure(Throwable t) {
                Log.i("HttpRequest-Post spell", "Failure");
            }
        });
    }

Answer 1

在活动中放置大量逻辑并不是特别糟糕，但你尝试让它只查看相关的东西是正确的。如果应用程序相对较小，则可能不值得将逻辑移出。还有一些overhead to using abstractions。

如果你的抽象没有提供显着的好处，你应该避免它们

我尝试将任何大数据对象保存在管理器类中，因此根据您的示例，可能值得创建一个 Battle 管理器类来保存其中涉及的所有逻辑，例如 postCastedSpell 函数。这样一来，所有的战斗信息都是自包含的，也可以在其他活动中使用。

请记住，如果您使用数据管理器类并且希望它们提示与 UI 进行某种交互，则必须使用 Callbacks 或 Bus pattern，因为战斗管理器不会可以访问您的 UI。例如，要调用postCastedSpell，调用将如下所示： BattleActivity：

BattleManager bm = BattleManager.getInstance(user1, user2);

onSpellClicked() {
   bm.castSpell(spellId, user1, callback)
}

BasicCallback callback = new BasicCallback() {
    @Override
    onComplete() {
        if (MyInfoFragment.this.isVisible()) {
            [Update UI]
        }
    }
};

注意：当使用像我的示例这样的回调时，当它最终被调用时，活动可能已经不在视野范围内并且已经被垃圾回收了。因此，在回调函数中，您需要先确保它仍然可见，然后再尝试修改可能不再存在的 UI。