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

    +72

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (doSomething() == true) {
    if (doSomething2()) {
        //...
    }
}

    Запостил: lotik, 10 Июня 2010

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

    • ava absolut 10.06.2010 12:24 # +1

      Если убрать ==true, то имеет место быть.
      Ответить

      • ava lotik 10.06.2010 13:47 # 0

        Этот код продолжение http://govnokod.ru/3439

        Я бы не только == true убрал...
        Ответить

    • ava Lure Of Chaos 10.06.2010 13:12 # 0

      if (doSomething2() == true) тоже. и так 100 ифов
      Ответить

    • ava This is obvious 10.06.2010 16:48 # +1

      if (doSomething() && doSomething2()) {} // K.O
      Ответить

      • ava Говногость 10.06.2010 17:02 # 0 

        if (doSomething() == true) {
    if (doSomething2()) {
        //...
    }
}

        !=
        if (doSomething() && doSomething2()) {}
        doSomething2 - не всегда можно исполнять, если не исполнился doSomething. Может здесь как раз такой случай.
        //К.О.
        Ответить

        • ava This is obvious 10.06.2010 17:05 # 0

          а может и нет. Капитан здесь бессилен.
          Ответить

          • ava Soul_re@ver 10.06.2010 18:02 # 0

            Капитан подсказывает, что выполнится ли doSomething2() зависит от компилятора
            Ответить

            • ava Говногость 10.06.2010 18:23 # +2 

              if (doSomething())
    if (doSomething2())
    {
        //...
    }

              Как бы думать на эту тему не нужно. Достаточно просто написать правильно, если есть возможная зависимость doSomething2 от doSomething.
              Ответить

            • ava absolut 10.06.2010 22:29 # −1

              По-моему вычисление выражения в условном операторе по короткой схеме прописано в стандарте.
              Ответить

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