4字节非负整数十六进制去小于5数位的程序开发求助

我看了你的代码，发现几个关键问题导致它没法正确工作，咱们一步步理清楚并修正：

你的代码里的核心问题 #

• 字符数值转换逻辑错误：你直接拿十六进制字符的ASCII值和5比较，比如字符'0'的ASCII是48，远大于5，这完全搞错了判断逻辑。得先把字符转换成对应的十六进制数值（比如'3'转成3，'a'转成10），再判断是否小于5。
• 使用非标准函数itoa：itoa不是C++标准库的函数，不同编译器（比如GCC）可能不支持，换成标准的snprintf更具可移植性。
• 循环中重复调用strlen：每次循环都调用strlen会降低效率，而且当你修改数组时，字符串长度变化可能导致逻辑出错，应该先缓存初始长度或者用更高效的指针方式处理。

修正后的代码 #

#include <iostream>
#include <cstdio>
#include <cstring>
#include <cctype>

using namespace std;

int main() {
    unsigned int x = 10012;
    char hexString[32];
    // 用标准的snprintf转换为十六进制字符串（小写形式）
    snprintf(hexString, sizeof(hexString), "%x", x);
    cout << "原始十六进制：" << hex << x << endl;

    int writeIdx = 0;
    int len = strlen(hexString);
    for (int readIdx = 0; readIdx < len; ++readIdx) {
        char c = hexString[readIdx];
        int value;
        // 将十六进制字符转换为对应的数值
        if (isdigit(c)) {
            value = c - '0';
        } else {
            // 统一处理大小写字母，转换为10-15的数值
            value = tolower(c) - 'a' + 10;
        }
        // 只保留数值>=5的字符
        if (value >= 5) {
            hexString[writeIdx++] = c;
        }
    }
    // 手动添加字符串结束符，避免残留字符干扰
    hexString[writeIdx] = '\0';

    cout << "处理后十六进制：" << hexString << endl;

    // 替换非标准的system("pause")，用标准输入等待用户操作
    cout << "按回车键继续..." << endl;
    cin.get();
    return 0;
}

代码关键改进说明 #

1. 标准字符串转换：用snprintf替代itoa，保证在所有C++编译器环境下都能正常运行；%x指定输出小写十六进制，若需要大写可改为%X。
2. 正确的数值转换：通过isdigit判断数字字符，直接减'0'得到对应数值；字母字符先统一转小写，再计算为10-15的十六进制数值。
3. 双指针高效处理：用readIdx遍历原字符串，writeIdx记录要写入的位置，比你原来的删除移位写法更高效，也避免了循环中修改索引的混乱。
4. 完善字符串收尾：处理完成后必须手动添加'\0'，否则字符串会包含原数组的残留字符，导致输出异常。
5. 可移植的暂停方式：用cin.get()等待用户输入，替代依赖系统的system("pause")，适配更多操作系统。

拿你的示例值x=10012测试，它的十六进制是271c：

• 数位对应数值：2（<5，删除）、7（>=5，保留）、1（<5，删除）、12（>=5，保留），所以处理后结果为7c，运行代码就能得到正确输出。

内容的提问来源于stack exchange，提问作者Notail