Discussion :

[domxaline]

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 :

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
 
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

[eulbobo]

Code :

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

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

[kolodz]

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 :

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
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.

[eulbobo]

Code :

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

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 :

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
 
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

[domxaline]

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 :

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
 
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

[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.

[domxaline]

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 :

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
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

[kolodz]

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 :

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
 
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 :

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
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.