Отказ от ответственности: протестировано с D2007.
Ваш код действительно создает XML (<label/>
), как показано в этой измененной функции:
function createXMLDocument(): TXMLDocument;
var
res: TXMLDocument;
rootNode: IXMLNode;
sl : TStringList;
begin
res := TXMLDocument.Create(nil);
res.Active := true;
rootNode := res.AddChild('label');
// create string for debug purposes
sl := TStringList.Create; // not needed
sl.Assign(res.XML); // Not true: sl is empty after this assignment
ShowMessage(sl.text);// sl is NOT empty!
sl.Free; // don't forget to free it! use try..finally.. to guarantee it!
//add more elements
// generateDOM(rootNode);
Result := res;
end;
Но это требует много замечаний :
- Вам не нужна локальная переменная res, просто используйте Result.
- Вам не нужен дополнительный StringList для просмотра XML: Result.Xml.Text
- Не забудьте Free sl StringList, если вы его создадите.
- Возвращаемый вами XmlDocument не может использоваться вне функции и выдает AV, если вы попытаетесь .
Почему?
Это потому, что XMLDocument предназначен для использования в качестве компонента с владельцем или в качестве интерфейса в противном случае для управления его продолжительностью жизни .
Тот факт, что вы используете интерфейс для хранения rootNode, приводит к его уничтожению в конце функции CreateXmlDocument. И если вы посмотрите на код в TXMLNode._Release
, вы увидите, что он вызывает TXMLDocument._Release
, который вызывает Destroy, если только нет владельца для XMLDocument (или интерфейса, содержащего ссылку на него).
Вот почему XMLDocument является допустимым и заполняется внутри функции CreateXMLDocument, но недоступен за ее пределами, если вы не вернете интерфейс или не предоставите владельца .
См. альтернативные решения ниже :
function createXMLDocumentWithOwner(AOwner: TComponent): TXMLDocument;
var
rootNode: IXMLNode;
begin
Assert(AOwner <> nil, 'createXMLDocumentWithOwner cannot accept a nil Owner');
Result := TXMLDocument.Create(AOwner);
Result.Active := True;
rootNode := Result.AddChild('label');
OutputDebugString(PChar(Result.Xml.Text));
//add more elements
// generateDOM(rootNode);
end;
function createXMLDocumentInterface(): IXMLDocument;
var
rootNode: IXMLNode;
begin
Result := TXMLDocument.Create(nil);
Result.Active := True;
rootNode := Result.AddChild('label');
OutputDebugString(PChar(Result.Xml.Text));
//add more elements
// generateDOM(rootNode);
end;
procedure TForm7.Button1Click(Sender: TObject);
var
doc: TXmlDocument;
doc2: IXMLDocument;
begin
ReportMemoryLeaksOnShutdown := True;
doc := createXMLDocument;
// ShowMessage( doc.XML.Text ); // cannot use it => AV !!!!
// already freed, cannot call doc.Free;
doc := createXMLDocumentWithOwner(self);
ShowMessage( doc.XML.Text );
doc2 := createXMLDocumentInterface;
ShowMessage( doc2.XML.Text );
end;