[Ben]

Hallo,
ich habe einen kleinen Fehler gefunden.

Wenn man einen Block mit geschweiften Klammern erstellt hat und beide Klammern existieren, also sowas hier

PHP-Code:

if(isset($var)) {
} 


dann nochmal in die if-Zeile hinter die Klammer geht und auf ENTER drückt, wird eine neue schließende Klammer eingefügt, obwohl diese gar nicht benötigt wird.
Ist auch der Fall, wenn die öffnende Klammer unter dem if steht

PHP-Code:

if(isset($var)) 
{
} 


Der Fehler ist reproduzierbar!

Ich nutze die Version Alpha 4.

[pago]

Jopp.

Analyse an der Stelle ist'n bisschen schwierig, weil ich dazu die ganze Datei nach Klammern durchsuchen müsste. Aber das ganze einfach nur einzurücken find ich wiederum etwas zu wenig. So wie andere Editoren einfach direkt hinter eine eingegebene "{" eine "}" zu setzen halte ich aber auch für wenig sinnvoll.
Kurzum: Irgendeinen guter Vorschlag auf Lager?

P.S.: Danke für's melden.

[Ben]

Ah, sorry.

Also ich hätte jetzt gedacht, dass das gar nicht so aufwendig wäre, da doch an der linken Seite (bei den Zeilennummern) auch angezeigt wird, welche öffnende zu welcher schließenden Klammer gehört.

Kannst du nicht prüfen, ob eine öffnende schon eine schließende hat und nur dann eine neue setzen?

Hm, ok.... merke gerade, dass man das ja auch aushebeln kann. Hm ..

[Artemis]

Hmm, es dürfte aber nicht so aufwendig sein.

Was mir gerade einfällt. Bei PHPeclipse gab/gibt es diesen "Bug" auch. PDT macht es aber so wie erwünscht. Vielleicht kannst du dich da ja inspirieren?

[mepeisen]

Zitat von Artemis

PDT macht es aber so wie erwünscht. Vielleicht kannst du dich da ja inspirieren?

Ne, gibt ein paar Sonderfälle, wo es das ebenfalls nicht korrekt hinbekommt
Zu SimpleEdit kann ich leider nix beitragen.

[pago]

Das Problem dabei ist im Prinzip, dass ich nen kompletten AST (Abstract Syntax Tree - Wikipedia, wenn's interessant klingt) brauche, d.h. ich muss das Teil parsen und gucken, wo syntaktisch die "}" fehlt - falls überhaupt eine fehlt.

Das heißt schonmal, dass ich das nicht generell für alle sinnvoll und brauchbar lösen kann. Ich kann natürlich ein paar Fälle dadurch abdecken, dass ich einfach die "{" und "}" zähle und nicht ergänze, wenn die Zahl identisch ist. Das hilft in ein paar Fällen, aber geht schon den Bach runter, wenn die Sprache, die man benutzt, Kommentare besitzt.

Für PHP im speziellen habe ich so einen AST zur Verfügung.
Jedenfalls sähe die Lösung so aus, dass ich den Parser drüberjage, die Fehlermeldungen analysiere (if missing token == "}"...), kontrolliere, ob die Positionen hinkommen könnten (gibt es zwischen der aktuellen Zeile und der fehlenden "}" keine einzige "{" ohne "}"?). Ich geh stark davon aus, dass ich selbst damit immer noch nicht alle Fälle abdecke.

Bin also etwas ratlos ob einer besseren (effizientieren, portableren) Lösung.


Wie ist denn die Meinung der Allgemeinheit zum Vorschlag, statt die "}" zu ergänzen, einfach nur bei "{" einzurücken und bei "}" eben uneinzurücken? Also so wie's bei jEdit ist.

[Ben]

Wie gesagt: Ne Lösung habe ich ja nicht, aber wenn PDT das jedenfalls teilweise realisiert hat, dann würd ich da auf jeden Fall mal reinschauen.

Zitat von pago

Wie ist denn die Meinung der Allgemeinheit zum Vorschlag, statt die "}" zu ergänzen, einfach nur bei "{" einzurücken und bei "}" eben uneinzurücken? Also so wie's bei jEdit ist.

Kann mir gerade nicht ganz vorstellen, was du machen würdest.

Quasi wenn ich { eintippe wird im Quelltext eingerückt, aber keine } eingefügt?
Hm, nä. Das fänd ich auch nicht so wirklich toll.