Discussion :

[lewisdu76]

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 :

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

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]);
}

[kwariz]

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

[lewisdu76]

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

[fred1599]

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

Code :

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

srand( time(0) );

[kwariz]

Bonjour,

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

Ce qui est pratique en mode debug

[fred1599]

Ce qui est pratique en mode debug

Ah ok

[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();
}

[Trademark]

[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();
}

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