Discussion :

[julieng31]

Bonjour,

j'ai écrit quelques lignes en C, mais je ne comprends pas pourquoi elles ne marchent pas. Les voici (en simplifiant pas mal le code) :

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
 
// Fonction mapping
void mapping(double * constelI, double * constelQ) {
    constelI = (double *) malloc(4*sizeof(double));
    consteQ = (double *) malloc(4*sizeof(double));
    for(i=0; i<4; i++) {
        constelI[i] = i;
        constelQ[i] = 2*i;
    }
}
 
// Fonction principale
double * constelI;
double * constelQ;
 
mapping(constelI, constelQ);

Après l'appel de mapping(), quand je fais :

Code :

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

1
2
 
printf("%f  %f\n", constelI[i], constelQ[i]);

ça fait un segfault :-(

Est-ce que quelqu'un pourrait m'expliquer pourquoi ! Merci d'avance.

[Obsidian]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
// Fonction mapping
void mapping(double * constelI, double * constelQ) {
    constelI = (double *) malloc(4*sizeof(double));
    consteQ = (double *) malloc(4*sizeof(double));
}
 
// Fonction principale
double * constelI;
double * constelQ;
 
mapping(constelI, constelQ);

Dans une fonction, les paramètres sont traités comme des variables locales. ConstelI et ConstelQ ne sont donc pas les mêmes que ceux de la fonction principale. Si tu veux affecter une valeur à un pointeur se trouvant en dehors de ta fonction, il faut passer à celle-ci l'adresse de ton pointeur pour qu'elle puisse écrire dedans. Autrement dit : un pointeur sur un pointeur (sur un double, en l'occurence). Puis passer explicitement l'adresse de ce pointeur à ta fonction. Donc :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
// Fonction mapping
void mapping(double ** constelI, double ** constelQ) {
    * constelI = (double *) malloc(4*sizeof(double));
    * consteQ = (double *) malloc(4*sizeof(double));
}

// Fonction principale
double * constelI;
double * constelQ;

mapping(&constelI, &constelQ);

Note qu'au niveau des étoiles vertes, on déclare le type de la variable, tandis qu'au niveau des étoiles oranges^WRouges, on déréférence le pointeur sur pointeur, pour accéder directement à l'objet pointé et déposer dedans le résultat de malloc().

[Emmanuel Delahaye]

Note qu'au niveau des étoiles vertes, on déclare le type de la variable, tandis qu'au niveau des étoiles oranges, on déréférence le pointeur sur pointeur, pour accéder directement à l'objet pointé et déposer dedans le résultat de malloc().

Les étoiles oranges, je ne les voie pas... C'est normal, comme beaucoup , je voie mal cette couleur...

[julieng31]

Merci pour vos réponses. Cela fonctionne bien. J'aurais eu le réflexe du passage de l'adresse pour une variable mais je ne l'ai pas eu pour un pointeur !

[Emmanuel Delahaye]

Merci pour vos réponses. Cela fonctionne bien. J'aurais eu le réflexe du passage de l'adresse pour une variable mais je ne l'ai pas eu pour un pointeur !

Passer l'adresse, c'est OK pour une variable . A partir de 2 ça devient lourd et peu évolutif. Je préfère une structure comme déjà indiqué.

[Emmanuel Delahaye]

Bonjour,

j'ai écrit quelques lignes en C, mais je ne comprends pas pourquoi elles ne marchent pas. Les voici (en simplifiant pas mal le code) :

Code :

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

1
2
3
4
5
6
 
// Fonction mapping
void mapping(double * constelI, double * constelQ) {
    constelI = (double *) malloc(4*sizeof(double));
    consteQ = (double *) malloc(4*sizeof(double));
<...>

http://emmanuel-delahaye.developpez....ge=Page3#LXIII

Dans ce cas, une structure est indiquée.

Code :

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

1
2
3
4
5
6
 
struct constel
{
   double * I; 
   double * Q;
};

puis

Code :

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

void mapping(struct constel *p)

et l'appel :

Code :

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

1
2
3
4
 
{
   struct constel c = {NULL, NULL};
   mapping (&c);