C语言中检测到非数字字符后如何终止程序运行?
解决字符串全数字校验后程序未终止的问题
你的核心问题是调用only_digits函数后没有处理它的返回值——即使函数检测到非数字字符并返回1,main函数依然会继续执行后续代码,不会终止。
修复步骤:
处理
only_digits的返回值
在main函数中调用only_digits后,判断其返回结果:如果返回非0(即检测到非数字),直接终止程序。修正无效的数值判断
原代码中else if (argv[1] < 0)完全无效——argv[1]是字符串指针,不是整数,无法直接和0比较。如果需要校验key为正整数,应该在确认全是数字后,将字符串转为整数再判断。
修改后的完整代码:
only_digits函数(保持逻辑,注意需包含<ctype.h>头文件)
#include <ctype.h> #include <string.h> #include <stdio.h> #include "cs50.h" int only_digits(string key) { for (int i = 0, n = strlen(key); i < n; i++) { if (!isdigit(key[i])) // 更简洁的写法,替代==0 { printf("Not a Digit\n"); return 1; } } return 0; }
main函数
#include <stdio.h> #include <stdlib.h> #include "cs50.h" int main(int argc, string argv[]) { if (argc != 2) { printf("Usage: ./caesar key\n"); return 1; } string dig = argv[1]; // 校验是否全为数字,非全数字则终止 if (only_digits(dig) != 0) { return 1; } // 额外校验key为正整数 int key = atoi(dig); if (key <= 0) { printf("Usage: ./caesar key\n"); return 1; } string plain = get_string("Enter Text: "); printf("Plaintext: %s\n", plain); return 0; }
为什么之前用break无效?
如果把return 1换成break,只会跳出for循环,函数最终会返回0——main函数接收到0的返回值,会认为校验通过,继续执行后续代码,所以必须让函数返回非0值,且在main中对返回值做判断。
内容的提问来源于stack exchange,提问作者Ryan Lewis
相关产品推荐
相关产品推荐

