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

    +144.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    string str=...;
    
    if(str!=null && str.Length>0)
    {
    ....
    }

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

    Запостил: AndrewK, 20 Октября 2009

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

    • ты ступил
      Ответить
    • нормальная проверка в плане производительности.
      можно заменить на string.IsNullOrEmpty(str)
      Ответить
      • видимо говнопостер считает проверку на null лишней, ведь у null'а длина 0...
        Ответить
        • Проверка на null обязательная, иначе будет NullReferenceException при попытке обратиться к свойству Length. Просто те, кто пишут под .NET, должны знать о функции string.IsNullOrEmpty(str) и использовать её, ну это моё субьективное мнение. Вот, в принципе, и суть гавнокода.
          Ответить
    • Проверка на null обязательная, иначе будет NullReferenceException при попытке обратиться к свойству Length. Просто те, кто пишут под .NET, должны знать о функции string.IsNullOrEmpty(str) и использовать её, ну это моё субьективное мнение. Вот, в принципе, и суть гавнокода.
      Ответить
      • а ее производительность хоть как-то отличается в лучшую сторону от такой проверки?
        Ответить
        • нет, там внутри такая же проверка. (проверено рефлектором)
          Ответить
    • По моему тут говнокод мог бы быть только если присвоение переменной - явное и константной строке.
      Вот так :
      string str= "Hello world";
      if(str!=null && str.Length>0)
      {
      ....
      }
      Ответить

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