1. C# / Говнокод #9484

    +964

    1. 1
    2. 2
    3. 3
    4. 4
    private static int CompareWidgetsByOrder(Widget x, Widget y)
{
      return x == null ? y == null ? 0 : 1 : y == null ? 0 : x.order > y.order ? -1 : x.order < y.order ? 1 : 0;
}

    Вот до чего доводит стремление к компактности кода.

    Запостил: nomad99111, 19 Февраля 2012

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

    • ava Lure Of Chaos 19.02.2012 00:22 # +1

      я переживаю за приоритет тернарных операций
      Ответить

      • ava nomad99111 19.02.2012 00:28 # 0

        Код работает правильно, точно так же, как до "рефакторинга".
        Ответить

      • ava bober_maniac 19.02.2012 00:33 # 0

        Вроде как у тернарного оператора высший приоритет.
        Ответить

    • ava Sauron 19.02.2012 02:02 # 0

      А вот если расставить скобки и переносы, то стало бы вполне читабельно!
      Ответить

    • ava eth0 19.02.2012 17:33 # 0

      Нужен, как минимум, четверичный оператор.
      Ответить

      • ava TarasB 19.02.2012 17:35 # +1

        Это типа как?
        Ответить

        • ava SmackMyBitchUp 19.02.2012 17:42 # +3

          да (с)
          Ответить

        • ava wvxvw 19.02.2012 17:51 # 0

          А почему, может быть. Это как расписать таблицу операций примерно следующим образом:

          _ | X | Y
X | A | B
Y | C | D

          Применение оператора выглядело бы так:
          foo = A join B ? A : B : C : D;

          Т.о. если X операция X дает true, выполняется A, если X операция Y дает true, выполняется B, и так далее :)
          Ответить

        • ava eth0 19.02.2012 20:08 # +1

          Ну, он как троичный, только четверичный.
          Ответить

    • ava 3.14159265 20.02.2012 16:45 # 0

      Лично я люблю тернарники и записал бы так:
      return  x == null
	? y == null 
	  ? 0 
	  : 1 
	: y == null 
	  ? 0 
	  : x.order > y.order 
	    ? -1 
	    : x.order < y.order 
	      ? 1 
              : 0
;
      Ответить

      • ava ICELedyanoj 24.02.2012 09:27 # 0

        Вплоть до строчек с x.order разбор кода шел нормально, но дальше я уже потерялся и перестал понимать кто там кому папа.
        Да еще и в прямом Return - запаришься проверять как работает метод, особенно если и в точке вызова результат напрямую передается в другой метод.
        Лучше все-таки соблюдать умеренность.
        Ответить

        • ava Steve_Brown 24.02.2012 13:34 # +2

          Возможно, самый тупой способ будет самым понятным:
          if (x == null && y == null)
  return 0;
if (x == null && y != null)
  return 1;
if (x != null && y == null)
  return 0;
return x.order > y.order ? 1 : 
  x.order < y.order ? -1 : 
  0;
          Ответить

        • ava bugmenot 24.02.2012 18:34 # +2

          > перестал понимать кто там кому папа
          If you don't have a DAD, you're a BASTARD.
          Ответить

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