Discussion :

[frantzgac]

Bonjour

j'ai lu les différentes contributions sur ce sujet sans toujours faire les mêmes constatations que mes prédécesseurs. Je rapporte ici un cas surprenant de divergences de comportement du même code entre deux projets différents que j’appellerai ANCIEN et NOUVEAU. la raison est que ANCIEN est un projet existant développé sous C++6 tandis que nouveau a été créé sous C++9. Les deux projets sont cependant executés sous C++9. L'ANCIEN a donc été converti automatiquement par le RAD.

Je précise que j'ai bien sûr essayé de faire varier la fameuse option "Mappage du _TCHAR" sans que cela fasse varier les observations suivantes :

Prenons le code inoffensif et fonctionnel sous C++ Builder 6

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
MessageBox(Application->Handle,
(Format("%s ",ARRAYOFCONST(("Toto")))).c_str(),
   "Erreur",
   MB_OK | MB_ICONERROR);

il ne se compile pas sous C++Builder 9 à cause du c_str(). Je le remplace donc par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
MessageBox(Application->Handle,
(Format("%s ",ARRAYOFCONST(("Toto")))).t_str(),
   "Erreur",
   MB_OK | MB_ICONERROR);

La compilation passe ... mais à l’exécution :

le projet NOUVEAU affiche

Toto

le projet ANCIEN affiche

????

Je présume que l'ouverture/conversion d'un projet C++6 sous C++9 entraîne quelques choix d'options qui expliqueraient ce comportement. Néanmoins je ne vois pas lesquelles ?

Après quelques recherches sur le forum je tente

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
MessageBox(Application->Handle,
AnsiString().Format("%s ",ARRAYOFCONST(("Toto"))).c_str(),
   "Erreur",
   MB_OK | MB_ICONERROR);

qui fonctionne depuis les deux projets.

Je pense à la lecture de la doc) que la fonction Format est surchargée, et que par conséquent ce n'est pas le même code qui est compilé dans les deux projets. Mais comment savoir quelle déclinaison est employée ?

D'ordinaire la discrimination de fonctions surchargées se fait par les paramètres d'appel. Mais ici ce sont les mêmes. Je pense aussi à l'inclusion automatique si elle se fait sur les composants visuels, elle n'a pas lieu sur de simples fonctions même de classe... ?

NB : j'ai réédité ce message après relecture car j'ai constaté après une nuit de sommeil qu'il était incompréhensible en l'état. Désolé pour ceux qui l'ont lu avant la mise à jour...

[ShaiLeTroll]

MessageBox, quelle idée !
Où je travailles, ils ont fait pareil par méconnaissance de l'outil C++ Builder !

Format peut importe le mappage fonctionne en String Delphi !
Donc AnsiString en D6 et UnicodeString en D2009 tout comme en XE2

Format est la fonction Delphi
AnsiString().Format est la méthode d'encapsulation dans l'objet C++ de la fonction Delphi

Ce n'est pas une histoire de surcharge mais une question de portée (Fonction ou Méthode) et de d'unité (~namespace) SysUtils.pas ou dstring.h
Il existe aussi la fonction Format dans System.AnsiStrings

Le Mappage ne concerne que les API Windows (MessageBoxA en l'occurence) et le type _TCHAR ou la macro _TEXT

Deplus, t_str est la pire erreur d'Embarcadero !
Migre au plus vite en XE2 car 2009 est, je pense, à considérer comme la version bêta pour l'Unicode !
t_str n'a pas vécu longtemps, apparu en 2009 et déprécié en 2010
Ton code n'aura pas le même comportement entre 2009 et XE2 avec t_str !

Il existe une fonction prévu pour cela dans la RTL\VCL : MessageDlg

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
MessageDlg(
  Format("%s ", ARRAYOFCONST(("Toto"))),
  mtError, TMsgDlgButtons() << mbOK, 0);

Un conseil évite le plus que possible d'utiliser c_str(), tu ne devrais y avoir besoin uniquement lors d'utilisation d'API Windows !
Avant vérifie si il n'existe pas dans la RTL une encapsulation comme TIniFile ou TRegistry
Je te dis ça parce que le code que je maintiens mélange ces objets et les API, du coup du char* et c_str() partout très pénible à maintenir

Avec une bonne encapsulation, ton code ne devrait jamais appelé l'API directement mais passer par un Wrapper ou une Classe, ainsi tu peux plus facilement géré et centralisé les appels !