1. C++ / Говнокод #3215

    +960

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    // Encode() takes in reference to data. We now pass in reference to data instead of the
// entire data object to Encode() as it was causing memory leak(with the data object copy
// not getting deleted.

  Data &tempData = *data;
  Data::Encode(tempData, &encodedRequest, requestSize, 0);

    Перевод:
    Encode() принимает ссылку на data. Давайте передадим туда ссылку,
    а то раньше мы передавали объект целиком, и это приводило к утечкам памяти (копия объекта не удалялась).


    Здесь впору процитировать "Бойцовский Клуб":
    -А в какой Вы фирме работаете?
    -В крупной...

    Запостил: belca, 13 Мая 2010

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

    • ava 3.14159265 13.05.2010 13:28 # 0

      В чем говно?
      Раньше копия не удалялась, сейчас копии нет, объект один, он удаляется, все нормально, leaka нет.
      Причем тут цитата?
      Да и сама цитата звучит немного по-другому
      Там была не фирма, а компания.
      И не крупной, а одной из крупнейших.
      Если я не прав - смело минусуйте.
      Ответить

      • ava belca 13.05.2010 15:19 # +1

        Тогда объясните, какая копия не удалялась? Та, которая была на стеке (и которую передавали "целиком" т.е. по значению)?
        (А ведь и правда, в одной из крупейших :))
        Ответить

        • ava 3.14159265 13.05.2010 17:55 # 0

          судя по коментарию именно та
          >>>которую передавали "целиком"
          Ответить

          • ava 3.14159265 13.05.2010 18:05 # 0

            или я чего то не пойму и leaka совсем не было,
            тогда какой смысл этого вообще?
            Ответить

    • ava Stormherz 13.05.2010 14:24 # 0

      Вместо "Давайте передадим туда ссылку, ..." будет
      "Теперь передается ссылка, вместо объекта целиком, что приводило к утечкам памяти (копия объекта не удалялась)." простите, не сдержался
      Ответить

    • ava absolut 13.05.2010 14:42 # 0

      Я так понимаю, что сигнатура функции не менялась, а в качестве первого параметра передавалось *data. Если так, то ГК имеет место быть.
      Ответить

      • ava belca 13.05.2010 16:01 # 0

        а в чем могла быть исходная проблема при любой сигнатуре? разве что деструктор неправильно написан.
        Ответить

        • ava absolut 13.05.2010 16:11 # 0

          Если сначала было 
          Data::Encode( *data, &encodedRequest, requestSize, 0);

          а потом стало
          Data &tempData = *data;
Data::Encode(tempData, &encodedRequest, requestSize, 0);
          .
          То ничего не изменилось с точки зрения работы Encode().
          Вообще, проблему ГК в первую очередь должен знать тот, кто его публикует :))
          Ответить

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