:= nil;
FSyncObj := nil;
end;
destructor TCEFComponentIdList.Destroy;
begin
if assigned(FList) then FreeAndNil(FList);
if assigned(FSyncObj) then FreeAndNil(FSyncObj);
inherited Destroy;
end;
procedure TCEFComponentIdList.AfterConstruction;
begin
inherited AfterConstruction;
FList := TList.Create;
FSyncObj := TCriticalSection.Create;
end;
это они зачем так?
продублирую субботний вопрос есть ли какой-то какой резон в такой двухэтапной инициализации?
Эта инициализация бесмысленна, т.к. при создания класса, переменные забиваются Default значениями FList := nil; FSyncObj := nil; Создание на afterconstraction нужно, чтобы при ошибке в конструкторе не произошло разрушение класса Но тогда нужно каждый раз проверять, был ли созданы объекты: FList и FSyncObj
с большой долей вероятности это писал не делфист код неграмотен, но на работу это не влияет
точно нет смысла в проверке Assigned перед FreeAndNil 🤷🏻♂️
Обниливать смысла тоже нет. Если инстанс только что создали то там и так гарантированно будет nil. Если же как-то умудрились до конструктора создать поля то будет утечека
почему сразу утечка)) и умудряца тоже ненадо - в лазаре можно создать с подготовленными полями емнип
Как после обниливания разрушить?
ну этоже не полный код, а кусок?
Ну я ж откуда знаю?)
полный практически одно присвоение невтемное из конструктора убрал
Обсуждают сегодня