For 循环未按预期执行

Question

我正在 C++ 上制作一个简单的程序，它读取数字并以英文文本输出所述数字。它可以读取从 0 到 9,999,999 的数字。我把它分为函数，一个函数，一个函数，一个函数，几十，几百，几千和几百万。

我在百万函数中有一个 for 循环，它只将整个数字中的一些数字传递给另一个字符串，但它只执行一次并且不复制数字：

`

string millionths(string num_i)
{
    string num_m;
    string num_l;
    string num_ln="000000";
    string num_ln5="00000";
    string num_ln4="0000";
    string num_ln3="000";
    string num_ln2="00";
    int it;
/*I just want the last 6 characters from num_i to be copied to num_ln
but the loop is only executing once and doesn't copy any value*/
    for(it=0;it<6;it++);
        {num_ln[it]=num_i[it+1];}
    num_m=units(num_i[0]);
    if(num_ln=="000000")
        return num_m+" million";
    else if(num_ln[0]=='0'&&num_ln[1]=='0'&&num_ln[2]=='0'&&num_ln[3]=='0'&&num_ln[4]=='0')
    {num_l=units(num_ln[5]);}
    else if(num_ln[0]=='0'&&num_ln[1]=='0'&&num_ln[2]=='0'&&num_ln[3]=='0')
    {for(it=0;it<2;it++);
        {num_ln2[it]=num_i[it+5];}
    num_l=tenths(num_ln2);}
    else if(num_ln[0]=='0'&&num_ln[1]=='0'&&num_ln[2]=='0')
    {for(it=0;it<3;it++);
        {num_ln3[it]=num_i[it+4];}
    num_l=hundreths(num_ln3);}
    else if(num_ln[0]=='0'&&num_ln[1]=='0')
    {for(it=0;it<4;it++);
        {num_ln4[it]=num_i[it+3];}
    num_l=thousanths(num_ln4, 1);}
    else if(num_ln[0]=='0')
    {for(it=0;it<5;it++);
        {num_ln5[it]=num_i[it+2];}
    num_l=thousanths(num_ln5, 2);}
    else
    {num_l=thousanths(num_ln, 3);}
    return num_m+" million "+num_l;
}

`

非常感谢您的帮助。

Answer 1

这里的第一行是一个空的 for 循环：

for(it=0;it<6;it++);     //You don't want to do this
    {num_ln[it]=num_i[it+1];}

结果是这个循环在没有实际执行任何操作的情况下运行，并以it == 6 结束。结果是num_ln[it]=num_i[it+1] 只执行一次，并且只对it=6 执行。这可能不是您想要的，因为它只复制一个元素。这可以通过以下方式修复：

for(it=0;it<6;it++){
    num_ln[it]=num_i[it+1];
}

通过使用更标准的大括号样式，这个问题得到了解决，它也将使其他程序员更容易阅读和理解您的代码。