Discussion :

[zac_7]

Salut tout le monde,
je viens de decouvrir votre forum . Peut etre pourriez vous m'aider car là je bloque un peu et ca m'empeche de continuer :

J'ai créé une structure Mon_Tampon :

Code :

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

1
2
3
4
5
typedef struct
    {     
       int tampon[TAILLE_TAMPON];   
       int dernier_element_lu;
    } Mon_Tampon;

Une fonction :

Code :

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

1
2
3
4
_Bool tampon_vide (Mon_Tampon le_tampon)    
    {        
         return  (le_tampon.dernier_element_lu==TAILLE_TAMPON);    
    }

et une autre dont la definition est :

Code :

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

void remplir_tampon (Mon_Tampon *le_tampon,char* nomdufichier)

Mon probleme c'est que je ne sais pas quoi mettre a la place des ??? dans la fonction si dessous, je pensais que c'était des adresses donc des "&" mais ca a pas l'air de marcher. Est ce du au fait que je considère des fois mon tampon comme "modifiable" et d autres fois non ?

Code :

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

1
2
3
4
5
6
7
8
9
10
int preleve_valeur_tampon (Mon_Tampon * le_tampon,char* nomdufichier)
    {        
           int res;
              if (tampon_vide ( ???le_tampon))
                   {
                         remplir_tampon (???le_tampon,nomdufichier);  
                          .... 
                        // le reste a pas d'importance     
 
    }

Merci de m'avoir lu.

[odsen.s]

Salut,

Pour ta fonction qui vérifie si le tampon est vide ou nom, tu peux passer le_tampon tel quel, car tu ne le modifies pas.

Pour remplir_tampon, tu dois passer l'adresse de ton tampon.
Attention, dans ce cas, à l'intérieur de la fonction, tu accèdes à un élement de ta structure de cette manière :

Code :

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

structure->element = ...;

et non pas

Code :

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

structure.element = ...;

comme dans le premier cas.

[Freed0]

Salut,

Je te conseille tout de même de passer une adresse de structure à ta fonction. Si toutefois tu ne souhaites pas la modifier, il suffit d'ajouter le mot-clé const

[odsen.s]

Salut freed0,

Peux-tu m'expliquer à quoi sert de passer l'adresse de la structure même quand il n'y a pas de modification envisagée ?
Merci.

[Freed0]

Salut freed0,

Peux-tu m'expliquer à quoi sert de passer l'adresse de la structure même quand il n'y a pas de modification envisagée ?
Merci.

Dans ce cas ci, je suis d'accord que ça peut paraître inutile, mais passer une adresse (4 bytes) au lieu d'une structure (qui peut aller de 1 à beaucoup de bytes) est plutôt une bonne chose. Ici on gagne 4 octets (waw ).
Je pense que c'est une bonne habitude à prendre, c'est pour ça que j'ai donné ce conseil

[Emmanuel Delahaye]

Peux-tu m'expliquer à quoi sert de passer l'adresse de la structure même quand il n'y a pas de modification envisagée ?

A limiter les copies au strict minimum (rappel : en C tous les passages de paramètres se font par valeur, donc par copie).

Quand on passe une adresse, on copie le contenu d'un pointeur (2, 4 bytes...). Quand on passe une structure, on copie toute la structure. Ca peut être pareil ou beaucoup plus.

Cette pratique est d'ailleurs interdite avec les tableaux, sans doute pour la même raison. La vrai question est "pourquoi est-elle autorisée en C avec les structures ?"

Probablement parce que la copie de petites structures peut suffire dans des cas simples (div(), idiv(), par exemple...)

[odsen.s]

Dans ce cas ci, je suis d'accord que ça peut paraître inutile, mais passer une adresse (4 bytes) au lieu d'une structure (qui peut aller de 1 à beaucoup de bytes) est plutôt une bonne chose.

A limiter les copies au strict minimum (rappel : en C tous les passages de paramètres se font par valeur, donc par copie).
...

Ok, merci

[zac_7]

odsens.s >> oui mais dans mon cas je veux appeler mes fonctions et je pensais qu'il fallait que je rentre leur adresse donc j'avais mis des &...

int preleve_valeur_tampon (Mon_Tampon * le_tampon,char* nomdufichier)
{
int res;
if (tampon_vide ( & le_tampon))
{
remplir_tampon (&le_tampon,nomdufichier);
....
// le reste a pas d'importance

}

et je rentre ma structure "my_struct" et non un element de ma structure...


Freed0>> Désolé mais je connais peut de choses et j'ai pas tres bien compris "passer une adresse de structure à ta fonction" ... tu veux dire dans ma fonction qui renvoie un booleen ?

[zac_7]

PS : il est vrai que si a la place de (tampon_vide (le_tampon))
je met (le_tampon->dernier_element_lu==TAILLE_TAMPON) ca marche donc j'ai plus de probleme avec le 1er "???" .

mais pour le deuxieme pouvez vous m'expliquer pourquoi :

remplir_tampon (&le_tampon,nomdufichier); dans ma fonction marche pas.





PS : voila la fonction remplir_tampon (je pense pas que ca vous aide plus)

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
void remplir_tampon (Mon_Tampon *le_tampon,char* nomdufichier)
    {
       FILE* monfichier;
        monfichier= fopen (nomdufichier,"r"); // ouvert en lecture seulement
 
        int i;
        int nombre;
 
        for (i=0;i<TAILLE_TAMPON; i++)
            {
                fscanf(monfichier,"%d",&nombre);
                (le_tampon->tampon)[i]=nombre;    // on stocke les nombres dans le tampon
            }
 
 
 fclose(monfichier);
 
    }

[Freed0]

PS : il est vrai que si a la place de (tampon_vide (le_tampon))
je met (le_tampon->dernier_element_lu==TAILLE_TAMPON) ca marche donc j'ai plus de probleme avec le 1er "???" .

mais pour le deuxieme pouvez vous m'expliquer pourquoi :

remplir_tampon (&le_tampon,nomdufichier); dans ma fonction marche pas.





PS : voila la fonction remplir_tampon (je pense pas que ca vous aide plus)

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
void remplir_tampon (Mon_Tampon *le_tampon,char* nomdufichier)
    {
       FILE* monfichier;
        monfichier= fopen (nomdufichier,"r"); // ouvert en lecture seulement
 
        int i;
        int nombre;
 
        for (i=0;i<TAILLE_TAMPON; i++)
            {
                fscanf(monfichier,"%d",&nombre);
                (le_tampon->tampon)[i]=nombre;    // on stocke les nombres dans le tampon
            }
 
 
 fclose(monfichier);
 
    }

Voici le code de ta fonction appelante (que j'ai repris dans ton premier post) :

Code :

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

1
2
3
4
5
6
7
8
9
10
int preleve_valeur_tampon (Mon_Tampon *le_tampon, char *nomdufichier)
{
    int res;
 
    if (tampon_vide (le_tampon))
    {
        remplir_tampon (le_tampon, nomdufichier);
        /* Etc etc. */
    }
}

Il suffit de regarder le type de la variable le_tampon : elle est du type Mon_Tampon * (c'est donc un pointeur [une adresse]).

Maintenant, regarde le prototype de tes "sous-fonctions" :

Code :

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

void remplir_tampon (Mon_Tampon *, char *);

Tu peux donc en déduire que les deux types correspondent ! Il ne faut donc pas rajouter de & ni de * ni quoi que ce soit