Discussion :

[syannic]

bonjour,

j'utilise une api fourni par le constructeur et mon programme plante violamment sur la derniere ligne lorsque j'essaie d'acceder a un champs de ma structure... Voici le code :

Code :

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

1
2
3
4
5
 
        MaStructure *pInfos;
        UINT16 LgInfos;
	retour = Info(&LgInfos,pInfos);
	int nb = pInfos->Nombre ;

Sachant que MaStructure contient bien un champs UINT16 Nombre.
Et que pInfos pointe sur la structure au retour des informations.

Quelqu'un a une idée ?

[ronan99999]

Ton pointeur pInfos n'a pas été initialisé, tu essai d'acceder à un espace non alloué.
Je ne connais pas la signature de ta fonction info mais ce ne serais pas plutot:

Code :

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

1
2
3
4
5
 
  MaStructure *pInfos;
  UINT16 LgInfos;
  retour = Info(&pInfos,LgInfos);
  int nb = pInfos->Nombre ;

[syannic]

non non c'est bien ce que j'ai mis...
c'est pas la fonction de lapi qui s'occupe dallouer lespace mémoire et qui te renvoie le pointeur ?

[Trap D]

Pas obligatoirement, ce doit être écrit dans la doc fournie par lle créateur de l'API.

[programan]

En complément à ce que dit Ronan, d'une manière générale, pour qu'une variable passée en paramètre soit réellement modifiée au sein d'une fonction, il faut que le paramètre soit l'adresse de cette variable.

Concrêtement, si tu veux que ton pointeur ait la bonne valeur après l'appel, il faut passer en paramètre l'adresse de ton pointeur, donc un truc du style:

Code :

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

1
2
 
retour = Info(&LgInfos, &pInfos);

Ce n'est pas ce que tu fais, donc pInfos a la même valeur avant et après l'appel, c'est à dire... une valeur indéterminée, qui n'est pas la bonne, forcément... Alors relie bien la doc de l'API.

[Trap D]

programan a tout a fait raison et ton compilo aurait dû te signaler l'erreur. Quel est le prototype de la fonction ?

[ronan99999]

Ou alors il faut allouer l'espace relatif à ton pointeur

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
  MaStructure *pInfos = (MaStructure *) 0;
  UINT16 LgInfos;
  pInfos = (MaStructure *)calloc(1,sizeof(MaStructure ));
  if(pInfos)
{
  retour = Info(LgInfos,pInfos);
  int nb = pInfos->Nombre ; 
}

En tout cas c'est surement un probléme dans l'utilisation de ton api.

[syannic]

Je viens d'essayer ca , ca ne marche pas...
Si j'arrive a le faire fonctionner un jour, je posterai l'explication...

[HRS]

Pourquoi pas

Code :

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

1
2
3
4
5
 
        MaStructure pInfos; 
        UINT16 LgInfos;
   retour = Info(&LgInfos,&pInfos); 
   int nb = pInfos.Nombre ;

au lieu de

Code :

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

1
2
3
4
5
 
        MaStructure *pInfos; 
        UINT16 LgInfos;
   retour = Info(&LgInfos,pInfos); 
   int nb = pInfos->Nombre ;

Dans le 1er cas, pas besoin de gérer des pointeurs, même si la fonction
utilise des pointeurs comme paramètres
------------------------------------
vérifier le proto de la fonction pour voir si le 1er argument est aussi un
pointeur

[syannic]

Parceque le compilateur ne veut pas.

Pour le prototype c'est :
Info(PUINT16 pLgInfos,struct MaStructure PTR pInfos)

[Emmanuel Delahaye]

j'utilise une api fourni par le constructeur et mon programme plante violamment sur la derniere ligne lorsque j'essaie d'acceder a un champs de ma structure... Voici le code :<...>

Je ne vois rien qui initialise pInfos, alors ce n'est pas très étonnant...

Code :

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

1
2
3
4
5
 
        MaStructure Infos;
        UINT16 LgInfos;
	retour = Info(&LgInfos,&Infos);
	int nb = Infos.Nombre ;

[Emmanuel Delahaye]

Ton pointeur pInfos n'a pas été initialisé, tu essai d'acceder à un espace non alloué.
Je ne connais pas la signature de ta fonction info mais ce ne serais pas plutot:

Code :

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

1
2
3
4
5
 
  MaStructure *pInfos;
  UINT16 LgInfos;
  retour = Info(&pInfos,LgInfos);
  int nb = pInfos->Nombre ;

Oh la la la la...

[Emmanuel Delahaye]

non non c'est bien ce que j'ai mis...
c'est pas la fonction de lapi qui s'occupe dallouer lespace mémoire et qui te renvoie le pointeur ?

Qui renvoi où ? Par quelle magie ? Regarde l'interface, c'est impossible. malloc(), il fait comment pour renvoyer l'adresse ?

[Emmanuel Delahaye]

programan a tout a fait raison et ton compilo aurait dû te signaler l'erreur. Quel est le prototype de la fonction ?

Mais non. Pourquoi tout le monde est aveuglé par le pointeur ? Un paramètre pointeur, c'est fait pour recevoir une adresse. Le plus simple est de passer l'adresse de l'instance d'un objet du type du pointeur.

Code :

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

1
2
3
4
 
   T a;
 
   f(&A);

avec

Code :

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

void f (T*);

Pourquoi aller chercher des trucs tordus ?

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
 
        MaStructure pInfos; 
        UINT16 LgInfos;
   retour = Info(&LgInfos,&pInfos); 
   int nb = pInfos.Nombre ;

Dans le 1er cas, pas besoin de gérer des pointeurs, même si la fonction
utilise des pointeurs comme paramètres

Ca y'est, y'en a un qui est reveillé; Pense quand même à retirer le 'p', ca fera moins nouille...

Code :

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

1
2
3
4
5
 
        MaStructure Infos; 
        UINT16 LgInfos;
   retour = Info(&LgInfos,&Infos); 
   int nb = Infos.Nombre ;

[Emmanuel Delahaye]

Parceque le compilateur ne veut pas.

Pour le prototype c'est :
Info(PUINT16 pLgInfos,struct MaStructure PTR pInfos)

C'est quoi cet interface à la gomme encore. T'as payé pour ça ?

[syannic]

Pas encore....

[Emmanuel Delahaye]

Pas encore....

Tu l'as volé alors ?

[syannic]

Non plus...