Discussion :

[lassault1]

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 :

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
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;
}

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

[lassault1]

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

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
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;
}

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

[ok.Idriss]

Salut.

Au niveau du code, quelques petites corrections :

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

[bizulk]

Code :

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

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 :

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

typedef char T_NOM[50] ;

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

Code :

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

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).

[lassault1]

Merc ia vous..

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

[ok.Idriss]

Re

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 :

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

[lassault1]

Ok .. merci