Discussion :

[Spotjaune]

Bonjour,

Je suis de façon autodidacte le cours de C++ sur internet et je tentes de comprendre la partie sur les boucles et les conditions.
Je tente de faire cette exercice:

Parmi les entiers entre 100 et 500, seuls 4 nombres peuvent être représentés pas la somme des cubes de leurs chiffres.

J'obtiens en ce moment ceci comme programme:

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
#include <iostream>
#include <cmath>
 
using namespace std;
 
int main()
{
    int nombre, sommeCube, valeur, Chiffre;
    int BORNE1 (100);
    int BORNE2 (500);
 
    for(nombre = BORNE1; nombre < BORNE2; nombre++)
        valeur = nombre;
        sommeCube = 0;
 
        while (valeur > 0 )
            Chiffre = valeur % 10;
            sommeCube = sommeCube + Chiffre * Chiffre * Chiffre;
            valeur = valeur / 10;
 
        if ( nombre == sommeCube)
      cout << "nombre = %5d, somme des cubes de ses chiffres = %5d\n" << nombre << sommeCube << endl;
 
 
    return 0;
}

Cependant, lorsque je fais exécuter mon programme, rien n'apparaît...
J'aurais besoins d'aide, car je ne sais pas ou mon erreur se situe

Merci

[foetus]

Tu viens de découvrir la boucle infinie ... parce que tu n'as pas saisi la notion des blocs

[Spotjaune]

Merci, mais ca m'aide pas plus à comprendre !lol

[foetus]

Pour affecter un entier ce n'est pas int BORNE1 (100) mais int BORNE1 = 100;

Indente ton code pour qu'on comprenne ce que tu as voulu coder.

Par exemple tu as voulu coder comme cela

Code :

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

1
2
3
    for(nombre = BORNE1; nombre < BORNE2; nombre++)
        valeur = nombre;
        sommeCube = 0;

Et tu aurais percuté qu'il te manque les accolades (la notion de bloc) et que seule la première ligne est prise en compte

De même

Code :

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

1
2
3
4
    while (valeur > 0 )
        Chiffre = valeur % 10;
        sommeCube = sommeCube + Chiffre * Chiffre * Chiffre;
        valeur = valeur / 10;

Et que comme ta boucle tant que ne prend que la première ligne et que valeur est toujours positif (*) elle tourne indéfiniment.

Apprends aussi à utiliser un débogueur : le mode pas à pas et le mode pause. (ainsi que quelques cout/ printf)


* -> Elle doit valoir BORNE2

[Spotjaune]

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
#include <iostream>
#include <cmath>
#include <string>
 
using namespace std;
 
int main()
{
    int nombre, sommeCube, valeur, Chiffre;
    int BORNE1 = 100;
    int BORNE2 = 500;
 
    for(nombre = BORNE1; nombre < BORNE2; nombre++)
    {
        valeur = nombre;
        sommeCube = 0;
    }
 
    while (valeur > 0 )
    {
        Chiffre = valeur % 10;
        sommeCube = sommeCube + Chiffre * Chiffre * Chiffre;
        valeur = valeur / 10;
    }
 
    if ( nombre == sommeCube)
    cout << "nombre = " << nombre << ", somme des cubes de ses chiffres = " << sommeCube << endl;
 
  return 0;
}

Merci pour les accolades, effectivement, j'avais oubliés de les mettres. La question que je me poses est, selon moi, mon cout que j'écris est mal ou que je ne saisi pas parce que selon moi, les calculs sont correctes ... enfin j'espères!lol

[foetus]

Ton code semble bon à vue de nez ... mais ton indentation de m*rd* m'a induit en erreur

Réfléchis un peu: papier crayon toussa

[Iradrille]

Merci pour les accolades, effectivement, j'avais oubliés de les mettres. La question que je me poses est, selon moi, mon cout que j'écris est mal ou que je ne saisi pas parce que selon moi, les calculs sont correctes ... enfin j'espères!lol

Le cout est bon, les calculs beaucoup moins.

[JolyLoic]

Merci pour les accolades, effectivement, j'avais oubliés de les mettres.

Tu les as mises, mais à un endroit différent de là où tu avais mis ton indentation dans le premier message... Initialement, tu avais une boucle while à l'intérieur d'une boucle for (sauf que ça ne marchait pas car en C++, les indentations sont uniquement décorative mais le compilateur les ignore). Après tes ajoutes d'accolade, tu as une boucle for, suivie d'une boucle while, ce qui est totalement différent...

À part ça, le reste du code m'a l'air correct.

[JolyLoic]

Pour affecter un entier ce n'est pas int BORNE1 (100) mais int BORNE1 = 100;

On parle ici d'une initialisation, pas d'une affectation, et pour un int, ces trois écritures sont équivalentes :

Code :

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

1
2
3
	int i(100);
	int j = 100;
	int k{ 100 };