学c的萌新 发表于 2023-8-24 22:21:44

do_while语句的练习题,水仙花数

void test02()
{
        //水仙花数 1的三次方+5的三次方+3的三次方=153
        //要求找出所有水仙花数字
        //定义一个数,然后设置一个门槛,100到1000之间,100为初始值,1000为结束条件,抽取个十百位的数然后用if为筛选条件
        //最后用while结尾,循环多次找到结果
        int num = 100;
        do
        {
                int a = num % 10;
                int b = num / 10 % 10;
                int c = num / 100;
                if(a*a*a+b*b*b+c*c*c==num)
                {
                        printf("%d\n", num);
                }
               
        }
while (num < 1000);
}
请大佬们班帮我看一下出现了什么问题,还有哪些需要改进的地方,谢谢大佬们

isdkz 发表于 2023-8-24 22:22:13

你的代码基本上是正确的,但是有一些可以改进的地方。

首先,你可以将计算水仙花数的逻辑封装成一个函数,这样可以提高代码的可读性和复用性。例如,你可以创建一个名为 isNarcissisticNumber 的函数,接受一个整数参数,并返回一个布尔值表示该数是否是水仙花数。

其次,你可以使用for循环来代替do-while循环,因为你已经明确知道循环的起始和结束条件。这样可以使代码更加简洁。

最后,你可以添加注释来解释代码的逻辑,这样其他人在阅读你的代码时可以更容易理解。

下面是改进后的代码示例:

#include <stdio.h>

// 判断一个数是否是水仙花数
int isNarcissisticNumber(int num)
{
    int a = num % 10;
    int b = num / 10 % 10;
    int c = num / 100;
    return a*a*a + b*b*b + c*c*c == num;
}

void test02()
{
    // 找出所有水仙花数字
    // 定义一个数,然后设置一个门槛,100到1000之间
    // 100为初始值,1000为结束条件
    // 抽取个十百位的数然后用if为筛选条件
    // 最后用for循环找到结果

    for (int num = 100; num < 1000; num++)
    {
      if (isNarcissisticNumber(num))
      {
            printf("%d\n", num);
      }
    }
}


希望对你有帮助!如果有任何问题,请随时提问。

陶远航 发表于 2023-8-24 22:22:29

你的代码基本上是正确的,并且可以用于找出范围在100到1000之间的水仙花数。不过有几个需要注意的地方:

1. 循环条件:你使用了do-while循环,这意味着循环体至少会执行一次。然而,你的循环条件num < 1000可能存在问题,因为当num等于1000时,循环体也会执行一次。为了符合题目要求的结束条件,你可以将循环条件改为num < 1000。

2. 输出格式:你已经正确地判断出了水仙花数并进行输出。不过,为了更好地展示结果,可以在每个水仙花数输出后加上一些提示信息,例如:"水仙花数是:"。

综合考虑以上两点,你可以稍微修改一下代码,如下所示:

void test02() {
    int num = 100;
    do {
      int a = num % 10;
      int b = num / 10 % 10;
      int c = num / 100;
      if (a * a * a + b * b * b + c * c * c == num) {
            printf("水仙花数是:%d\n", num);
      }
      num++;
    } while (num < 1000);
}


这样修改后,你就能得到范围在100到1000之间的所有水仙花数,并以"水仙花数是:"的形式进行输出。

希望这能够帮助你!如果你还有其他疑问,请随时提问。
如果问题已经解决,请设置最佳答案

langyawang 发表于 2023-8-25 10:15:20

你的代码基本上是正确的,但还有一些可以改进的地方:

1. 在循环结尾处,应该使用`while`关键字而不是`do...while`。因为你设置的初始值是100,所以当初始值不满足条件时,`do...while`循环会执行至少一次,但你并不希望输出不符合条件的数字。

2. 对于循环中的变量命名,可以更加具有描述性,这样有助于代码的可读性。例如,将`a`、`b`、`c`改为`unit`、`tenth`、`hundredth`。

3. 考虑将判断水仙花数的逻辑封装成一个函数,这样可以提高代码的复用性。例如,创建一个函数`isNarcissistic()`,输入一个整数,返回布尔值表示是否是水仙花数。

4. 注意在循环内更新`num`的值,否则循环将成为死循环。在每次迭代结束后,应该将`num`自增1。

下面是改进后的代码示例:

c

希望对你有所帮助!如果还有其他问题,请随时提问。

琅琊王朝 发表于 2023-8-25 10:18:16

你的代码基本是正确的,但有几个地方可以改进:

1. 在 `do...while` 循环中,应该在每次迭代结束后递增 `num` 的值,否则循环将陷入无限循环。在循环的最后加上 `num++` 语句。

2. 变量命名可以更具描述性,以提高代码的可读性。例如,将 `a`、`b`、`c` 改为 `unit`、`tenth`、`hundredth`。

3. 将判断是否为水仙花数的逻辑封装成一个函数,这样可以提高代码的可维护性和复用性。

下面是改进后的代码示例:

c

这样修改后的代码应该能够正确输出所有的水仙花数。

希望对你有所帮助!如果还有其他问题,请随时提问。

sfqxx 发表于 2023-8-25 10:19:35

琅琊王朝 发表于 2023-8-25 10:18
你的代码基本是正确的,但有几个地方可以改进:

1. 在 `do...while` 循环中,应该在每次迭代结束后递增 `n ...

《c》

琅琊王朝 发表于 2023-8-25 10:27:32

sfqxx 发表于 2023-8-25 10:19
《c》

脚本太聪明了
{:5_104:}

liuhongrun2022 发表于 2023-8-25 10:38:31

这段代码的目标是找到所有的水仙花数。水仙花数是指一个三位数,其个位、十位和百位上的数字的立方和等于该数本身。

这里的代码有一个小问题,即在 `do-while` 循环中,变量 `num` 的更新被遗漏了,会导致无限循环。应该在找到水仙花数后,将 `num` 增加1,以继续寻找下一个数字。同时,为了保证循环能够正常结束,应将结束条件修改为 `num <= 1000`。

以下是修正后的代码:

void test02()
{
    // 水仙花数 1的三次方 + 5的三次方 + 3的三次方 = 153
    // 要求找出所有水仙花数字
    // 定义一个数,然后设置一个门槛,100到1000之间,100为初始值,1000为结束条件,抽取个十百位的数然后用if为筛选条件
    // 最后用while结尾,循环多次找到结果
   
    int num = 100;
    do
    {
      int a = num % 10;
      int b = num / 10 % 10;
      int c = num / 100;
      
      if(a*a*a + b*b*b + c*c*c == num)
      {
            printf("%d\n", num);
      }
      
      num++;
    }
    while (num <= 1000);
}


这样修改后,代码应该可以正确地找到所有的水仙花数。希望能对你有所帮助!如有任何进一步的疑问,请随时提问。此内容为ChatGPT回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!

liuhongrun2022 发表于 2023-8-25 10:38:54

琅琊王朝 发表于 2023-8-25 10:27
脚本太聪明了

哈哈哈哈

琅琊王朝 发表于 2023-8-25 10:40:12

liuhongrun2022 发表于 2023-8-25 10:38
哈哈哈哈

cao

琅琊王朝 发表于 2023-8-25 11:34:23

在你提供的代码中,主要问题是循环条件的设置和一些改进的地方。下面是对代码进行修改和改进的建议:

1. 循环条件问题:当前的代码使用了`do-while`循环,并将循环条件设置为`num < 1000`。然而,根据题目要求,水仙花数的范围应该是100到999,即`num <= 999`。所以,需要将循环条件改为`num <= 999`。

2. 改进建议:
   - 变量命名:可以使用更具有描述性的变量名称,例如使用`units`代替`a`,`tens`代替`b`,`hundreds`代替`c`。
   - 增加注释:通过添加注释,可以更好地理解代码的逻辑和目的。
   - 提前声明变量:可以在函数开头或循环之前声明变量,以提高代码的可读性。

下面是修改后的代码:

void test02() {
    // 水仙花数 1的三次方+5的三次方+3的三次方=153
    // 要求找出所有水仙花数字
    // 定义一个数,然后设置一个门槛,100到1000之间,100为初始值,1000为结束条件,抽取个十百位的数然后用if为筛选条件
    // 最后用while结尾,循环多次找到结果
    int num = 100;

    do {
      int units = num % 10;
      int tens = num / 10 % 10;
      int hundreds = num / 100;

      if (units * units * units + tens * tens * tens + hundreds * hundreds * hundreds == num) {
            printf("%d\n", num);
      }

      num++;
    } while (num <= 999);
}


希望这些修改和改进能对你的代码有帮助!如果还有其他问题,请随时提问。
此内容为ChatGPT回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!

琅琊王朝 发表于 2023-8-25 12:18:42

在你提供的代码中,主要问题是你没有在循环体内更新 `num` 的值,导致进入死循环。除此之外,其他部分看起来是正确的。下面是对代码进行修改和改进的建议:

void test02() {
    // 水仙花数 1的三次方+5的三次方+3的三次方=153
    // 要求找出所有水仙花数字
    // 定义一个数,然后设置一个门槛,100到1000之间,100为初始值,1000为结束条件,抽取个十百位的数然后用if为筛选条件
    // 最后用while结尾,循环多次找到结果
    int num = 100;

    do {
      int a = num % 10;
      int b = num / 10 % 10;
      int c = num / 100;

      if (a * a * a + b * b * b + c * c * c == num) {
            printf("%d\n", num);
      }

      num++; // 更新 num 的值
    } while (num < 1000);
}


修改后的代码通过增加 `num++` 表达式来更新 `num` 的值,以避免进入死循环。这样,循环会逐步增加 `num` 的值并检查每个值是否符合水仙花数的条件。

希望这些修改和改进能对你的代码有帮助!如果还有其他问题,请随时提问。
此内容为ChatGPT回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!

琅琊王朝 发表于 2023-8-25 15:10:02

在你提供的代码中,主要问题是你没有在循环体内更新 `num` 的值,导致进入死循环。除此之外,其他部分看起来是正确的。下面是对代码进行修改和改进的建议:

void test02() {
    // 水仙花数 1的三次方+5的三次方+3的三次方=153
    // 要求找出所有水仙花数字
    // 定义一个数,然后设置一个门槛,100到1000之间,100为初始值,1000为结束条件,抽取个十百位的数然后用if为筛选条件
    // 最后用while结尾,循环多次找到结果
    int num = 100;

    do {
      int a = num % 10;
      int b = num / 10 % 10;
      int c = num / 100;

      if (a * a * a + b * b * b + c * c * c == num) {
            printf("%d\n", num);
      }

      num++; // 更新 num 的值
    } while (num < 1000);
}


修改后的代码通过增加 `num++` 表达式来更新 `num` 的值,以避免进入死循环。这样,循环会逐步增加 `num` 的值并检查每个值是否符合水仙花数的条件。

希望这些修改和改进能对你的代码有帮助!如果还有其他问题,请随时提问。
此内容为ChatGPT回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!

sfqxx 发表于 2023-8-29 20:15:29

琅琊王朝 发表于 2023-8-25 15:10
在你提供的代码中,主要问题是你没有在循环体内更新 `num` 的值,导致进入死循环。除此之外,其他部分看起 ...

《连发3次》
页: [1]
查看完整版本: do_while语句的练习题,水仙花数