Discussion :

[SAKDOSS]

Bonjour,

dans un programme (oui oui toujours le même ^^) je dois stocker des noms de pays.
Ils sont stockés dans une variable :
"char nomsPays[NMAX][NMAX];"
(NMAX = 100)

Le problème est que cela prend de la place (enfin en tout cas ce n'est pas optimisé).
Pour le 2eme NMAX le problème peux se résoudre avec l'allocation dynamique.

Pour le premier je n'arrive pas à voir comment m'en sortir.

Voici un schéma de mon programme :

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
#include<*.h>
#define NMAX 100
void example(char*[NMAX]);
 
main(){
  int nbPays;
  char *nomsPays[NMAX];
/*  On saisie le nombre de Pays  */
  ...
/* On saisie le noms des differents pays  */
  ...
}
 
void example(char* nomsPays[NMAX]{
  ...
}

Je pense qu'une solution est de faire entrer "nbPays" par l'utilisateur et ensuite déclarer les variables (ça pas de problème) ainsi que les prototype et les déclaration des fonctions.
Afin de pouvoir avoir :
void example(char*[nbPays]);

Voila je ne vois pas trop comment m'y prendre pour le faire de manière élegante.

Merci d'avance pour vos réponses

[diogene]

- Alloue dynamiquement le tableau nomsPays. Ce sera un tableau de char* donc le pointeur chargé de récupérer l'adresse de ce tableau sera du type char **
- Initialise les éléments de ce tableau à NULL (peut être qu'il ne sera pas totalement rempli)
- Alloue chaque chaîne de caractères (strlen donnera la longueur de la chaîne nom qu'on veut mettre. Ajouter 1 pour le 0 terminal) et met l'adresse de la chaîne allouée dans un élément libre du tableau (un élément à NULL)
- Copie la chaîne nom dans la chaîne allouée (strcpy)

[ToTo13]

Bonjour,

je garderai plutôt ta première solution : char *nompays[NMAX].
Je pense que c'est plutôt difficile de gérer une taille variable pour NMAX, car si tu dois faire une re-allocation à cause du nombre de pays, tu devras connaitre (ou recalculer) la somme de l'occupation mémoire de ton tableau.

Donc tout simplement :

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#define NMAX = ???
int nbPays = ???
char **Pays ;
...
Pays = malloc(nbPays * sizeof(char *)) ;
assert(nbPays != NULL) ;
for (i=0 ; i < nbPays ; i++)
{
Pays[i] = malloc(NMAX * sizeof(char)) ;
assert(Pays[i] != NULL) ;
}

Si nécessaire, tu fais un realloc...

[Médinoc]

Le code en général est à peu près bon, mais ces assertions sont inappropriées. Une allocation mémoire peut toujours échouer suite à des facteurs extérieurs, il faut donc un vrai traitement d'erreur si une allocation échoue.

Un traitement d'erreur qui évitera à l'application de crasher, et sera également présent en mode Release.

[diogene]

...
Je pense que c'est plutôt difficile de gérer une taille variable pour NMAX, car si tu dois faire une re-allocation à cause du nombre de pays, tu devras connaitre (ou recalculer) la somme de l'occupation mémoire de ton tableau.
...

J'avoue ne pas avoir compris ce que tu voulais dire par cette phrase (même et surtout en regardant ton code).

Accessoirement :

Code :

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

assert(Pays != NULL) ;

[Melem]

dans un programme (oui oui toujours le même ^^) je dois stocker des noms de pays.
Ils sont stockés dans une variable :

Code :

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

char nomsPays[NMAX][NMAX];

Je propose :

Code :

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

char nomsPays[NPAYS][NOMMAX];

Je pense qu'une solution est de faire entrer "nbPays" par l'utilisateur et ensuite déclarer les variables

Dans ce cas c'est :

Code :

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

char (*nomsPays)[NOMMAX]

Les parenthèses sont obligatoires pour dire que nomsPays est un pointeur. Sans les parenthèses :

Code :

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

char * nomPays[NPAYS];

on aurait plutôt un tableau de pointeurs.

Donc finalement on a :

Code :

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

1
2
3
4
5
6
7
8
 
int nbPays;
char (*nomsPays)[NOMMAX];
...
nomsPays = malloc(...);
...
example(nomsPays, nbPays);
...

Code :

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

1
2
3
4
5
 
void example(char (*p)[NOMMAX], int nbPays)
{
    ...
}

[diogene]

Donc finalement on a :

Code :

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

1
2
3
4
5
6
7
8
 
int nbPays;
char (*nomsPays)[NOMMAX];
...
nomsPays = malloc(...);
...
example(nomsPays, nbPays);
...

Code :

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

1
2
3
4
5
 
void example(char (*p)[NOMMAX], int nbPays)
{
    ...
}

Pourquoi se priver de l'allocation dynamique sur les noms de pays alors que le posteur dit explicitement qu'il a résolu ce point et qu'il désire améliorer l'occupation mémoire de ses données ?