Discussion :

[vincenet]

Bonjour,

J'ai beau lire des exemples et des cas similaires au mien, je patauge.

Voila un code qui tombe en marche et son warning. J'aimerai trouver une écriture plus propre et comprendre la gravité du warning dans mon cas :

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
int loadPlaylist(ChaisesMusicales* pJeu){
        TypeAudio *audioPtr;
        pJeu->songs = calloc(pJeu->nbSongs, sizeof(TypeAudio));
        // ...
        audioPtr = &pJeu->songs[i];
        (*audioPtr).pathFile = getPathFromElementAudioNameJeu(&pJeu->baseParams, songsNames[i]);
        // ...
}
// avec précédement déclaré :
 
/// \struct TypeAudio
typedef struct{
    char *pathFile;
}TypeAudio;
 
typedef struct ChaisesMusicales
{
    TypeAudio **songs;
    // ...
}
ChaisesMusicales;
 
char* getPathFromElementAudioNameJeu(TypeJeu* pJeu, const char* elementName);

Code :

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

1
2
3
chaisesMusicales.c:223:50: warning: assignment from incompatible pointer type [enabled by default]
                                         audioPtr = &pJeu->songs[i];
                                                  ^

Auriez-vous une idée pour corriger ma syntaxe ? Merci

[diogene]

pJeu est un  ChaisesMusicales*
pJeu->songs est un  TypeAudio **
pJeu->songs[i] est un  TypeAudio *
&pJeu->songs[i] est un TypeAudio **

Or

audioPtr est un TypeAudio *

Et l'assignation correspond à " TypeAudio * = TypeAudio ** "
les deux types sont différents et il n'y a pas de conversion implicite entre ces deux types.
Surtout ne pas tenter un cast pour masquer cet avertissement : ce simple avertissement traduit pratiquement toujours une erreur fatale dans le code.

Quant à corriger cette erreur, ne connaissant rien de ce que tu veux faire, c'est difficile.

[Sve@r]

J'aimerai trouver une écriture plus propre

Bonjour

Déjà nomme tes structures s_xxx et tes types t_xxx. Sinon, comme le dit Diogene, &truc[i] (correspondant à truc + i) est donc forcément du même type que truc...

[vincenet]

Jusque là j'arrive à suivre, mais le retour de fonction getPathFromElementAudioNameJeu() qui est un char * ne peut être récupéré sans plantage.
J'essaie de décomposer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
        TypeAudio *audioPtr;
        // audioPtr est un TypeAudio *
        // *audioPtr est un TypeAudio
        // (*audioPtr).pathFile est du type de pathFile soit un char *
        char * tmp;
        // tmp est un char *
 
        audioPtr = pJeu->songs[i];
        msg(DEBUG, MSG_PARCOURS, "toujours en vie");
        tmp = getPathFromElementAudioNameJeu(&pJeu->baseParams, songsNames[i]);
        msg(DEBUG, MSG_PARCOURS, "toujours en vie"); // ligne affichée
        (*audioPtr).pathFile = tmp; // y a t-il un problème ici ?
        msg(DEBUG, MSG_PARCOURS, "toujours en vie"); // ligne PAS affichée

Aurai-je encore fait une bourde ?

[ternel]

Tu compiles bien avec un maximum de warning?

Ca ressemble à un pointeur nul, au pif audioPtr.

Un crash n'intervient qu'a l'accès à la mémoire, donc sur un * ou un ->.
pas sur une copie d'un pointeur, qui est une variable locale, propre, mais contient une adresse (possiblement valide)

[vincenet]

Ca ressemble à un pointeur nul, au pif audioPtr.

pJeu->songs vaut 228280
pJeu->songs[0] vaut 0
&pJeu->songs[0] vaut 228280

Code :

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

1
2
3
audioPtr = pJeu->songs[i]; // pas de warning mais vaut 0 et crash
audioPtr = &pJeu->songs[i]; // warning à la compilation mais pas de crash
audioPtr = pJeu->songs + i*sizeof(TypeAudio); // warning à la compilation mais pas de crash

[ternel]

Mais puisqu'il vaut 0, c'est qu'il faut lui donner une valeur AVANT tout ce code.

Tu n'aurai pas oublié de charger/créer la song?

[Sve@r]

Code c :

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

1
2
pJeu->songs vaut 228280
&pJeu->songs[0] vaut 228280

Evidemment puisque comme je l'ai dit précédemment, &pJeu->songs[i] vaut pJeu->songs + i. Donc fatalement &pJeu->songs[0] vaut pJeu->songs + 0 !!!

Code c :

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

1
2
audioPtr = pJeu->songs[i];
(*audioPtr).pathFile = tmp; // y a t-il un problème ici ?

Oui il y a un problème si audioPtr (donc si ptJeu->songs[i]) ne contient pas une adresse valide. Mis à part ça, ça s'écrit aussi audioPtr->pathFile = tmp...

audioPtr = pJeu->songs[i]; // pas de warning mais vaut 0 et crash

Ben voilà. Si audioPtr vaut 0 tu ne peux évidemment pas aller voir *audioPtr !!!

J'ai précédemment reservé un espace pour N structures TypeAudio

Code c :

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

pJeu->songs = calloc(pJeu->nbSongs, sizeof(TypeAudio));

Et tu ne sais pas que calloc() met tout à 0 ???
Fatalement ptJeu->songs[i] vaut 0 donc audioPtr vaut 0 donc aller remplir (*audioPtr).pathFile pose un sérieux problème !!!