Tableaux et fonctions

Version imprimable

Bonjour j'ai fait un programme avec un menu sur les tableaux et fonctions :
- Saisie du nombre maximum de cas du tableau de flottants générés aléatoirement, avec saisie du minimum et du maximum souhaités
- Affichage du tableau généré
- Affichage à l'envers du tableau généré
- Inversion du tableau généré
- Recherche du maximum du tableau
- Recherche du max et du min
- Recherche des deux max
- Tri croissant du tableau

Est-ce que vous pouvez me dire si je peux améliorer mon programme ? Merci !

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <float.h>
 
#define NMAX 50
 
void affmenu()
{
  printf("\n\n\t\t\tTABLEAUX ET FONCTIONS\n\n");
  printf("\t\t1...Affichage a l'envers d'un tableau de flottants\n\n");
  printf("\t\t2...Inversion d'un tableau de flottants\n\n");
  printf("\t\t3...Recherche du maximum d'un tableau de flottants\n\n");
  printf("\t\t4...Recherche du maximum et du minimum d'un tableau de flottants\n\n");
  printf("\t\t5...Recherche des deux plus grandes valeurs d'un tableau de flottants\n\n");
  printf("\t\t6...Tri croissant d'un tableau de flottants\n\n");
  printf("\t\t7...Sortie\n\n\n");
  printf("\t\t\tVotre choix : ");
}
 
void saisie(double tableau[],int* n)
{
    int i,ok,max,min;
 
    do
    {
     printf( "Entrer le nb d'elements du tableau ( <= %d ) : ", NMAX );
     ok = scanf( "%d", n );
	 while( getchar( ) != '\n' );
   }while( !ok || *n < 0 || *n > NMAX );
 
    do
    {
     printf( "Entrer la valeur max des flottants generes ( <= %e ) : ", DBL_MAX );
     ok = scanf( "%d", &max );
	 while( getchar( ) != '\n' );
   }while( !ok ||  max > DBL_MAX );
 
    do
    {
     printf( "Entrer la valeur min des flottants generes ( >= %e ) : ", -DBL_MAX );
     ok = scanf( "%d", &min );
	 while( getchar( ) != '\n' );
   }while( !ok || min < -DBL_MAX);
 
    srand (time (NULL));
    for(i=0;i<*n;i++)
        tableau[i]=(rand()/(double)RAND_MAX )*(max-min)+min;
}
 
void affiche(double tableau[],int n)
{
    int i;
    printf("Voici le tableau genere : \n");
    for(i=0;i<n;i++)
    {
        printf("%lf\t",tableau[i]);
        if ( (i+1) % 10 == 0 ) printf( "\n" );
    }
 
}
 
void affiche_envers(double tableau[],int n)
{
    int i;
    for (i=n-1;i>=0;i--)
        printf("%lf\t",tableau[i]);
}
 
void inversion(double tableau[],int n)
{
    int i;
    double temp;
    for (i=0;i<(n/2);i++)
    {
        temp=tableau[i];
        tableau[i]=tableau[n-1-i];
        tableau[n-1-i]=temp;
    }
}
 
double recherche_max(double tableau[],int n)
{
    int i;
    double max;
 
    max=tableau[0];
    for(i=0;i<n;i++)
        if(tableau[i]>=max)
            max=tableau[i];
    return (max);
}
 
void recherche_max_min(double tableau[],int n,double* min,double* max)
{
    int i;
 
    *max=*min=tableau[0];
    for(i=0;i<n;i++)
    {
        if(tableau[i]>=*max)
            *max=tableau[i];
        if(tableau[i]<=*min)
            *min=tableau[i];
    }
}
 
void recherche_max1_max2(double tableau[],int n,double* max1,double* max2)
{
    int i,indice;
 
    if(tableau[0]>=tableau[1])
    {
        *max1=tableau[0];
        *max2=tableau[1];
    }
    else
    {
        *max1=tableau[1];
        *max2=tableau[0];
    }
 
    for(i=2;i<n;i++)
    {
        if(tableau[i]>=*max1)
        {
            if(tableau[i]>*max2)
            {
                *max2=*max1;
                *max1=tableau[i];
            }
            else
                *max1=tableau[i];
        }
        else if (tableau[i]>*max2)
            *max2=tableau[i];
    }
}
 
void tri_selection(double tableau[],int n)
{
    int i,j,indice;
    double petit,temp;
 
    for(i=0;i<n-1;i++)
    {
        indice=i;
        petit=tableau[indice];
        for(j=i+1;j<n;j++)
        {
            if(tableau[j]<petit)
            {
                indice=j;
                petit=tableau[indice];
            }
        }
        tableau[indice]=tableau[i];
        tableau[i]=petit;
    }
}
 
void tri_bulles(double tableau[],int n)
{
    int i,j;
    double temp;
 
    i=0;
    while(i<n)
    {
        j=0;
        while(j<n-1-i)
        {
            if(tableau[j]>tableau[j+1])
            {
                temp=tableau[j];
                tableau[j]=tableau[j+1];
                tableau[j+1]=temp;
            }
            j=j+1;
        }
        i=i+1;
    }
}
 
int main()
{
    int n,ok,choix;
    double max,min,max1,max2;
    double tableau[NMAX];
 
    saisie(tableau,&n);
    affiche(tableau,n);
 
    do
    {
        do
        {
          affmenu();
          ok=scanf("%d",&choix);
          while( getchar( ) != '\n' );
        }while( !ok || choix < 1 || choix > 7 );
        switch ( choix )
        {
            case 1:
            {
                affiche_envers(tableau,n);
                break;
            }
 
            case 2 :
            {
                inversion(tableau,n);
                affiche(tableau,n);
                break;
            }
 
            case 3 :
            {
                max=recherche_max(tableau,n);
                printf("maximum=%lf\n",max);
                break;
            }
 
            case 4 :
            {
                recherche_max_min(tableau,n,&min,&max);
                printf("minimum=%lf maximum=%lf",min,max);
                break;
            }
 
            case 5 :
            {
                recherche_max1_max2(tableau,n,&max1,&max2);
                printf("max1=%lf max2=%lf\n",max1,max2);
                break;
            }
 
            case 6 :
            {
                tri_selection(tableau,n);
                affiche(tableau,n);
                break;
            }
 
            case 7 : break;
 
            default : printf("ERREUR DE CHOIX ( 1 a 7 )\n");
        }
    }while(choix!=7);
 
    return 0;
}

08/12/2015, 13h25
dalfab

Bonjour,

L'améliorer en quoi ?
Y ajouter des fonctionnalités ou bien améliorer sa lisibilité.
Pour la seconde possibilité : c'est clair, c'est propre, c'est robuste. Ca manque peut-être un peu de commentaires.
Pour la première : ajouter le remplissage aléatoire d'un tableau, l'ajout de vecteurs,...

Le test tri_bulles() n'est pas appelé, mais j'ai l'impression qu'il manque une étape.
08/12/2015, 15h39
benjamin_malaussene

Bonjour Dafalb,

Je voulais savoir s'il était possible de l'améliorer au niveau du code. Et sinon le tri à bulles marche, je l'ai testé, je l'ai laissé quand même. Oui il faut que je prenne l'habitude de commenter !!
Pour le remplissage aléatoire, c'est fait dans mon code, qu'entends tu par là ? Et par l'ajout de vecteurs ?

Merci !
08/12/2015, 16h11
dalfab

Lecture trop rapide de ma part, je n'avais pas vu la géneration aléatoire des tableaux.

En ajout de fonctionnalité si on pose : tableau <=> vecteur
On peut alors ajouter : Addition de 2 tableaux de même taille, ajout d'une constante à un tableau, multiplication scalaire, multiplication par une constante, etc...

Sincérement pour l'amélioration du code lui-même, c'est clean.
Exemple : on ne devrait jamais utiliser de constante dans un code. j'essaie de chipoter en disant que le 10 pourrait être un define NB_ELEMENT_PAR_LIGNE.
Il serait plutôt 'trop' que 'pas assez'.

Attention : s'il existait un code qui faisait exactement ce que l'on veut sans être buggué, son prix serait infini :lol:

Citation:

Envoyé par benjamin_malaussene

Tu pourrais faire quel que chose de ce genre:

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
 
choix=1;    
  while ( choix )
        { if(choix==1)
                     affiche_envers(tableau,n);
           else if(choix==2)
                     { inversion(tableau,n);
                        affiche(tableau,n);
                      }
           else if(choix==3)
                      {    max=recherche_max(tableau,n);
                           printf("maximum=%lf\n",max);
                      }
           else if(choix==4)
                      {  recherche_max_min(tableau,n,&min,&max);
                          printf("minimum=%lf maximum=%lf",min,max);
                       }
           else if(choix==5)
            {  recherche_max1_max2(tableau,n,&max1,&max2);
                printf("max1=%lf max2=%lf\n",max1,max2);
            }
           else if(choix==6)
            {  tri_selection(tableau,n);
                affiche(tableau,n);
            }
       }

et ta fonction d'affichage va devenir

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void affmenu()
{
  printf("\n\n\t\t\tTABLEAUX ET FONCTIONS\n\n");
  printf("\t\t1...Affichage a l'envers d'un tableau de flottants\n\n");
  printf("\t\t2...Inversion d'un tableau de flottants\n\n");
  printf("\t\t3...Recherche du maximum d'un tableau de flottants\n\n");
  printf("\t\t4...Recherche du maximum et du minimum d'un tableau de flottants\n\n");
  printf("\t\t5...Recherche des deux plus grandes valeurs d'un tableau de flottants\n\n");
  printf("\t\t6...Tri croissant d'un tableau de flottants\n\n");
  printf("\t\t0...Sortie\n\n\n");
  printf("\t\t\tVotre choix : ");
}

Citation:

Envoyé par Abacar94

Tu pourrais faire quel que chose de ce genre:

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
choix=1;    
  while ( choix )
        { if(choix==1)
                     affiche_envers(tableau,n);
           else if(choix==2)
                     { inversion(tableau,n);
                        affiche(tableau,n);
                      }
           else if(choix==3)
                      {    max=recherche_max(tableau,n);
                           printf("maximum=%lf\n",max);
                      }
           else if(choix==4)
                      {  recherche_max_min(tableau,n,&min,&max);
                          printf("minimum=%lf maximum=%lf",min,max);
                       }
           else if(choix==5)
            {  recherche_max1_max2(tableau,n,&max1,&max2);
                printf("max1=%lf max2=%lf\n",max1,max2);
            }
           else if(choix==6)
            {  tri_selection(tableau,n);
                affiche(tableau,n);
            }
       }

Bonjour

Tu ne penses pas qu'un switch/case est quand-même un outil bien approprié pour ce cas là ???

Citation:

Envoyé par Abacar94

et ta fonction d'affichage va devenir

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void affmenu()
{
  printf("\n\n\t\t\tTABLEAUX ET FONCTIONS\n\n");
  printf("\t\t1...Affichage a l'envers d'un tableau de flottants\n\n");
  printf("\t\t2...Inversion d'un tableau de flottants\n\n");
  printf("\t\t3...Recherche du maximum d'un tableau de flottants\n\n");
  printf("\t\t4...Recherche du maximum et du minimum d'un tableau de flottants\n\n");
  printf("\t\t5...Recherche des deux plus grandes valeurs d'un tableau de flottants\n\n");
  printf("\t\t6...Tri croissant d'un tableau de flottants\n\n");
  printf("\t\t0...Sortie\n\n\n");
  printf("\t\t\tVotre choix : ");
}

C'est quand-même dommage une fonction qui affiche un menu sans renvoyer le choix de l'utilisateur !!!