Discussion :

[nicolo011]

Bonjour,

Je suis à la recherche d'un outil me permettant une automatisation de tâche de refactoring de code C++.
Je dispose de nombreuses variables accédées directement (en public) dans un projet.
Je souhaite passer ces variables en private, et donc remplacer :

• les accès en lecture par des appels aux nouvelles méthodes Get() correspondantes
• les accès en écriture par des appels aux nouvelles méthodes Set() correspondantes (avec un argument qui est la valeur à donner, actuellement après le "=" de l'affectation)

Je cherche donc un outil/script qui fasse :
- recherche des occurrences de ma variable
- distinction entre accès en lecture et en écriture
- remplacement

ça sent bien les expressions régulières.

Mais je me dis qu'un tel outil existe certainement déjà...

Merci d'avance pour vos réponses

Nicolo

[3DArchi]

Salut,
Mon avis : si c'est pour rajouter Get/Set, pourquoi ne pas les laisser en public ?
Sinon, beaucoup d'I.D.E ont des options de refactoring : avec lequel est ton projet ? Une solution intégrée est peut être possible.

[Luc Hermitte]

Mon avis : si c'est pour rajouter Get/Set, pourquoi ne pas les laisser en public ?
Sinon, beaucoup d'I.D.E ont des options de refactoring : avec lequel est ton projet ? Une solution intégrée est peut être possible.

Pareil x 2.
1- Si c'est juste pour encapsuler, il ne faut pas se mentir, Déméter ne s'en portera pas mieux: le code que tu maintiens est pensé en données et non en services, et c'est déjà trop tard pour faire de l'OO joli/propre, véritable penseront certains.

2- Au pire, remplace tes attributs dans la classe, fabrique toi deux macros pour transformer en get/set, et finalement compile+applique les macros jusqu'à ce que tu n'aies plus rien oublié.

[Emmanuel Deloget]

C'est un refactoring particulier dont tu as besoin là. En fait, il vaut mieux casser ce refactoring en plusieurs étapes :

* création de la méthode GetX
* replacement de X par GetX là ou ça fait sens
* création de la méthode SetX
* remplacement des derniers X par SetX
* passage de X en private

Il y a un problème toutefois : si ton code contient des lignes du genre

a = X = c;

Alors tu ne peut remplacer X ni par un GetX, ni par un SetX (à moins que SetX ne renvoie la nouvelle valeur).

Je te conseille d'utiliser Eclipse et ses refactoring tools - ils sont puissants, même en C++. Certes, il ne feront pas tout le travail, mais il t'en laisseront moins à faire toi même.