1. Python / Говнокод #19632

    −43

    1. 1
    2. 2
    damage, self.magic_shield_hp = sorted([0, magic_shield_hp - damage])
damage = -damage

    Интуитивно понятный способ реализовать урон, пробивающий защиту.

    Запостил: Soul_re@ver, 15 Марта 2016

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

    • ava Soul_re@ver 16.03.2016 13:07 # 0

      Сегодня на ревью прислали исправленную версию:
      damage, self.magic_shield_hp = map(abs, sorted([0, self.magic_shield_hp - damage]))

      Пойду, пообщаюсь с человеком лично.
      Ответить

      • ava kegdan 16.03.2016 14:02 # 0

        Передай от меня ногой в пах
        Ответить

      • ava 3_14dar 16.03.2016 19:32 # 0

        А чо не так? Ну можно переписать в лестницу из ифов.
        Ответить

        • ava bormand 16.03.2016 19:42 # 0

          Лестницу из одного ифа.
          Ответить

        • ava 1024-- 16.03.2016 19:47 # 0

          Если я не ошибаюсь, это сводится к
          x = min(self.magic_shield_hp, damage)
damage -= x
self.magic_shield_hp -= x

          что явно проще и легче читается.
          Ответить

          • ava bormand 16.03.2016 19:47 # 0

            Ответить

          • ava bormand 16.03.2016 19:48 # +2

            Дубово, зато логика сразу понятна:
            if damage > magic_shield_hp:
    # пробили щит
    damage -= magic_shield_hp
    magic_shield_hp = 0
else:
    # не пробили щит
    magic_shileld_hp -= damage
    damage = 0
            Ответить

            • ava 1024-- 16.03.2016 19:49 # 0

              Кстати, я сначала map/sort именно в этот вариант перевёл, а затем пошёл на min.
              Ответить

            • ava kegdan 16.03.2016 19:55 # 0 

              magic_shileld_hp -= damage
if (magic_shileld_hp < 0) {
hp +=magic_shileld_hp;
magic_shileld_hp = 0;
}

              сделаем вид, что героя бьет не уроном, а щитом
              Ответить

    • ava nihau 16.03.2016 14:53 # 0

      Я ничего не понимаю в ваших питонах.
      Ответить

      • ava 1024-- 16.03.2016 16:14 # 0

        Вот, что-то похожее на ES6ES2015:
        var damage;
[damage, this.magicShieldHp] = [0, magicShieldHp - damage].sort((x,y) => x-y);
damage = -damage;
        var damage;
[damage, this.magicShieldHp] = [0, this.magicShieldHp - damage]
    .sort((x,y) => x-y)
    .map(Math.abs);
        Ответить

      • ava 3_14dar 16.03.2016 21:08 # 0

        пошел напитон!
        Ответить

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