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);
}
请大佬们班帮我看一下出现了什么问题，还有哪些需要改进的地方，谢谢大佬们 你的代码基本上是正确的，但是有一些可以改进的地方。
首先，你可以将计算水仙花数的逻辑封装成一个函数，这样可以提高代码的可读性和复用性。例如，你可以创建一个名为 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之间的所有水仙花数，并以"水仙花数是："的形式进行输出。
希望这能够帮助你！如果你还有其他疑问，请随时提问。
如果问题已经解决，请设置最佳答案 你的代码基本上是正确的，但还有一些可以改进的地方：
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
这样修改后的代码应该能够正确输出所有的水仙花数。
希望对你有所帮助！如果还有其他问题，请随时提问。 琅琊王朝 发表于 2023-8-25 10:18
你的代码基本是正确的，但有几个地方可以改进：
1. 在 `do...while` 循环中，应该在每次迭代结束后递增 `n ...
《c》 sfqxx 发表于 2023-8-25 10:19
《c》
脚本太聪明了
{:5_104:} 这段代码的目标是找到所有的水仙花数。水仙花数是指一个三位数，其个位、十位和百位上的数字的立方和等于该数本身。
这里的代码有一个小问题，即在 `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回答，如果胡说八道，请不要理会
如果对你有帮助，请设置一个最佳答案！ 琅琊王朝 发表于 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回答，如果胡说八道，请不要理会
如果对你有帮助，请设置一个最佳答案！ 琅琊王朝 发表于 2023-8-25 15:10
在你提供的代码中，主要问题是你没有在循环体内更新 `num` 的值，导致进入死循环。除此之外，其他部分看起 ...
《连发3次》
页:
[1]