demeterとplaneshiftの法律

Question

このゲームのソースコードはオープンソースなので、私はそれを調べることにしました。その中で、私のようなものを見つけました：

// This ActionManager is basically a controller like in the MVC pattern. 
void ActionManager::HandleQueryMessage(csString xml, Client* client) 
{ 
    //check the two hands as a start. 
    psItem* item = client->GetCharacterData()->Inventory().GetInventoryItem(PSCHARACTER_SLOT_RIGHTHAND); 
    if(!item || !item->GetItemCommand().Length()) 
     item = client->GetCharacterData()->Inventory().GetInventoryItem(PSCHARACTER_SLOT_LEFTHAND); 
} 

アイテムを取得するための最初のラインがはっきりとデメテルの法則に違反します。しかし、たとえそれがclient->GetCharacterData()->GetInventoryItem(PSCHARACTER_SLOT_RIGHTHAND);に変更されたとしても、それは（私が知る限り）デメテルの法則に違反することになります。

これについて何ができるのですか？それともLODが適用されないこの場所の1つですか？

クライアントはcharacterとは関係がないため、GetInventoryItemをclientクラスに移動することは私の視点では意味がありません。

clientクラスのラッパーをすべてのxxメソッドに対して作成すると、characterクラスは過剰なようです。

どのような考えですか？この追加の間接はそれだけの価値

Item* Client::GetCharacterInventoryItem(int itemID) 
{ 
    return characterData->getInventoryItem(itemId); 
} 
/* ... */ 
Item* CharacterData::getInventoryItem(int itemID) 
{ 
    return inventory->getItem(itemId) 
} 
/* ... */ 
Item* Inventory::getItem(int itemID) 
{ 
    assert_valid_itemID(itemID); 
    return inventory_table[itemId]; 
} 

Answer 1

あなた自身を示唆しているように、あなたは完全にLODをフォローしたい場合は、あなたのような機能を必要とするだろう...か？私は、その事件、あなたの個人的な好みなどに依存しています。コメントに示されているように、LODは法律ではなくガイドラインとみなすべきです。一方、私の個人的な経験では、頻繁にそれを壊して、あなたはトラブルになるでしょう:) :)