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

    • ava guest 22.10.2009 05:52 # 0

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

      • ava guest 22.10.2009 05:54 # 0

        да, забыл, внутренние 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;
        }
        Ответить

        • ava guest 22.10.2009 05:55 # 0

          при этом еще жизненно необходим коммент к строке init = ..., т.к. очень трудноосиливаемое условие.
          Ответить

          • ava guest 25.03.2010 17:55 # 0

            пистец, эта похоже гавнорефакторинг Ы :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;
            }
            Ответить

            • ava guest 25.03.2010 17:59 # 0

              Ой, можно даже проще Ы

              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;
              }
              Ответить

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