Когда нужно память освобождать ?

[niockasd]

Всем привет, часто сталкиваюсь при работе с JSON с такой проблемой

Procedure AddAuthorizeLog (stat,text: string);
var
  JSObj: TJSONObject;
begin
  try
    JSObj:=TJSONObject.Create as TJSONObject;
    JSObj.AddPair('stat',TJSONString.Create(stat));
    JSObj.AddPair('data',TJSONString.Create(DateTimeToStr(now)));
    JSObj.AddPair('text',TJSONString.Create(text));
    AuthorizLogArry.AddElement( JSObj );
  finally
   //  FreeAndNil(JSObj);
  end;


end;

Если раскоментировать FreeAndNil(JSObj); программа крашится при обращении к AuthorizLogArry. Без освобождения все работает.

В моем понимании, если есть create должны и освободить по завершении операции. Или это не так?

[p51x]

Ну вы то операцию не завершили. Вы только отдали объект кому-то попользоваться. Читайте доки очищает ли AuthorizLogArry свои элементы сам или потом вам надо чистить.

[niockasd]

Сообщение от p51x

Ну вы то операцию не завершили. Вы только отдали объект кому-то попользоваться. Читайте доки очищает ли AuthorizLogArry свои элементы сам или потом вам надо чистить.

AuthorizLogArry - это переменная глобальная JSONArry. При создании формы инициируется и при дестройте освабождается...
Какая разница JSObj что дальше будет с отданой информацией )?

[p51x]

Сообщение от niockasd

Какая разница JSObj что дальше будет с отданой информацией )?

Никакой. Но вы ведь сам объект отдаете в пользование.

[Pavia]

Практика такая где выделили там и освобождаем. Иногда это тело в функции. Но иногда объекты живут дольше и это уже не тело функции, а тело класса или даже весь юнит. Класс TMyClass1 создал объект он его и уничтожает. Или ваш Юнит.
Это не строгие правила, а правила хорошего тона. Известные как RAII. Но оно спасает от многих ошибок.

У вашей библиотеке JSON кривая архитектура.

Сообщение от niockasd

AuthorizLogArry - это переменная глобальная JSONArry. При создании формы инициируется и при дестройте освабождается...
Какая разница JSObj что дальше будет с отданой информацией )?

Так не информация передаётся, а лишь ссылка(JSObj). А вы память освобождаете.
Разумеется когда AuthorizLogArry захочет обратиться к JSObj он по пробует перейти по ссылке он попадает в молоко. Так как данные вы уже освободили во finally.

Поэтому в вашем случае вы должны освобождать память когда он вам будет не нужен в AuthorizLogArry. Тогда вы его изъимите из AuthorizLogArry, а так же перед удалением AuthorizLogArry.

[niockasd]

Сообщение от Pavia

Практика такая где выделили там и освобождаем. Иногда это тело в функции. Но иногда объекты живут дольше и это уже не тело функции, а тело класса или даже весь юнит. Класс TMyClass1 создал объект он его и уничтожает. Или ваш Юнит.
Это не строгие правила, а правила хорошего тона. Известные как RAII. Но оно спасает от многих ошибок.

У вашей библиотеке JSON кривая архитектура.

Так не информация передаётся, а лишь ссылка(JSObj). А вы память освобождаете.
Разумеется когда AuthorizLogArry захочет обратиться к JSObj он по пробует перейти по ссылке он попадает в молоко. Так как данные вы уже освободили во finally.

Поэтому в вашем случае вы должны освобождать память когда он вам будет не нужен в AuthorizLogArry. Тогда вы его изъимите из AuthorizLogArry, а так же перед удалением AuthorizLogArry.

неожиданно, печально.
Я хотел сделать что то типа записи логов в массив JSON, в моем случае код приведенный выше работает, могут ли возникнуть какие то проблемы в дальнейшем?

Получается при каждой новой записи данных в лог вызывается

 JSObj:=TJSONObject.Create as TJSONObject;

старые ссылки которые отправлены ранее в AuthorizLogArry получается должны уничтожится? Почему это тогда работает?

[p51x]

Сообщение от niockasd

старые ссылки которые отправлены ранее в AuthorizLogArry получается должны уничтожится?

С чего вдруг? У вас есть полка с книгами, вы ставите новую. Все остальные на этой полке уничтожаются?

Сообщение от niockasd

Почему это тогда работает?

А почему не должно?