1. PHP / Говнокод #20334

    +10

    1. 1
    if ($invitationType == '1' || '2') {}

    Indian validation. WHYYYYY?

    Запостил: madfishGovnokod, 06 Июля 2016

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

    • ava Dummy00001 06.07.2016 19:23 # +7

      идея специально для пыха: новый оператор `|||`.

      логично еще предположить оператор `&&&` но я еще не придумал что он будет делать.
      Ответить

      • ava roman-kashitsyn 06.07.2016 19:26 # +4 

        (&&&) :: a b c -> a b c' -> a b (c, c')
        Fanout: send the input to both argument arrows and combine their output.

        The default definition may be overridden with a more efficient version if desired.

        — http://tinyurl.com/control-arrow-ands
        Ответить

      • ava Soul_re@ver 06.07.2016 19:27 # +5

        if ($invitationType != '1' && '2') {}

        На поверхности же лежит.
        Ответить

    • ava inkanus-gray 06.07.2016 21:32 # +3

      Идея специально для крестов: перегрузить оператор ==, чтобы он возвращал значение другого типа, для которого перегружен оператор ||.
      Ответить

      • ava gost 07.07.2016 12:10 # +2

        Набросал: https://ideone.com/xwMvx7 . Добавить поддержку &&, != и посыпать шаблончиками - будет ваще красота.
        Ответить

        • ava Soul_re@ver 07.07.2016 12:24 # +3

          Посыпал шаблончиками: http://coliru.stacked-crooked.com/a/e5ec0a5f334b22e6

          Делать как положено было лень, поэтому тот ещё говнокод. Из ценного — гетерогенное сравнение.
          Ответить

          • ava inkanus-gray 07.07.2016 13:10 # +1

            Это то, что надо! Я джва года ждал такой синтаксис!
            Ответить

            • ava Soul_re@ver 07.07.2016 13:39 # +3

              Только сегодня, поддержка != и && в подарок: http://coliru.stacked-crooked.com/a/5ab4ab755cef7ff4

              На самом деле куча проблем: Если случайно написать не тот оператор, ни одного варнинга не будет. Придётся принудительно кастовать в бул, если хочется смешать сравнение с набором и ещё одну часть логического условия.

              Более безопасным будет синтаксис вроде

              if( x == (set || 10 || 11 || 12))
              Ответить

              • ava Dummy00001 07.07.2016 14:30 # +2

                вместо "set", "any" для "||" и "every" для "&&"
                Ответить

              • ava roman-kashitsyn 07.07.2016 14:46 # +2

                > Более безопасным будет синтаксис вроде
                Как всё-таки люди любят усложнять себе жизнь...
                #include <iostream>

template <class ValueType>
constexpr bool anyOf(ValueType &&) { return false; }

template <class ValueType, class FirstType, class... Rest>
constexpr bool anyOf(ValueType && x, FirstType && value, Rest && ... rest) {
    return x == value || anyOf(std::forward<ValueType>(x), std::forward<Rest>(rest)...);
}

int main() {
    std::cout << std::boolalpha;
    int a = 5;
    std::cout << anyOf(a, 7, 19l, 5, 21u) << "\n"
              << anyOf(a, 1, 2, 3, 4, 7) << "\n";
    return 0;
}

                Ещё пара обобщений и на утро у нас выйдет моноид.
                Ответить

                • ava Soul_re@ver 07.07.2016 14:50 # +1

                  Так же не интересно. Задача была заставить работать код подобный коду из топика.
                  Ответить

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