12小时制转24小时制C++程序输出空白问题排查

我帮你仔细排查了这段代码，发现几个关键问题导致输出空白或者结果错误，下面逐个拆解：

1. AM/PM判断逻辑不完整且字符处理有误 #

你用s[8]==80判断PM，虽然ASCII中'P'的编码确实是80，但更直观且不易出错的写法是s[8] == 'P'。但更大的问题是：你只处理了PM的情况，完全忽略了AM场景。比如输入12:30:45AM时，你的代码会跳过所有处理逻辑，导致p是空字符串，最终返回的结果自然是空白。

2. 字符转数字的方式完全错误 #

你直接用s[0]*10 + s[1]计算小时数，这是典型的字符与数字混淆的错误。s[0]是字符类型（比如'1'的ASCII值是49），这样计算出来的不是十进制的小时数，而是ASCII值的运算结果（比如'1'+'2'会得到49+50=99，而不是12）。正确的做法是先把字符转为对应数字：(s[0] - '0') * 10 + (s[1] - '0')。

3. 未初始化字符串p导致未定义行为 #

你声明了string p;但没有给它分配空间，直接执行p[i]=s[i]会访问未分配的内存，这会导致程序行为异常——可能输出空白、崩溃，或者产生随机乱码。

4. PM小时处理的边界错误 #

当PM的小时是12时（比如12:15:30PM），不需要加12，否则会得到24，而24小时制里应该保持12。你的代码没有处理这个边界情况，会导致错误的输出。

5. 多余代码导致编译错误 #

main函数里的enter code here是无效代码，会直接导致编译失败，这也是程序无法正常运行的原因之一。

修正后的代码 #

这里保留你原有的代码框架，修复了所有问题：

#include <iostream>
#include <string>
#include <cstdlib> // 用于getenv
using namespace std;

string timeConversion(string s) {
    string result;
    char period = s[8]; // 获取AM/PM的第一个字符
    // 正确转换小时为数字
    int hour = (s[0] - '0') * 10 + (s[1] - '0');

    if (period == 'P') { // 处理PM情况
        if (hour != 12) {
            hour += 12;
        }
        // 更新原字符串的小时位
        s[0] = (hour / 10) + '0';
        s[1] = (hour % 10) + '0';
    } else { // 处理AM情况
        if (hour == 12) { // 12AM转为00
            s[0] = '0';
            s[1] = '0';
        }
    }
    // 移除最后两位的AM/PM
    result = s.substr(0, 8);
    return result;
}

int main() {
    ofstream fout(getenv("OUTPUT_PATH"));
    string s;
    getline(cin, s);
    string result = timeConversion(s);
    fout << result << "\n";
    fout.close();
    return 0;
}

修正说明 #

• 正确处理了AM/PM的所有边界情况（比如12AM转00，12PM保持12）；
• 修复了字符转数字的逻辑，确保小时计算正确；
• 给字符串result正确赋值，避免未初始化的问题；
• 移除了无效的enter code here代码，保证编译通过。

内容的提问来源于stack exchange，提问作者D.Sheshma