有沒有更好的可讀方式來寫這個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()返回非空這就保證了getMaster()和getPlayer()也不會返回NULL ....

Player* player = getPlayer(); 

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

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