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

学c的萌新

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

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

首先，你可以将计算水仙花数的逻辑封装成一个函数，这样可以提高代码的可读性和复用性。例如，你可以创建一个名为 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);
        }
    }
}

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

陶远航

你的代码基本上是正确的，并且可以用于找出范围在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

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

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

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

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

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

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

c

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

琅琊王朝

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

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

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

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

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

c

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

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

sfqxx

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

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

《c》

琅琊王朝

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

脚本太聪明了

liuhongrun2022

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

这里的代码有一个小问题，即在 `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:27
脚本太聪明了

哈哈哈哈

琅琊王朝

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

cao

琅琊王朝

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

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回答，如果胡说八道，请不要理会
如果对你有帮助，请设置一个最佳答案！

琅琊王朝

在你提供的代码中，主要问题是你没有在循环体内更新 `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回答，如果胡说八道，请不要理会
如果对你有帮助，请设置一个最佳答案！

琅琊王朝

在你提供的代码中，主要问题是你没有在循环体内更新 `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-25 15:10
在你提供的代码中，主要问题是你没有在循环体内更新 `num` 的值，导致进入死循环。除此之外，其他部分看起 ...

《连发3次》