Refactoring - Bezeichner umbenennen (F2)

[Joh]

Moin,

bei umbenennen von Variablen via F2 werden diese sogar in auskommentierten Bereichen umbenannt. => Finde ich gut.
Aber wenn ich deaktivierte Bereiche mit {$IFDEF ...} habe, beiben diese alten Bezeichner stehen. gibt es dafür irgendeinen grund oder ist das ein Bug?

Gerade im aktuellen Fall blöd:
ich wollte eine neue Variable zeile (int) einfügen; spalte (int) gabs schon. Aber es gab eine Variable zeile als String... Diese umbenannt als zeilenText, wie es eigentlich auch sein sollte; aber in dem Bereich mit Kompilerdirektive {$ifdef wird nix umbenannt.
Das fällt einem latürnich später auf die Füße, wenn man die Kompilerdirektive (de)aktiviert.

lg
Jochen

[anse]

Kann ich bestätigen - deaktivierte Code-Bereiche zwischen IFDEF's werden beim Umbenennen mit F2 nicht abgearbeitet.

Vorkommen in {} Kommentaren werden dabei übrigens umbenannt, obwohl das für mich noch weniger Sinn ergibt.

[af0815]

Vorkommen in {} Kommentaren werden dabei übrigens umbenannt, obwohl das für mich noch weniger Sinn ergibt.

Ist sinnvoll, weil auch Objektnamen zwischen den Kommentarklammern vorkommen.

Das bei IFDEFs nicht geht - ist per Design - dazu gabs schon öfter Diskussionen. Vor allen dann ein Problem wenn da was umbenannt wird, wenn man größere Refactorings macht oder für verschiedene Plattformen IFDEFs hat.

Nur so nebenbei, es ist nicht Zeile umbenennen. Es heist genaugenommen "Find or rename Identifier". Daher finden oder umbennen von Bezeichnern.

[PascalDragon]

Aber wenn ich deaktivierte Bereiche mit {$IFDEF ...} habe, beiben diese alten Bezeichner stehen. gibt es dafür irgendeinen grund oder ist das ein Bug?

Das ist absichtlich so, da ein deaktiviertes $IFDEF Code enthalten könnte, den die CodeTools nicht verstehen (zum Beispiel nicht unterstützte Sprachfeatures aus Delphi) und dies dann letztlich zu einem Absturz führen könnte.

[Joh]

Nur so nebenbei, es ist nicht Zeile umbenennen. Es heist genaugenommen "Find or rename Identifier". Daher finden oder umbennen von Bezeichnern.

uups... Da lag mir im Kopf, das ich die Variable zeile umbenennen wollte...

Ich hab das Thema mal umbenannt!

[Joh]

Das ist absichtlich so, da ein deaktiviertes $IFDEF Code enthalten könnte, den die CodeTools nicht verstehen (zum Beispiel nicht unterstützte Sprachfeatures aus Delphi) und dies dann letztlich zu einem Absturz führen könnte.

ok... wenn da Sinn dahintersteckt. Ich nutze die $IFDEFs allerdings zur Entwicklungszeit auch, um aktuell irrelevanten Code nicht auszuführen: das fällt mir dann besser auf, als ein im Code versteckten if false.

Andererseits wäre eine 2. Option zu
- auch in Kommentaren suchen
, wie
- auch in {IFDEF}-Bereichen suchen
eventuell angebracht.