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

    • ava guest 20.10.2009 17:18 # 0

      ты ступил
      Ответить

    • ava guest 20.10.2009 17:24 # +1

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

      • ava guest 20.10.2009 17:59 # 0

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

        • ava guest 20.10.2009 19:32 # +1

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

    • ava AndrewK 20.10.2009 19:33 # 0

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

      • ava guest 20.10.2009 23:36 # 0

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

        • ava guest 21.10.2009 06:56 # +1

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

    • ava guest 21.10.2009 20:19 # 0

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

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