himXML (gesprochen himix ML)

[p80286]

@nahpets
:oops:das kommt davon wenn man sich nicht mehr tagesaktuell mit etwas beschäftigt. In einer Ansi-Umgebung halte ich Umlaute immer noch für tödlich, spätestens wenn du abweichende Codepages nutzt.

Gruß
K-H

[CodeX]

Ich habe jetzt ein paar Wochen testweise mit himXML gearbeitet und bin grundsätzlich sehr positiv überrascht. Super Arbeit, himitsu!

So wie ich verstanden habe, möchtest Du das ganze Projekt grundlegend überarbeiten, findest dafür aber schon seit Längerem keine Zeit. Sofern sich das in absehbarer Zeit nicht ändert: Was spricht denn dagegen, den Grundcode so zu belassen und lediglich an diversen Stellen Verbesserungen vorzunehmen und Fehler zu beheben? Der Aufwand dafür wäre vielleicht eher überschaubar. Wären kleine Verbesserungen nicht besser als aufgrund einer Riesenaufgabe gar nicht mehr Hand anzulegen?

Ich habe nachfolgend ein paar meiner Verbesserungsvorschläge und gefundene Fehler zusammengefasst.


Bugs:
1. Speichert man ein "&" in einem Attribut, steht anschließend in der XML-Datei "&" statt "&". Liest man das Attribut ein und speichert es gleich wieder, dann wird daraus "&" usw. (das ist für mich aktuell das kritischste Problem)

2. Löscht man einen erzeugten Knoten wieder, wird beim Speichern eine EAccessViolation geworfen:

Delphi-Quellcode:

  XML := TXMLFile.Create;
  try
    xn := XML.RootNode.AddNode('test');
    // ...
    XML.RootNode.Nodes.Delete(xn);
    XML.SaveToFile('test.xml'); // -> EAccessViolation
  finally
    XML.Free;
  end;

Verbesserungsvorschläge:
1. Per Option festlegen können, dass statt True/False die standardkonforme Alternative 1/0 gespeichert wird. Ist kürzer und kann dann je nach Einsatzzweck und persönlicher Präferenz individuell gewählt werden.

2. Es wäre sehr hilfreich, wenn SaveToFile() zurückgeben würde, ob der Speichervorgang erfolgreich war oder nicht.

3. Ich speichere und lade oft TGUIDs und TDateTime. Es wäre schön, wenn man dafür nicht zusätzlich String-Umwandlungen durchführen müsste. TGUID geht ohne GUIDToString/StringToGUID gar nicht und TDateTime wird als Kommawert (also für einen menschlichen Betrachter der XML-Datei unlesbar) gespeichert. D.h. man muss zwecks Lesbarkeit zusätzlich mit DateTimeToStr/StrToDateTime arbeiten. Schön wäre es, wenn man in den TXMLFile-Optionen das Datumsformat als TFormatSettings einstellen könnte und das Lesen/Schreiben von Datum/Zeit dann automatisch erfolgen würde.

4. Dokumentation: Ich weiß, dass man da immer am wenigsten Lust drauf hat, aber ein paar essentielle Dinge sind nicht selbsterklärend und im Quelltext nicht dokumentiert. Ich musste mir erst hier im Forum die Antworten zusammensuchen (z.B.: *NF=Not Filtered, Was die einzelnen Optionen genau bewirken, Knoten verschieben, ...)

5. Last but not least: Vollverschlüsselung
Ich weiß, dass man mit dem Stream manuell anstellen kann, was man will, aber wenn das von Dir sowieso schon vorgesehen war (29.09.2011: "Eine integrierte Verschlüsselung über die gesammte Datei kommt erst in der nächsten Version."), dann wäre das echt super, wenn die Komponente das entsprechend selbst könnte.

[CodeX]

Hallo himitsu, liest du hier überhaupt noch mit? Wollte dir eine private Nachricht senden und nachfragen, aber das geht wohl nicht weil dein Posteingang voll ist...
Könntest du bitte auf mein letztes Posting antworten? Danke!

[himitsu]

autsch

Wegen dem Bug schau ich mal nach. (hab ja bald paar Tage nichts zu tun :stupid:)

[CodeX]

Zitat:

Zitat von himitsu (Beitrag 1389177)

Wegen dem Bug schau ich mal nach. (hab ja bald paar Tage nichts zu tun :stupid:)

Na, bist Du damit schon erfolgreich ins neue Jahr gestartet? :)

[himitsu]

Joar. Hatte mich bis kurz vor Mitternacht nochmal ins Bett gelegt, damit ich den Wechsel überlebe.
Aber so konnte ich mein neues elektrisches Wärmeunterbett ausprobieren, nachdem ich die uralte winzige knochenharte Heizdecke entsorgt hatte. Bissl Ruhe und Wärme tut dem schmerzenden Rücken garnicht so schlecht. :feuerchen:

[CodeX]

Hi himitsu,
hast Du Dir die beiden Bugs in der Zwischenzeit denn mal angeschaut?

[himitsu]

Upps, dachte ich hätte schon geantwortet.

Die EAccessViolation konnte ich nicht nachvollziehen.
> himxml_164 und XE3
> heute Abend kann ich nochmal schnell XE und 10.1 testen

Wie sieht dein System denn aus?
> Versionen XML und Delphi

Hab den Test nochmal schnell zusammengeklöppelt

Delphi-Quellcode:

var XML: TXMLFile;
  Node:  TXMLNode;
  S:     String;
begin
  // EAccessViolation

  XML := TXMLFile.Create;
  try
    Node := XML.RootNode.AddNode('test');
    XML.RootNode.Nodes.Delete(Node);
    XML.SaveToFile('test.xml');
    //  <?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
    //  <xml/>
  finally
    XML.Free;
  end;

  // Node "&amp;amp;" statt "&amp;"

  XML := TXMLFile.Create;
  try
    XML.RootNode.AddNode('test').Text := '&';
    XML.SaveToFile('test.xml');
    S := TFile.ReadAllText('test.xml');
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  <?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
    //  <xml>
    //    <test>&amp;</test>
    //  </xml>
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    S := '"' + XML.RootNode.Node['test'].Text + '"' + sLineBreak
      + '"' + XML.RootNode.Node['test'].Text_S + '"' + sLineBreak
      + '"' + XML.RootNode.Node['test'].Text_D + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "&"
    //  "&"
    //  "&"
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    XML.SaveToFile('test.xml');
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    S := '"' + XML.RootNode.Node['test'].Text + '"' + sLineBreak
      + '"' + XML.RootNode.Node['test'].Text_S + '"' + sLineBreak
      + '"' + XML.RootNode.Node['test'].Text_D + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "&"
    //  "&"
    //  "&"
  finally
    XML.Free;
  end;

  // Attribute "&amp;amp;" statt "&amp;"

  XML := TXMLFile.Create;
  try
    XML.RootNode.AddNode('test').Attributes.Add('attr', '&');
    XML.SaveToFile('test.xml');
    S := TFile.ReadAllText('test.xml');
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  <?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
    //  <xml>
    //    <test attr="&amp;"/>
    //  </xml>
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    S := '"' + XML.RootNode.Node['test'].Attribute['attr'] + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "&"
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    XML.SaveToFile('test.xml');
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');
    S := '"' + XML.RootNode.Node['test'].Attribute['attr'] + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "&"
  finally
    XML.Free;
  end;

  // DateTime

  XML := TXMLFile.Create;
  try
    XML.RootNode.AddNode('test').Text := Now;
    XML.RootNode.AddNode('test').Attributes.Add('attr', Now);
    XML.SaveToFile('test.xml');
    S := TFile.ReadAllText('test.xml');
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  <?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
    //  <xml>
    //    <test>2018-02-19T10-05-51.276</test>
    //    <test attr="2018-02-19T10-05-51.598"/>
    //  </xml>
  finally
    XML.Free;
  end;

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');

    S := '"' + DateTimeToStr(XML.RootNode.Node['test'].Text) + '"' + sLineBreak;
    //S := S + '"' + XML.RootNode.Node['test'].Text + '"' + sLineBreak;  // EVariantTypeCastError: Could not convert variant of type (UnicodeString) into type (Date)
    S := S + '"' + XML.RootNode.Node['test'].Text_S + '"' + sLineBreak + sLineBreak;

    S := S + '"' + DateTimeToStr(XML.RootNode.Node['test'].Attribute['attr']) + '"'+ sLineBreak;
    S := S + '"' + XML.RootNode.Node['test'].Attribute['attr'] + '"' + sLineBreak + sLineBreak;

    S := S + '"' + TXHelper.DateTimeToXML(Now, 3) + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "19.02.2018 10:14:41"
    //  "2018-02-19T10-14-41.622"
    //
    //  "30.12.1899"
    //  ""
    //
    //  "2018-02-19T10-14-48"
  finally
    XML.Free;
  end;

So ging alles.

Diesmal auch das Datum probiert und ...

Zitat:

EVariantTypeCastError: Could not convert variant of type (UnicodeString) into type (Date)

Bug im XE3?
> Der Cast ist ja wohl andersrum, also Variant(DateTime) > String :stupid:

Und bei den Attributen klappt das Auslesen nicht ... muß aber noch schauen warum.
Mir fiel grade ein warum es nicht ging. :oops:
> Attribut im zweiten Node, aber den ersten Node ausgelesen

Delphi-Quellcode:

  // richtig

  XML := TXMLFile.Create;
  try
    XML.LoadFromFile('test.xml');

    S := '"' + DateTimeToStr(XML.RootNode.Node['test'].Text) + '"' + sLineBreak;
    //S := S + '"' + XML.RootNode.Node['test'].Text + '"' + sLineBreak;  // EVariantTypeCastError: Could not convert variant of type (UnicodeString) into type (Date)
    S := S + '"' + XML.RootNode.Node['test'].Text_S + '"' + sLineBreak + sLineBreak;

    S := S + '"' + DateTimeToStr(XML.RootNode.Node['test'].Attribute['attr']) + '"'+ sLineBreak;
    S := S + '"' + XML.RootNode.Node['test'].Attribute['attr'] + '"' + sLineBreak + sLineBreak;

    S := S + '"' + DateTimeToStr(XML.RootNode.Nodes[1].Attribute['attr']) + '"'+ sLineBreak;
    //S := S + '"' + XML.RootNode.Nodes[1].Attribute['attr'] + '"' + sLineBreak + sLineBreak;  // EVariantTypeCastError: Could not convert variant of type (UnicodeString) into type (Date)
    S := S + sLineBreak;

    //S := S + '"' + DateTimeToStr((XML.RootNode.XPath['test[2]'] as TXMLNode).Attribute['attr']) + '"'+ sLineBreak;
    //S := S + '"' + (XML.RootNode.XPath['test[2]'] as TXMLNode).Attribute['attr'] + '"' + sLineBreak + sLineBreak;

    S := S + '"' + TXHelper.DateTimeToXML(Now, 3) + '"';
    OutputDebugString(PChar(S));  //ShowMessage(S);
    //  "19.02.2018 10:28:40"
    //  "2018-02-19T10-28-40.633"
    //
    //  "30.12.1899"
    //  ""
    //
    //  "19.02.2018 10:28:40"
    //
    //  "2018-02-19T10-28-45"
  finally
    XML.Free;
  end;

[Ghostwalker]

Moinmoin,

erstmal, super Lib, im Großen und Ganzen bin ich sehr zufrieden damit.

Bei meinem neusten Projekt bekomm ich aber ein Problem.

Schnippsel der Daten

Code:

<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<Label1 Display="Label1" Kind="tkClass" Value="" Typename="Vcl.StdCtrls.TLabel" Parentname="" Qualified="Vcl.StdCtrls.TLabel">
  <Align Display="Align" Kind="tkEnumeration" Value="alBottom" Typename="TAlign" Parentname="TLabel" Qualified="Vcl.Controls.TAlign"/>
  <Alignment Display="Alignment" Kind="tkEnumeration" Value="taCenter" Typename="TAlignment" Parentname="TLabel" Qualified="System.Classes.TAlignment"/>
  <Anchors Display="Anchors" Kind="tkSet" Value="[akLeft,akRight,akBottom]" Typename="TAnchors" Parentname="TLabel" Qualified="System.UITypes.TAnchors">

Beim Zugriff auf die einzelnen Nodes (nach dem Laden) bekomme ich zwar den Node, aber nicht seine Attribute:

Delphi-Quellcode:

  xml := TXMLFile.Create(NIL,true);
  xml.LoadFromFile(filename);
  node := ftree.GetFirst();
  ndata := Ftree.GetNodeData(node);
  path := ndata^.Display;
  xnode := xml[path];
  if (xnode <> NIL) then
  begin
    :

Hab einen Haltepunkt beim if gesetzt, und mir xnode.Attributes.count anzeigen lassen....ist 0.
Ich hab das ganze nicht nur mit dem Rootnode sonder auch mit Subnodes auspropiert. Den Node bekomm ich,
die Attribute nicht.

Wo liegt das Problem ?

[himitsu]

"path" stimmt auch?
siehe

Delphi-Quellcode:

xnode.NodeType

,

Delphi-Quellcode:

xnode.Name

,

Delphi-Quellcode:

xnode.FullPath

Standardmäßig ist xoNodeAutoCreate in den Options aktiv.

Entweder deaktivieren oder erst prüfen, vorm Zugreifen.

Delphi-Quellcode:

xml.Options := xml.Options - [xoNodeAutoCreate];

Delphi-Quellcode:

if xml.Nodes.Exists(path) then begin
  xnode := xml[path];