Discussion :

[Invité]

Bonjour à tous,

Voulant apprendre le codage avec Java, j'effectue plusieurs exercices "simples" pour m'initier.
Dans ce programme, j'essaye de calcul la somme des n nombres.
Cependant, quelque soit le chiffre/nombres indiqué, j'obtiens un résultat égal à 0.

J'imagine que l'erreur provient de ma boucle "for" mais je n'arrive pas à voir d'où ça vient.

Pourriez-vous m'aiguiller je vous prie ?

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
import java.util.Scanner;
 
public class PremiersNombres 
{
 
	public static void main(String[] args) 
	{
		int n;
		int i;
		int j=0;
 
		System.out.print("Veuillez indiquer le nombre/chiffre souhaité : ");
		Scanner clavier = new Scanner (System.in);
		n = clavier.nextInt();
 
		for(i=1; i==n; i++)
		{
			j+=i;
		}
 
		System.out.print("La somme de " + n + " est " + j);
		clavier.close();
	}
 
}

Cordialement,

[foetus]

Tu n'as pas besoin de boucle La somme des n premiers nombres est égale à n * (n + 1) / 2.

Sinon ton problème, c'est ton test : i==n - on boucle tant que la condition est vraie.
Comme l'égalité est fausse seulement pour n = 1, alors tu auras toujours 0 comme résultat.

Sinon , tes noms de variables ne sont pas terrible et ta variable j peut être initialisée dans ta boucle.

[Invité]

Bonjour/Bonsoir foetus,

Déjà merci de prendre la peine de me répondre. J'apprécie grandement.
Pour la petite histoire, je commence à apprendre la programmation via Java depuis deux jours. J'imagine que mon code va en choquer plus d'un

Tu n'as pas besoin de boucle La somme des n premiers nombres est égale à n * (n + 1) / 2.

Effectivement, je suis d'accord avec toi. Le but était plus d'apprendre à utiliser la boucle "for". Mais je suis sûr que tu avais compris la démarche.

Sinon ton problème, c'est ton test : i==n - on boucle tant que la condition est vraie.
Comme l'égalité est fausse seulement pour n = 1, alors tu auras toujours 0 comme résultat.

Oui, je n'avais pas bien compris qu'on boucle tant que la condition est vraie.

Code :

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

1
2
3
4
for(i=1; i<=n; i++)
{
	j+=i;
}

Je pense que c'est mieux ainsi ?

Sinon , tes noms de variables ne sont pas terrible et ta variable j peut être initialisée dans ta boucle.

En reprenant mon cas, comment aurais-tu nommé mes variables ?
Je n'arrive pas à voir comment initialiser la variable j dans la boucle ?

D'avance merci.
Cordialement,

[foetus]

Code :

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

1
2
3
4
5
6
7
 
    int sum, nb, n; // <- nb, bof mais cela passe
 
    for(nb=1, sum=0; nb <= n; nb++)
    {
        sum += nb;
    }

[Invité]

Merci pour le coup de main.

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
import java.util.Scanner;
 
public class PremiersNombres 
{
 
	public static void main(String[] args) 
	{
		int saisieClavier;
		int nb;
		int sum;
 
		System.out.print("Veuillez indiquer le nombre/chiffre souhaité : ");
		Scanner clavier = new Scanner (System.in);
		saisieClavier = clavier.nextInt();
 
		for(nb=1, sum=0; nb<=saisieClavier; nb++)
		{
			sum+=nb;
		}
 
		System.out.print("La somme de " + saisieClavier + " est " + sum);
		clavier.close();
	}
 
}

Bizarrement, Eclipse m'indique un warning à la ligne 13 "clavier is never closed". Pourtant j'effectue cette démarche à la ligne 22.
Sauriez-vous pourquoi ?

[thelvin]

Hello,

Je ne vais pas installer Eclipse pour vérifier (surtout qu'il faudrait avoir exactement la même version que toi et tu n'as pas indiqué la version),

mais, on peut envisager que le code que tu montres fasse un warning, mais pas exactement le warning dont tu parles.

Si un warning est fait, ça devrait être que tu n'es pas 100% sûr de fermer le Scanner correctement. Et non pas qu'il n'est jamais fermé.
Pour information, quand on veut correctement fermer une ressource après l'avoir ouverte, il faut utiliser un try-with-resource. Ça ressemble à ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
try(Scanner clavier = new Scanner(System.in))
{
    int saisieClavier = clavier.nextInt();
 
    int sum = 0;
    for(int nb=1; nb<=saisieClavier; nb++)
    {
        sum+=nb;
    }
 
    System.out.print("La somme de " + saisieClavier + " est " + sum);
}

Cela dit, dans cet exemple très précis, il est assez importun de faire une alarme que le Scanner n'est pas fermé. On doit être responsable de fermer les ressources qu'on ouvre, or le Scanner utilise System.in et ce n'est pas ton code qui l'ouvre. Il ne devrait donc pas non plus prendre la responsabilité de le fermer.

Ce n'est pas grave de le fermer quand même pour qu'Eclipse se taise avec ses warnings, mais je dirais qu'Eclipse a tort de le demander.

[Invité]

Merci pour la précision.
En reprenant tes remarques, nous arrivons à ce 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
import java.util.Scanner;
 
public class PremiersNombres 
{
 
	public static void main(String[] args) 
	{
		int sum;
		int saisieClavier;
 
		System.out.print("Veuillez indiquer le nombre/chiffre souhaité : ");
 
		try(Scanner clavier = new Scanner (System.in))
		{
			saisieClavier = clavier.nextInt();
			sum = saisieClavier * (saisieClavier + 1) / 2;
		}
 
		System.out.print("La somme de " + saisieClavier + " est " + sum);
	}
}

[thelvin]

Il n'était pas nécessaire de remplacer la boucle par la formule mathématique ad hoc, mais oui, c'est le principe de fermer les ressources.