鱼C论坛

 找回密码
 立即注册
查看: 1117|回复: 13

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

[复制链接]
发表于 2023-8-24 22:21:44 | 显示全部楼层 |阅读模式

马上注册,结交更多好友,享用更多功能^_^

您需要 登录 才可以下载或查看,没有账号?立即注册

x
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);
}
请大佬们班帮我看一下出现了什么问题,还有哪些需要改进的地方,谢谢大佬们
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复

使用道具 举报

发表于 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);
        }
    }
}

希望对你有帮助!如果有任何问题,请随时提问。
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 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之间的所有水仙花数,并以"水仙花数是:"的形式进行输出。

希望这能够帮助你!如果你还有其他疑问,请随时提问。
如果问题已经解决,请设置最佳答案
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 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
希望对你有所帮助!如果还有其他问题,请随时提问。
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-8-25 10:18:16 | 显示全部楼层
你的代码基本是正确的,但有几个地方可以改进:

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

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

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

下面是改进后的代码示例:
c
这样修改后的代码应该能够正确输出所有的水仙花数。

希望对你有所帮助!如果还有其他问题,请随时提问。
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-8-25 10:19:35 | 显示全部楼层
琅琊王朝 发表于 2023-8-25 10:18
你的代码基本是正确的,但有几个地方可以改进:

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

《c》
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-8-25 10:27:32 | 显示全部楼层

脚本太聪明了
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 0 反对 1

使用道具 举报

发表于 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回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-8-25 10:38:54 | 显示全部楼层

哈哈哈哈
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-8-25 10:40:12 | 显示全部楼层
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 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回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 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回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 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回答,如果胡说八道,请不要理会
如果对你有帮助,请设置一个最佳答案!
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

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

《连发3次》
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2024-6-17 06:44

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表