Discussion :

[lucaordi]

Bonsoir,

J'ai déclaré une structure de la manière suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 typedef struct{
    float x;
    float y;
    float a;
    float b;
    float x_result;
    float y_result;
    float length_result;
  } vector;
  vector pt_rec;

.


Ensuite, je désire simplement créer une fonction effectuant certaines opérations sur les variables de cette structure; par exemple:

Code :

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

1
2
3
4
5
 
static float DoSth(float pt_rec.x, float pt_rec.a, float pt_rec.y, float pt_rec.b){
(...)
return x_result;
}

.
Seulement, j'obtiens, à la compilation:
error: expected ‘;’, ‘,’ or ‘)’ before ‘.’ token
1) Comment puis-je faire?
En effet, d'habitude, j'accède à un élément d'une structure par structure.élément.

2) Je souhaiterais simplement que la fonction calcule x_result et y_result. Est-il ``mieux'' de déclarer la fonction comme static float, et de lui faire retourner x_result et y_result (si cela est possible), ou vaut-il mieux la déclarer ne retournant rien (void)?

Merci.

[tonton fred]

Salut,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 typedef struct{
    float x;
    float y;
    float a;
    float b;
    float x_result;
    float y_result;
    float length_result;
  } vector;
  vector pt_rec;

.
pt_rec.x etc ... est un float, donc tu peux declarer ta fonction comme suit:

Code :

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

static float DoSth(float x, float a, float y, float b) {

Et l'appel

Code :

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

DoSth(pt_rec.x,pt_rec.a, pt_rec.y, pt_rec.b);

Tu peux aussi faire

Code :

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

1
2
3
4
5
static float DoSth(vector rec) {
/* tu peux acceder aux aux membres rec.x rec.y etc... */
}
 
DoSth(pt_rec);

Ceci dit, les operations de copie de structures sont assez cheres. Moi je ferais avec des pointeurs

Code :

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

1
2
3
4
5
static float DoSth(vector *p_rec) {
/* tu peux acceder aux aux membres p_rec->x p_rec->y etc... */
}
 
DoSth(&pt_rec);

[lucaordi]

Merci.
J'en arrive donc à:

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
 
  typedef struct{
    float x;
    float y;
    float a;
    float b;
    float x_result;
    float y_result;
    float length_result;
  } vector;
  vector pt_rec;
 
static vector DoSth(float, float, float, float); // declaration
 
static vector DoSth(float x, float a, float y, float b){
(...)
return pt_rec;
}

Le problème est que je reçois:

foo.c:71: warning: unused parameter ‘x’
foo.c:71: warning: unused parameter ‘a’
foo.c:71: warning: unused parameter ‘y’
foo.c:71: warning: unused parameter ‘b’, bien que le programme fonctionne (et retourne les bonnes valeurs).

Que faire? Merci.

[tonton fred]

La a priori il n'y a pas de probleme donc il va falloir que tu postes le code en entier

Ah et je n'avais pas vu cette question:

2) Je souhaiterais simplement que la fonction calcule x_result et y_result. Est-il ``mieux'' de déclarer la fonction comme static float, et de lui faire retourner x_result et y_result (si cela est possible), ou vaut-il mieux la déclarer ne retournant rien (void)?

Une fonction ne peut retourner qu'une seule valeur. Mais il y a des astuces:
- retourner une structure (ce que tu as fait)
- passer les variables en parametres via des pointeurs
- retourner un pointeur vers un structure
- et surement d'autres
Personnellement je prefere la deuxieme solution pour pouvoir retourner un code d'erreur.