Nit: What about changing method code to this
```
if (g_Assist.DoesEntityObjectExist(_entityId))
{
    GameEntity* pEntity = pRtsGame->Self()->GetEntity(_entityId);
    assert(pEntity);
    return pEntity->Attr(EOATTR_IsMoving);
}
```

Does this looks cleaner for you?