Discussion :

[klakman]

Bonjour,

Un problème que je n'arrive jamais à résoudre, alors cette fois-ci au lieu de trouver des méthodes détournées pour essayer de passer outre ce problème, je vais l'aborder de plein fouet.

Mais j'ai beau réfléchir, je n'y comprends pas grand chose, et j'aurai besoin d'un peu d'aide pour faire les choses correctement.

Mon problème :

Je déclare une structure :

Code :

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

1
2
3
4
5
6
7
 
typedef struct {
 
	TCHAR *parametre1;             
	TCHAR *parametre2;         
 
} MaStr;

Je déclare ensuite un tableau :

Code :

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

1
2
 
MaStr *tableau = new MaStr[MAX_STRUCT];

J'ai ensuite une fonction qui est chargée de remplir ce tableau.
Donc je dois passer en paramètre de ma fonction un pointeur sur mon tableau. ( si je ne me trompe pas? )

voici le prototype de ma fonction :

Code :

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

1
2
 
int MaFonction( MaStr *(tableau)[MAX_STRUCT] );

Et l'appel :

Code :

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

1
2
 
MaFonction( &tableau );

Le problème se situe au niveau du passage en paramètre je pense. Car dans le corps de ma fonction, si j'utilise le pointeur, le programme compile mais plante.

Ma conclusion c'est donc que le pointeur a une mauvaise valeur. Du coup je tape dans un espace mémoire réservé... Pourtant je ne vois pas mon erreur.

Merci d'avance pour ceux qui pourront m'aider

[ram-0000]

le prototype :

Code :

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

int MaFonction( MaStr *tableau);

l'appel :

Code :

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

MaFonction( tableau );

Tout simplement (ou alors, j'ai pas bien compris)

[souviron34]

d'une part que vient faire ce "new" ?? Tu programmes en C++ ??


Une manière normale en C serait :

soit

Code C :

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

MaStr tableau[MAX_STRUCT];

soit

Code C :

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

MaStr *tableau= malloc(MAX_STUCT*sizeof(MaStr)) ;

Ensuite, c'est simple :

Code C :

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

1
2
3
4
5
6
7
int MaFonction ( MaStr **table) 
{
..
}
 
 
 i = MaFonction ( &tableau);

Qui ne marche que dans le 2ième cas (alocation).

Si tu dimensionnes réellement , alors c'est :

Code C :

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

1
2
3
4
5
6
7
int MaFonction ( MaStr *table) 
{
..
}
 
 
 i = MaFonction ( tableau);

[klakman]

Ça ne marche toujours pas.

J'ai utilisé :

Code :

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

1
2
 
MaStr *tableau= malloc(MAX_STUCT*sizeof(MaStr)) ;

Code :

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

1
2
3
4
5
6
7
8
 
int MaFonction ( MaStr **table) 
{
..
}
 
 
 i = MaFonction ( &tableau);

Et lorsque je fais dans le corps de ma fonction :

Code :

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

1
2
 
wcscpy_s( (wchar_t *)*(tableau)[index]->parametre1, sizeof(buffer), buffer ); // bug ici

Ça compile correctement, mais le programme plante à l'exécution.

A noter la modification suivante :

Code :

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

1
2
3
4
5
6
7
 
typedef struct {
 
	TCHAR parametre1[MAX_LENGHT];             
	TCHAR parametre2[MAX_LENGHT];         
 
} MaStr;

Il me semble avoir tout essayé, et j'ai plus d'idée sur l'origine du problème.

[Neitsa]

Un exemple:

Code C :

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
#define MAX_LENGTH 256
#define MAX_STRUCT 100
 
CONST TCHAR s1[] = _T("SENTENCE 1");
CONST TCHAR s2[] = _T("SENTENCE 2");
 
typedef struct {
 
	TCHAR parametre1[MAX_LENGTH];             
	TCHAR parametre2[MAX_LENGTH];         
 
} MASTR;
 
 
void foo(MASTR* tab)
{
	int i = 0;
	for(i = 0; i < MAX_STRUCT; i++)
	{
		_tcscpy_s(tab[i].parametre1, MAX_LENGTH, s1);
		_tcscpy_s(tab[i].parametre2, MAX_LENGTH, s2);
	}
}
 
int _tmain(int argc, _TCHAR* argv[])
{
	MASTR* tab = (MASTR*) malloc(MAX_STRUCT * sizeof(MASTR));
 
	foo(tab);
 
	free(tab);	
 
	return 0;
}

Le problème avec les tableaux de taille fixe (MAX_LENGTH) c'est qu'on perd une place folle si les chaînes sont plus courtes. Je ferais plutôt de façon à allouer à chaque fois la bonne taille.

L'inconvénient, dans ce dernier cas de figure, est que le code devient plus "complexe" étant donné qu'il faudra allouer et libérer pour chaque structure plutôt que globalement.

Petites remarques:

wcscpy(), c'est pour le type wchar_t, pas pour le type TCHAR (qui peut désigner soit des des caractères Unicode, soit MBCS).

Même chose pour le cast en wchar_t plutôt dangereux si on est en MBCS.

[jeroman]

Ça compile correctement, mais le programme plante à l'exécution.

Commençons déjà par une chose essentielle :
Si ton "new" passait à la compilation, c'est que manifestement tu compiles non pas en C mais en C++. Configure déjà ton projet de manière à ce qu'il compile en C, car un code C doit être compilé avec un compilo C.

[diogene]

Code :

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

1
2
3
4
int MaFonction ( MaStr **tableau) 
{
 ....
wcscpy_s( (wchar_t *)*(tableau)[index]->parametre1, sizeof(buffer), buffer ); // bug ici

Le problème est sur l'argument *(tableau)[index]->parametre1 :

1- on devrait avoir (*tableau) pour avoir un pointeur sur le tableau de MaStr

2- Un élément de ce tableau est alors (*tableau)[index]

3- Cet élément est une structure. Pour accéder à son membre parametre1 : (*tableau)[index].parametre1

[klakman]

En effet, je perds beaucoup de place, mais ça permet aussi de sécuriser mon application en évitant les chaines trop longues.

De plus c'est une petite application, donc sauf si le PC date de 1995, donc la mémoire n'est pas trop mon soucis premier ici.

Mais je note et je retiendrais pour une application plus gourmande.

wcscpy(), c'est pour le type wchar_t, pas pour le type TCHAR (qui peut désigner soit des des caractères Unicode, soit MBCS).

Tu as encore raison, mais comme je code en Unicode, TCHAR = wchar_t. Mais je devrais être plus rigoureux là dessus à l'avenir, merci du conseil.

( J'avoue que c'est un peux pénible leur trus TCHAR = wchar_t ou TCHAR = char* )

Pour tout ceux qui m'ont fait la remarque pour les bouts de code C++, oui je code en C++.
Je suis venu sur le forum C, car il me semble que se problème relevait d'avantage du C "pur" plutôt que d'un problème objet, ou inhérent au C++.
Et je pense vraiment que le problème n'a rien à voir avec un mot clé C++ dans mon code, ou alors un compilo C++ qui serait ( je ne sais pas pourquoi ) incapable de compiler du C.


Pour finir, merci à vous tous, et surtout à diogene qui m'a fourni la solution sur un plateau!

Ça marche parfaitement.