Discussion :

[tapiou]

Bonjour,

Je dois migrer un projet de Visual Studio 2005 à 2008 (avec un changement de la version du SDK de V5 à V6.0A) et il y a un bout de code qui ne compile plus :

Code :

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

1
2
3
4
5
6
7
8
9
10
    wchar_t WDirectoryToShare[MAX_PATH];
    wchar_t WSharename[MAX_PATH];
 
    swprintf_s(WDirectoryToShare,L"%S",DirectoryToShare);
    swprintf_s(WSharename,L"%S",Sharename);
 
    si502.shi502_netname = (char *)WSharename;     // ici erreur de compilation
    si502.shi502_max_uses = SHI_USES_UNLIMITED;
    si502.shi502_current_uses = 0;
    si502.shi502_path = (char *)WDirectoryToShare;	 // ici erreur de compilation

J'ai parcouru les 3 fichiers du SDK concernés et il y a le typedef suivant :

Code :

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

 typedef __nullterminated WCHAR *NWPSTR, *LPWSTR, *PWSTR;

puis

Code :

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

#define LMSTR   LPWSTR

et un typedef struct :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
typedef struct _SHARE_INFO_502 {
    LMSTR     shi502_netname;
    DWORD     shi502_type;
    LMSTR     shi502_remark;
    DWORD     shi502_permissions;
    DWORD     shi502_max_uses;
    DWORD     shi502_current_uses;
    LMSTR     shi502_path;
    LMSTR     shi502_passwd;
    DWORD     shi502_reserved;
    PSECURITY_DESCRIPTOR  shi502_security_descriptor;
} SHARE_INFO_502, *PSHARE_INFO_502, *LPSHARE_INFO_502;

Le compilateur n'admet pas donc pas le cast d'un wchar_t en WCHAR.
Il met 2 erreurs de compilation du type :
error C2440: '=' : cannot convert from 'char *' to 'LPWSTR'

Que dois-je faire alors ?

J'ai joint les 3 fichiers du SDK concernés.

Merci.

Tapiou.

[Rewpparo]

Code :

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

si502.shi502_netname = (char *)WSharename;

netname est une LPWSTR, il ne faut pas caster en char* (ce qui est une très mauvaise facon de faire dans tous les cas)

Code :

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

si502.shi502_netname = WSharename.c_str();

[tapiou]

Merci bcp une fois de plus

Tapiou.

[tapiou]

Bonjour,

Hélas çà ne compile pas avec l'erreur suivante :
error C2228: left of '.c_str' must have class/struct/union
type is 'wchar_t [260]'

Je ne sais pas si Rewpparo est là alors si quelqu'un veut bien m'aider, son aide sera la bienvenue.

Merci.

Tapiou.

[dragonjoker59]

Et bien tu as donc une chaine WSharename qui est déclarée en tant que wchar_t [] donc qui est déjà un LPWSTR. Tu as donc juste à enlever le cast pour que ça fonctionne

[tapiou]

Merci beaucoup, çà marche.

Mais un peu plus loin, j'ai la même erreur de compilation sur

Code :

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

1
2
3
4
5
6
7
 
nas = NetShareAdd(
	    (Server ? (char *)WServer : Server),
             502,           
            (LPBYTE)&si502,
            NULL            
            );

L'erreur est :

"error C2664: 'NetShareAdd' : cannot convert parameter 1 from 'char *' to 'LPWSTR'
Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or function-style cast"

alors j'ai retiré le (char *) devant WServer mais là, j'ai l'erreur de compilation :
"error C2446: ':' : no conversion from 'LPTSTR' to 'wchar_t *'
Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or function-style cast"

Il ne veut pas de char et il ne veut pas de wchar_t.

Je ne comprends pas ce qu'il veut.

D'avance, merci ...

Tapiou.


Nota : NetShareAdd est défini dans LMShare.h joint en début de topic comme çà :

Code :

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

1
2
3
4
5
6
7
 
NetShareAdd (
    __in_opt IN  LMSTR   servername,
    IN  DWORD   level,
    __inout IN  LPBYTE  buf,
    __out_opt OUT LPDWORD parm_err
    );

[Rewpparo]

La variable Server c'est quoi ?

[tapiou]

argh réveil nocturne

LPTSTR Server;

[Rewpparo]

"error C2446: ':' : no conversion from 'LPTSTR' to 'wchar_t *'

Bizarre. Normalement LPTSTR est un wchar_t*.
Tu compiles bien avec l'option unicode ?

[dragonjoker59]

(Server ? WServer : Server)Je pense que ce test est faux :
Si Server n'est pas nul on prend WServer mais s'il est nul c'est Server (donc NULL) qu'on prend.
Ne voulais-tu pas faire l'inverse plutôt (ce qui me semblerait plus logique)
(Server ? Server : WServer)Par contre ça ne résoudra pas ton problème de conversion.
Je te suggère de vérifier si tu compiles bien en Unicode, comme te l'a dit Rewpparo.

[tapiou]

Merci tous les 2.

Mais je compile bien en Unicode :
Je suis sous Visual 2008 et j'ai Yes dans "Use UNICODE Response Files" du champ General de Linker.

Je ne dois pas changer le test même s'il est faux (...) mais juste compiler sous Visual 2008 ce qui compilait avant sous Visual 2005.

Et cette ligne, donc :
(Server ? (char *)WServer : Server),

ne compile pas en me disant :
error C2664: 'NetShareAdd' : cannot convert parameter 1 from 'char *' to 'LPWSTR'

Gloups !

Tapiou.

[Rewpparo]

Déjà enlèves le char*, ca c'est sur. C'est juste débile de caster un wchar_t* en char* sans traitement, même les caractères français comme é buguent avec ca.
Si ca compile toujours pas, je vois vraiment pas, normalement ca marche. Un message comme "cannot convert parameter 1 from 'wchar_t *' to 'LPWSTR'" n'a pas de sens car LPWSTR est un wchar_t*.
Peut etre un include qui manque ? windows.h ?

[tapiou]

Salut Rewpparo et dragonjoker59,

Pour windows.h, le include est entre commentaires en haut du fichier mais avec sans, même erreur de compil.

Mais j'ai mis :

Code :

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

1
2
3
4
5
6
nas = NetShareAdd(
          (Server ? WServer : (LPWSTR)Server),         
           502,         
          (LPBYTE)&si502,
           NULL            
        );

et là çà compile.

Qu'est-ce que vous en dites ?

Sinon dragonjoker59, je vais dire à mon chef que le test est peut-être faux ... çà pourrait servir de le savoir

Tapiou.

[Rewpparo]

Oui, c'est normal que ca marche. Le problème c'est que ca ne résout rien, ca cache juste le message d'erreur à la compilation. Le cast du C ne garanti pas du tout la compatibilité des données entre elles, et il vaut mieux le remplacer par static_cast (ou un autre *_cast) quand c'est possible.
En même temps le code contenait à la base un cast vers char*, ce qui n'est pas beaucoup mieux de toutes facons, donc bon....

[mala92]

Je dois migrer un projet de Visual Studio 2005 à 2008 (avec un changement de la version du SDK de V5 à V6.0A)

Salut,
J'arrive un peu tard, mais là tu cummules 2 migrations : IDE (compilo) + SDK.

Il aurait fallu y aller par étapes :
- Origine : VS 2005 + SDK 5 (quelle horreur !!!)
- 1er étape : VS 2005 + SDK 6.0A
- 2e étape : VS 2008 + SDK 6.0A

Généralement, un changement de SDK n'est pas très problématique.
Par contre, le changement de version de Visual, c'est plus contraignant.

[tapiou]

Bonjour Rewpparo,

Merci pour ta réponse : je vais me plonger dans les static_cast pour comprendre ce que devrais mettre à la place de

Code :

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

   (Server ? WServer : (LPWSTR)Server),

sinon mala92, je vais dire à mon chef qu'il est nul mais il risque de pas apprécier ....

Tapiou.

[dragonjoker59]

Moi j'aimerais quand même bien voir la ligne de déclaration de WServer et Server.
Ensuite, le champ "Use UNICODE Response Files" ne définit pas la compilation en Unicode mais juste que les fichiers générés (logs, toussa) seront en Unicode.
Pour savoir si tu compiles en Unicode, il faut aller voir dans les propriétés du projet, "Général", "Jeu de caractères".

[tapiou]

dragonjoker59,

Dans les propriétés du projet, "Général", "Jeu de caractères" : j'ai mis maintenant Use Unicode Character Set (c'était à Not set avant) et du coup

Code :

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

(Server ? WServer : Server),

compile

Merci !

Avec des dizaines d'autres projets que j'ai migré de Visual Studio 2005 à Visual Studio 2008, qui ont tous Use Unicode Character Set à la valeur Not set, je n'ai pas eu de problème.

Qu'est-ce qu'il faut que je fasse ? Est-ce que c'est grave si ils n'ont pas "Use Unicode Character Set" de positionné ?

Merci.

Tapiou.

[Rewpparo]

Il y a un type qui est pas mal utilisé dans les API Windows, c'est le TCHAR. Si tu compiles sans unicode activé, TCHAR est un char. Avec Unicode, ca devient wchar_t.
LPTSTR est un TCHAR* et est apparemment utilisé pas mal dans ton code. Si tes données sont des char* majoritairement, il vaut mieux ne pas utiliser Unicode. si c'est des wchar_t* majoritairement, utilises Unicode.
Mais l'idéal si tu as plusieurs projets c'est d'avoir la même convention pour tous. Donc je te conseille d'en choisir un et de t'y tenir, éventuellement avec les conseils de ton boss.
Mais dans les problèmes qu'on a résolu avant, il me semble que tu étais avec Unicode activé. Vérifie tes projets, mais je pense que dans ton cas il vaut mieux utiliser Unicode.

[tapiou]

Merci

Tapiou.