Question

我正在尝试检查用户提供的号码是否为armstrong number。但是有些事情是错的，我无法理解。

感谢任何帮助。

下面的代码。

#include<stdio.h>

int fun(int);

int main()
{
    int x,a,b,y=0;

    printf("enter the number you want to identify is aN ARMSTRONG OR NOT:");
    scanf("%d",&a);

    for(int i=1 ; i<=3 ; i++)
    {
        b = a % 10;
        x = fun(b);
        y = x+y;
        a = a/10;
    }

    if(y==a)
        printf("\narmstrong number");
    else
        printf("\nnot an armstrong number");

    return 0;
}

int fun(int x)
{
    int a;
    a=x*x*x;
    return (a);
}

Answer 1

主要问题是您没有记录您开始使用的号码。您将a重复除以10（结束为0），然后将0与153进行比较。这些不相等。

你的另一个问题是你不能寻找4位数或更长的阿姆斯壮数字，也不能寻找除1之外的1位数字。你的函数fun()会更好地命名为cube();在我的下面的代码中，它被重命名为power()，因为它被推广为处理N位数字。

我决定，对于所考虑的功率范围，没有必要使用更复杂的power()算法 - 一个除以2的算法等。可以节省6-10位数字，但你无法在这种情况下衡量它。如果使用-DDEBUG编译，它包括诊断打印 - 用于向我保证我的代码正常工作。还要注意答案回应输入;这是确保您获得正确行为的基本技术。我已经将代码包装成一个函数来测试一个数字是否是一个阿姆斯特朗数，这是从主程序迭代调用的。这使得测试更容易。我已将检查添加到scanf()以阻止问题，这是另一项重要的基本编程技术。

我已经检查了大部分阿姆斯特朗的数字，最高可达146511208，看起来是正确的。 370和371对很有趣。

#include <stdio.h>
#include <stdbool.h>

#ifndef DEBUG
#define DEBUG 0
#endif

static int power(int x, int n)
{
    int r = 1;
    int c = n;
    while (c-- > 0)
        r *= x;
    if (DEBUG) printf("    %d**%d = %d\n", x, n, r);
    return r;
}

static bool isArmstrongNumber(int n)
{
    int y = 0;
    int a = n;
    int p;
    for (p = 0; a != 0; a /= 10, p++)
        ;
    if (DEBUG) printf("    n = %d, p = %d\n", n, p);
    a = n;
    for (int i = 0; i < p; i++)
    {
        y += power(a % 10, p);
        a /= 10;
    }
    return(y == n);
}

int main(void)
{
    while (1)
    {
        int a;
        printf("Enter the number you want to identify as an Armstrong number or not: ");
        if (scanf("%d", &a) != 1 || a <= 0)
            break;
        else if (isArmstrongNumber(a))
            printf("%d is an Armstrong number\n", a);
        else
            printf("%d is not an Armstrong number\n", a);
    }

    return 0;
}

Answer 2

一个问题可能是您正在更改a（因此它将不再具有原始值）。它也只匹配1, 153, 370, 371, 407。这是一个提示，在a为零之前替换for和test，并将函数更改为数字。

Answer 3

#include<stdio.h>
#include <math.h>

int power(int, int);
int numberofdigits(int);

//Routine to test if input is an armstrong number.
//See: http://en.wikipedia.org/wiki/Narcissistic_number if you don't know
//what that is. 

int main()
{
    int input;
    int digit;
    int sumofdigits = 0;

    printf("enter the number you want to identify as an Armstrong or not:");
    scanf("%d",&input);

    int candidate = input;
    int digitcount = numberofdigits(input);

    for(int i=1 ; i <= digitcount ; i++) 
    {
        digit = candidate % 10;
        sumofdigits = sumofdigits + power(digit, digitcount);
        candidate = candidate / 10;
    }

    if(sumofdigits == input)
        printf("\n %d is an Armstrong number", input);
    else
        printf("\n %d is NOT an Armstrong number", input);
    return 0;
}

int numberofdigits(int n);
{
  return log10(n) + 1;
}

int power(int n, int pow)
{
  int result = n;
  int i=1;
  while (i < pow) 
  {
    result = result * n; 
    i++;
  }
}

代码有什么问题：

不使用有意义的变量名称，使代码的含义难以理解;记住代码是为人类而不是编译器编写的。
请勿使用令人困惑的代码此代码：int x,a,b,y=0;令人困惑，所有变量都设置为0或仅y。始终将已初始化的变量放在单独的行上。它使阅读更容易。加倍努力是明确的，从长远来看，它将带来巨大的回报。
使用评论：如果您不知道什么是强号，那么从您的代码中很难说出来。添加一些有意义的评论，以便人们知道您的代码假设要做什么。这将使您和其他人更容易，因为他们知道您的意图和可以看到您实际做了什么，并在需要时解决差异。
使用有意义的例程名称 WTF会fun(x)做什么？从不命名任何事物fun()这就像事实上的自由科学一样，有什么意义呢？
不要硬编码，你的例行程序只接受了armstrong3号码，但如果你可以硬编码那么为什么不进行return (input == 153) || (input == 370) || ....

Answer 4

/* 
Name: Rakesh Kusuma

Email Id:  rockykusuma@gmail.com

Title: Program to Display List of Armstrong Numbers in 'C' Language

*/



#include<stdio.h>

#include<math.h>

int main()

{

int temp,rem, val,max,temp1,count;

int num;

val=0;

num=1;

printf("What is the maximum limit of Armstrong Number Required: ");

scanf("%d",&max);

printf("\nSo the list of Armstrong Numbers Before the number %d are: \n",max);

while(num <=max)

    {         
        count = 0;

        temp1 = num;

        while(temp1!=0)

        {
            temp1=temp1/10;

            count++;
        }   

        if(count<3)

        count = 3;

            temp = num;

            val = 0;

            while(temp>0)

            {

                rem = temp%10;

                val = val+pow(rem,count);

                temp = temp/10;

            }

            if(val==num)

            {

                printf("\n%d", num);

            }

     num++; 

    }

 return 0;

 }

Answer 5

检查号码是阿姆斯壮还是不使用 C语言

#include<stdio.h>
#include<conio.h>
void main()
{
    A:
    int n,n1,rem,ans;
    clrscr();
    printf("\nEnter No. :: ");
    scanf("%d",&n);

    n1=n;
    ans=0;
    while(n>0)
    {
        rem=n%10;
        ans=ans+(rem*rem*rem);
        n=n/10;
    }

    if(n1==ans)
    {
        printf("\n Your Entered No. is Armstrong...");
    }
    else
    {
        printf("\n Your Entered No. is not Armstrong...");
    }

    printf("\n\nPress 0 to Continue...");
    if(getch()=='0')
    {
        goto A;
    }
    printf("\n\n\tThank You...");
    getch();
}

Answer 6

如果您尝试查找armstrong number，则您发布的解决方案会丢失数字大于3的情况... armstrong数字可以大于3个数字（例如9474）。这是Python中的代码，逻辑很简单，可以转换为任何其他语言。

def check_armstrong(number):
    num = str(number)
    total=0
    for n in range(len(num)):
        total+=sum(int(num[n]),len(num))       

    if (number == total):
        print("we have armstrong #",total)

def sum(input,power):
    input = input**power
    return input

check_armstrong(9474)

Answer 7

这是一种检查号码是否为阿姆斯特朗（strongstrong）的方法

t=int(input("nos of test cases"))
while t>0:
    num=int(input("enter any number = "))
    n=num
    sum=0
    while n>0:
        digit=n%10
        sum += digit ** 3
        n=n//10

    if num==sum:
        print("armstronng num")
    else:
        print("not armstrong")
    t-=1

Answer 8

这是我为Armstrong号码检测制作并见过的最简单的代码：

def is_Armstrong(y):
    if y == 0:
        print('this is 0')
    else:
        x = str(y)
        i = 0
        num = 0
        while i<len(x):
            num += int(x[i])**(len(x))
            i += 1
            if num == y:
                print('{} is an Armstrong number.'.format(num))
                break
            else:
                print('{} is not an Armstrong number.'. format(y))
is_Armstrong(1634)

确定号码是否为阿姆斯壮号码的问题

8 个答案: