1. C++ / Говнокод #56

    +246

    1. 1
    2. 2
    3. 3
    4. 4
    if (result == true)
  return true;
else
 return result;

    этпипец... из разряда индокода, только в реале

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

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

    • ava guest 08.12.2008 17:38 # +16.4

      Sap:
      Похоже, сначала возвращалось не булевское значение, а како-0то код ошибки и было что-то вроде:
      if (result == true)
  return -1;
else
 return result;

      А потом по ходу отладки решили возвращать булевское - поставили заплатку, да так она и осталась. В больший проектах и не такое встречется.
      Ответить

      • ava guest 16.12.2011 01:20 # 0

        неа, надо так:
        if (result == true)
        return true;
        else
        {
        if (result == false) return false;
        else
        {
        return (!true) && (!false); // ну надо же хоть что-то вернуть....
        }
        }
        Ответить

    • ava guest 10.02.2009 20:18 # −7

      показать все, что скрытоили автор кода подразумевал оператор === вместо ==, тогда вполне вменяемый смысл приобретается :)
      Ответить

    • ava guest 04.03.2009 21:23 # +4

      tigerox:
      да с === код не скомпилируется-смысл понятен=)
      Ответить

    • ava guest 10.04.2009 22:57 # +1

      Звездочёт:
      Ну да! Конечно! Автор же не знает значение result! Вот и надо узнать! А то он ещё возвратит result непонятно какой...)
      Ответить

    • ava guest 25.06.2009 18:26 # +2

      Andrei:
      мг...похожее обычно получается после нескольких мелких изменений в коде в разное время. А потом как-нибудь смотришь на весь фрагмент в целом и понимаешь что данунах.
      Ответить

    • ava guest 06.08.2009 19:24 # −4

      AntON-LINE:
      я иногда так куски html комментирую
      ..
      if(0){
      ?>
      <table><tr..<?=$var?>
      строк 30
      </table>
      <?
      }
      главное по view source всего этого не будет ))

      так что в процессе многоразового исправления и не такое "накапливается"
      Ответить

      • ava gorsash 02.02.2012 10:41 # 0

        а нефиг на сыром пхп писать.
        smarty - наше всё
        Ответить

    • ava guest 11.08.2009 10:14 # 0

      Индокод реален, %username%
      Ответить

    • ava guest 11.08.2009 18:08 # +1

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

    • ava guest 12.08.2009 13:01 # 0

      Если для result переопределён опреатор ==, то смысл есть. Хотя, конечно:
      return result == true;
      Или ещё православней использовать преобразование типа
      Ответить

    • ava guest 29.09.2009 17:59 # +2

      Довольно часто так делают что в вставить берйкпоинт перед возвратом из функции
      Ответить

      • ava guest 28.12.2009 13:16 # +2

        Поддерживаю, иногда возникают задачи, когда BP нужно поставить по определенному условию. Еще ладно, когда это условие выполняется чаще, чем не выполняется... Но если только вероятность - 1%, то приходится прибегать к таким конструкциям... Возможно, человек просто забыл убрать :-)
        Ответить

        • ava guest 15.11.2011 00:53 # 0

          что то забыл, а Condition в BP не вариант использовать?
          Ответить

    • ava guest 12.05.2011 01:37 # 0

      Если result == 1, тогде его результат вернет true ( typeof true == boolean ), а если просто заменить на return result, то получим 1 ( typeof true == number )

      Так-что все кто сразу заржали, что автор полный мудак ВОЗМОЖНО тоже мудаки
      Ответить

    • ava Scipetr 06.07.2011 18:59 # 0

      В некоторых языках это может быть нормальным кодом. В if (result == true) result не обязательно булев тип, а только приводится к нему. То есть если result равен true, то возвращаем true, иначе возвращаем result.

      А вообще надо в таких случаях писать так:
      return not result or true
      Ответить

    • ava Dg_INC 24.04.2012 02:23 # 0

      Например в павне будет работать такая штука.
      new variable = 0;
      if(varible)
      {
      return varible = true;
      }
      else if(!varible)
      {
      return varible = false;
      }
      Ответить

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