1. C++ / Говнокод #11608

    +25

    1. 1
    2. 2
    3. 3
    4. 4
    uint32 originalDamage = damage;

    if (absorb && originalDamage > damage)
        *absorb += (originalDamage - damage);

    Вытащил из 1 эмулятора игры World of WarCraft

    Запостил: Heisenberg, 19 Августа 2012

    Комментарии (14) RSS

    • ava absolut 20.08.2012 09:53 # +3

      что было во 2ой строке?
      Ответить

    • ava Heisenberg 20.08.2012 23:05 # −1

      ничего, пустая строка, просто так сместилось
      Ответить

    • ava guest 25.08.2012 01:36 # −1

      Если вкопаться в TrinityCore по глубже, и не такой говнокод увидишь.
      Ответить

    • ava TarasB 27.08.2012 11:29 # 0

      где говно?
      Ответить

      • ava absolut 29.08.2012 10:31 # 0

        то есть присваивание с последующим сравнением тебя не смущают?
        Ответить

    • ava sidsukana 17.10.2012 06:27 # +1

      А ничего что вся функция выглядит вот так?
      void Unit::DealDamageMods(Unit *pVictim, uint32 &damage, uint32* absorb)
{
    if (!pVictim->isAlive() || pVictim->IsTaxiFlying() || (pVictim->GetTypeId() == TYPEID_UNIT && ((Creature*)pVictim)->IsInEvadeMode()))
    {
        if (absorb)
            *absorb += damage;
        damage = 0;
        return;
    }

    uint32 originalDamage = damage;

    //Script Event damage Deal
    if (GetTypeId() == TYPEID_UNIT && ((Creature *)this)->AI())
        ((Creature *)this)->AI()->DamageDeal(pVictim, damage);
    //Script Event damage taken
    if (pVictim->GetTypeId() == TYPEID_UNIT && ((Creature *)pVictim)->AI())
        ((Creature *)pVictim)->AI()->DamageTaken(this, damage);

    if (absorb && originalDamage > damage)
        *absorb += (originalDamage - damage);
}


      И переменные проверяются после модифицирования, так как они ссылочные. Автор не позорься.
      Ответить

    • ava Heisenberg 17.10.2012 07:40 # 0

      Не надо меня учить) Не знаю где вы взяли этот код, но в мастер ветке тринити (я брал его оттуда, увидел в коммите) такого и близко нету. Вот что там
      https://github.com/TrinityCore/TrinityCore/blob/4.3.4/src/server/game/Entities/Unit/Unit.cpp#L543
      вот после удаления

      а вот этот коммит https://github.com/TrinityCore/TrinityCore/commit/8eba4aebfb56bf86a74fc9841af75359b9cf2346
      Ответить

    • ava sidsukana 17.10.2012 09:23 # 0

      Тогда извиняюсь. Я код взял из мангоса и других его форков, там код выше присутствует вместе с вызовами виртуалок. Видимо тринити когда-то убрали скриптовые ивенты отсюда и забыли про мертвый код.
      Ответить

    Добавить комментарий