Variable qui change de valeur "toute seule"

Bonjour,

j'ai un petit problème avec une variable qui avant l'appel d'une fonction vaut 7 et qui après vaut 3.

Non, évidemment je ne l'ai pas passé en référence, elle n'est qu'en entrée de la fonction.

Le plus drôle c'est que quand je l'affiche à la fin de la fonction elle vaut bien 7.

C'est dans le programme principal qu'elle vaut 3.

La variable en question s'appelle nbS

L'entête de la fonction :

Code:

---

1 2 3 4 5

void Mafonction( int tab[TailleMax][TailleMax], int nbS, int T[N][TailleMax]){   // J'affiche nbS à la sortie de cette fonction, elle vaut 7 }

---

Dans le programme principal :

Code:

---

1 2 3 4

cout<<"nbS = "<<nbS<<"\n"; // Ici nbS = 7 mafonction(tab, nbS, T); cout<<"nbS = "<<nbS<<"\n"; //Ici nbS = 3

---

Avez vous rencontré un problème similaire ?

Merci,

GDM.

Salut

Il nous manque des infos : si tu as alloué tes tableaux n'importe comment, l'écriture d'une valeur dedans peut écraser celle de nbS. Montre nous comment tu crées et initialise l'ensemble des variables utilisées, et aussi ce que fait la fonction :)

Le comportement est il systématique ? Qu'est ce que ça donne en mode debug ?

Donc,

J'ai défini mes constantes pour les bornes des tableaux :

Code:

---

1 2 3

#define TailleMax 50 #define N 1

---

Ma fonction est définie comme cela:

Code:

---

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

void mafonction( int tab[TailleMax][TailleMax], int nbS, int T[N][TailleMax]){   int i,j; int d;   cout<<"nbS = "<<nbS<<"\n"; \\ça c'est juste pour tester nbS   for(i=0; i<nbS;i++){ deg = Degre(tab, nbS,i);   T[0][i] = i; T[1][i] = d;   } cout<<"nbS = "<<nbS<<"\n"; \\ça c'est juste pour tester nbS   }

---

Dans le programme principal, voici comment je déclare les tableaux:

Code:

---

1 2 3 4

int tab[TailleMax][TailleMax]; int nbS; // Nbre d'éléments effectif de tab. int T[N][TailleMax];

---

Est-ce que cela vous suffit ? Par contre oui, la fonction mafonction appelle encore une autre fonction.... mais je n'ai pas eu de problème particulier avec celle ci, donc ce n'est peut-être pas la peine que je la décrive ?

Je n'ai pas essayé le mode debug, je vais voir ça.

Merci pour votre aide,

GDM

Hoa,

Avec N = 1, le seul premier indice de ton tableau T est 0 :

T[1][...] n'existe pas. :(

Ensuite, si ma mémoire est bonne, certains compilateur/Allocateur alloue la mémoire de façon contigüe (ça a peu être changé depuis la dernière fois ou je me suis penché sur le sujet sérieusement).

comme tu déclare

Citation:

---

int tab[TailleMax][TailleMax];
int nbS; // Nbre d'éléments effectif de tab.
int T[N][TailleMax];

---

nbS se situe en mémoire juste dérrière tab,
si jamais en parcourant tab tu dépasse les indice max tu ira écrire la mémoire qui est derrière, donc nbs puis T.
en fonction des compilateur/options de compilation parfois cela core dump, parfois cela passe inaperçu et deviens un bug "dormant" qui se réveille en général au mauvais moment.

As-tu utilisé un outil (type valgrind/purify/....) pour vérifier les fuites mémoires et autre choses joyeuse du même style?

Citation:

---

droggo Hoa,

Avec N = 1, le seul premier indice de ton tableau T est 0 :

T[1][...] n'existe pas.

---

Effectivement, c'est une erreur stupide, avec N = 2 ça marche très bien ! Merci beaucoup !

J'avais oublié qu'il fallait toujours réserver n+1 cases (en comptant le 0).

Ensuite,

Citation:

---

As-tu utilisé un outil (type valgrind/purify/....) pour vérifier les fuites mémoires et autre choses joyeuse du même style?

---

non, en fait, je ne programme pas très souvent, juste quand j'y suis contraint comme c'est le cas aujourd'hui, alors je ne connais pas ces outils. Mais merci de me les avoir signalés, ça pourra toujours me servir.

En tous les cas merci beaucoup, grace à vous ma variable reste bien à 7 !

GDM

Salut,

Citation:

---

Envoyé par GDMINFO

Effectivement, c'est une erreur stupide, avec N = 2 ça marche très bien ! Merci beaucoup !

J'avais oublié qu'il fallait toujours réserver n+1 cases (en comptant le 0).

---

D'où est ce que ca vient, ca :question:

Tu dois réserver n+1 cases pour... les chaines de caractères, parce que, classiquement, les fonctions C qui les manipulent considère que le caractère '\0' représente le ce qui suit le dernier caractère.

Par contre, si tu as besoin de N eniters ou de N Trucs, il n'y a strictement aucune raison de réserver N+1 éléments :P

Par contre, tu dois t'assurer que la logique prenne le cas particulier où N vaut 1 en compte...

Classiquement, étant donné que les tableaux C style ne connaissent pas leur taille, tu devrait donc, également, faire en sorte que la valeur de N soit transmise en argument à la fonction, et ne tenter le deuxième accès (T[1][i] = d; ) que... si N est exclusivement plus grand que 1 ;)

Bon bon bon... merci beaucoup pour ces infos supplémentaires...

La prochaine fois je réfléchirai davantage avant de parler :roll:

GDM.

Citation:

---

Envoyé par GDMINFO

Bon bon bon... merci beaucoup pour ces infos supplémentaires...

La prochaine fois je réfléchirai davantage avant de parler :roll:

GDM.

---

Excuses moi si je t'ai un peu semblé agressif dans ma réponse :P

Mais il me semblait important d'attirer ton attention sur un fait particulier:

Lorsque tu manipule une chaine de caractères "C style" (car, en C++, on préférera quand même utiliser la classe string, disponible dans l'espace de noms std), le caractère '\0' (caractère à valeur nulle :D) représente la fin de la "chaine utile" (la chaine qui représente effectivement quelque chose).

On pourrait assez facilement comparer la mémoire d'un ordinateur à une sorte de grande bande sur laquelle se trouvent les différentes cases mémoires, et il faut comprendre que les différentes fonctions qui manipulent les chaines de caractères "C style" se "baladent" sur cette bande d'une case à l'autre.

Le fait est que, lorsque les fonctions arrivent à la dernière adresse mémoire accessible sur le système, elle vont recommencer au début de celle-ci sans se poser plus de question que cela.

La seule chose qui fait que ces fonctions s'arrêtent, c'est la présence du caractère nul.

C'est ainsi, si tu "oublie" de donner la valeur 0 au caractère qui suit le dernier caractère utile de ta chaine que tu peut arriver, lors d'un simple affichage à l'affichage d'une série de caractères plus ou moins cabalistiques, suivi d'un "THIS PROGRAM CANNOT BE RUN blablabla", simplement parce que la fonction d'affichage n'a pas croisé le caractère nul et qu'elle a continué à afficher... la mémoire occupée par une autre application :aie:

Et si le malheur veut qu'il n'y ait aucun équivalent de caractère nul dans tout ce qui est assimilable à de la mémoire dans de ton ordinateur, tu est littéralement parti pour profiter de l'affichage en boucle de l'ensemble de ton système (mémoire, disques durs et périphériques compris) :aie:

Mais ce qu'il est important de comprendre, c'est que la présence de ce caractère nul n'est requise que... pour les fonctions de manipulation de chaines de caractères, parce que l'idée de base est de pouvoir se contenter de demander à l'utilisateur de passer un pointeur sur char sans forcément l'obliger à fournir la taille de la chaine de caractères en plus (bien qu'il existe des versions de ces fonctions qui le demande).

Autrement, si tu as besoin d'un tableau de N éléments, quels qu'ils soient, tu peux parfaitement le déclarer sous la forme de type tab[N].

tu disposera effectivement des N éléments, même si tu y accédera avec des indices compris entre 0 et N-1 ;)

Encore une fois merci beaucoup de ces compléments fort intéressants :ccool:

Et pas de soucis, je n'avais pas trouvé la précédente réponse "agressive", c'est juste que je ne me suis pas trouvé très malin sur ce coup là, d'où ma réponse ;)

GDM.