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

    +142

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    return status == TestStatus.Skipped
                    ? TestResultStatus.Skiped
                    : status == TestStatus.Inconclusive
                    ? TestResultStatus.Skiped
                    : status == TestStatus.Failed
                    ? TestResultStatus.Failed
                    : status == TestStatus.Passed
                    ? TestResultStatus.Passed 
                    : TestResultStatus.Failed;

    Запостил: dirtygopher, 13 Июня 2012

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

    • ava bormand 13.06.2012 13:31 # 0

      > TestStatus.Skipped
      > TestResultStatus.Skiped
      Граммотеи блин.
      Ответить

    • ava krypt 13.06.2012 15:21 # +2

      А главное - как читабельно отформатировано.
      Ответить

      • ava vistefan 13.06.2012 15:58 # 0

        В форматировании смущает отсутствие тёплых ламповых точек с запятой в конце строк. Даже не по себе как-то...
        Ответить

      • ava Lure Of Chaos 13.06.2012 23:27 # +3

        но это не помогает понять приоритеты
        Ответить

    • ava guest 15.06.2012 10:22 # +2

      А если забыть про правило одной точки выхода из метода и чуть=чуть переписать, то и понятнее, и видно, что условие status == TestStatus.Failed при желании можно выкинуть.
      if(status == TestStatus.Skipped || status == TestStatus.Inconclusive)
      return TestResultStatus.Skiped;
      if(status == TestStatus.Failed)
      return TestResultStatus.Failed;
      if(status == TestStatus.Passed)
      return TestResultStatus.Passed;
      return TestResultStatus.Failed;
      Ответить

      • ava defecate-plusplus 15.06.2012 10:30 # +4

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

        • ava guest 15.06.2012 12:42 # +1

          Не возражаю. Тем более, в свитче переменная status будет фигурировать лишь один раз, что не помешает человеку, чередующему Skipped и Skiped.
          Просто мне при чтении этого проще было не сразу в один свитч делать, а по кускам выносить отдельные условия в if-ы.
          Ответить

    • ava guest 22.06.2012 09:27 # −2

      2.71828 18284 59045 23536 02874 71352 66249 77572 47093 69995 95749 66967 62772 40766 30353 54759 45713 82178 52516 64274 27466 39193 20030 59921 81741 35966 29043 57290 03342 95260 59563 07381 32328 62794 34907 63233 82988 07531 95251 01901 15738 34187 93070 21540 89149 93488 41675 09244 76146 06680 82264 80016 84774 11853 74234 54424 37107 53907 77449 92069 55170 27618 38606 26133 13845 83000 75204 49338 26560…
      Ответить

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