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

    • ava bormand 29.08.2013 15:41 # +4

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

    • ava krypt 29.08.2013 15:49 # +4

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

    • ava kegdan 29.08.2013 16:00 # −2

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

      • ava kegdan 29.08.2013 16:10 # +4

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

        • ava kegdan 29.08.2013 16:23 # 0

          хотя, по логике вещей, капитан был прав
          Ответить

          • ava kegdan 29.08.2013 16:23 # +1

            гребанное разделение личностей
            Ответить

          • ava Vindicar 29.08.2013 21:45 # +1

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

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

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

            • ava blackhearted 30.08.2013 15:41 # −1

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

              • ava guest 31.08.2013 10:35 # −3

                Нахуй пошёл, дедуля.
                Ответить

              • ava Vindicar 31.08.2013 14:00 # +2

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

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