In einem TreeView einen Knoten erstellen

[Delbor]

Hi zusammen

Mit folgendem Code zeige ich in meinem PathfinderFrame ein Popup an, um unter einem selectierten Parent einen neuen TreeNode (und schliesslich einen neuen Ordner) zu erstellen:

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerContextPopup(Sender: TObject;
  MousePos: TPoint; var Handled: Boolean);
  var P: TPoint; AnItem: TTreeNode;
      X,Y: integer;
begin
  X := MousePos.X;
  Y := MousePos.Y;
  AnItem := TVFilesExplorer.GetNodeAt(MousePos.X, MousePos.Y);
  FPopUpNode := AnItem;
  FPopUpPath := PNameRec(AnItem.Data)^.RealName;
...
  P := ClientToScreen(Point(x,y));
  PopupMenu1.Popup(P.X, P.Y);
end;

procedure TOpenFileFrame.Ordnererstellen1Click(Sender: TObject);
  var Node: TTreeNode;  LIsdirectory: Boolean;
      NameRec: PNameRec; LCaption, LRealName, X,Y: String;
begin
  LCaption:= PNameRec(FPopUpNode.Data)^.Caption;
  LRealName := PNameRec(FPopUpNode.Data)^.RealName;
  LIsdirectory := PNameRec(FPopUpNode.Data)^.IsDirectory;
  if LIsdirectory then
  begin
    Node.ImageIndex := 3;
    Node.SelectedIndex := 3;
    Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neu');
    X := Node.Text;
    Node.EditText;
//    TVFilesExplorer.Selected := Node;    
//    TVFilesExplorer.Selected.EditText;
  end
  else
  begin
    Showmessage('Der Ordner kann nicht erstellt werden.');
  end;
end;

Der Knoten wird zwar erstellt, aber er klebt links oben in der Ecke des Treeviews. Was mache ich falsch?

Gruss
Delbor

[Redeemer]

Du fügst den ja auch zu den Items hinzu und nicht zu dem gewünschten Parent.

Ich kapiere deinen Code auch nicht.
- Du weist erst einem Node Bilder zu und überschreibst dann genau diesen Node, bevor du irgendwas damit machst. Dazu ist Node noch ein var-Parameter, die Referenz ist also futsch und du hast ein Speicherleck. Objekte sollte man wenn überhaupt mit Assign zuweisen.
- Was soll uns die obere Methode sagen?

/Edit: Okay, ich sehe, wo du den Parent zuweist und dass du die obere Methode dafür benutzt. Ob die rechtzeitig aufgerufen wird, keine Ahnung, aber dein komisches Zuweisen bleibt.

[Delbor]

Hi Redeemer

Delphi-Quellcode:

procedure TOpenFileFrame.AddNewNode(ParentNode: TTreeNode; const aCaption,
  aRealName: string; aIsdirectory: Boolean);
  var Node: TTreeNode; NameRec: PNameRec;
      Lextension: String;
begin
  Node := TVFilesExplorer.Items.AddChild(ParentNode, aCaption);
  if aIsdirectory then
    begin
      Node.ImageIndex := 1;
      Node.SelectedIndex := 1;
      (* Dummy-Knoten anlegen *)         //   'alle Dateien', '.pdf', '.Docx', '.bmp', '.nef', '.jpg', '.png
      TVFilesExplorer.Items.AddChild(Node, 'dummy');
    end
  else
    begin
      Node.ImageIndex := 2;
      Node.SelectedIndex := 2;
      FPathlist.Add(aRealName);
      Lextension := ExtractFileExt(aRealName);   // Hier wird die Übereinstimmung mit dem aktuellen
      Node.Enabled := False;                     // Datefilter (Dateiendung) festgestellt. Wenn nein,
      if (FFileType = Lextension)  then            // ist der eingefügte Node nicht Enabled
        Node.Enabled := true;
      if (FFileType = 'alle Dateien') then
        Node.Enabled := true;
    end;
  FReportlist.Add('procedure TOpenFileFrame.AddNewNode');
  FReportlist.Add('Node.Text := ' + Node.Text);
  New(NameRec);
  NameRec^.RealName := aRealName;
  NameRec^.Caption := aCaption;
  NameRec^.IsDirectory := aIsdirectory;
  Node.Data := NameRec;
end;

Mit dieser Methode werden die von der TFilesearcher-Komponente gefundenen Ordner und Dateien dem Treeview hinzugefügt. Und danach beabsichtige ich, mich zu orientieren.
Ich hab mir auch schon überlegt, in meiner eigenen Methode lediglich das neue Verzeichnis zuu erstellen uund danach eine neue Suche anzustossen.
Etwas müsste ich mir allerdings hier einfallen lassen, da bestimmte Verzeichnisse spezielle Icons erhalten sollen - aktuell soll ein Verzeichnis für SQLite-DBs mit eigenem Icon erstellt werden. Ausserdem sollen Verzeichnisse dieser Art jeweils an erster Stelle unter dem entsprechenden Parent angezeigt werden, wofür sich ja 'AddChildFirst' anbietet. Ob das dann auch so bleibt, wenn Knoten mal erstellt ist, weiss ich nicht.

Gruss
Delbor

[Redeemer]

Ich sehe gerade erst, dass Node eine lokale Variable ist. Finde das komisch, hinter dem "var" für lokale Variablen was zu schreiben, weil das dann optisch nicht sofort von Parametern zu unterscheiden ist.
Dann müsste der Compiler doch meckern, dass Node.SelectedIndex undefiniert sein kann. Da müsste dann eigentlich auch eine AV stattfinden.

Hast du mal geprüft, ob FPopupNode nil ist? Also entweder liefert GetNodeAt nil zurück oder deine obere Methode wird zu spät aufgerufen. Letzteres glaube ich zwar nicht, aber in ich habe deinen Code mal mit meinem verglichen und ich mache etwas ähnliches wie du in TPopupMenu.OnPopup und nicht in TControl.OnContextPopup.

[Delbor]

Hi Redeemer

Ich sehe gerade, dass die Methode Ordnererstellen nicht mehr der Aktualität entspricht. Zur Zeit sieht die so aus:

Delphi-Quellcode:

procedure TOpenFileFrame.Ordnererstellen1Click(Sender: TObject);
  var Node: TTreeNode;  LIsdirectory: Boolean; NameRec: PNameRec;
      LDirectoryPath, LCaption, LRealName, X,Y: String;
begin
  if FPopUpNode <> nil then
  begin
    LRealName := PNameRec(FPopUpNode.Data)^.RealName;     //PNameRec(FSelectedNode.Data)^.RealName
    LIsdirectory := PNameRec(FPopUpNode.Data)^.IsDirectory;
    LCaption:= PNameRec(FPopUpNode.Data)^.Caption;
    LDirectoryPath := FPopUpPath + 'Neuer Ordner';

  ShowMessage('FSelectedNode := ' + FSelectedNode.Text +sLineBreak+
              'FPopUpNode := ' + FPopUpNode.Text +sLineBreak+
              'LRealName := ' + LRealName +sLineBreak+
              'LDirectoryPath := ' + LDirectoryPath);
  end;
  if LIsdirectory then
  begin
    Node := TVFilesExplorer.Items.AddChildFirst(FPopUpNode, 'Neu');
    Node.ImageIndex := 3;
    Node.SelectedIndex := 3;
    Node.EditText;
//    TVFilesExplorer.Selected := Node;
//    TVFilesExplorer.Selected.EditText;
  end
  else
  begin
    Showmessage('Der Ordner kann nicht erstellt werden.');
  end;
end;

Zitat:

ich mache etwas ähnliches wie du in TPopupMenu.OnPopup und nicht in TControl.OnContextPopup.

Das seltsame ist im Moment: ich weiss nicht mehr, wie ich auf OnContextPopup gekommen bin.

Gruss
Delbor

[Redeemer]

Die Methode sieht besser aus. Und deine ShowMessage kommt? Text ist, was du denkst?

Zitat:

Zitat von Delbor (Beitrag 1450685)

Zitat:

ich mache etwas ähnliches wie du in TPopupMenu.OnPopup und nicht in TControl.OnContextPopup.

Das seltsame ist im Moment: ich weiss nicht mehr, wie ich auf OnContextPopup gekommen bin.

Vielleicht, weil da die Mausposition gleich mit kommt.

[Delbor]

Hi Redeemer
Ich hab mal noch einige Showmessages eingebaut. Das sieht dann so aus:

Zitat:

procedure TOpenFileFrame.TVFilesExplorerContextPopup
FPopUpPath := := P:\
FPopUpNode :=P:\ [Elements]

Zitat:

TOpenFileFrame.PopupMenu1Popup

Nur der Methodenrumpf verlangt noch ein klein wenig Hirnschmalz, um die korrekte Position des PopUps zu setzen:

Delphi-Quellcode:

procedure TOpenFileFrame.PopupMenu1Popup(Sender: TObject);
begin
  ShowMessage('TOpenFileFrame.PopupMenu1Popup');
end;

Da werde ich wohl noch einige Felder einführen müssen...

Zitat:

Vielleicht, weil da die Mausposition gleich mit kommt.

Eher, weil das Event vom Treeview veröffentlicht wird - aber klar, dass da die Mouseposition dabei ist, macht die Sache scheinbar einfacher.

Gruss
Delbor

[Redeemer]

Jetzt steig ich durch die ganzen Codefetzen nicht mehr durch und weiß auch nicht mehr genau, ob du noch ein Problem hast oder nicht.

[Delbor]

Hi Redeemer

Mein Problem ist nach wie vor ungelöst. Ich habe die prozedur TVFilesExplorerContextPopup bis auf den Prozedurkopf auskommentiert. Damit sollte sie unassigned sein - auszuführen gibts da eh nichts mehr.

Dafür habe ich die Prozedure Mousedown angepasst:

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerMouseDown(Sender: TObject;
  Button: TMouseButton; Shift: TShiftState; X, Y: Integer);
  var AnItem: TTreeNode; P: TPoint;
begin
  if Button =(mbRight) then
  begin
    AnItem := TVFilesExplorer.GetNodeAt(X, Y);
    if AnItem <> nil then
    begin
      FPopUpNode := AnItem;
      FPopUpPath := PNameRec(AnItem.Data)^.RealName;
      FPopUpPoint := Point(x,y);
    end;
    P := ClientToScreen(Point(x,y));

    ShowMessage('procedure TOpenFileFrame.TVFilesExplorerMouseDown' +sLineBreak+
                'FPopUpPath := ' + FPopUpPath +sLineBreak+
                'FPopUpNode :=' + FPopUpNode.Text);
    PopupMenu1.PopupComponent := TVFilesExplorer;
    PopupMenu1.Popup(P.X, P.Y);
  end;
end;

Anschliessnd wird PopupMenu1Popup aufgerufen, wo ich per Showmessage einige Ausgaben mache.
Interessant wirds dann aber bei den Menuklicks. Hier das wesentliche Schnipsel aus 'OrdnererstellenClick':

Delphi-Quellcode:

  if LIsdirectory then
  begin
//    Node := TVFilesExplorer.Items.AddChildFirst(FPopUpNode, 'Neu');
    Node := TVFilesExplorer.Items.Add(FPopUpNode, 'Neu');
//    Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neu');
    Node.ImageIndex := 3;
    Node.SelectedIndex := 3;
    Node.EditText;
  end
  else
  begin
    Showmessage('Der Ordner kann nicht erstellt werden.');
  end;
end;

Interessant hier: Beide auskommentirten Zeilen erzeugen den Knoten zwar, zeigen ihn aber aber in der Linken oberen Ecke des Treeviews an.
Die nicht auskommentierte Zeile zeigt den Knoten zwar an, aber nicht mit dem Knoten als Parent, den ichh ausgewählt hatte ( Hier 'P:\'), sondern eigentlich darunter, also auf gleicher Ebene.
Anhang 51766
Das Problem ist also nicht die Prozedur TVFilesExplorerContextPopup.

Gruss
Delbor

[Redeemer]

Na ja, Add ist auf jeden Fall falsch („Der Knoten wird als letzter gleichrangiger Knoten des Parameters Sibling hinzugefügt.“ willst du ja nicht). Habe jetzt deinen Code mal in ein Projekt kopiert und mit AddChild geht es.
Damit das mit dem EditText funktioniert, musst du vorher noch FPopupNode.Expand(egal) aufrufen, aber darum geht's hier nicht.

Kompletter Implementation-Abschnitt:

Delphi-Quellcode:

procedure TForm1.TVFilesExplorerMouseDown(Sender: TObject; Button: TMouseButton;
  Shift: TShiftState; X, Y: Integer);
var AnItem: TTreeNode; P: TPoint;
begin
  if Button =(mbRight) then
  begin
    AnItem := TVFilesExplorer.GetNodeAt(X, Y);
    if AnItem <> nil then
    begin
      FPopUpNode := AnItem;
      //FPopUpPath := PNameRec(AnItem.Data)^.RealName;
      //FPopUpPoint := Point(x,y);
    end;
    P := ClientToScreen(Point(x,y));

    ShowMessage('procedure TOpenFileFrame.TVFilesExplorerMouseDown' +sLineBreak+
                'FPopUpNode :=' + FPopUpNode.Text);
    PopupMenu1.PopupComponent := TVFilesExplorer;
    PopupMenu1.Popup(P.X, P.Y);
  end;
end;

procedure TForm1.uwas1Click(Sender: TObject);
var
  Node: TTreeNode;
begin
  begin
// Node := TVFilesExplorer.Items.AddChildFirst(FPopUpNode, 'Neu');
    Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neu');
    FPopupNode.Expand(True);
// Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neu');
    Node.ImageIndex := 3;
    Node.SelectedIndex := 3;
    Node.EditText;
  end
  //else
  //begin
  //  Showmessage('Der Ordner kann nicht erstellt werden.');
  //end;
end;

D2009. Wenn's nicht geht, liegt's vielleicht an D5. Probier doch mal nur das Hinzufügen in einem Minimalbeispiel ganz ohne Popup.

[Delbor]

Hi Redeemer

Zitat:

Zitat von Redeemer (Beitrag 1450712)

D2009. Wenn's nicht geht, liegt's vielleicht an D5. Probier doch mal nur das Hinzufügen in einem Minimalbeispiel ganz ohne Popup.

Zuerst war ich überrascht, was Delphi5 angeht - aber dann bemerkte ich, dass das so im Eingangspost steht. Meine aktuell eingesetzte Version kannst du jedem meiner Beiträge entnehmen.
Zum aktuellen Stand: Dank deines Codescnipsels funktioniert es nun soweit. Allerdings erst, nachdem ich den bewussten Knoten wieder zugeklappt habe - der neue Knoten klebt dann als letzter im Childbaum.

Grundsätzlich habe ich zwei Möglichkeiten:

Delphi-Quellcode:

procedure TOpenFileFrame.Verzeichniserstellen1Click(Sender: TObject);
  var LRealName, LDirectoryPath, NewString: string; ClickedOK: Boolean;
begin
  if FSelectedNode <> nil then
  begin
    LRealName := PNameRec(FSelectedNode.Data)^.RealName;     //PNameRec(FSelectedNode.Data)^.RealName
    NewString := 'Geben sie einen Verzeichnisnamen ein ';
    ClickedOK := InputQuery('Input Box', 'Prompt', NewString);
    if ClickedOK then            { NewString contains new input string. }
    begin
      LDirectoryPath := FPopUpPath +'\' + NewString;
    end;
    TDirectory.CreateDirectory(LDirectoryPath);
  end;
end;

Die Prozedur war eigentlich nur ein Test. Der Vorteil: für die angelegte Datei wird bei aufklappen des Parentnodes auch 'automatisch' ein Knoten erzeugt. Automatisch heisst hier: Wenn der ParentKnoten (In meinen Beispielen 'P:\') aufgeklappt wird, startet die Komponente TFileSearcher eine Dateisuche und ruft jedesmal, wenn ein Ordner oder eine Datei gefunden wurde, die Prozedur AddNewNode des PathfinderFrames auf:

Delphi-Quellcode:

procedure TOpenFileFrame.AddNewNode(ParentNode: TTreeNode; const aCaption,
  aRealName: string; aIsdirectory: Boolean);
  var Node: TTreeNode; NameRec: PNameRec;
      Lextension: String;
begin
  Node := TVFilesExplorer.Items.AddChild(ParentNode, aCaption);
  if aIsdirectory then
    begin
      Node.ImageIndex := 1;
      Node.SelectedIndex := 1;
      (* Dummy-Knoten anlegen *)         //   'alle Dateien', '.pdf', '.Docx', '.bmp', '.nef', '.jpg', '.png
      TVFilesExplorer.Items.AddChild(Node, 'dummy');
    end
  else
    begin
      Node.ImageIndex := 2;
      Node.SelectedIndex := 2;
      FPathlist.Add(aRealName);
      Lextension := ExtractFileExt(aRealName);   // Hier wird die Übereinstimmung mit dem aktuellen
      Node.Enabled := False;                     // Datefilter (Dateiendung) festgestellt. Wenn nein,
      if (FFileType = Lextension)  then            // ist der eingefügte Node nicht Enabled
        Node.Enabled := true;
      if (FFileType = 'alle Dateien') then
        Node.Enabled := true;
    end;
  New(NameRec);
  NameRec^.RealName := aRealName;
  NameRec^.Caption := aCaption;
  NameRec^.IsDirectory := aIsdirectory;
  Node.Data := NameRec;
end;

Der Nachteil: Ich habe keine Möglichkeit, den neu angelegten direkt umzubenennen, wie es zB. im Explorer der Fall ist, sondern ich muss den neuen Namen per Inputbox eingeben.

Die zweite Möglichkeit wäre (gewesen?), einen Knoten zu erzeugen, diesen in den Editmodus zu versetzen und analog zum Explorer umzubennen - aber gerade dies scheint nicht zuverlässig zu funktionieren.

Gruss
Delbor

[Redeemer]

Zitat:

Zitat von Delbor (Beitrag 1450717)

Zum aktuellen Stand: Dank deines Codescnipsels funktioniert es nun soweit. Allerdings erst, nachdem ich den bewussten Knoten wieder zugeklappt habe - der neue Knoten klebt dann als letzter im Childbaum.

Was klappst du zu? Versteh ich nicht.

[Delbor]

Hi Reedemer
Das hier war dein Beispiel aus Beitrag #10 (um die auskommentierten Zeilen gekürzt):

Delphi-Quellcode:

procedure TForm1.uwas1Click(Sender: TObject);
var
  Node: TTreeNode;
begin
  begin
    Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neu');
    FPopupNode.Expand(True);                                   // <=== Die von dir eingefügte Zeile
    Node.ImageIndex := 3;
    Node.SelectedIndex := 3;
    Node.EditText;
  end
end;

Mit der von dir eingefügten Zeile wird der Knoten aufgeklappt. Ob das nun wirklich so war oder nicht; der Knoten schien nicht da zu sein - aber er war es definitiv, nachdem ich ihn erst zu- und dann wieder aufgeklappt hatte.

Im Moment allerdings steht mir Delphi nicht wirklich zur Verfügung, da ich den Debugmodus nicht starten kann; Kaspersky hat mir die von Delphi erstellte EXE als Virus eingestuft und hindert Delphi daran, eine neue zu erstellen. Und wie ich dem Kerl nun beibringe, dass er das gefälligst lassen soll, hab ich noch nicht mit letzter Sicherheit herausgefunden.

Gruss
Delbor

[Redeemer]

Kaspersky als Delphi-Entwickler... Kein Kommentar.

[Delbor]

Hi Redeemer

Zitat:

Zitat von Redeemer (Beitrag 1450868)

Kaspersky als Delphi-Entwickler... Kein Kommentar.

Sorry,was heisst das genau? Telefoniert Kaspersky zu oft nach Hause? Oder wie muss ich das verstehen?
Der hat mir schon öfters EXE-Dateien vor allem von älteren Beispielen gelöscht, aber dass ich das Projekt nicht mehr kompilieren konnte, ist mir noch nie passiert.

Gruss
Delbor

[Delbor]

Hi zusammen

Ich habe die Lösung gefunden und denke, sie könnte für den einen oder anderen von Interesse sein. Das 'Zauberwort' heisst Treeview.Expanded. Doch der Reihe nach:

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerMouseDown(Sender: TObject;
  Button: TMouseButton; Shift: TShiftState; X, Y: Integer);
  var AnItem: TTreeNode; P: TPoint;
begin
  if Button =(mbRight) then
  begin
    AnItem := TVFilesExplorer.GetNodeAt(X, Y);
    if AnItem <> nil then
    begin
      FPopUpNode := AnItem;
      FPopUpPath := PNameRec(AnItem.Data)^.RealName;
      FPopUpPoint := Point(x,y);
    end;
    P := ClientToScreen(Point(x,y));
    PopupMenu1.PopupComponent := TVFilesExplorer;
    PopupMenu1.Popup(P.X, P.Y);
  end;
end;

Damit zeige ich ein Popupmenü an. Ursprünglich hatte ich dazu das Ereignis OnContextPopup benutzt.

Delphi-Quellcode:

procedure TOpenFileFrame.Ordnererstellen1Click(Sender: TObject);
  var Node: TTreeNode;  LIsdirectory: Boolean; NameRec: PNameRec;
      LDirectoryPath, LCaption, LRealName, X,Y: String;
begin
  if FPopUpNode <> nil then
  begin
    LIsdirectory := PNameRec(FPopUpNode.Data)^.IsDirectory;
    if LIsdirectory then
    begin
  //    Node := TVFilesExplorer.Items.AddChild(FPopUpNode, 'Neuer Ordner');
  //    Node.ImageIndex := 3;
  //    Node.SelectedIndex := 3;
  //    //    Node.EditText;
      FPopupNode.Expand(True);
    end;
  end;
end;

Die auskommentierten Zeilen zeigen meinen Fehler: Wenn FPopUpNode expandiert wird, startet der TFilesearcher eine Suche, die alle Dateien und Ordner zurückliefert, die sich im Pfad des FPopUpNode befinden. Dabei wird pro Datei/Ordner ein Knoten erzeugt und der oben in den auskommentiertenn Zeilen erzeugte Knoten offenbar verworfen - auf jeden Fall war der Knoten nachher nicht da.
Das hab ich wie folgt lösen können:

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerExpanded(Sender: TObject;
  Node: TTreeNode);
begin
  Node := TVFilesExplorer.Items.AddChildFirst (FPopUpNode, 'St.Gallen in der Ostschweiz');
  Node.ImageIndex := 3;
  Node.SelectedIndex := 3;
  //    Node.EditText;
end;

Das erzeugt mir unmittelbar unter FPopUpNode den gewünschten Knoten. Wie beabsichtigt, soll der Standardname des Knotens von 'Neuer Ordner' wie im Explorer in den gewünschte Namen geändert werden. Also nicht über einen Inputdialog. Dazu werde ich die jetzt auskommentierte Zeile aktivieren.
Der eigentliche Ordner im Verzeichnis FPopUpNode wird dann erstellt, wenn der Editmodus des aktuellen Node beendet wird.

Gruss
Delbor