1. Java / Говнокод #128

    −60

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    if (!account_number.equals("") && cycleT != null) {
        if (!account_number.equals(num)) {
            doWrite(bufferedOut);
            account_number = num;
            cycleT = c;
            vT = v;
        } else if (!cycleT.equals(c)) {
            doWrite(bufferedOut);
            account_number = num;
            cycleT = c;
            vT = v;
        } else if (vT != v) {
            doWrite(bufferedOut);
            account_number = num;
            cycleT = c;
            vT = v;
        }
    } else {
        account_number = num;
        cycleT = c;
        vT = v;
    }

    Запостил: guest, 12 Декабря 2008

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

    • лучше заменить повторяющийся блок на init = true; и перед проверками поставить boolean init = false; а потом
      if (init)
      {
      account_number = num;
      cycleT = c;
      vT = v;
      }
      Ответить
      • да, забыл, внутренние if-ы тоже в один лучше объединить.
        Вот конечный код:

        boolean init = false;

        if (!account_number.equals("") && cycleT != null)
        {
        init = !account_number.equals(num) || !cycleT.equals(c) || vT != v;

        if (init)
        {
        doWrite(bufferedOut);
        }
        }

        if (init)
        {
        account_number = num;
        cycleT = c;
        vT = v;
        }
        Ответить
        • при этом еще жизненно необходим коммент к строке init = ..., т.к. очень трудноосиливаемое условие.
          Ответить
          • пистец, эта похоже гавнорефакторинг Ы :D жесть Ы
            Код ваще бесмыслено написан и не эквивалентен оригиналу Ы :)

            Он эквивалетнен этаму:
            boolean isProcсes = false;
            if (!account_number.equals("") && cycleT != null) {
            if (!account_number.equals(num) || !cycleT.equals(c) || vT != v) {
            doWrite(bufferedOut);
            isProcсes = true;
            }
            } else {
            isProcсes = true;
            }
            if (isProcсes) {
            account_number = num;
            cycleT = c;
            vT = v;
            }
            Ответить
            • Ой, можно даже проще Ы

              boolean isProcсes = true;
              if (!account_number.equals("") && cycleT != null) {
              if (!account_number.equals(num) || !cycleT.equals(c) || vT != v) {
              doWrite(bufferedOut);
              }
              isProcсes = false;
              }
              if (isProcсes) {
              account_number = num;
              cycleT = c;
              vT = v;
              }
              Ответить

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