fonction d'échange de 2 structures qui prend trop de resource : segmentation fault

Bonjour,

mon code

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

struct structure_student { char name[50]; int size; bool sex; //sex=0 it's a boy -- sex=1 it's a girl };   //****** function exchangeStudent ***** //* use to exchange the position of 2 students in a table of structure_student void exchangeStudent(structure_student* studentS, int a, int b) { structure_student temp; //put the data of the student a into the temp strcpy(temp.name, studentS[a].name); temp.size=studentS[a].size; temp.sex=studentS[a].sex; //put the data of the student b into the data of the student b strcpy(studentS[a].name, studentS[b].name); studentS[a].size=studentS[b].size; studentS[a].sex=studentS[b].sex; //put the data of the temp into the data of the student b strcpy(studentS[b].name, temp.name); studentS[b].size=temp.size; studentS[b].sex=temp.sex; }

---

Ma fonction exchangeStudent marche très bien si je l'appel une dizaine de fois mais si je l'appelle quelque chose comme 200 fois, et ben :evilred: Segmentation Fault !!!

Alors je comprend pas, peut être que mon "structure_student temp" ne de détruit pas à chaque foi, du coup il reste en mémoire ?

Faut-il le déclarer en dynamique, puis le détruire ?
et alors comment faire ?

Merci pour toute réponse

Bonjour,

Je pense que le plantage vient des paramètres que tu donnes à ta fonction.
En particulier, si tu lui donne à manger des indices invalides de ton tableau, ça a des chances de planter.

par exemple :

Code:

---

1 2 3 4 5 6 7

structure_student a[10]; // ceci peut planter car le nom est trop long : a[0].name = "123456789012345678901234567890123456789012345678901"; // ceci peut planter à cause du 45 : exchangeStudent(&a[0], 4, 45); // ceci peut planter à cause du 10 : exchangeStudent(&a[0], 4, 10);

---

Es-tu en train d'apprendre le C++ ?
Est-ce à partir d'un livre ? d'un cours ?

Si tu es en train d'apprendre à gérer la mémoire, je comprends que tu codes comme ça et c'est très bien de commencer par se heurter un petit peu aux problèmes d'assignations de mémoire.

Mais attention : une fois que tu maitriseras les allocations, les pointeurs, les tableaux, il faudra absolument arréter de programmer comme ceci. Ton code présente d'énormes risques d'erreurs, et c'est justement dans une de ces erreurs que tu as du tomber.

Un code parfait ne peut pas planter, même s'il est mal utilisé par quelqu'un qui ne connais rien à son fonctionnement. Un code correct essaie d'être parfait.

Par exemple, ta fonction :

Code:

---

void exchangeStudent(structure_student* studentS, int a, int b)

---

suppose que le :

• studentS pointe sur un tableau de student alloué (et pas sur NULL ou sur une zone émmoire non-allouée)
• a et b sont des indices valides pour le dit tableau

Courage

Merci pour réponse,

Oui je suis en train d'aprendre le C++.

Effectivement, il y a des grosses lacunes dans mon code. Est-ce que tu peux me donner quel que truc pour pouvoir proteger mon code ?

Merci !

Si tu fais du C++, envisage d'utiliser les outils du C++, à savoir par exemple les STL Containers pour les tableaux dynamiques, listes, ...
Voila le site que j'utilise lorsque j'oublie des détails à propos des fonctions, ... http://www.cplusplus.com/reference/stl/
Mais c'est pas l'idéal pour une première aproche je pense.
Et puis il existe les transferts de variables par référence en c++.

Bonne continuation

Greg