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.

[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]

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

[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

[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]

merci ! : )
tout n'est pas encore au point mais je devrais arriver a gerer le reste.
disons que j'avais un peu tout essayé à la place des "???" et rien n'avait l'air de marcher: ni & ni * ni rien ni ** ni &* ... LOL

et remplir_tampon (le_tampon,nomdufichier); n'avait pas l'air de marcher mais si tu me dit que c'est la bonne syntaxe je devrais arriver a rectifier cela.

c'est peut etre du au fait qye je pensais que dans remplir_tampon le fscanf parcourait les n premieres cases et que si je rapellais une seconde fois remplir_tampon le fscanf allait parcourir les n cases suivantes, ce qui ne semble pas etre le cas, cela remplit le tampon avec toujours les n 1eres nombres du fichiers (tjrs les meme).

[odsen.s]

et remplir_tampon (le_tampon,nomdufichier); n'avait pas l'air de marcher mais si tu me dit que c'est la bonne syntaxe je devrais arriver a rectifier cela.

Attention, on disait justement qu'il fallait passer l'adresse du tampon, donc :

Code :

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

remplir_tampon (&le_tampon,nomdufichier);

[Freed0]

c'est peut etre du au fait qye je pensais que dans remplir_tampon le fscanf parcourait les n premieres cases et que si je rapellais une seconde fois remplir_tampon le fscanf allait parcourir les n cases suivantes, ce qui ne semble pas etre le cas, cela remplit le tampon avec toujours les n 1eres nombres du fichiers (tjrs les meme).

Exactement, chaque fois que la fonction se termine, ses variables locales sont libérées. C'est à dire qu'à chaque appel de ta fonction, la variable i se remet à 0, et donc Mon_Tampon->tampon[i] commence toujours au même endroit

[Freed0]

Attention, on disait justement qu'il fallait passer l'adresse du tampon, donc :

Code :

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

remplir_tampon (&le_tampon,nomdufichier);

Hop hop, doucement monsieur
La variable le_tampon de la fonction preleve_valeur_tampon est déjà un pointeur vers la structure Mon_Tampon. Il est donc inutile d'ajouter l'opérateur d'adresse

[zac_7]

Ok merci pour les explications. grrrrr il aurait pu gérer ca tout seul mon ordi : ) boh c'est vrai que c'est qu'une machine, et avec moi au commandes ....... le pauvre lol

En tout cas vous zetes sympa
et vous expliquez tout bien, je comprends tout (ou presque). Car , souvent quand tu demandes des trucs les gens te parles comme si tu savais tout : hé hop ils te sortent des phrases incompréhensibles. Et puis si tu comprends pas alors là ils te font bien comprendre que t'es une m**** heuu que tu es trop nul.

Enfin bon, vous m'avez bien aidé ya pas un smiley qui fait une hola pour vous remercier ah si j'ai trouvé : lol

bonne soarée a tous
puis si j'ai d'autres pblm je reviens vous voir : )

[odsen.s]

La variable le_tampon de la fonction preleve_valeur_tampon est déjà un pointeur vers la structure Mon_Tampon.

Ouip, désolé, inattention.