Discussion :

[Bibibis]

Voilà, je fais actuellement ma première fonction, qui est une fonction qui donne la factorielle d'un nombre. J'y arrive a peu près, mais j'ai deux problèmes:

1) Je n'arrive pas à faire en sorte que le programme écrive "La factorielle de x est y".

2) Quand j'écris une lettre au lieu d'un nombre, le programme m'écrit en boucle "Entrez un nombre en 0 et 255 : Ce nombre est invalide"

Voici le 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
22
23
24
// Factorielle.cpp*: définit le point d'entrée pour l'application console.
//
 
#include "stdafx.h"
 
 
int main(void)
{
	unsigned int x, y=1;
	printf("Calcul de la factorielle\n");
	negatif:;
	printf("Entrez un nombre entre 0 et 255 : ");
	scanf("%i",&x);
	if(x>255) printf("Ce nombre est invalide.\n");
	if(x>255) goto negatif;
	while(x>1)
	{
		y=x*y;
		x=x-1;
	}
	printf("La factorielle de &i est &i.\n", x, y);
	goto negatif;
	return 0;
}

[Aleph69]

Bonsoir,

tu n'es pas dans le bon forum : ton code est écrit en C.
Petit conseil personnel : fais rapidement disparaître les "goto" de ton algorithme avant de poster ton code autre part...

[Bibibis]

D'accord mais quelles sont les caractéristiques qui font que ce code est écrit en C plutôt qu'en C++?
Parce que c'est un devoir, et je dois l'écrire en C++, et je suis un tutoriel qui fait le C et le C++ en "parallèle"...
Sinon pour le goto j'ai cherché une autre chose qui pourrait faire sauter la fonction while mais je n'ai pas trouvé...

[koala01]

Salut,

D'accord mais quelles sont les caractéristiques qui font que ce code est écrit en C plutôt qu'en C++?
Parce que c'est un devoir, et je dois l'écrire en C++, et je suis un tutoriel qui fait le C et le C++ en "parallèle"...
Sinon pour le goto j'ai cherché une autre chose qui pourrait faire sauter la fonction while mais je n'ai pas trouvé...

En gros, les fonctions printf, scanf et consort sont des fontions de C.

Si elle sont présentes en C++, ce n'est que dans un soucis de comptabilité, mais il est globalement déconseillé d'y recourir en C++.

En C++, printf sera remplacé par l'utilisation de cout et scanf par celle de cin (tous deux présents dans l'espace de noms std par inclusion du fichier d'en-tête <iostream>)

Ceci dit, tu devrais essayer de réfléchir à l'utilisation de goto... Sauf cas particulier est excessivement rare en C++, tu devrais préférer l'utilisation des différents types de boucles à celle de goto

[Bibibis]

Voilà j'ai réussi à supprimer le code C et ça marche, j'ai juste un problème: quand je met une lettre au lieu d'un nombre le programme met
"La factorielle de 0 est 1"
"La factorielle de 0 est 1"
"La factorielle de 0 est 1"
...

Et je ne sais pas pourquoi.

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
// Factorielle.cpp*: définit le point d'entrée pour l'application console.
//
 
#include <StdAfx.h>
#include <iostream>
 
 
int main(void)
{
	std::cout << "Calcul de la factorielle" << std::endl;
	negatif:;
	unsigned int x=0, y=1, n=0;
	std::cout << "Entrez un nombre entre 0 et 30 : ";
	std::cin >> x;
	n=x;
	if(x>30) printf("Ce nombre est invalide.\n");
	if(x>30) goto negatif;
	while(x>1)
	{
		y=x*y;
		x=x-1;
	}
	std::cout << "La factorielle de " << n << " est " << y << std::endl;
	goto negatif;
	return 0;
}

[Aleph69]

Bonsoir,

c'est l'avant denière ligne de ton code

Code :

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

1
2
 
goto negatif;

qui renvoie au début de ton code qui explique ce comportement.