Есть ли лучший читаемый способ написать эту цепочку операторов 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(); 
}