如何确保字符串为纯数字?现有验证函数处理「11.」存在异常
问题分析与解决方法
原函数的核心bug是循环只检查第一个字符就返回结果:只要第一个字符是数字,直接返回true,后面的字符(比如"11."里的'.')根本没机会被检查,这才导致误判。之前改判断条件或者用取模的思路没用,都是因为没先修复这个循环逻辑的根本问题。
修复方案1:修正循环逻辑
遍历字符串的每一个字符,只要发现任何一个不是数字的字符,立刻返回false;只有当所有字符都验证通过后,才返回true。同时要注意,isdigit的参数需要是unsigned char类型(避免有符号char的未定义行为):
/* Checks if the inventory is valid. Criteria: Valid if and only if it contains only digits */ bool isValidInventory(string token) { // 空字符串直接判定无效 if (token.empty()) { return false; } for (char ch : token) { // 转换为unsigned char避免isdigit的未定义行为 if (!isdigit(static_cast<unsigned char>(ch))) { return false; } } return true; } // End of isValidInventory() function.
修复方案2:用C++标准库算法简化代码
利用<algorithm>里的all_of算法,代码更简洁,逻辑和上面一致:
#include <algorithm> // 需要包含这个头文件 bool isValidInventory(string token) { if (token.empty()) { return false; } return all_of(token.begin(), token.end(), [](char ch) { return isdigit(static_cast<unsigned char>(ch)); }); }
修复方案3:正则表达式匹配
如果需要更灵活的规则(比如后续可能调整验证逻辑),可以用正则表达式匹配^[0-9]+$(表示整个字符串由一个或多个数字组成):
#include <regex> // 需要包含这个头文件 bool isValidInventory(string token) { std::regex digit_regex("^[0-9]+$"); return std::regex_match(token, digit_regex); }
为什么之前的尝试无效?
- 改
isdigit(ch) && !ispunct(ch)没用:因为原循环只检查第一个字符,就算判断条件对了,后面的字符没遍历,还是会误判。 - 取模运算不行:把"11."转成浮点数后会变成11.0,数值上是整数,但这是数值验证,不是字符串格式验证,和需求不符。
内容的提问来源于stack exchange,提问作者Brother Bear
相关产品推荐
相关产品推荐

