Discussion :

[Topeur]

Bonjour a tous,

Je suis a la recherche d'une explication au sujet de l'allocation.
Je sais allouer un tableau de structure dans mon main, mais quand je cherche à faire la même chose dans une fonction, je rencontre des erreurs

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
22
23
24
25
26
27
28
29
30
 
typedef struct
{
   int number;
}MOY;
 
 
void AllocMOY( MOY **moy )
{
   int qte = 10;
   moy = /* (MOY*) */ malloc( qte * sizeof(MOY) );
   moy[2]->number = 10; //Ici, erreur de violation d'acces !
 
   /* Utilisé ca aussi, mais ceci provoque une erreur au moment du printf en sortie de fonction
   moy= malloc (11 * sizeof(MOY*) );
   for(i=0; i<11; i++)
   {
	moy[i] = malloc(sizeof(MOY));
   }
 
   */
}
 
 
void main()
{
   MOY *moy;
   AllocMOY( &moy );
   printf("%d",moy[2]);
}

En quoi ce code ne fonctionne pas ?
Ou ai je fais une erreur ?
Je ne trouve pas de solution, c'est pourquoi je viens demander votre aide
Merci d'avance pour vos reponses !

[poukill]

Euh, t'es sûr que c'est dans la rubrique C++ que tu voulais poster ?
malloc, printf, c'est que du C ça !

Dis moi et je déplace si besoin!

[JolyLoic]

moy est un pointeur sur un pointeur sur un MOY. Ce code est absolument non typique de ce qu'on ferait en C++. Pour le corriger à très court terme :

Code :

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

1
2
3
4
5
6
void AllocMOY( MOY **moy )
{
   int qte = 10;
   *moy = (MOY*) malloc( qte * sizeof(MOY) );
   (*moy)[2].number = 10;
}

Maintenant, en tenant compte du fait que la structure MOY peut évoluer, et que faire malloc dessus risque de ne plus être correct :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
struct MOY
{
   int number;
};
 
 
void AllocMOY( MOY **moy )
{
   int qte = 10;
   *moy = new MOY[qte];
   (*moy)[2].number = 10;
}

Ensuite, on utilisera en C++ le passage par référence, plutôt que de passer un pointeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void AllocMOY( MOY *&moy )
{
   int qte = 10;
   moy = new MOY[qte];
   moy[2].number = 10;
}
int main() // Main DOIT retourner un entier
{
   MOY *moy;
   AllocMOY( moy );
   cout << moy[2]; // cout est plus simple à utiliser que printf
}

Et enfin, de toute manière, en C++, on n'alloue généralement pas de mémoire manuellement. C'est trop aisé de se tromper.

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
#include <vector>
 
using namespace std;
 
vector<MOY> AllocMOY()
{
   int qte = 10;
   vector<MOY> result(qte);
   moy[2].number = 10;
}
int main()
{
   vector<MOY> moy = AllocMOY();
   cout << moy[2];
}

[Topeur]

Veuillez m'excuser d'avoir poster dans le forum C++ (au depart, depuis ca a été deplacé !)
Je travaille en C++ mais la personne qui avait besoin de cette reponse bosse elle en C.

J'ai testé et ca marche, en revanche j'aurai encore une tout petite question sur votre solution en C

Pourquoi ceci marche

Code :

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

(*moy)[2].number = 10;

Mais pas ca

Code :

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

moy[2]->number = 10;

Cela normalement reviens au même je pensais et pourtant, avec la flèche, cela plante le code.
Merci pour votre rapidité de reponse en tout cas

[diogene]

Code :

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

(*moy)[2].number = 10;

implique que moy est un pointeur sur un tableau de structures

Code :

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

moy[2]->number = 10;

implique que moy est un tableau de pointeurs sur structure

Ce qui n'est pas la même chose.

En terme des opérateurs . (point) et *, le premier est équivalent à

Code :

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

(*(*moy+2)).number = 10

et le second à

Code :

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

(**(moy+2)).number

[Topeur]

Merci pour ces explications

La difference est assez subtil mais j'ai finis par comprendre le truc
Je crois que je vais garder ca quelquepart de coté !

Merci encore