[Debutant] [Algo] [Debug]

**IDE** · 13/02/2006, 00h26

Voici mon code

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
 
 /* Données en entrée */
 
       int nbr = 0;
       int max = 0; 
       int min = 0;
 
    /* Résultats */
 
    /* Données modifiées */
 
    /* Données de travail */
 
    /* Traitement */
 
    /* Entrée des données */    
 
       while ( nbr != -1 )
{
 
        nbr = Lire.i(" entrez le nombre : ");
 
       if (nbr == -1)
   {
        break;
   }
 
        if ( nbr > max )
    {
          max = nbr;
    }
 
}
       if ( nbr < max )
       {
          min = nbr;       
       }
    /* Calculs */
 
    /* Sortie des résultats */
 
       System.out.println("La valeur de max : " + max);
       System.out.println("la valeur de min : " + min);

et voici mon résultat

entrez le nombre : 4
entrez le nombre : 5
entrez le nombre : -9
entrez le nombre : -1
La valeur de max : 5
la valeur de min : -1

ici je ne veux pas qu'il me compte le -1, donc le plus petit est -9

j'espere que j'ai été clair merci pour toutes vos réponses

Michael

[ Modéré par Viena ]
Ajout d'un tag et modification du titre : Pour la compréhension de tous, utilisez un titre clair et les tags ! merci.

Les Règles du Forum

**sylk974** · 13/02/2006, 00h42

Code :

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

1
2
3
4
5
 
if ( nbr < MAX )
{
   min = nbr;       
}

ca ca marchera beaucoup mieux je pense

Code :

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

1
2
3
4
5
 
if ( nbr < MIN )
{
   min = nbr;       
}

De plus le fait que cette portion de code fait qu'il est exécuté même en dehors de la boucle ... Indentes bien ton code, ca evitera ce genre de problème

**IDE** · 13/02/2006, 00h56

Merci pour ta réponse mais j'ai tjr le meme probleme, regarde ceci

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
 
/* Données en entrée */
 
       int nbr = 0;
       int max = 0; 
       int min = 0;
       int PrePosMax = 0;
       int i;
    /* Résultats */
 
    /* Données modifiées */
 
    /* Données de travail */
 
    /* Traitement */
 
    /* Entrée des données */  
 
 
 
       for ( i = 1 ; nbr != -1 ; i++ ){
 
        nbr = Lire.i(" entrez le nombre : ");
 
        if ( nbr > max ){
 
          max = nbr;
          PrePosMax = i;
    }
 
}
         if ( nbr < min ){
 
            min = nbr;       
       }
    /* Calculs */
 
    /* Sortie des résultats */
 
       System.out.println("La valeur de max : " + max);
       System.out.println("la valeur de min : " + min);
       System.out.println("Max est à la position : " + PrePosMax);

et voici le résultat

entrez le nombre : 5
entrez le nombre : 3
entrez le nombre : 8
entrez le nombre : 1 Voici la valeur MIN
entrez le nombre : -1
La valeur de max : 8
la valeur de min : -1 je ne veux pas qu'il me dise que c'est - 1 la valeur min car cette valeur me sert simplement pour sortir du Pgr
Max est à la position : 3

**sylk974** · 13/02/2006, 01h06

Comme je le disais au message précédent

Code :

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

1
2
3
4
5
 
if ( nbr < min ){
 
            min = nbr;       
       }

n'est pas dans la boucle.

min n'est donc pas mis à jour.
quand on sort de la boucle, min = 0, nbr = -1
donc il fait if nbr < min
ce qui est le cas vu (-1 < 0)
donc il fait l affectation min = nbr; // avec nbr = -1;

Regardes tes accolades, la mise à jour de min n'est pas dans la boucle.

**IDE** · 13/02/2006, 01h23

Je me permet de t'écrire par ici car meme si je met min dans la boucle le probleme est tjr le meme

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
 
/* Entrée des données */  
 
 
 
       for ( i = 1 ; nbr != -1 ; i++ ){
 
        nbr = Lire.i(" entrez le nombre : ");
 
        if ( nbr > max ){
 
          max = nbr;
          PrePosMax = i;
        }  
 
         else if ( nbr == max ){ // si max est égale a nbr
 
          effmax = effmax + 1;
       }
 
          if ( nbr < min ){
 
            min = nbr;       
       }
}               
    /* Calculs */
 
    /* Sortie des résultats */
 
       System.out.println("La valeur de max : " + max);
       System.out.println("la valeur de min : " + min);
       System.out.println("Max est à la position : " + PrePosMax);
       System.out.println("\neffectif du maximum est : " + effmax);

Résulat
---------

entrez le nombre : 8
entrez le nombre : 8
entrez le nombre : 8
entrez le nombre : 8
entrez le nombre : 0
entrez le nombre : -1
La valeur de max : 8
la valeur de min : -1
Max est à la position : 1

effectif du maximum est : 4

je ne comprend vraiment pas, on va dire que je suis débutant hihihi, merci pour tes réponses c'est vraiment super sympa de ta part

**sylk974** · 13/02/2006, 01h30

ta condition de ta boucle for nbr != -1
est evalué en début de boucle

quand tu vas saisir -1
tu vas continuer dans ta boucle
et donc passer par la condition

if nbr < min

etant donné que ton min est initialisé à 0
il fera l effectation min = -1
et sortira de la boucle juste après.

la tu pourras remettre la condition if (nbr == -1) break; apres le Lire
et il sortira de la boucle juste après que t'ais saisi -1

**IDE** · 13/02/2006, 01h58

Bon, revoici encore une fois mon code

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
 
 
for ( i = 1 ; nbr != -1 ; i++ ){
 
         nbr = Lire.i(" entrez le nombre : ");
 
        if ( nbr > max ){
 
          max = nbr;
          PrePosMax = i;
        }  
 
         else if ( nbr == max ){ // si max est égale a nbr
 
          effmax = effmax + 1;
       }
         if (nbr == -1){
             break;
         }
 
         if ( nbr < max ){
 
            min = nbr;       
       }
}

j'ai encore un petit probleme quand je met le min en premiere position, il n'y tiens pas compte ex :

entrez le nombre : 1
entrez le nombre : 2
entrez le nombre : 3
entrez le nombre : 4
entrez le nombre : 5
entrez le nombre : -1
La valeur de max : 5
la valeur de min : 0
Max est à la position : 5
Max apparait : 1 fois

par contre si je met min par ex a la 3 emes positon comme ceci

entrez le nombre : 3
entrez le nombre : 2
entrez le nombre : 1
entrez le nombre : 5
entrez le nombre : 5
entrez le nombre : -1
La valeur de max : 5
la valeur de min : 1
Max est à la position : 4
Max apparait : 2 fois

il me dit bien que min est 1

**Malo** · 13/02/2006, 03h16

Il faut mettre if(nbr<min) et non pas if(nbr<max) dans ta derniere condition.

**IDE** · 13/02/2006, 04h19

Quand je fais ceci sa ne fonctionne pas non plus, j'ai tjr la meme 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
 
 
 for ( int i = 1 ; nbr != -1 ; i++ ){
 
         nbr = Lire.i(" entrez le nombre : ");
 
        if ( nbr > max ){
 
          max = nbr;
          PrePosMax = i;
        }  
 
         else if ( nbr == max ){ // si max est égale a nbr
 
          effmax = effmax + 1;
       }
         if (nbr == -1){
             break;
         }
 
         if ( nbr < min ){
 
            min = nbr;       
       }

**benratti** · 13/02/2006, 11h01

ton probleme avec min = 0 est un probleme classique lors de la recherche du minimum ou du maximum. tu choisis un valeur par defaut pour initialiser ta valeur min et par malchance, cette valeur est plus petite ou plus grande que toute les autres a valeur. Pour regler ce genre deux problemes, tu as deux solutions possibles :

- soit tu initialises ton min et ton max a la premiere iteration de la boucle et non avant
- soit tu initialises min ( resp max ) avec la valeur la plus grande ( resp. la plus petite ) autorisé par le type de donnée que tu utilises. Comme ca, le min et le max vont forcement ce mettre a la premiere valeur.

**woorant** · 13/02/2006, 13h12

Salut! essaye ce code ci:

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
 
		int max = 0;
		int min = 0;
		int nbr = 0;
		int effmax = 0;
		int PrePosMax = 0;
 
		while (nbr != -1) {
			nbr = lire.i(" entrez le nombre : ");
                        ++PrePosMax;			
 
			if (nbr>max) {
				max = nbr;
			}else if (nbr == max){
				effmax = effmax + 1;
			}else if (nbr<min) {
				min = nbr;
			}
		}
                System.out.println("La valeur de max : " + max);
                System.out.println("la valeur de min : " + min);
                System.out.println("Max est à la position : " + PrePosMax);
                System.out.println("\neffectif du maximum est : " + effmax);

Bon courage!

**IDE** · 14/02/2006, 22h45

Bonjour, pouvez vous me montrez ou je dois initialisé max car je n'ai pas trop bien compris, voici mon code

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
47
48
49
50
51
52
53
 
 
/* Données en entrée */
 
       int nbr = 0;
       int max = 0;
       int min = 0;
       int PrePosMax = 0;
       int PrePosMin = 0;
       int effmax = 1;
 
    /* Résultats */
 
    /* Données modifiées */
 
    /* Données de travail */
 
    /* Traitement */
 
    /* Entrée des données */  
 
 
       for ( int i = 1 ; nbr != -1 ; i++ ){
 
         nbr = Lire.i(" entrez le nombre : ");
 
        if ( nbr > max ){
 
          max = nbr;
          PrePosMax = i;
        }  
 
         else if ( nbr == max ){ // si max est égale a nbr
 
          effmax = effmax + 1;
       }
         if (nbr == -1){
             break;
         }
 
         if ( nbr < min ){
 
            min = nbr;       
       }
}               
    /* Calculs */
 
    /* Sortie des résultats */
 
       System.out.println("La valeur de max : " + max);
       System.out.println("Max est à la position : " + PrePosMax);
       System.out.println("Max apparait  : " + effmax + " fois " );
       System.out.println("la valeur de min : " + min);

**woorant** · 15/02/2006, 09h46

Salut!
Essaye de retirer le "else" de

Code :

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

1
2
 
else if ( nbr == max ){ // si max est égale a nbr

ou met en a chaque condition ce qui serait celon moi, plus logique!

As tu essayé le bout de code que je t'es envoyé?*

Bon courage!

[Debutant] [Algo] [Debug]

Langage Java

Discussions similaires

Partager

Partager