Aide allocation dynamique

Version imprimable

Bonsoir a tous :)

Avec ce code qui se trouve dans le chapitre "L'allocation dynamique" , j'ai voulu entrer les noms des Amis au lieu des ages mais ça plante a chaque fois.. avez des indices a me donner?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
int main(int argc, char *argv[])
{
    int nombreDAmis = 0, i = 0;
    int* ageAmis = NULL; 
 
 
    printf("Combien d'amis avez-vous ? ");
    scanf("%d", &nombreDAmis);
 
    if (nombreDAmis > 0) 
    {
        ageAmis = malloc(nombreDAmis * sizeof(int)); 
        if (ageAmis == NULL) 
        {
            exit(0); 
        }
 
 
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("Quel age a l'ami numero %d ? ", i + 1);
            scanf("%d", &ageAmis[i]);
        }
 
 
        printf("\n\nVos amis ont les ages suivants :\n");
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("%d ans\n", ageAmis[i]);
        }
 
 
        free(ageAmis);
    }
 
    return 0;
}

22/04/2010, 00h15
ram-0000

Tu as un problème sur les noms de tes amis et le code que tu présentes gère l'age de tes amis, pas leur nom. Montre nous le code qui gère leur nom et on t'aidera à debugger.

Voici ce que j'ai fais avec un plantage..

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
int main(int argc, char *argv[])
{
    int nombreDAmis = 0, i = 0;
    char* nomAmis = NULL;
 
 
    printf("Combien d'amis avez-vous ? ");
    scanf("%d", &nombreDAmis);
 
    if (nombreDAmis > 0)
    {
        nomAmis = malloc(nombreDAmis * sizeof(char));
        if (nomAmis == NULL)
        {
            exit(0);
        }
 
 
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("Quel est le nom de l'amis  %d ? ", i + 1);
            scanf("%s", nomAmis[i]);
        }
 
 
        printf("\n\nVos amis ont les noms suivants :\n");
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("%s\n", nomAmis[i]);
        }
 
 
        free(nomAmis);
    }
 
    return 0;
}

22/04/2010, 11h59
Médinoc

Tu dois utiliser autre chose que nombreDAmis comme taille maximale de la chaîne de caractères nomAmis.

De plus, sans doute aurais-tu besoin d'un tableau de chaînes de caractères (ou d'un tableau de structures représentant tes amis)

Salut.

Au niveau du code, quelques petites corrections :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
# include <stdio.h>
# include <stdlib.h>
 
int main(void) /* Les paramètres de main sont inutiles => on met void à la place */
{
    size_t nombreDAmis = 0, i = 0; /* malloc attend un paramètre du type size_t, nombreDAmis doit donc être de type size_t. On laisse aussi i de type size_t pour ne pas gêner les comparaisons avec nombreDAmis dans les test if qui suivent */
    char* nomAmis = NULL;
 
 
    printf("Combien d'amis avez-vous ? ");
    scanf("%u", &nombreDAmis);
 
    if (nombreDAmis > 0)
    {
        nomAmis = malloc (nombreDAmis * sizeof(char)); /* l'expression en paramètre est bien du type size_t maintenant */
        if (nomAmis == NULL)
        {
            exit(0);
        }
 
 
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("Quel est le nom de l'amis  %d ? ", i + 1);
            scanf("%s", nomAmis); /* Pas d'indice en utilisant le format %s ici : il s'agit d'une chaîne de caractère et non d'un tableau de chaîne (ce dont tu aurais peut être besoin d'ailleurs) */
        }
 
 
        printf("\n\nVos amis ont les noms suivants :\n");
        for (i = 0 ; i < nombreDAmis ; i++)
        {
            printf("%s\n", nomAmis); /* Pareil que précédemment */
        }
        free(nomAmis);
    }
 
    return 0;
}

Mais, cela n'aura pas le comportement attendu pour des raisons que Médinoc a souligné ...

Cordialement,
Idriss

22/04/2010, 12h31
bizulk
Code:

nomAmis = malloc(nombreDAmis * sizeof(char));

SI on considère que "nomAmis" est la liste des noms de tes amis, alors tu n'as alloué qu'un seul caractère pour chaque nom. Il te faut réserver une chaine et non un caractère avec ta liste.

Ne connaissant pas la taille de la chaine pour chaque nom, tu pourrait définir un type avec une "une taille suffisante" pour un nom genre :

Code:

typedef char T_NOM[50] ;

Puis ensuite allouer l'espace suivante le nombre d'amis :
Code:

1 2 3 T_NOM * nomAmis ; nomAmis = (T_NOM *)malloc(nombreDAmis * sizeof(T_NOM));
N'oublie pas d'initialiser le contenu de ta mémoire aussi, et lors de la saisie de limiter la lecture à la taille définie (avec fgets par exemple).
22/04/2010, 17h46
lassault1

Merc ia vous..

ok.Idriss ton code source affiche 3 fois le même nom..!
22/04/2010, 17h53
ok.Idriss

Re

Citation:

Envoyé par lassault1

ok.Idriss ton code source affiche 3 fois le même nom..!

Ce n'est pas mon code mais le tien corrigé des quelques erreurs et warnings à la compilation (il faut lire les commentaires) ... Je t'ai indiqué en dessous que le comportement obtenu n'est pas celui que tu attendait et on t'a déjà expliqué pourquoi ...

Là tu utilises une chaîne de caractère unique qui est re-saisie à chaque itération de la boucle for, normal que tu aies plusieurs fois le même nom. Maintenant c'est à toi de résoudre se problème, et si tu as des difficultés on sera la pour t'aider à te corriger mais on ne fait pas tout à ta place ;) .

Un petit rappel :

Citation:

Envoyé par Médinoc

sans doute aurais-tu besoin d'un tableau de chaînes de caractères (ou d'un tableau de structures représentant tes amis)

Cordialement,
Idriss
23/04/2010, 20h05
lassault1

Ok .. merci