Си диез / Говнокод #16125 Ссылка на оригинал

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
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
public bool Enabled(string elementId)
        {
            bool flag = false;
            bool flagFalse = false;
            bool flagTrue = false;
            bool mainFlag = false;
            IWebElement input = Driver.FindElement(By.Id(elementId));
            if (input.Enabled == true)
            {
                flagTrue = true;                
            }
            else
                if (input.Enabled == false)
                {
                    flagFalse = false;                    
                }

            if (flagTrue = true)
            {
                mainFlag = flagTrue;
            }

            else if (flagFalse == false)
            {
                mainFlag = flagFalse;
            }
            return mainFlag;
        }

Астрологи объявили неделю флагов. Их количество увеличивается вдвое.

Запостил: alexCoder2007 alexCoder2007, (Updated )

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

    • ava 3.14159265 # Ссылка на оригинал 0 ⇈

      Не думаю. Эта строка говорит в пользу реальности кода:
      > IWebElement input = Driver.FindElement(By.Id(elementId));

      Порог у шарпа такой низкий, что кодить на нем могут даже вот такие экземпляры.
      Ответить

  • ava someone # Ссылка на оригинал 0

    Больше всего доставляет, что flagFalse вообще никогда не может быть true, а просто flag нигде не используется.

    А ещё

    if (flagTrue = true)


    присваивание? Получается, что из-за опечатки в условии вся эта писанина эквивалентна return true (а без неё была бы эквивалентна return Driver.FindElement(By.Id(elementId)).Ena bled).
    Ответить

    • ava TauSigma # Ссылка на оригинал 0 ⇈

      На такой код будет warning, так что если умный проектировщик в автобилдах или в решении поставил флаг "Threat warning as errors", то такой код дальше машины разработчика никуда не уйдёт.
      Ответить

  • ava absolut # Ссылка на оригинал 0

    введение дополнительных уровней косвенности прям ... на каждом из которых бул.
    p.s. нужен еще nullInputFlag.
    Ответить

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

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

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

    ↑
    8
    ↓