有没有更好的可读方式来编写这个 if 语句链？

Question

我有以下代码：

Creature::cancelWalk()
{
    Player* player = getPlayer();

    if (!player) {
        if (getMonster() && getMonster()->getMaster() && getMonster()->getMaster()->getPlayer()) {
            player = getMonster()->getMaster()->getPlayer();
        }
    }

    if (player) {
        player->sendCancelMessage(ret);
        player->sendCancelWalk();
    }
}

经过简单分析，很容易理解我想要实现一些简单的东西：

如果creature 是player 本身，那么sendCancelMessage 和sendCancelWalk。否则，如果 creature 是一个 monster 并且还有一个主控是 player，则将相同的内容发送给客户端。

有没有更好的方法来编写此代码不在Monster、Creature 和Player 类上添加其他方法？

Monster 和 Player 都是源自 Creature 的“兄弟姐妹”。

Answer 1

假设各种函数对多次调用没有不同的影响，尝试引入一些临时函数。

Player* player = getPlayer();

if (!player)
{
    Monster monster = getMonster();
    if (monster)
    {
        Master *master = monster->getMaster();
        if (master) player = master->getPlayer()) {
    }
}

if (player) {
    player->sendCancelMessage(ret);
    player->sendCancelWalk();
}

除此之外，您可能需要更仔细地查看您的设计。如果您有很多嵌套指针需要在取消引用之前顺序检查 NULL，则可能值得指定和强制执行指针不为 NULL 的不变量（这意味着只有在可以创建所有组件部分时才构造父对象，并且永远不会如果对象只能部分构造，则构造对象）。

例如，如果我们假设 getMonster() 返回非 NULL，则保证 getMaster() 和 getPlayer() 也不会返回 NULL ....

Player* player = getPlayer();

if (!player)
{
    player = getMonster()->getMaster()->getPlayer());
}

if (player)
{
    player->sendCancelMessage(ret);
    player->sendCancelWalk();
}