Discussion :

[Erakis]

Bonjour,

Je developpe une application sous Windows CE.
J'ai un petit problème avec cette ligne, je ne trouve pas la solution.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void CHeaderCtrlEx::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
{
    ASSERT(lpDrawItemStruct->CtlType == ODT_HEADER);

    HDITEM hdi;
    TCHAR  lpBuffer[256];

    hdi.mask = HDI_TEXT;
    hdi.pszText = lpBuffer;
    hdi.cchTextMax = 256;

    ::DrawText(lpDrawItemStruct->hDC, lpBuffer, strlen(lpBuffer), &lpDrawItemStruct->rcItem, uFormat);
}

Voici l'erreur du compilateur :

Erreur 1 error C2664: 'strlen'*: impossible de convertir le paramètre 1 de 'TCHAR [256]' en 'const char *'

[stephdim]

salut,

tu mixes les TCHAR et les char ... ça se trouve t'es en unicode et il veut des wchar_t

utilises _tcslen(lpBuffer)

char -> strlen()
wchar_t -> wcslen()

et

TCHAR -> _tcslen

qui est une macro qui dirige soit vers strlen, soit wcslen
suivant les directives de compilation

idem pour TCHAR qui est soit char, soit wchar_t

et il y a un paquet d'autres macros comme ça , toutes commencent par _t (voir include TCHAR.H)

@+

[bla69136]

Problème assez classique ...
A bien y regarder, lpBuffer n'est pas un pointeur sur des char mais un tableau de char et c'est celà que le compilateur dit.
Il suffit de caster lpBuffer en const char *.
Ceci étant dit, la remarque de setphdim sur l'utilisation de _tcslen est tout à fait judicieuse.
Je suggère donc :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void CHeaderCtrlEx::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
{
    ASSERT(lpDrawItemStruct->CtlType == ODT_HEADER);
 
    HDITEM hdi;
    TCHAR  lpBuffer[256];
 
    hdi.mask = HDI_TEXT;
    hdi.pszText = lpBuffer;
    hdi.cchTextMax = 256;
 
    ::DrawText(lpDrawItemStruct->hDC, (const TCHAR *)lpBuffer, _tcslen((const TCHAR *)lpBuffer), &lpDrawItemStruct->rcItem, uFormat);
}

[Spout]

Il suffit de caster lpBuffer en const char *

Avoir une fonction avec un paramètre en const char* ne signifie pas que le paramètre doit obligatoirement être de type const. Cela veut juste dire que l'objet passé ne sera pas modifié par la fonction.

@Erakis:
D'après ce que' j'ai vu de la fonction DrawText, le buffer passé en paramètre doit contenir du texte, ce que je ne vois pas dans ton code: tu lui donnes un tableau de char avec des valeurs indéfinies.
De plus, pour les 2 et 3e paramètres, tu n'as que 2 combinaisons possibles:

• Ton tableau de char contient du texte terminé par 0. Le 3e paramètre peut alors être -1 car la fonction s'arrêtera au 0 lors de la lecture du buffer.
• Ton tableau de char contient du texte qui n'est pas terminé par 0. Le 3e paramètre doit alors contenir la taille exacte du texte pour savoir ou s'arrêter dans la lecture du buffer.

En aucun cas tu peux faire un _tcslen() sur un buffer qui n'est pas terminé par 0, car la fonction va compter les caractères jusqu'au 0. S'il n'y en a pas avant la fin de ton buffer (256 caractères), tu auras un dépassement de mémoire et une erreur (CRT je crois).

[3DArchi]

Le dépassement mémoire peut passer inaperçu car si on reste dans un espace adressable (et en lecture seule), aucune erreur ne sera générée. Mais le risque existe.

[Erakis]

@spoutspout : Désolé, le contenu de mon tableau n'est pas indéfinie, c'est que j'ai retiré le code non utile pour corriger mon problème de compilation. Voici ce qui manque au début...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void CHeaderCtrlEx::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
{
   ASSERT(lpDrawItemStruct->CtlType == ODT_HEADER);

   HDITEM hdi;
   TCHAR  lpBuffer[256];

   hdi.mask = HDI_TEXT;
   hdi.pszText = lpBuffer;
   hdi.cchTextMax = 256;

   GetItem(lpDrawItemStruct->itemID, &hdi);
	
   CDC* pDC;
   pDC = CDC::FromHandle(lpDrawItemStruct->hDC);

   //DRAW THE TEXT
   ::DrawText(lpDrawItemStruct->hDC, (const TCHAR *)lpBuffer, _tcslen((const TCHAR *)lpBuffer), &lpDrawItemStruct->rcItem, uFormat);

   ...

Bref, j'ai toujours de la misère à démêler tout ces LPCSTR, WCHAR, _TSCHAR, LPWSTR, etc... Quelle misère !

Merci à tous pour votre aide.

[bla69136]

Ceci étant dit, ce que disait spoutspout:

Avoir une fonction avec un paramètre en const char* ne signifie pas que le paramètre doit obligatoirement être de type const. Cela veut juste dire que l'objet passé ne sera pas modifié par la fonction.

est tout à fait vrai ( honte à moi sur ce coup), il suffit de faire un cast en TCHAR *

Code :

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

::DrawText(lpDrawItemStruct->hDC, (TCHAR *)lpBuffer, _tcslen((TCHAR *)lpBuffer), &lpDrawItemStruct->rcItem, uFormat);

le cast que je t'avais proposé signifiant littéralement que lpBuffer est un pointeur sur un(des) caractere(s) constant(s).
On peut également contourner ce genre d'écueil en utilisant &lpBuffer[0]. Dans tous les cas, il serait bon de renommer ton tableau pour ne pas "enduire le futur mainteneur avec de l'erreur". Ca pourrait donner :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
void CHeaderCtrlEx::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
{
   ASSERT(lpDrawItemStruct->CtlType == ODT_HEADER);
 
   HDITEM hdi;
   TCHAR  buffer[256];
 
   hdi.mask = HDI_TEXT;
   hdi.pszText = &buffer[0];
   hdi.cchTextMax = 256;
 
   GetItem(lpDrawItemStruct->itemID, &hdi);
 
   CDC* pDC;
   pDC = CDC::FromHandle(lpDrawItemStruct->hDC);
 
   //DRAW THE TEXT
   ::DrawText(lpDrawItemStruct->hDC, &buffer[0], _tcslen( &buffer[0]), &lpDrawItemStruct->rcItem, uFormat);
 
   ...

[stephdim]

il suffit de faire un cast en TCHAR *

On peut également contourner ce genre d'écueil en utilisant &lpBuffer[0].

non c'est inutile.
ce qu'avait fait Erakis était correct. (peut etre juste le nom de la variable qui était trompeur ... )

Code :

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

1
2
3
4
 
TCHAR buffer[256];
LPCTSTR lpBuffer=buffer;  // et pas de cast ni de &[0]
LPTSTR lpBuffer2=buffer;  // version non const

et pour

Code :

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

1
2
 
hdi.cchTextMax = 256;

c'est préférable de faire ça : (macro très utile)

Code :

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

1
2
3
 
// compte du nombre d'éléments d'un tableau
#define countof(array) (sizeof(array)/sizeof(array[0]))

puis

Code :

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

1
2
 
hdi.cchTextMax = countof(buffer);

ça évite de se planter, ou de voir des sizeof() par erreur (qui serait faux pour du wchar_t)

mais bon, les gouts et les couleurs ça ne se discutent pas

@+

[Spout]

Code :

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

1
2
3
 
// compte du nombre d'éléments d'un tableau
#define countof(array) (sizeof(array)/sizeof(array[0]))

+1
C'est exactement ce que j'utilise. Ca évite bien des erreurs... Attention toutefois lorsque des pointeurs entrent en jeu dans l'objet du tableau.

&lpBuffer[0]

Question piège: quelle est la différence entre &lpBuffer[0] et lpBuffer?

[farscape]

c'est pareil ...
l'adresse du premier caractère , c'est l'adresse du buffer.

[Spout]

c'est pareil ...
l'adresse du premier caractère , c'est l'adresse du buffer.

Exactement .
D'où l'inutilité de l'écriture de &lpBuffer[0] car, au final, qu'est-ce qu'un tableau? C'est un pointeur.