OPEARTIOS SUR TABLEAU

ctx_man Messages postés 285 Date d'inscription mardi 28 décembre 2004 Statut Membre Dernière intervention 20 janvier 2013 - 26 févr. 2010 à 17:49
ctx_man Messages postés 285 Date d'inscription mardi 28 décembre 2004 Statut Membre Dernière intervention 20 janvier 2013 - 26 févr. 2010 à 17:49
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/51355-opeartios-sur-tableau

ctx_man Messages postés 285 Date d'inscription mardi 28 décembre 2004 Statut Membre Dernière intervention 20 janvier 2013 3
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.
Rejoignez-nous