当我的程序使用类似60的命令行参数运行时,第一次计算是错误的,但其余的似乎是正确的。我想我在main()函数中有一个bug,但我找不到它。在Linux 60上提供约30000步,在Windows上显示〜-7000000步。
#include <iostream>
#include <cstdlib>
using namespace std;
void printcollatz( int i ) {
int count;
int originalnumber = i;
int workingnumber = i;
while( workingnumber >= 1 ) {
if( workingnumber == 1 ) {
cout << originalnumber << " takes " << count << " steps" << endl;
count = 0;
break;
}
if( workingnumber % 2 == 0 ) {
workingnumber /= 2;
count++;
}
else {
workingnumber = ( 3 * workingnumber ) + 1;
count++;
}
}
}
int main( int argc, char* argv[] ) {
int incrementnumber;
if( argc > 1 ) {
if( atoi( argv[1] ) == 0 ) {
incrementnumber = 2;
}
else {
incrementnumber = atoi( argv[1] );
}
}
while( 1 ) {
printcollatz( incrementnumber );
incrementnumber++;
}
return 0;
}
答案 0 :(得分:-1)
您需要初始化计数:
int count = 0;
此外,您可能希望在循环中添加终止条件。