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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
string str=...;

if(str!=null && str.Length>0)
{
....
}

Без комментариев

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

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

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

    нормальная проверка в плане производительности.
    можно заменить на string.IsNullOrEmpty(str)
    Ответить

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

      видимо говнопостер считает проверку на null лишней, ведь у null'а длина 0...
      Ответить

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

        Проверка на null обязательная, иначе будет NullReferenceException при попытке обратиться к свойству Length. Просто те, кто пишут под .NET, должны знать о функции string.IsNullOrEmpty(str) и использовать её, ну это моё субьективное мнение. Вот, в принципе, и суть гавнокода.
        Ответить

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

    Проверка на null обязательная, иначе будет NullReferenceException при попытке обратиться к свойству Length. Просто те, кто пишут под .NET, должны знать о функции string.IsNullOrEmpty(str) и использовать её, ну это моё субьективное мнение. Вот, в принципе, и суть гавнокода.
    Ответить

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

      а ее производительность хоть как-то отличается в лучшую сторону от такой проверки?
      Ответить

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

        нет, там внутри такая же проверка. (проверено рефлектором)
        Ответить

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

    По моему тут говнокод мог бы быть только если присвоение переменной - явное и константной строке.
    Вот так :
    string str= "Hello world";
    if(str!=null && str.Length>0)
    {
    ....
    }
    Ответить

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

Переведи на "PHP", guest!

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

    ↑
    8
    ↓