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

    +82

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    bool isOperator( char c ) {
	return c == '+'
		|| c == '-'
		|| c == '*'
		|| c == '/'
		|| c == '('
		|| c == ')';
}

    Запостил: Fai, 05 Сентября 2012

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

    • ava Fai 05.09.2012 23:18 # −2

      Авторское форматирование сохранено.
      Ответить

      • ava ErmineMD 06.09.2012 00:42 # +2

        Если табы по 4 пробела, смотрится нормально.
        Ответить

    • ava WGH 06.09.2012 00:54 # +2

      Вот так?
      strchr("+-*/()", c) != NULL

      А вообще - не говнокод это.
      Ответить

    • ava bormand 06.09.2012 05:18 # +1

      Где здесь говно, Fai?
      Совершенно нормальный код. Работает быстрее strchr(). Единственное говно которое я вижу - в использовании табов для отступов. Но это же придирка по оформлению, а не по логике, да и лечится astyle'ом за секунды.
      Ответить

    • ava govnomonad 06.09.2012 07:04 # 0

      Говно в кривой грамматике?
      Бинарные операции вместе со скобками выглядат странно. Или автор любитель какого-нибудь J?
      Ответить

      • ava LispGovno 06.09.2012 08:33 # 0

        Вы считаете, что в J кривая грамматика? Я с вами категорически не согласен.
        Ответить

    • ava Steve_Brown 06.09.2012 10:23 # +2 

      a = some * (very * long) * expression + 
    + other * long * expression;

      Problems? :7
      А если серьезно, то два варианта:
      is_a = some * (very * long) * expression == a && 
    other * long * expression == 0;

      is_a = some * (very * long) * expression == a
    && other * long * expression == 0;

      Второй даже, возможно, более читаем, потому что логическая операция сразу видна. И не возникает дискомфорта, как в случае с "0 == other * long * expression" или, тем более, ", EnumElementN".
      Ответить

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