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

    +83

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    private static boolean getIsDeclinedByRules(boolean oldIsSatisfy, boolean oldIsDeclined, boolean newIsSatisfy) {
    if (!oldIsSatisfy &&  oldIsDeclined && !newIsSatisfy) return true;
    if (!oldIsSatisfy &&  oldIsDeclined &&  newIsSatisfy) return false;
    if (!oldIsSatisfy && !oldIsDeclined && !newIsSatisfy) return true;
    if (!oldIsSatisfy && !oldIsDeclined &&  newIsSatisfy) return false;
    if ( oldIsSatisfy &&  oldIsDeclined && !newIsSatisfy) return true;
    if ( oldIsSatisfy &&  oldIsDeclined &&  newIsSatisfy) return true;
    if ( oldIsSatisfy && !oldIsDeclined && !newIsSatisfy) return true;
    if ( oldIsSatisfy && !oldIsDeclined &&  newIsSatisfy) return false;
    return true;
  }

    Запостил: glprizes, 25 Марта 2013

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

    • ava SSSandman 25.03.2013 13:03 # +6

      Вот если бы входных булеанов было штук 6, автор этого кода одумался бы, набирая x-десятую строку такого говнеца
      Ответить

      • ava krypt 25.03.2013 13:50 # +1

        Вы просто не общались с чем-нибудь, типа моего бывшего отдела R&D - вот там - настоящий взрыв мозга, и только такие простыни и спасают.
        Ответить

        • ava QuickNick 16.04.2013 11:24 # 0

          Если перед этим не провести упрощение блок-схемы - то да.
          Ответить

          • ava krypt 16.04.2013 12:32 # 0

            Ага. Только смотришь на ту схему, и получается именно такой костыль ;)
            А пользователям со всем этим ещё как-то жить надо...
            Ответить

      • ava 3.14159265 25.03.2013 16:13 # +3

        Не одумался. Инфа 90%.
        Существуют совсем неизлечимые поциенты. Только лоботомия, только хардкор.
        Ответить

      • ava QuickNick 16.04.2013 11:29 # 0

        http://govnokod.ru/11936

        Специально обученный программист на 6 похожих кнопок навесил 6 отдельных функций.
        Ответить

    • ava wvxvw 25.03.2013 14:27 # +5 

      return !newIsSatisfy || oldIsSatisfy && oldIsDeclined;

      Вроде как, не?
      Ответить

      • ava VseGovnoOdinYaKrut 25.03.2013 15:41 # +1

        Да! http://ideone.com/9o85yR
        Ответить

      • ava myaut 25.03.2013 15:41 # −3

        Случай, когда ни один из if'ов не отработал не учтен
        Ответить

        • ava wvxvw 25.03.2013 16:10 # +3

          Ну это уже к Хейзенбергу тогда... мне кажется, что Гослинг в таком случае окажется не у дел.
          Ответить

    • ava bormand 25.03.2013 16:08 # +6

      Таблица истинности прям.
      Ответить

    • ava TarasB 16.04.2013 11:45 # 0

      У меня есть такое но оформлено чуть иначе, поэтму логарифм вместо эн!
      function GetTrumDir: TDir;
begin
  with TrumAttr do begin
    if ModelReversed then begin
      if Head then begin
        if Tail then Result := dHTR
                else Result := dHR;
      end else begin
        if Tail then Result := dTR
                else Result := dMR;
      end;
    end else begin         
      if Head then begin
        if Tail then Result := dHTD
                else Result := dHD;
      end else begin
        if Tail then Result := dTD
                else Result := dMD;
      end;
    end;
  end;
end;
      Ответить

      • ava defecate-plusplus 16.04.2013 12:15 # 0

        у тебя 8 разных результатов - совсем другое
        собсно, можно за O(1)
        Ответить

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