VPF::Утечка памяти про работе со строками

Ibragim

Дата 22.1.2007, 17:10 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 177
Регистрация: 28.9.2004
Где: Киев

Репутация: нет
Всего: нет

День добрый.
Есть участок кода, выполняется в цикле, который вызывает утечку памяти:

Код


    try
      SiteContent := ht.Get(Url);
    except
      ht.Free;
      State := 'Error';
      Synchronize(ShowState);
      Continue;
    end;
    ht.Free;
    SiteContent := LowerCase(SiteContent);

Как говорит MemProof, не удаляется объект SiteContent, то есть при следующей итерации цикла место из-под прошлого SiteContent не освобождается, а создается новая область в куче.
Как корректно освободить место из-под SiteContent? Пробую DisposeStr(@SiteContent); но функция, как говорит помощь, корректно работает только со строками, созданными NewStr. Да и опыт показал, что так нестабильно.
Как правильно решается такая задача?
Заранее спасибо.

MetalFan

Дата 22.1.2007, 17:15 (ссылка)

(нет голосов)

Загрузка ...

Аццкий Сотона

Профиль
Группа: Комодератор
Сообщений: 3815
Регистрация: 2.10.2006
Где: Moscow

Репутация: 62
Всего: 128

все супер. в приведенном куске кода никаких ошибок не наблюдается, с чем автора и поздравляю.

--------------------

There are always someone smarter than you...

Ibragim

Дата 22.1.2007, 17:34 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 177
Регистрация: 28.9.2004
Где: Киев

Репутация: нет
Всего: нет

н-да действительно, в "голом" проекте нет утечки...
черт, где же собака порылась...

А можно вот так?

Код


s := ConLinks(Url, s);

ConLinks - моя функция, s, Url: string;

ShadeSt

Дата 22.1.2007, 17:39 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 256
Регистрация: 9.8.2005
Где: Россия, г. Ставро поль

Репутация: нет
Всего: 7

Либо так:

Код


  S := ''

либо так

Код


  Finalize(S);

если я правильно понимаю, то утечка может быть в том случае, если ht.Get() выделяется новый участок памяти под строку.
попробуй так:

Код


      SiteContent := '';
      SiteContent := ht.Get(Url);

--------------------

It's nice to be importand, but it's more importand to be nice © Scooter

Ibragim

Дата 22.1.2007, 17:41 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 177
Регистрация: 28.9.2004
Где: Киев

Репутация: нет
Всего: нет

Код


SiteContent := '';
SiteContent := ht.Get(Url);

Так пробовал, не помогает. Сча попробую Finalize

MetalFan

Дата 22.1.2007, 17:42 (ссылка)

(нет голосов)

Загрузка ...

Аццкий Сотона

Профиль
Группа: Комодератор
Сообщений: 3815
Регистрация: 2.10.2006
Где: Moscow

Репутация: 62
Всего: 128

Цитата(Ibragim @ 22.1.2007, 17:34

)

А можно вот так?

что вот так?

абсолютно неинформативные сообщения. такое ощущение, что сам с собой автор разговаривает)

--------------------

There are always someone smarter than you...

Ibragim

Дата 22.1.2007, 17:52 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 177
Регистрация: 28.9.2004
Где: Киев

Репутация: нет
Всего: нет

гм, сорри

Не вызовет ли утечку памяти, если в работе со строками сделать следующее.
1. Создать функцию, положим

Код


function ConLinks(Url: string; s: string);
begin
   s := LeftStr(s,20);
   Result := LeftStr(Url,5) + s;
end;

2. И вызывать эту функцию следующим образом

Код


s := 'test2test2test2test2test2test2test2';
Url := 'test1test1test1test1test1test1test1test1';
s := ConLinks(Url, s);

aktuba

Дата 23.1.2007, 01:37 (ссылка)

(нет голосов)

Загрузка ...

Смышленный

Профиль
Группа: Завсегдатай
Сообщений: 1915
Регистрация: 24.4.2006
Где: Планета Земля

Репутация: 16
Всего: 38

Я так понимаю, это код из DLL. Ты случаем не передаешь данные из/в программу?

--------------------

Stern87

Дата 25.2.2008, 03:24 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 48
Регистрация: 27.11.2007
Где: Украина

Репутация: нет
Всего: нет

Если я не ошибаюсь, то строки в параметрах надо писать через const.

MetalFan

Дата 25.2.2008, 09:23 (ссылка)

(нет голосов)

Загрузка ...

Аццкий Сотона

Профиль
Группа: Комодератор
Сообщений: 3815
Регистрация: 2.10.2006
Где: Moscow

Репутация: 62
Всего: 128

Цитата(Stern87 @ 25.2.2008, 03:24

)

Если я не ошибаюсь, то строки в параметрах надо писать через const.

смотря какая цель преследуется...

Цитата(Ibragim @ 22.1.2007, 17:52

)

Не вызовет ли утечку памяти, если в работе со строками сделать следующее.

не вызовет. частенько так делаю, чтобы не заводить явно локальные переменные.

--------------------

There are always someone smarter than you...

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Delphi: Общие вопросы \| Следующая тема »