Discussion :

[Darknesse]

Quelqu'un peut-il m'aider? Je ne trouve vraiment pas mon erreur...

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
39
40
41
42
43
44
45
46
/* Déterminer si un vecteur v de taille n est trie par ordre croissant ou non */
 
#include <stdio.h>
 
void ordre(int v[], int n)
 
{
    int i, a = 1;
 
    for (i = 0; i < n; i++ )
    {
        if (v[i] <= v[i+1])
            a += 1;
 
    }
 
    if (a == n)
       printf ("Le vecteur est trie par ordre croissant");
 
    else
        printf ("Le vecteur n'est pas trie par ordre croissant");
 
    return;
}
 
/* Essayons la fonction avec un exemple concret */
 
int main ()
 
#define TAILLE_VECTEUR_MAX 99
 
{
    int t[TAILLE_VECTEUR_MAX], b;
 
    printf ("Entrez la taille du vecteur: ");
    scanf ("%d", &b);
 
    printf ("Entrez les composantes du vecteurs:");
    scanf ("%d", &t);
 
    ordre (t, b);
 
    return 0;
}
 
/* Erreur quelquepart, il dit toujours que le vecteur n'est pas trie par ordre croissant */

[matafan]

1) Si ton tableau a n éléments, v[i+1] n'existe pas pour i = n-1
2) Tu n'enregistres qu'une valeur dans ton tableau. Il te faut une boucle pour lire les n données.

A part ça, tu peux arrêter la boucle dès que tu trouve deux éléments non triés. Pas la peine de continuer à comparer les éléments suivants, le tableau n'est de toute façon pas trié.

[Darknesse]

1) Si ton tableau a n éléments, v[i+1] n'existe pas pour i = n-1

for (i = 0; i <= n; i++ )
{
while (v[i] <= v[i+1])
a += 1;

En mettant le "<=" ça irait alors?

2) Tu n'enregistres qu'une valeur dans ton tableau. Il te faut une boucle pour lire les n données.

for (i = 0; i <= b ; i++ )
ordre (t[i], b);

De cette façon là il affiche une erreur.
Je ne vois pas ce que tu veux dire, ou plutôt, comment faire...?!

[matafan]

En mettant le "<=" ça irait alors?

Je ne sais pas. Tu devrais essayer de taper des trucs au hasard jusqu'à ce que ça tombe en marche.

Pour ma deuxième remarque, je parlais du scanf("%d", &t). Il n'enregistre que la première valeur. Il ta faut une boucle pour lire les b valeurs entrées par l'utilisateurs, et les stocker dans les &(t[i]).

[Invité]

Bonsoir,

En mettant le "<=" ça irait alors?

Au contraire: avec ça, la boucle for fera n+1 itérations au lieu de n-1.

for (i = 0; i < n-1; i++ )

[quetzacoatl]

En fait ce qui te manque surtout c'est le vidage du buffer d'entree, je m'explique lorsque que tu fais:

scanf ("%d", &b);

ce scanf ne va pas prendre le '\n' qui correspond à la touche entrée que tu vas presser pour signifier la fin de ta saisie.
Ainsi tu dois faire:

Code :

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

getchar();

Pour pouvoir prendre ce caractère. Cela suppose quand même que l'utilisateur sait ce qu'il fait, le mieux est de faire:

Code :

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

while(getchar()!='\n');

dans le cas des entrées érronées.
Après comme le disent les autres intervenants, il te faut également une boucle pour pouvoir remplir ton tableau.

[diogene]

En fait ce qui te manque surtout c'est le vidage du buffer d'entree, je m'explique lorsque que tu fais:

Code :

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

scanf ("%d", &b);

ce scanf ne va pas prendre le '\n' qui correspond à la touche entrée que tu vas presser pour signifier la fin de ta saisie.
....

C'est vrai, mais sans importance pour une boucle sur un tel scanf("%d",...) : les 'blancs' qui précèdent le nombre sont ignorés dans la conversion et le '\n' est considéré comme un caractère 'blanc'.
Par contre, ce '\n' peut poser problème pour un scanf() derrière la boucle avec un format %c ou %[].