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

    +128

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 29 Августа 2013

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

    • > ExceptionPartiallyDefinedResourceForMess ageTemplate
      Написано сотрудником эпла, недавно перешедшим на сторону M$? :)
      Ответить
    • А зачем здесь else? Куда его впихнуть? Тут если оба не нул - возвращаем результат, а если нул только 1 - кидаем ошибку. Если нул оба - ничего не делаем.
      Ответить
    • Второй if заменяется строкой else (Ваш капитан)
      Ответить
      • Ваш капитан не правильно прочитал условия. (ваш прапорщик( )
        Ответить
        • хотя, по логике вещей, капитан был прав
          Ответить
          • гребанное разделение личностей
            Ответить
          • Если сомневаешься, составь таблицу истинности.
            Первое условие:
            оба null - 0
            первый null - 0
            второй null - 0
            оба не null - 1

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

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

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