Tableau à valeurs aléatoires et tri à bulles

Bonjour, je suis un débutant en language et j'ai cet exercice à rendre seulement je suis perdu... C'est l'exercice 11 du fichier que je vous joint. J'ai essayer mais en vain, ce doit être plein d'erreurs :

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 25 26 27 28 29 30 31 32 33 34 35 36 37 38

#include <stdio.h> #include <stdlib.h>   int main() { int n=10, A[100]; alea (A,n); affichage (A,n); Tri (A,n); affichage (A,n); return (0); } void alea(int A[],int n) { int i; srand(1); for (i=0;i<n;i++) A[i]=rand()/100; } void Tri (int A[],int n ) { int j; int i; for( j=0,j<n-1,j++) for(i=0,i<n-j-1,i++) if(A[i]>A[i+1])echange (&A(i),&A(i+1)); } void echange (int*x,int*y) { int k,h; h=*x; *x=*y; *y=k; } void affichage(int n,float A[100]) { int i; for (i=0;i< n;i++) printf("A[%d] = %f\n",i,A[i]); }

---

Bonjour,

après un coup d'œil en diagonale il y a plusieurs points à regarder :

• Utilise les balises CODE pour insérer du code dans un message
• Dans la fonction Tri la notation A(i) est incorrecte pour accéder à l'élément i du tableau A (le compilateur doit râler)
• Dans la fonction echange : c'est quoi k ? le compilateur doit râler aussi. Un simple test manuel doit te montrer qu'elle ne fonctionne pas correctement.
• Dans la fonction affichage ... partout tu as des tableaux d'int et là tu utilises un tableau de float ? pas cohérent et le compilateur doit râler aussi
• En plus du typage il faut faire attention à l'ordre des paramètres (cf affichage ...). C'est bien de garder une certaine cohérence (d'abord le tableau puis la taille comme dans l'exemple du TD).
• Déclare correctement tes fonctions en dehors du main

Merci pour ton aide ! Mon travail est donc bourré de fautes ! Je suis complètement paumé dans le language C, je débute. Je vais tenter de modifier tout ca

srand(1) va renvoyé à chaque exécution du programme les même nombres aléatoires, je conseille plutôt

Code:

---

srand( time(0) );

---

Bonjour,

Citation:

---

Envoyé par fred1599

srand(1) va renvoyé à chaque exécution du programme les même nombres aléatoires,[...][/code]

---

Ce qui est pratique en mode debug :)

Citation:

---

Ce qui est pratique en mode debug

---

Ah ok :lol:

[code]srand( time(0) );

Personellement,
je crois que votre fonction est un peu lente...et je trouve celle plus rapide
void alea(k)
{
int i;
for(i=0;i<k;i++)
T[i]=rand();
}

:zoubi:

Citation:

---

Envoyé par oualidosphelix

[code]srand( time(0) );

Personellement,
je crois que votre fonction est un peu lente...et je trouve celle plus rapide
void alea(k)
{
int i;
for(i=0;i<k;i++)
T[i]=rand();
}

:zoubi:

---

Salut,

Pourquoi dis-tu que la tienne est plus rapide ? C'est la même mais en moins bien vu qu'il y a une utilisation de variable globale :aie: