ファインディング・アームストロングナンバー使用してC++

-1

私は与えられた数かどうかを見つけるためのプログラムを書いているがArmstrong Numberです：私はそれを実行するとファインディング・アームストロングナンバー使用してC++

int main() 
{ 
    int n,rem,sum=0; 
    cout<<"Enter the Number for checking"<<endl; 
    cin>>n; 

    while(n!=0) 
    { 
     rem=n%10; 
     sum=sum+(rem*rem*rem); 
     n=n/10; 
    } 

    if(sum==n) 
    { 
     cout<<"Armstrong Number"<<endl; 
    } 
    else 
    { 
     cout<<"It's not a armstrong number"; 
    } 

    return 0; 
}

が、それは常に報告入力に関わらず、「これは、アームストロングの番号ではありません」。

コードを次のように変更し、正しい結果を得ました。しかし、なぜ私はn1に入力を割り当てて操作を行う必要があるのか理解できません。なぜnで直接操作できませんか？あなたのプログラムがsumとnを比較if (sum==n)ラインで

int main() 
{ 
    int n,rem,sum=0,n1; 
    cout<<"Enter the Number for checking"<<endl; 
    cin>>n; 
    n1=n; 

    while(n1!=0) 
    { 
     rem=n1%10; 
     sum=sum+(rem*rem*rem); 
     n1=n1/10; 
    } 

    if(sum==n) 
    { 
     cout<<"Armstrong Number"<<endl; 
    } 
    else 
    { 
     cout<<"It's not a armstrong number"; 
    } 

    return 0; 
}

出典

2016-06-27 Hola

。 2番目のプログラムでは、nはユーザーが入力した最初の番号です。しかし、最初のプログラムn==0（上のループを参照）。

したがって、最初のプログラムでは、if (sum==n)のチェックはif (sum==0)として機能します。しかし、sumの値は0になりません（ユーザーが0を入力した場合を除く）。したがって、最初のプログラムは常に「それはアームストロングナンバーではない」と返します。

スタイルについて：ロジック全体を1つのmain()関数に入れるのではなく、関数を使用するほうがはるかに優れています。あなたのプログラムがはるかに簡単になりますと、あなたが最初に持ってミスを犯すのは難しいだろう。この場合、

int getSumOfCubesOfDigits(int n) 
{ 
    int sum = 0; 
    while (n) 
    { 
     const int rem = n % 10; 
     sum += rem * rem * rem; 
     n = n/10; 
    } 
}

：たとえば、あなたはアームストロング数のchechingのための中間和を計算するための関数を作成することができますあなたの質問のプログラム：

int main() 
{ 
    int n; 
    cout << "Enter the Number for checking" << endl; 
    cin >> n; 
    if(getSumOfCubesOfDigits(n) == n) 
     cout<<"Armstrong Number"<<endl; 
    else 
     cout<<"It's not a armstrong number"; 
    return 0; 
}

出典

2016-06-27 03:36:32 Ilya