I think it should be outsourced a bit.
The clienthandle is already very overloaded
For example:
if(SomeClass::IsFood(Item->m_ItemID))
{
if(m_Player->GetInventory().RemoveItem( Item )
m_Player->Feed(SomeClass::GetSaturation(Item->m_ItemID));
break;
}
This would make the cClientHandle a bit clearer
The clienthandle is already very overloaded
For example:
if(SomeClass::IsFood(Item->m_ItemID))
{
if(m_Player->GetInventory().RemoveItem( Item )
m_Player->Feed(SomeClass::GetSaturation(Item->m_ItemID));
break;
}
This would make the cClientHandle a bit clearer