PHP Inspection Unit

[mirage228]

Hi,

Danke erstmal für euer Feedback. :)

@DSeven:
Ich werde mal schauen, wie sich das am besten realisieren lässt, um nur die Unterschiede in der Datei zu berücksichtigen. Das ist auf jeden Fall eine gute Idee!
Ich hatte auch erst überlegt ein VirtualTreeView in die Beispielanwendung zu packen, aber es ist nur eine Beispielanwendung, die jeder "mal eben so" kompilieren und leicht nachvollziehen soll. Optimale Geschwindigkeit war nicht in erster Linie das Ziel des Beispielprogrammes. Aber mit einem VirtualTreeview wäre es natürlich schneller und besser - aber leider hat nicht jeder das VirtualTreeview :-(

@Pr0g:
Ja, das habe ich gestern auch feststellen können. Der Stacküberlauf wird von der verwendeten PCRE.pas verursacht undzwar bei dem Pattern der CheckIsCommented() funktion, wo auf Kommentare oder Strings geprüft wird. Ich werde das nochmal genauer unter die Lupe nehmen...

mfG
mirage228

[mirage228]

Hi,

so die neue Version ist fertig (Version 1.4)

Ich konnte das Problem mit EStackOverflow behoben. Der eine RegEx beim Prüfen von Kommentaren war "fehlerhaft" und verursachte den gennanten Fehler.

Auch habe ich die Erkennung von Kommentaren stark optimiert. Anstatt bei jedem Durchlauf den gesamten Quelltext erneut nach Kommentaren zu parsen, werden nun alle Kommentare am Anfang der Analyse im Vorraus bestimmt (neue Klassen TPHPComment und TPHPComments)
Bei meinem Test (viele Klassen mit schön viel Kommentar :) ) kam ich auf das Ergebnis von ~640ms (jetzt) gegen ~16500ms (alt) für das Analysieren des Codes. Ich werde aber noch versuchen bzgl. Speed etwas mehr rauszuholen.
Das Darstellen im TreeView habe ich nicht mit eingeschlossen, da das etwas länger dauern könnte. Als Lösung kann ich empfehlen, wie bereits vorgeschlagen wurde, das VirtualTreeView von Mike Lischke zu verwenden. Um jedoch so wenige Zusatzkomponenten wie möglich voraussetzen / mitliefern zu können, findet sich im Beispiel ein normales "TreeView" (sollte für Testzwecke aber ausreichend sein).

Aber erstmal viel Spaß mit der neuen Version! (Downloads, wie gehabt, im ersten Beitrag) :)

mfG
mirage228

[DSeven]

Zitat:

Zitat von mirage228

Hi,
Ich werde mal schauen, wie sich das am besten realisieren lässt, um nur die Unterschiede in der Datei zu berücksichtigen. Das ist auf jeden Fall eine gute Idee!

Also mein Parser tut das prinzip zur Zeit nur zur helfte verwirklichen.
Erstmal läuft der gesammte Parser in einem Thread.
Ich parse eigentlich auch immer die gesammte Datei :D (deshalb verwirkliche ich es ja nur zur helfte) aber ich überprüfe beim erstellen des Trees (VirtualTreeView) ob es den Eintrag bereits gibt, wenn ja überspringe ich den eintrag und gehe zum nächsten. So muss ich nur neue einträge erstellen anstatt das gesammte Tree erneut zu erstellen.

Bringt finde ich erheblich viel Zeit.
Ich versuche zur Zeit den Parser so umzustellen das er nur nach änderungen im Quelltext sucht was aber nur teilweise funktioniert.

Es Funktioniert nur dann wenn man auch die Editor Kompo dem entsprechend umprogrammiert. So das sie sich merkt was geändert wurde.

Also ziemlich kompliziert.
Aber eigentlich lässt sich die benötigte Zeit auch beim erstellen des TreeViews rausholen finde ich.
Teste die geschwindigkeit deines Parsers am besten gegenüber anderen Produkten wie phpEdit. Obwohl der auch sehr langsam ist.

@Pr0g: Arbeite gerade am Stable release :D
mfg,
Nexis

[mirage228]

Hi,

ich habe auch versucht zu implementieren, dass wirklich nur unterschiedliche Methoden etc. geparsed werden. Um aber zu untersuchen, dass was geändert wurde, musste ich schon fast das ganze Parsing einmal erledigen, was das ganze Prozedere ca. 1,5x langsamer machte.

Ich hab gestern noch das im Netz gefunden:
http://www.users.on.net/johnson/delphi/textdiff.html

Das sieht ganz brauchbar aus - jetzt muss ich nur schauen, wie sich das auf den Speed auswirkt. Ich hab nämlich das Gefühl das bei kürzeren Sourcen der komplette Neuaufbau am schnellsten ist.

Ich werde mir heute aber dann erstmal phpEdit anschauen und ihn mit meinem Parser vergleichen :)

Danke für die Tipps :)

mfG
mirage228

[mirage228]

Hi,

ich habe heute den Quelltext noch einmal so optimiert, um auch ohne Text-Differenz mehr rauszuholen.
Daraus resultiert die neue Version 1.5 :)

Dabei habe ich folgendes gemacht:

• Die Reguläre Ausdrücken werden einmalig im initialization Abschnitt erstellt, sodass dies nicht immer wieder geschehen muss. Das spart besonders bei langen Sourcen Zeit.
• Kommentare werden nur bis zu der Stelle geparsed, an der irgendetwas (Methode, Klasse, ...) gefunden wurde. Das spart Zeit bei Source, welcher nur am Anfang z.B. Includes, Methoden etc. stehen hat und der rest dadrunter normaler Code ist.
• Zudem habe ich den Kommentar-Algorithmus selbst ein wenig verfeinert (es wird jetzt kein Zweiter durchlauf mehr benötigt). Das spart auch etwas Zeit.

Insgesamt konnte ich eine Geschwindigkeitserhöhung von teils über 20% (variiert nach oben und unten - je nach Source) herausholen!

Aber das mit der Text-Differenz werde ich mir auf jeden Fall noch ansehen @DSeven.

Downloadlinks, wie gehabt, im ersten Beitrag!

mfG
mirage228

[Pr0g]

Gibt es irgend eine Möglichkeit die Unit ohne Beigabe der Dll nutzen zu können. Durch eine andere Lib, oder sonstwas?

MfG Pr0g

[DSeven]

Guten Tag,

Also ich habe heute einen Test mit deinem Parser 1.3 gemacht und einer Datei mit 410 Kb. Es befinden sich da verschiedene Klassen und ein paar variablen und Funktionen.

Dein Parser hat 30 Sekunden gebraucht meiner 132 ms.
ALso du siehst du must noch einiges machen.

PHPEdit hat nur zwei sekunden mehr gebraucht.
OK mein Parser part nicht so viele sachen wie deiner, aber der von phpEdit schon also solltest du dich am speed von PHPEdit orientieren.

mfg,
Nexis

[mirage228]

Zitat:

Zitat von Pr0g

Gibt es irgend eine Möglichkeit die Unit ohne Beigabe der Dll nutzen zu können. Durch eine andere Lib, oder sonstwas?

MfG Pr0g

Hi,

wenn Du mir eine Lib nennen kannst, die PCRE vollständig unterstützt und ohne DLL auskommst, dann wäre das möglich. :)
Das Problem mit TRegExpr (native Delphi) z.B. ist das einige PCRE Befehle nicht unterstüzt werden (Modifier /R für Rekursion z.B.).
Aber ich denke, dass die 200KB der PCRE.dll nicht viel ausmachen...

Zitat:

Zitat von DSeven

Guten Tag,

Also ich habe heute einen Test mit deinem Parser 1.3 gemacht und einer Datei mit 410 Kb. Es befinden sich da verschiedene Klassen und ein paar variablen und Funktionen.

Dein Parser hat 30 Sekunden gebraucht meiner 132 ms.
ALso du siehst du must noch einiges machen.

PHPEdit hat nur zwei sekunden mehr gebraucht.
OK mein Parser part nicht so viele sachen wie deiner, aber der von phpEdit schon also solltest du dich am speed von PHPEdit orientieren.

mfg,
Nexis

Also das Ergebnis mit der Version 1.3 überrascht micht nicht sehr. Dort wird bei jedem gefundenen Eintrag die Funktion CheckIsCommented() aufgerufen, die jedesmal den gesamten Quelltext nach Kommentaren und Strings(!) parsed. Das dauert auch so schon teils lange, aber die Funktion wurde je nach gefundenen Einträgen mehrmals aufgerufen. :?
Das hatte ich in der Version 1.4 verbessert. Damit kam ich auf einen Geschwindigkeitszuwachs von ~16000ms auf ~640ms in meinem Testbeispiel. Mit der Version 1.5 von gestern sollte es noch schneller gehen.

Weisst du zufällig auch, wo ich eine dementsprechend große PHP Datei herbekomme? Die größte, die ich hier habe ist ~75 KB.
Dann werde ich mir heute mal phpEdit runterladen und vergleichen...


Danke nochmal für euer Feedback :)

mfG
mirage228

[DSeven]

Guten Tag,

Wow die Version 1.5 brauch für das erste parsen nur ~900 ms und beim zweiten mal gehts noch viel viel schneller. Also das ist perfekt für den gebrauch. Ich glaube da brauchst du nicht mehr viel zu optimieren. Weil es eigentlich genauso wenn nicht schneller ist als bei PHPEdit so viel ich weis. PHPEdit parst die 450 KB große Datei nicht mal :D

Also die Datei habe ich mir selber zusammengestellt. Einfach eine klasse genommen die mir gefällt und immer wieder hintereinander rein kopiert. So teste ich meinen parser immer :D

Naja ok werde meinen Parser im Programm durch deinen ersetzen weil deiner PHP 5 Support hat und schneller ist. Muss das ganze nur noch auf VirtualTree umstellen und in einen Thread packen.

Echt klasse dein Parser hoffe das du weiter dran arbeiten wirst!!!!

@Edit: währe es eigentlich möglich ohne viel Zeit zu verlieren auch Variablen zu parsen die keinen klassen bezug haben wie in PHPEdit? Währe das letzte Feature was ich vermisse.
PHPDoc support währe auch klasse.

mfg,
DSeven

[Pr0g]

Zitat:

Zitat von mirage228

Hi,

wenn Du mir eine Lib nennen kannst, die PCRE vollständig unterstützt und ohne DLL auskommst, dann wäre das möglich. :)
Das Problem mit TRegExpr (native Delphi) z.B. ist das einige PCRE Befehle nicht unterstüzt werden (Modifier /R für Rekursion z.B.).
Aber ich denke, dass die 200KB der PCRE.dll nicht viel ausmachen...

Es geht mit eigentlich weniger um die 200Kb, sondern mehr darum, dass ich ungern eine zusätliche Dll dem Programm befüge, wenn es irgendwie eine Möglichkeit gibt dies zu vermeiden. Ich werde mich mal umschauen, ob ich eine passenden Lib finden kann.