Externes Programm zum durchsuchen von *.dfm Dateien und Speicherung von Actions

[Axti]

Hallo zusammen,
ich stehe vor folgendem Problem:

Ich soll ein Programm schreiben, welches die Formulardateien(*.dfm) eines Delphiprojektes durchsucht.
Dort sollen folgende Dinge gefunden und später in einer Datenbank abgelegt werden: Formularname, TActions,
Caption-Eigenschaft der TActions und die Menüstruktur in der die Actions hinterlegt sind.

Kein allzu großes Problem dachte ich mir am anfang. Irgendwann, als ich dann die Routine für das durchsuchen
der Menüstruktur geschrieben habe bin ich dann aber zu dem Problem gekommen, dass Actions, die in im
3. Untermenü liegen, nicht mehr erfasst werden.

Delphi unterscheidet leider nicht zwischen Menü und den entsprechenden MenüItems. Es gibt lediglich 'TMainMenu' und 'TMenuItem'.
Die Menüstruktur sieht also folgendermaßen aus:

MainMenu:
Datei-----------------------------Bearbeiten----------Hilfe------------
-MenuItem1----------------------MenuItem1---------MenuItem1-
-MenuItem2->MenuItem1-----MenuItem2---------MenuItem2-
-MenuItem3---MenuItem2-----MenuItem3---------MenuItem3-
------------------MenuItem3--------------------------------------------


Wenn ich also eine den Menüpfad zu einer Action haben möchte, die bei MainMenü->Datei->MenuItem2->MenuItem1 hinterlegt habe,
ist der Übergang im Code von Datei zu MenuItem2 genau der gleiche wie von MenuItem2 zu MenuItem1 im Untermenü. Mir fehlt
also ein spezifischer Anhaltspunkt der mir sagt: "Hier hat ist einer der Menüpunkte noch ein Menü". Mein bisheriger Ansatz
scheiterte genau an dem Fehlen eines solchen speziellen Anhaltspunkt. Die Variable, in der der erste Menüpunkt gespeichert wurde,
wurde jedesmal überschrieben da die Abfrage ob ein Menüpunkt vorhanden in beiden Fällen identisch sein muss, andernfalls speichert
er die Menüpunkte gar nicht ab.

Fällt jemanden eine anderer Lösungsansatz ein oder bei mir ein Denkfehler auf? Habe heute schon den ganzen Tag geknobelt und komme
nicht drauf

[TigerLilly]

Im DFM sieht das doch so aus:

Code:

object frmUserMgmt: TfrmUserMgmt
  ...
  Menu = mmMain
  ...
  object mmMain: TMainMenu
    Left = 344
    Top = 56
    object Extras1: TMenuItem
      Caption = 'Extras'
      object Datenimportierenexportieren2: TMenuItem
        Action = acDataTrans
      end
      object BenutzerEinstellungen2: TMenuItem
        Action = acUserSettings
      end
    end
  end
  ...

Du musst beim Scannen die Verschachtelung berücksichtigen.

Oder verstehe ich dich falsch?

[Axti]

Nein du verstehst mich wohl richtig.

Was genau meinst du mit berücksichtigen der Verschachtelung? Kannst du mir ein Beispiel geben?

[TigerLilly]

Naja, schau dir das DFM an. Du siehst, dass die eingerückten TMenuItems innerhalb eines anderen TMenuItems sind. Also verschachtelt. Das gibt deine Menüstruktur wieder.

Wie soll die Menüstruktur denn in der Datenbank abgelegt werden? Ein Parent würde sich anbieten, also der Name des übergeordneten Menüitems.
Dann hättest du in etwa so eine Tabelle:

- Formular
- Menüeintrag
- Action
- übergeordneter Menüeintrag

Am Beispiel:
frmUserMgmt, Extras1, null, null
frmUserMgmt, Datenimportierenexportieren2, acDataTrans, Extras1
frmUserMgmt, BenutzerEinstellungen2, acUserSettings, Extras1
etc

[Axti]

Die Tabelle soll so aussehen:

Form | Action | Caption | Menüpfad
frmBsp| acBsp | DoStuff | Bearbeiten->BeispielMenü->BeispielKnopf

Den Pfad am Ende bekomme ich soweit raus, dass er mir bei nur 2 Menüpunkten(Bearbeiten->Beispiel) den richtigen Pfad zurückgibt.
Sobald aber das Menü 3 fach verschachtelt ist, nimmt er den 3 Punkt nicht(Bearbeiten->BeispielMenü->BeispielKnopf.

Die verschachtelung in Delphi wäre dann so:

Code:

object frmUserMgmt: TfrmUserMgmt
  ...
  Menu = mmMain
  ...
  object mmMain: TMainMenu
    Left = 344
    Top = 56
    object Extras1: TMenuItem
      Caption = 'Extras'
      object Datenimportierenexportieren2: TMenuItem
        Caption = 'Dateien Importieren/Exportieren'
        object DatenImportieren: TMenuItem
          Action = acImport
        object DatenExportieren: TMenuItem
         Action = acExport
      end
      object BenutzerEinstellungen2: TMenuItem
        Action = acUserSettings
      end
    end
  end

Um genau den Punkt, die dritte Verschachtelung, um den geht es mir. Wie ich von der zweiten Verschachtelung in die dritte übergehe und das speichern kann. In diesem Falle eben "DatenImportieren" bzw. "DatenExportieren". Da hakt es leider bei mir

[Dennis07]

Zum Glück gibt es dafür ja mehr als genügend Möglichkeiten in Delphi, ohne groß Code schreiben zu müssen. Was wohl daran liegt, dass Delphi selbst ja in Delphi geschrieben ist und sowohl die ganze IDE, als auch die Forms zur Laufzeit direkt mit den DFM-Ressourcen arbeiten. Insofern ist das Lesen, Schreiben und sogar Anwenden und Generieren dieser ein Leichtes, wenn man sich ein wenig auskennt.

In deinem Fall dürfte TParser (was eigentlich TLexer heißen müsste) wohl die richtige Anlaufstelle sein.

[TigerLilly]

Zitat von Axti:

Um genau den Punkt, die dritte Verschachtelung, um den geht es mir. Wie ich von der zweiten Verschachtelung in die dritte übergehe und das speichern kann. In diesem Falle eben "DatenImportieren" bzw. "DatenExportieren". Da hakt es leider bei mir

Arbeite das DFM zeilenweise ab und merke dir die Einrückung in einem Level.
Kommt ein object erhöhe den Level um 1.
Kommt ein end verringere den Level um 1
wenn der Level 0 wird speicher den Datensatz und leere den Menüpfad, sonst
hänge die Caption der zeile an den Menüpfad dran.

Da das eine Baumstruktur ist, kannst du auch mit Rekursionen arbeiten oder, wie Dennis07 schreibt, den Parser benutzen, was mE aber jetzt nicht dein Problem löst, sondern dir nur den Zugriff erleichtert.

[himitsu]

Zitat:

Im DFM sieht das doch so aus:

Nein, grundsätzlich ist die DFM binär, aber seit mehreren Jahren wird sie als Text gespeichert und auch in der IDE bekommt man die Text-Darstellung.
Das ist aber optional und seit 'ner Weile ist es nur per Default das Text in der Datei.

Für dich sind also die Funktionen TestStreamFormat, ObjectBinaryToText und ObjectTextToBinary aus der Classes.pas interessant.
DFM-Datei in einen Stream einlesen/öffnen, mit TestStreamFormat nachsehen und eventuell mit ObjectBinaryToText oder ObjectTextToBinary konvertieren, jenachdem, ob du als Text oder binär drüberlaufen willst.
TReader und TWriter können de binären Stream verarbeiten und im Prinzip ist das Format auch seh einfach, denn dieses Format sagt dir immer, was als nächstes kommt (bestimmt auch fehlerunanfälliger als den Text selber parsen zu wollen)
Es ist immer TypByte, Daten, TypByte, Daten, TypByte, Daten, .... (Daten mit Längenangabe bei dynamischen Typen ala String)

[Axti]

Okay ich hab mir den Ratschlag zu herzen genommen und das ganze mal gebastelt. Ich gehe jede Zeile der *.dfm durch und isoliere den TMainMenu block. Dieser wird dann in einer StringList TempMenuData gespeichert. Diese Liste gehe ich von oben bis unten für jede Datei durch und speicher dann entsprechend den Pfad. Am Ende gehe ich die fertige Tabelle durch und hänge an der entsprechenden Stelle der Action den Pfad dran. Zumindest sollte es so sein. Leider macht er nicht genau das sondern speichert nur den zuerst gefundenen Menüpunkt. Alle darauffolgenden nicht mehr. In dem Falle nur "Datei" und alle seine Items. Der nächste Menüpunkt wäre Bearbeiten. Diesen Block speichert er schon nicht mehr ab.

Delphi-Quellcode:

    for linePath in TempMenuData do
    begin
      if (MenuLevel = 1)
        and (TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('object')
        and  TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('TMenuItem'))
      then
      begin
        tempMenuTop := TempMenuData.Strings[TempMenuData.IndexOf(linePath)];
        tempMenuTop := tempMenuTop.Replace(' ', EmptyStr);
        tempMenuTop := tempMenuTop.Replace('object', EmptyStr);
        tempMenuTop := tempMenuTop.Replace(':TMenuItem', EmptyStr);

        MenuLevel := 2;
      end
      else if (MenuLevel = 2)
        and (TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('object')
        and  TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('TMenuItem'))
      then
      begin
        tempPath := tempMenuTop + '->' + TempMenuData.Strings[TempMenuData.IndexOf(linePath)];
        tempPath := tempPath.Replace(' ', EmptyStr);
        tempPath := tempPath.Replace('object', EmptyStr);
        tempPath := tempPath.Replace(':TMenuItem', EmptyStr);

        if TempMenuData.Strings[TempMenuData.IndexOf(linePath)+1].Contains('Action') then
        begin
          tempMenuAction := TempMenuData.Strings[TempMenuData.IndexOf(linePath)+1];
          tempMenuAction := tempMenuAction.Replace('Action = ', '');
          tempMenuAction := tempMenuAction.Replace(' ', '');
        end;

        MenuLevel := 3;
      end
      else if (MenuLevel = 3)
        and (TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('object')

        and  TempMenuData.Strings[TempMenuData.IndexOf(linePath)].Contains('TMenuItem'))
      then
      begin
        tempPath := tempPath + '->' + TempMenuData.Strings[TempMenuData.IndexOf(linePath)];
        tempPath := tempPath.Replace(' ', EmptyStr);
        tempPath := tempPath.Replace('object', EmptyStr);
        tempPath := tempPath.Replace(':TMenuItem', EmptyStr);

        if TempMenuData.Strings[TempMenuData.IndexOf(linePath)+1].Contains('Action') then
        begin
          tempMenuAction := TempMenuData.Strings[TempMenuData.IndexOf(linePath)+1];
          tempMenuAction := tempMenuAction.Replace('Action = ', '');
          tempMenuAction := tempMenuAction.Replace(' ', '');
        end;
      end;

      if not linePath.Contains('end') then
      begin
        for lineNewPath in FinishedTable do
        begin
          if FinishedTable[FinishedTable.IndexOf(lineNewPath)].Contains(tempMenuAction)
           and not FinishedTable[FinishedFile.IndexOf(lineNewPath)].Contains('Path')
          then
          begin
            FinishedTable.Strings[FinishedTable.IndexOf(lineNewPath)]
                := FinishedTable.Strings[FinishedTable.IndexOf(lineNewPath)] +
                   ' Path: ' + tempPath;
            break;
          end;
        end;
      end
      else if linePath.Contains('end') then
      begin
        MenuLevel := MenuLevel-1;
      end;

      if MenuLevel = 0 then
      begin
        tempPath := EmptyStr;
      end;

    end;

[freimatz]

Hallo,
ich habe zwar jetzt nicht so die Ahnung im konkreten Fall, aber weil seit einem Tag sich niemand gemeldet hat versuche ich mal zu helfen:
Wenn ich an einem Code bin der nicht funktioniert und nicht weiß warum, dann räume ich erst mal auf. Das könntes Du auch versuchen, vielleicht kommst Du vielleicht selber drauf oder kannst uns dann den präsentieren.
z.B:
1.
- am Anfang ein i:Integer; und ein line: string;
- dann statt "for linePath in TempMenuData do" ein "for i in TempMenuData do".
- zwei Zeilen später dann ein "line := TempMenuData.Strings[i]
- dann alle "TempMenuData.Strings[TempMenuData.IndexOf(linePath)]" durch "line" ersetzen
2.
- die jeweils drei Zeilen mit "tempMenuTop.Replace" in eine separate Methode verpacken (z.B. CleanMenItem und CleanAction)
3.
- das zweite "if linePath.Contains('end')" scheint mir überflüssig zu sein.
4.
- Kurze Zeilen mit if zusammenziehen

Dann wird übersichtlicher und Du oder wir sehen eher woran es hakt.

Ansonsten teile ich die Meinung von TigerLilly und meine eine zeilenweise Analyse ist in Deinem Fall zielführend. Für mich riecht es danach, also wäre das nur ein Tool firmenintern. Da einen Lexer oder mit binären DFM zu hantieren halte ich für übertrieben. (Ich gehe davon aus, dass jeder, der professionell Software entwickelt, dfm als Text speichert, weil man sonst Versionskontrolle nicht sinnvoll benutzen kann.)