Discussion :

[seif393]

Bonsoir

Alors voila , je ne suis pas habituer a utiliser les tableaux dynamique , mais je me suis dit qu'il était temps de m'y mettre . Voici 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
17
18
19
20
21
22
23
24
25
26
27
28
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <conio.h>
#include <malloc.h>
 
void main()
{
    char *chaine;
    char **tab;
    int i,n;
    printf("saisir n :");
    scanf("%d",&n);
    chaine=(char*)malloc(10);
    tab=(char**)malloc(n*sizeof(char*));
    for(i=0;i<n;i++)
    {
        printf("saisir la chaine ");
        scanf("%s",chaine);
        strcpy(*(tab+i),chaine);
    }
    for (i=0;i<n;i++)
    {
        printf("%s",*(tab+i));
    }
    free(tab);
    free(chaine);
}

Le seule problème c'est que l’exécution du programme se bloque et je n sais pas d'ou ça vient

Toute aide serai la bien venu et merci d'avance

[Invité]

Bonsoir,

Cette ligne :

Code :

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

tab=(char**)malloc(n*sizeof(char*));

ne fait qu'allouer de la mémoire pour stocker n pointeurs sur char, tels que chaine.

Or si tu souhaites avec strcpy(*(tab+i),chaine); y copier la chaine tapée, il faut alors pour chaque pointeur tab[i] allouer de la mémoire, de la même manière que chaine, et bien entendu la libérer par la suite
Tu peux te servir de la fonction strlen sur chaine pour calculer le nombre de caractères exact à allouer.

Notes :

• le cast de malloc n'est pas nécessaire en C
• les deux inclusions

  Code :

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

  1
  2
  #include <conio.h>
  #include <malloc.h>

  sont inutiles.
• On préfèrera utiliser fgets à la place de scanf pour une saisie sécurisée.

[seif393]

Merci mille fois problème résolu

[Sve@r]

Code c :

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

1
2
3
4
5
6
7
8
9
10
void main()
{
    char **tab;
    int i,n;
    ...
    for (i=0;i<n;i++)
    {
        printf("%s",*(tab+i));
    }
}

Bonsoir

Ecrire *(tab+i) ne fera pas aller ton programme plus vite qu'écrire tab[i] car dans les deux cas, le code se cale sur tab pour décaler de i éléments. Si tu veux aller plus vite, il te faut utiliser un pointeur intermédiaire qui sera, lui, calé sur l'élément en cours

Code c :

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

1
2
3
4
5
6
7
8
9
int main()
{
    char **tab;
    char **pt
    int i,n;
    ...
    for (i=0, pt=tab;i<n;i++, pt++)
        printf("%s",*pt);
}

Et accessoirement la fonction main() est de type "int".

Code c :

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

1
2
3
4
5
6
7
8
 
    char *chaine;
    char **tab;
    chaine=(char*)malloc(10);
    tab=(char**)malloc(n*sizeof(char*));
    ...
    free(chaine);
    free(tab);

Allouer tab se justifie vu que tu ne connais pas sa taille lorsque tu écris le code. Mais allouer chaine ne se justifie pas tellement pour une taille fixe...

Code c :

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

1
2
3
4
5
6
 
    char chaine[10];
    char **tab;
    tab=malloc(n*sizeof(char*));
    ...
    free(tab);