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

    +119

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if (this.bmp == null) {
    				} else if (this.bmp.Size.Height < this.ViewSize.Height) {
    					DisposeBmp();
    				} else if (this.bmp.Size.Width < this.ListData.RowSize.Width) {
    					DisposeBmp();
    				}

    Еще один Американский перл.... Присмотритесь к 1-му "if". И так по всему коду.

    Запостил: Dmitriy, 31 Мая 2010

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

    • "обратная польская запись" в мозгах
      Ответить
    • if (bmp != null) DisposeBmp();
      Ответить
    • > Присмотритесь к 1-му "if". И так по всему коду.

      компилятор, кстати, разворачивает это в

      if (bmp != null)
          {
                   if (this.bmp.Size.Height < this.ViewSize.Height)
                         DisposeBmp();
                   else if (this.bmp.Size.Width < this.ListData.RowSize.Width)
                         DisposeBmp();
          }


      так что говнокод разве что если для восприятия человеком
      Ответить
      • Ну да... Просто когда пытаешься понять что было ранее написано, очень легко упустить что они имели ввиду, поскольку, кажется, что остальные If, содержатся в истинном условии условии.
        Ответить
      • Хм... Откуда инфа, что почитать? (не секрет если ;)
        Ответить
        • Инфа из проекта над которым я сейчас работаю, после того как над ним поработали американцы. А читать нужно "Рефакторинг" Фаулера и "Совершенный код" Макконнела.
          Ответить
        • > Хм... Откуда инфа, что почитать? (не секрет если ;)

          скомпилируй код и посмотри в рефлекторе :)
          Ответить
          • Да я не сомневаюсь что компилятор сделает все правильно. "Код понятный компьютеру может написать любой. Искусство писать код понятный человеку". (С) Фаулер. А когда такого кода много, очень хочется взять карту и стереть Америку.
            ----
            P.S. Сорри, думал что комментарий был к моему сообщению, но только когда отправил этот комментарий понял к кому относилось. Жаль что удалить свой комментарий невозможно.
            Ответить
          • Ну, тоже вариант. Без практики теория бесполезна...
            Ответить
    • показать все, что скрытоЭто не перл, это Ц#.
      Ответить
      • А ты гость.
        Ответить
        • И я гость
          Ответить
          • Мы два гостя
            Ответить
            • если каждый гость = два гостя, то сколько чашек чая?
              Ответить
              • присовокупи к этому что эти гости - капитаны очевидности

                картина мира в мою голову не умещается
                Ответить
                • картину просто нарисуй поменьше или голову увелич
                  Ответить

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