Discussion :

[eclectique_95470]

Bonsoir, je suis actuellement en deuxième année de classe préparatoire en informatique et ai une série de programmes en C à coder.

Je suis actuellement sur un exercice de cryptage.
Dans cet exercice , on a un tableau de 26 lignes sur 5 colones.
Les poste 1 des colones contiennent tous codes ascii des lettres de a à z (97 à 122).

Je dois faire remplir ce tableau de code numériques générés aléatoirement. En sachant que si le poste reçoit 0, tous les postes de la même ligne succédant ce poste doivent contenir aussi 0.

L'algorithme ci dessous fonctionne bien pour les 9, 10 premières lignes mais après le programme plante et je ne sais pourquoi.

Si vous pouviez m'aider, ce serait vraiment cool. Cela fais 4 heures que je cherche mais sans réponses .....

[|PaRa-BoL]

Code :

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

tabcompare=realloc(tabcompare,1*sizeof(int));

realloc prend en paramètre la nouvelle taille entière et non pas un rajout.

[diogene]

Préférer la forme suivante qui permet de gérer le cas de l'échec de réallocation et d'éviter les fuites mémoires :

Code :

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

1
2
3
4
5
6
7
p = realloc(tabcompare,.....);
if(p != NULL)
{
   tabcompare = p;
   .......
}
else.... // erreur de realloc. l'ancienne allocation est toujours pointée par tabcompare

[|PaRa-BoL]

Préférer la forme suivante qui permet de gérer le cas de l'échec de réallocation et d'éviter les fuites mémoires :

Code :

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

1
2
3
4
5
6
7
p = realloc(tabcompare,.....);
if(p != NULL)
{
   tabcompare = p;
   .......
}
else.... // erreur de realloc. l'ancienne allocation est toujours pointée par tabcompare

Là n'est pas son problème, inutile de l'embrouiller
Par ailleurs je trouve assez discutable cette méthode concernant la lisibilité du code (à la limite en utilisant un wrapper).

[uknow]

Là n'est pas son problème, inutile de l'embrouiller

Bonjour,

Je ne pense pas que cela l'embrouillerait, c'est un moyen efficace pour enlever un risque (et indirectement une hypothèse sur le bogue rencontré) .

[jeroman]

Là n'est pas son problème, inutile de l'embrouiller
Par ailleurs je trouve assez discutable cette méthode concernant la lisibilité du code (à la limite en utilisant un wrapper).

Désolé de te contredire, mais c'est justement la bonne façon de faire. Quand on apprend à programmer, autant bien apprendre dès le départ, non ? Car si apprend mal dès le départ, c'est justement là que les sales habitudes apparaissent. S'il existe autant de logiciels buggués, ce n'est pas un hasard...

[|PaRa-BoL]

Je savais que j'allais me faire taper dessus en disant ça

Évidemment qu'il faut vérifier que l'allocation n'a pas échouée.
C'est un cas d'école et je suppose qu'il apprendra au fur et à mesure à gérer ce genre d'erreur.

A ce moment là quitte à montrer les bonnes manière dès le départ, autant montrer quelque chose de lisible/générique.

Son bug était clairement à un autre endroit.

En regardant son code, il y a d'autres trucs à revoir et à pointer du doigt mais qui ne sont pas en rapport avec le bug qu'il a rencontré.

[eclectique_95470]

Problème résolu, merci beaucoup.