Discussion :

[julien.63]

salut,

je suis sur un petit programme où l'utilisateur peut rentrer plusieurs fois la même option.
J'aimerai connaitre votre avis sur la meilleure façon de stocker ces données.
J'ai pensé au tableau biensur mais le souci c'est que si on défini une taille fixe, on prend le risque d'avoir un souci. Et je ne vois pas trop comment définir la taille dynamiquement.

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
 
int main(int argc, char **argv) 
{
int           carg;
int           i=0
int           r=0;
int           restricted_taxid[10];
int           ignored_taxid[10];
while ((carg = getopt(argc, argv, "r:i:")) != -1) {
 
     switch (carg) {
        case 'r':
          restricted_taxid[r] = optarg;
          r++;		
          break;
 
        case 'i':
          ignored_taxid[i] = optarg;
          i++;
          break;
     }
}

[straasha]

pour te mettre sur la voie :
cherche autour des pointeurs et des fonctions malloc et realloc

[julien.63]

salut,
merci pour tes conseils. J'ai donc cherché du coté de realloc et voici le résultat (simplifié) (ça compile et les premiers tests ne montrent pas de bugs...)

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
 
#define REALLOC(chunk,size,error_message) \
        realloc((chunk),(size),(error_message),__FILE__,__LINE__)
 
int main(int argc, char **argv) 
{
int           carg;
int           r=0;
int32_t	    *restricted_taxid = NULL;
 
    while ((carg = getopt(argc, argv, "r:")) != -1) {
 
     switch (carg) {
        case 'r': 
          restricted_taxid = realloc(restricted_taxid,sizeof(int32_t));
          sscanf(optarg,"%d",&restricted_taxid[r]);
          r++;		
          break;
    }
}

Est-ce correct?

Je me demande aussi s'il vaut mieux faire :

Code :

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

restricted_taxid = realloc(restricted_taxid,sizeof(int32_t));

ou

Code :

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

restricted_taxid = realloc(restricted_taxid,sizeof(int32_t)*r);

[Médinoc]

Je n'ai pas les détails, mais je pense qu'il faut au moins du sizeof(int32_t)*(r+1) si tu veux que ça passe.

De plus, tu dois passer par un pointeur temporaire, car si realloc() échoue, ton programme leake la mémoire.

[straasha]

Je me demande aussi s'il vaut mieux faire :

Code :

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

1
2
 
restricted_taxid = realloc(restricted_taxid,sizeof(int32_t));

ou

Code :

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

1
2
 
restricted_taxid = realloc(restricted_taxid,sizeof(int32_t)*r);

la 2e solution est meilleure, la 1ere est fausse puisque tu alloue 1 seul int32_t donc l'acces a &restricted_taxid[r] sera incorrect pour r>0
ensuite j'aurai plutot allouer 10 int32_t en init (10 etant une taille min arbitraire pour mon tableau) et ensuite si r%10==0 je realloue en rajoutant 10 int32_t
ainsi tu ne fais pas de reallocation a chaque tour d'ou un gain de temps mais c'est vrai que du coup tu as un tableau un peu trop grand

[Jean-Marc.Bourguet]

S'il y a des chances qu'il y ait beaucoup d'elements, il vaut mieux ne pas reallouer en augmentant la taille de 1 a chaque fois mais en multipliant l'ancienne taille (je conseille un facteur de 1.5, facilement utilisable: new_size = old_size + old_size/2 -- naturellement, il faut commencer avec une taille differente de 1 :-)), quitte a faire une derniere reallocation ramenant a la taille precise a la fin.

[julien.63]

merci beaucoup pour vos réponses et vos précisions.