Algorithme qui calcule la somme des n nombres

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:

---

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,

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

Sinon ton problème, c'est ton test :mrgreen: : 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 :whistle:, tes noms de variables ne sont pas terrible et ta variable j peut être initialisée dans ta boucle.

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

Citation:

---

Envoyé par foetus

Tu n'as pas besoin de boucle :mrgreen: 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.

Citation:

---

Envoyé par foetus

Sinon ton problème, c'est ton test :mrgreen: : 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:

---

1 2 3 4

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

---

Je pense que c'est mieux ainsi ?

Citation:

---

Envoyé par foetus

Sinon :whistle:, 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,

Code:

---

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

---

Merci pour le coup de main.

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

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 ?

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:

---

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.

Merci pour la précision.
En reprenant tes remarques, nous arrivons à ce code :

Code:

---

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

---

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.