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

    • ava Lure Of Chaos 31.05.2010 19:01 # −1

      "обратная польская запись" в мозгах
      Ответить

    • ava guest 31.05.2010 19:10 # −1

      if (bmp != null) DisposeBmp();
      Ответить

    • ava guest 31.05.2010 19:14 # +2

      > Присмотритесь к 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();
    }


      так что говнокод разве что если для восприятия человеком
      Ответить

      • ava Dmitriy 31.05.2010 19:21 # +1

        Ну да... Просто когда пытаешься понять что было ранее написано, очень легко упустить что они имели ввиду, поскольку, кажется, что остальные If, содержатся в истинном условии условии.
        Ответить

      • ava Stalker 31.05.2010 20:47 # 0

        Хм... Откуда инфа, что почитать? (не секрет если ;)
        Ответить

        • ava Dmitriy 31.05.2010 20:59 # +1

          Инфа из проекта над которым я сейчас работаю, после того как над ним поработали американцы. А читать нужно "Рефакторинг" Фаулера и "Совершенный код" Макконнела.
          Ответить

        • ava guest 31.05.2010 21:02 # +1

          > Хм... Откуда инфа, что почитать? (не секрет если ;)

          скомпилируй код и посмотри в рефлекторе :)
          Ответить

          • ava Dmitriy 31.05.2010 21:16 # +4

            Да я не сомневаюсь что компилятор сделает все правильно. "Код понятный компьютеру может написать любой. Искусство писать код понятный человеку". (С) Фаулер. А когда такого кода много, очень хочется взять карту и стереть Америку.
            ----
            P.S. Сорри, думал что комментарий был к моему сообщению, но только когда отправил этот комментарий понял к кому относилось. Жаль что удалить свой комментарий невозможно.
            Ответить

          • ava Stalker 02.06.2010 12:34 # 0

            Ну, тоже вариант. Без практики теория бесполезна...
            Ответить

    • ava guest 01.06.2010 10:03 # −5

      показать все, что скрытоЭто не перл, это Ц#.
      Ответить

      • ava guest 01.06.2010 15:49 # −4

        А ты гость.
        Ответить

        • ava guest 01.06.2010 16:17 # −4

          И я гость
          Ответить

          • ava guest 01.06.2010 16:45 # −4

            Мы два гостя
            Ответить

            • ava guest 01.06.2010 19:20 # −4

              если каждый гость = два гостя, то сколько чашек чая?
              Ответить

              • ava guest 01.06.2010 21:24 # −4

                присовокупи к этому что эти гости - капитаны очевидности

                картина мира в мою голову не умещается
                Ответить

                • ava guest 01.06.2010 23:24 # −4

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

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