projet c:gestion d'un forum

bonjour,
voila ,je dois rendre un projet c portant sur la gestion d'un forum .
on doit pouvoir ajouter des thèmes de discussions,des sujets initiaux ,répondre a des messages....
j'ai deja commencer le programme mais y'a un problème lors de l'exécution ca bugge dans la fonction répondre si vous pouvez m'aider s'il vous plait .merciii d'avance

1- La présence de ces trois variables globales

Code:

---

1 2	theme *T[TAILLE]; int n=0,code=1;

---

me décourage complétement d'aller explorer le code (aller chercher tous les n dans le code...).
De plus la présentation du code (indentation aléatoire) ne facilite pas la lecture.

2- Quelques remarques faites par mon compilateur et qui auraient dû être faites par le tien si il était bien réglé.

- Appels de fonctions sans prototype : il faut savoir que, par exemple

Code:

---

void afficher_msg_hierar();

---

ne constitue pas un prototype puisque le type des arguments n'est pas spécifié. On doit écrire :

Code:

---

void afficher_msg_hierar(void);

---

- Une erreur

Code:

---

1 2 3 4

void menu_deposer_sujet() { ..... while(choix==1)

---

Même erreur dans void menu_after_afficher()

- La fonction msg* trouver_message(int code_msg) doit renvoyer une valeur , ce qui n'est pas toujours le cas

Code:

---

1 2	if(msg_rech==NULL) {printf("ce message n'existe pas");menu_after_afficher();} else return msg_rech;

---

idem pour main() qui doit retourner un int


3- Toutes ces fonctions qui n'ont pas de paramètres! Il faut les utiliser au lieu de passer par des variables globales

4- Il y a d'autres erreurs après un coup d'oeil rapide.
- Dans la première fonction appelée :

Code:

---

1 2 3 4 5

void debut() { char choix; .... scanf("%d",&choix);

---

choix n'est pas un int, comme le suppose le format "%d" --> plantage.
Avec scanf, il faut que la variable de destination soit parfaitement en accord avec le format.
Pourquoi cette construction baroque pour cette fonction au lieu d'avoir :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

void debut() { int choix; do { MenuPrincipal(); scanf("%d",&choix); switch(choix) { case 1: menu_ajout_theme(); break; case 2: menu_deposer_sujet(); break; case 3: menu_afficher(); break; } getch(); }while(choix!=0); }

---

- void afficher_theme_ordre_alphabetique() :

Code:

---

1 2 3 4

char *copie[n]; if(n==0) printf("aucun theme n'a encore ete saisi"); else {

---

Le test est trop tard : si n==0 on a essayé de créer un tableau de taille nulle ce qui n'est pas permis

Code:

---

1 2 3 4

if(n==0) printf("aucun theme n'a encore ete saisi"); else { char *copie[n];

---

Ceci ne peut pas marcher :

Code:

---

1 2	char *p=(char *)malloc(sizeof(char)); strcpy(p,T[i]->nomdetheme);

---

On n'a réservé que 1 char avec le malloc(), juste de quoi mettre le '\0' terminal. Le strcpy() va faire planter sauf si la chaine à copier est...vide

-

Code:

---

1 2 3 4

void tri_nom(char *tab[]) : char *ech; .... strcpy(ech,tab[i]);

---

va planter : on n'a réservé cette fois aucune place pour faire la copie

etc.

voila le code aprés modification .mais je trouve tjr pas le problème de la fonction repondre (j'ai essayer de voir si le probleme étai dans la recherche du msg )je trouve pas !!

Tu n'as pas pris en compte ces remarques importantes :
Ceci fait planter le programme

Citation:

---

- void afficher_theme_ordre_alphabetique() :
.....

Code:

---

1 2	char *p=(char *)malloc(sizeof(char)); strcpy(p,T[i]->nomdetheme);

---

---

ni celle-ci qui va faire aussi planter le programme

Citation:

---

Code:

---

1 2 3 4

void tri_nom(char *tab[]) : char *ech; .... strcpy(ech,tab[i]);

---

---

Et il y a toujours ces globales !!!

pour la première remarque j'ai mis l'allocation de la mémoire à l'interieur du while

Code:

---

1 2 3 4 5 6 7 8 9

while(i<n) { char *p=(char*)malloc(sizeof(char)); strcpy(p,T[i]->nomdetheme); copie[i]=p; i++;   }

---

sinon pour le seconde je ne vois pas ce que je peux faire
ech est bien un pointeur sur char
et tab[i] aussi nn ??


pour les variables globales aussi ,elles sont utilisées tout au long du programme ,je ne vois pas comment les enlever !!

Citation:

---

pour la première remarque j'ai mis l'allocation de la mémoire à l'interieur du while

---

Mais chaque allocation alloue 1 unique byte. Dans un tel tableau, on ne peut mettre qu'une chaine vide

Citation:

---

ech est bien un pointeur sur char

---

Mais un pointeur n'est pas fait pour stocker des chaines de caractères mais pour stocker des adresses. Il n'y a pas de tableau dans lequel stocker les caractères.

Citation:

---

et tab[i] aussi nn ??

---

Oui, mais tab[i] est censé contenir l'adresse d'un tableau de caractères. C'est dans ce tableau que le strcpy() va chercher les caractères à copier et il veut les copier à l'adresse qui est dans ech. Mais dans ech, il n'y a pas l'adresse du début d'un tableau !

Citation:

---

Mais chaque allocation alloue 1 unique byte. Dans un tel tableau, on ne peut mettre qu'une chaine vide

---

pour cela il suffit de mettre une allocation de mémoire de 30*(sizeof(char)) (30 étant le nombre max de caractère du thème ,c'est cela ??

Code:

---

1 2 3 4 5 6 7 8

while(i<n) { char *p=(char*)malloc(sizeof(char)*30); strcpy(p,T[i]->nomdetheme); copie[i]=p; i++; }

---

sinon pour le ech je lui ai aussi allouer de la mémoire

Code:

---

char *ech=(char*)malloc(sizeof(char));

---

c'est cela ???

Citation:

---

pour cela il suffit de mettre une allocation de mémoire de 30*(sizeof(char)) (30 étant le nombre max de caractère du thème ,c'est cela ??

---

OUi, si T[i]->nomdetheme ne peut excéder 29 caractères + le zéro terminal.
Maintenant, il est possible d'attribuer la taille suffisante et nécessaire à ce stockage qui est de strlen(T[i]->nomdetheme)+1 :

Code:

---

1 2	char *p = malloc(strlen(T[i]->nomdetheme)+1); // sizeof(char) vaut toujours 1 strcpy(p,T[i]->nomdetheme);

---

Citation:

---

sinon pour le ech je lui ai aussi allouer de la mémoire

Code:

---

char *ech=(char*)malloc(sizeof(char));

---

c'est cela ???

---

:calim2:

On vient de dire, au post précédent, que ceci alloue 1 caractère.
Ce n'était pas assez hier pour y mettre une chaine non vide et il n'y a aucune raison pour que ce soit assez aujourd'hui.
Cette erreur va se répéter combien de fois ? :mur:

:furieux:

voila mon programme aprés modifications mais la fonction repondre ne marche tjr pas. il y'a un problème au niveau de la fontion trouver_message