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

    +120

    1. 1
    2. 2
    3. 3
    4. 4
    string tmpS;
//идут всякие монипуляции с этой строкой
//. . .
tmpS.Remove(0, tmpS.Length);

    Реальный код из коммерческого продукта.

    Запостил: inser, 04 Марта 2011

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

    • ava absolut 04.03.2011 13:47 # +2

      Всё правильно. Поработал - убери за собой.
      Ответить

      • ava ReL 04.03.2011 13:49 # −2

        борьба с утечками по памяти...
        Ответить

      • ava inser 04.03.2011 14:22 # 0

        А почему бы не написать просто tmpS = "" ?
        Ответить

        • ava TarasB 04.03.2011 14:24 # 0

          Вот ты присвоил tmpS на новую строку, а староя-то строка висит! Надо ей явно написать Remove.
          Ответить

          • ava diablo 04.03.2011 14:42 # +10

            Люди, шо вы гонете?))) этот код на имеет смысла, поскольку строки в шарпе неизменяемы. Вот если б он написал tmpS = tmpS.Remove(0, tmpS.Length); , тогда tmpS указывала бы на новую строку(результат выполнения Remove). А старый объект зохавал бы сборщик мусора.
            Ответить

          • ava Em1ss1oN 05.03.2011 11:43 # −1

            GC существует для подобных вещей.
            Ответить

    • ava Анонимус 04.03.2011 21:56 # +1

      этот код не делает ничего
      Ответить

    • ava FofanovIS 12.03.2011 22:01 # −2

      Если написано в проекте .NET 4.0, то это - некритичный говнокод, ибо к StringBuilder добавили кошерный метод Clear() :)
      Ответить

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