我是c ++的初学者,我在使这段代码按照我想要的方式工作时遇到了问题。任务是编写一个程序,将所有自然数乘以加载数n。
为了打印正确的结果,我将x
除以n
(请参阅下面的代码)。如何将其打印x
而不必将其除以n
以获得正确答案?
#include<iostream>
using namespace std;
int main(){
int n,x=1;
int i=0;
cout<<"Enter a number bigger than 0:"<<endl;
cin>>n;
while(i<n){
i++;
x=i*x;
};
cout<<"The result is: "<<x/n<<endl;
return 0;
}
答案 0 :(得分:6)
首先,您最好尽快习惯:始终检查用户输入是否正确!
cin >> n;
if(cin && n > 0)
{
// valid
}
else
{
// appropriate error handling
}
不确定,为什么需要while循环?在这种情况下,for循环肯定更好:
int x = 1;
for(int i = 2; i < n; ++i)
x *= i;
如果你还想要while循环:从i == 2
开始(无论如何都是1),然后再增加:
i = 2;
while(i < n)
{
x *= i;
++i;
}
如果是n == 1
,则不会输入循环(任何一种变体),你很好......
答案 1 :(得分:2)
你已经有两个非常好的选择了,但是当你在编程中足够轻松时,你可能想看看另一个选项:
unsigned factorial(unsigned value)
{
if (value <= 1)
{
return 1;
}
else
{
return value * factorial(value - 1);
}
}
它是一个递归函数,在适当的时刻使用时有点整洁(不幸的是,这可能不是这样的情况,因为执行堆栈可能会变得如此之大,以至于你在完成之前填满了你的记忆。但您可以查看它以了解有关递归函数的更多信息)
当你的内存已满时,你会崩溃你的应用程序实际上是一个堆栈溢出。
答案 2 :(得分:1)
我怎样才能这样做,以便在最后一个cout中我只能放x并且不必将x除以n得到正确的答案?
最好使用for
循环。
// This stops when i reaches n.
// That means, n is not multiplied to the result when the loop breaks.
for (int i = 1; i < n; ++i )
{
x *= i;
}
cout << "The result is: " << x <<endl;