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

    • > TestStatus.Skipped
      > TestResultStatus.Skiped
      Граммотеи блин.
      Ответить
    • А главное - как читабельно отформатировано.
      Ответить
      • В форматировании смущает отсутствие тёплых ламповых точек с запятой в конце строк. Даже не по себе как-то...
        Ответить
      • но это не помогает понять приоритеты
        Ответить
    • А если забыть про правило одной точки выхода из метода и чуть=чуть переписать, то и понятнее, и видно, что условие 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;
      Ответить
      • свитч смотрится на порядок нагляднее
        Ответить
        • Не возражаю. Тем более, в свитче переменная status будет фигурировать лишь один раз, что не помешает человеку, чередующему Skipped и Skiped.
          Просто мне при чтении этого проще было не сразу в один свитч делать, а по кускам выносить отдельные условия в if-ы.
          Ответить
    • 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…
      Ответить

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