ctx_man
Messages postés285Date d'inscriptionmardi 28 décembre 2004StatutMembreDernière intervention20 janvier 20133 26 févr. 2010 à 17:49
Toutes tes boucles for sont fausses.
Les indexes de tableau ca commence à 0, pas à 1, tu sautes à chaque fois le premier élément.
De plus, quand tu arrive à i = n, tu lis un élément hors tableau.
Une boucle de parcourt de tableau de dimension N : for(int i = 0; i < N; i++)
Personnellement sur la fonction de suppression je m'attend a donner l'index à supprimer et non pas la valeur, puisque le tableau peut contenir plusieurs fois la même valeur.
Tu n'as fait aucune protection contre l'entrée utilisateur, ce qui rends tes fonctions ultra dangereuses. Exemple, ta fonction d'ajout, rien n'empêche l'utilisateur de dire que le tableau fait 1 000 000 000 d'éléments. Or, la taille de ton tableau est fixe, puisque définie en dur à 10. Ce qui rend d'ailleur les fonctions d'ajout et de suppression très peu logiques puisque tu ne redimensionne pas ton tableau.
26 févr. 2010 à 17:49
Les indexes de tableau ca commence à 0, pas à 1, tu sautes à chaque fois le premier élément.
De plus, quand tu arrive à i = n, tu lis un élément hors tableau.
Une boucle de parcourt de tableau de dimension N : for(int i = 0; i < N; i++)
Personnellement sur la fonction de suppression je m'attend a donner l'index à supprimer et non pas la valeur, puisque le tableau peut contenir plusieurs fois la même valeur.
Tu n'as fait aucune protection contre l'entrée utilisateur, ce qui rends tes fonctions ultra dangereuses. Exemple, ta fonction d'ajout, rien n'empêche l'utilisateur de dire que le tableau fait 1 000 000 000 d'éléments. Or, la taille de ton tableau est fixe, puisque définie en dur à 10. Ce qui rend d'ailleur les fonctions d'ajout et de suppression très peu logiques puisque tu ne redimensionne pas ton tableau.