Jawa / Говнокод #128 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 guest, (Updated )

Комментарии (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 = ..., т.к. очень трудноосиливаемое условие.
        Ответить
        • пистец, эта похоже гавнорефакторинг Ы 😀 жесть Ы
          Код ваще бесмыслено написан и не эквивалентен оригиналу Ы 🙂

          Он эквивалетнен этаму:
          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;
            }
            Ответить
              • Надо так 🙂

                boolean isProcсes = true;
                if (!account_number.equals("") && cycleT != null) {
                if (!account_number.equals(num) || !cycleT.equals(c) || vT != v) {
                doWrite(bufferedOut);
                } else {
                isProcсes = false;
                }
                }
                if (isProcсes) {
                account_number = num;
                cycleT = c;
                vT = v;
                }
                Ответить
                • Купить: закладки, амфетамин, скорость соль кристаллы, кокаин
                  Купить, закладки, мефедрон, MDMA, MDA, MDPV, 3md мука.


                  Моментальный магазин закладок LegalRF.

                  [u]Купить закладки через онлайн магазин или телеграм:
                  Скорость соль кристаллы (a-pvp, a29, MDPV, 3md) , гашиш и шишки (бошки, haze, гарик, гаш), кокаин (VHQ, HQ, MQ, первый, орех), амфетамин (фен, амф, порох, кеды), мефедрон (меф, мяу-мяу), лирика (300), героин и метадон (хмурый, гера, гречка, мёд, мясо), спайс (микс, россыпь, ТВ), реагент (рега, каннабиноид, 1к15, 1к30, 1к45, 1к70, 1к100), лсд (lsd 25, марки, сибирь), грибы (психоделики), экстази (MDMA, ext, круглые, диски, таблы) [u]
                  ________________
                  Ответить

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?


    8