mon while fonctionne pas comme je veux

Version imprimable

Bonjour
je m'explique en exécutant mon prg,
quand je saisi 3eme no:0,
il me demande 4eme no aussi;
normalement dès qu'on saisi le 0, la saisi des nombre doit arrêter.
veuillez m'aidez svp

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
 
import java.util.Scanner;
public class Testing3 
{	
	public static void main(String[] args) 
	{
			      int PG=0; //PG veut plus grand
			      int PPG=0;  //PPG veut dire position de plus grand
 
			      for(int i=1; i<5; i++)
			      {
			    	System.out.println("saissisez un no  :");
			        Scanner valeur=new Scanner(System.in);
			        //int N=valeur.nextInt();
			        int N;
					while ((N= valeur.nextInt()) != 0) 
					{
			          if(i==1 || N>PG)
			          {
			 		   PG=N;
			           PPG=i;
			          }
			         break;
			        }
			    } 
			    System.out.println("Le nombre le plus grand était: "+PG) ;
			    System.out.println("Il a été saisi en position numéro: "+PPG) ;
		}
	}

vous voyez mon output:
saissisez un no :
4
saissisez un no :
6
saissisez un no :
0
saissisez un no :
0
Le nombre le plus grand était: 6
Il a été saisi en position numéro: 2

02/06/2015, 13h36
eulbobo
Code:

1 2 for(int i=1; i<5; i++) {
Tu as fait une boucle à qui tu demandes de tourner 4 fois quoi qu'il arrive là. Faudrait peut-être rajouter une autre condition de sortie comme la dernière saisie

Et dans la structure de ton programme, vu que tu veux que la saisie se fasse au moins une fois, tu devrais t'orienter vers une boucle de type do{...} while();

Ton while n'est pas nécessaire est n'est pas logique dans ton cas.

Ce que tu veux, c'est lire au maximum 4 fois la console et annoncé le plus grand à la fin.

Voici le code commenté de ce que tu devrais avoir:

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
package org.k.developpez.forum;
import java.util.Scanner;
 
public class TrouvePlusGrand {
	public static void main(String[] args) {
		// On met dans plusGrand la valeur la plus petite possible pour être sur qu'on met mettra à jour cette valeur.
		int plusGrand = Integer.MIN_VALUE;
		int positionPlusGrand = 0;
 
		// On déclare le scanner en dehors de la boucle. Cela évite de le redéclarer à chaque itération.
		Scanner console = new Scanner(System.in);
		// On utilse <= pour que le nombre donnée soit le nombre de boucle à réaliser.
		for (int i = 1; i <= 4; i++) {
			System.out.println("saissisez un no  :");
			// On récupère ce que l'utilisateur nous donne.
			int n = console.nextInt();
			// Ici on considère que 0 n'est pas à prendre en compte.
			if (n == 0) {
				break;
			}
			// Mise à jours du plus grand.
			if (n > plusGrand) {
				plusGrand = n;
				positionPlusGrand = i;
			}
			// On pourrait prendre en compte la valeur 0 en placant le test pour sortir de la boucle ici.
		}
		// On ferme le flux à la fin pour être propre.
		console.close();
		System.out.println("Le nombre le plus grand était: " + plusGrand);
		System.out.println("Il a été saisi en position numéro: " + positionPlusGrand);
	}
}

Cordialement,
Patrick Kolodziejczyk.

Citation:
Envoyé par kolodz
Code:

1 2 3 4 5 // Ici on considère que 0 n'est pas à prendre en compte. if (n == 0) { break; }
Non.... Pitié... Pas un break là dedans...
Faut arrêter d'utiliser des break/continue qui ne sont que des solutions moches pour forcer la sortie d'un algo qui peut être propre !

Ce qu'on veut faire, c'est ça :
- Lire un chiffre
- vérifier s'il est le plus grand, si oui on stocke son rang et sa valeur
Jusqu'à ce que le chiffre saisi soit 0
Afficher la valeur et son rang

Du coup, on fait ça
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public static void main(String[] args) { int PG = 0; // PG veut plus grand int PPG = 0; // PPG veut dire position de plus grand int nbSaisi = 0; int i = 1; Scanner valeur = new Scanner(System.in); do { System.out.println("saissisez un no :"); nbSaisi = valeur.nextInt(); if (i == 1 || nbSaisi > PG) { PG = nbSaisi; PPG = i; } i++; } while (nbSaisi != 0 || i < 5); valeur.close(); System.out.println("Le nombre le plus grand était: " + PG); System.out.println("Il a été saisi en position numéro: " + PPG); }
Plus court, plus simple, plus lisible, plus proche de ce qu'on veut faire réellement

Citation:
Envoyé par eulbobo
Non.... Pitié... Pas un break là dedans...
Faut arrêter d'utiliser des break/continue qui ne sont que des solutions moches pour forcer la sortie d'un algo qui peut être propre !

Ce qu'on veut faire, c'est ça :
- Lire un chiffre
- vérifier s'il est le plus grand, si oui on stocke son rang et sa valeur
Jusqu'à ce que le chiffre saisi soit 0
Afficher la valeur et son rang

Du coup, on fait ça
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public static void main(String[] args) { int PG = 0; // PG veut plus grand int PPG = 0; // PPG veut dire position de plus grand int nbSaisi = 0; int i = 1; Scanner valeur = new Scanner(System.in); do { System.out.println("saissisez un no :"); nbSaisi = valeur.nextInt(); if (i == 1 || nbSaisi > PG) { PG = nbSaisi; PPG = i; } i++; } while (nbSaisi != 0 || i < 5); valeur.close(); System.out.println("Le nombre le plus grand était: " + PG); System.out.println("Il a été saisi en position numéro: " + PPG); }
j'attire l'attention de
Plus court, plus simple, plus lisible, plus proche de ce qu'on veut faire réellement
mais avez vous vérifier output de votre prg, ça donne même résultat que la mienne
output est:
saissisez un no :
2
saissisez un no :
3
saissisez un no :
0
saissisez un no :
0
Le nombre le plus grand était: 3
Il a été saisi en position numéro: 2

02/06/2015, 15h18
wax78

Parce que la condition du while n'est pas bonne, à cause du OU. Un cas basique que tu aurais put toi même corrigé. Mais il manquera le cas du quand on tappe 0 comme premier chiffre a traiter en plus.

Citation:

Envoyé par kolodz

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
package org.k.developpez.forum;
import java.util.Scanner;
 
public class TrouvePlusGrand {
	public static void main(String[] args) {
		// On met dans plusGrand la valeur la plus petite possible pour être sur qu'on met mettra à jour cette valeur.
		int plusGrand = Integer.MIN_VALUE;
		int positionPlusGrand = 0;
 
		// On déclare le scanner en dehors de la boucle. Cela évite de le redéclarer à chaque itération.
		Scanner console = new Scanner(System.in);
		// On utilse <= pour que le nombre donnée soit le nombre de boucle à réaliser.
		for (int i = 1; i <= 4; i++) {
			System.out.println("saissisez un no  :");
			// On récupère ce que l'utilisateur nous donne.
			int n = console.nextInt();
			// Ici on considère que 0 n'est pas à prendre en compte.
			if (n == 0) {
				break;
			}
			// Mise à jours du plus grand.
			if (n > plusGrand) {
				plusGrand = n;
				positionPlusGrand = i;
			}
			// On pourrait prendre en compte la valeur 0 en placant le test pour sortir de la boucle ici.
		}
		// On ferme le flux à la fin pour être propre.
		console.close();
		System.out.println("Le nombre le plus grand était: " + plusGrand);
		System.out.println("Il a été saisi en position numéro: " + positionPlusGrand);
	}
}

Cordialement,
Patrick Kolodziejczyk.

merci beaucoup

Citation:
Envoyé par eulbobo
Non.... Pitié... Pas un break là dedans...
Faut arrêter d'utiliser des break/continue qui ne sont que des solutions moches pour forcer la sortie d'un algo qui peut être propre !

Ce qu'on veut faire, c'est ça :
- Lire un chiffre
- vérifier s'il est le plus grand, si oui on stocke son rang et sa valeur
Jusqu'à ce que le chiffre saisi soit 0
Afficher la valeur et son rang

Du coup, on fait ça
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 public static void main(String[] args) { int PG = 0; // PG veut plus grand int PPG = 0; // PPG veut dire position de plus grand int nbSaisi = 0; int i = 1; Scanner valeur = new Scanner(System.in); do { System.out.println("saissisez un no :"); nbSaisi = valeur.nextInt(); if (i == 1 || nbSaisi > PG) { PG = nbSaisi; PPG = i; } i++; } while (nbSaisi != 0 || i < 5); valeur.close(); System.out.println("Le nombre le plus grand était: " + PG); System.out.println("Il a été saisi en position numéro: " + PPG); }
Plus court, plus simple, plus lisible, plus proche de ce qu'on veut faire réellement
Sauf que dans ton cas, tu ne maitrise pas quand tu réalise ton test à l'intérieur de la boucle. Ce qui fait que si tu ne veux pas prendre en compte la valeur 0... Tu ne peux pas.

Les break et les continue ne sont pas des solutions moches. Pas plus que la logique du return en milieu de fonction. Pour rappel, le standard Java en est truffé ! Et on ne peux pas dire que le code de l'API standard soit "moche". (A ce niveau là c'est du foutage de gueule.)

D'autant plus que ta proposition ne retiens rien des bonnes pratiques.
Le while n'est pas à utiliser dans le cas où tu incrémente un variable à chaque passage dans la boucle. Car on se retrouve avec l'incrémentation dans le corps de la boucle. Ce fameux i++; n'est là par un défaut de choix dans la structure de boucle utilisé.

De même, le cas spécifique pour i==0, que j'avais corrigé et expliqué et présent dans ton code. Or quand on fait du code propre, on évite les cas spécifique le plus possible.

A la limite, si tu considère que le break n'est pas propre tu devrais avoir un code ressemblant à cela :
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 package org.k.developpez.forum; import java.util.Scanner; public class TrouvePlusGrand { public static void main(String[] args) { // On met dans plusGrand la valeur la plus petite possible pour être sur qu'on met mettra à jour cette valeur. int plusGrand = Integer.MIN_VALUE; int positionPlusGrand = 0; // On déclare le scanner en dehors de la boucle. Cela évite de le redéclarer à chaque itération. Scanner console = new Scanner(System.in); // On utilse <= pour que le nombre donnée soit le nombre de boucle à réaliser. boolean finBoucle = false; for (int i = 1; i <= 4 && !finBoucle; i++) { System.out.println("saissisez un no :"); // On récupère ce que l'utilisateur nous donne. int n = console.nextInt(); // Ici on considère que 0 n'est pas à prendre en compte. if (n == 0) { finBoucle=true; } // Mise à jours du plus grand. if (n > plusGrand) { plusGrand = n; positionPlusGrand = i; } // On pourrait prendre en compte la valeur 0 en placant le test pour sortir de la boucle ici. } // On ferme le flux à la fin pour être propre. console.close(); System.out.println("Le nombre le plus grand était: " + plusGrand); System.out.println("Il a été saisi en position numéro: " + positionPlusGrand); } }
Mais dans tout les cas, on ne code au grand jamais un while pour faire un for.

Après, si tu veux continuer à affirmer que les break et continue sont à proscrire. Merci de me citer une source qui justifie cela. Car, je n'ai jamais entendu cela. Les goto rendaient le code incompréhensible. Mais ceux-ci ne sont pas présent en Java pour cette raison.

Cordialement,
Patrick Kolodziejczyk.