Discussion :

[saubin]

mon souci c'est d'ajouter des chaines dans mon tableau ( chaine de longueur variable ) .

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
 
 
 char *saisie; 
    char * Tableau[10];
    int i = 0;
    int choix;
    int position1;
    int position2;
    int position;
 
    printf("Entrez le %d er  ",i+1);
    scanf("%s",&saisie);
    Tableau[i] = malloc(sizeof(saisie));
    Tableau[i] = saisie;
    while ( i <9 )
    {
    i = i+1;
    printf("Entrez le %d eme  ", i+1);
    scanf("%s",saisie);
    Tableau[i] = malloc(sizeof(saisie));
    Tableau[i] = saisie;
    }

Voila je sais que c'est faux, je suis un novice en C, et je n'est pas bien compris comment on utilisait malloc.

merci de votre aide

[muad'dib]

Wahou il y a un paquet d'erreurs. Je vais en citer quelques unes pour commencer:

• Tu veux scanner un caractère avec scanf mais tu met %s, le format pour un charactère est %c
• Tu veux stocker ce caractère sur un pointeur de caractère qui ne pointe sur rien! il faut avoir un emplacement mémoire valable. A savoir définir saisie en tant que char (et non char*) et faire scanf("%c", &saisie);
• L'utilisation de scanf est très dangereuse. Mieux vaut utiliser fgets
• Tu ne libères jamais la mémoire allouée
• Pourquoi ne pas stocker directement tes caractères dans un tableau de caractères statique?

[saubin]

Ma variable saisie est une chaine qui peux varier ( je demande 10 saisies ) elle peut contenir de 60 à 10 caracteres.

Et à chaque fois que je saisie ma chaine, je veux la mettre dans mon tableau de chaine.

[matafan]

Ta variable "saisie" est un pointeur qui pointe "dans le vide"... Tu ne peux rien mettre dedans tant qu'elle ne pointe pas, soit vers une zone de mémoire allouée par malloc, soit vers une variable (un tableau en l'occurence) local.

Ensuite scanf("%s") est extrèmement dangereux puisqu'il n'y a aucun contrôle sur la longueur de la chaine lue. Utilise plutôt fgets.

Enfin, sizeof saisie n'est pas la longueur de la chaine. C'est la taille d'un pointeur, c'est à dire d'une adresse mémoire.

[gl]

Et j'ajouterais que la copie de chaîne en C ne se fait pas avec = mais avec str(n)cpy.

Le code suivant

Code :

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

1
2
Tableau[i] = malloc(sizeof(saisie));
Tableau[i] = saisie;

alloue un espace mémoire dont l'adresse est mise dans tableau[i] puis cette adresse est remplacée par celle de saisie.
D'où plusieurs problèmes : Tableau[i] ne contient pas ce que tu crois et la mémoire alloué par malloc() n'est plus atteignable donc n'est pas libérable.

[Emmanuel Delahaye]

Et j'ajouterais que la copie de chaîne en C ne se fait pas avec = mais avec str(n)cpy.

J'éviterais de parler de strncpy() à un débutant, car c'est une source d'erreurs importantes...

Je n'ai d'ailleurs jamais bien compris à quoi servait cette fonction...

Préférer strncat().