Си диез / Говнокод #13701 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
if (null != this.messageTemplateResourceName && null != this.messageTemplateResourceType)
{
    return ResourceStringLoader.LoadString(this.messageTemplateResourceType.FullName,
        this.messageTemplateResourceName,
        this.messageTemplateResourceType.Assembly);
}
if (null != this.messageTemplateResourceName || null != this.messageTemplateResourceType)
{
    throw new InvalidOperationException(Resources.ExceptionPartiallyDefinedResourceForMessageTemplate);
}

Microsoft EnterpriseLibrary
else? нет, не слышал

Запостил: NeoN NeoN, (Updated )

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

  • ava bormand # Ссылка на оригинал 0

    > ExceptionPartiallyDefinedResourceForMess ageTemplate
    Написано сотрудником эпла, недавно перешедшим на сторону M$? 🙂
    Ответить

  • ava krypt # Ссылка на оригинал 0

    А зачем здесь else? Куда его впихнуть? Тут если оба не нул - возвращаем результат, а если нул только 1 - кидаем ошибку. Если нул оба - ничего не делаем.
    Ответить

  • ava kegdan # Ссылка на оригинал 0

    Второй if заменяется строкой else (Ваш капитан)
    Ответить

    • ava kegdan # Ссылка на оригинал 0 ⇈

      Ваш капитан не правильно прочитал условия. (ваш прапорщик( )
      Ответить

        • ava Vindicar # Ссылка на оригинал 0 ⇈

          Если сомневаешься, составь таблицу истинности.
          Первое условие:
          оба null - 0
          первый null - 0
          второй null - 0
          оба не null - 1

          Второе условие:
          оба null - 0
          первый null - 1
          второй null - 1
          оба не null - 1

          Вывод - от второго if'а не избавиться (см. случай когда оба null), но ставить его в else первого имеет смысл для ясности - из-за return выше он все равно выполнится тогда и только тогда когда есть хотя бы один null.
          Ответить

          • ava blackhearted # Ссылка на оригинал 0 ⇈

            Вах, готовишься к пересдаче и выучил что такое ТИ?
            Похвально, сынок.
            Ответить

            • ava Vindicar # Ссылка на оригинал 0 ⇈

              Я-то своё отсдавал, а вот капитаны и прапорщики очевидных войск в комментах выше никак не договорятся.
              Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓