连接问题 +伊托阿

Question

我有以下代码：

char stringHour[50], stringMinute[50], stringSecond[50];

// lots of code...
itoa(hour, stringHour, 10);
itoa(minute, stringMinute, 10);
itoa(second, stringSecond, 10);

strcat(":", stringSecond);
strcat(":", stringMinute);
strcat(stringMinute, stringSecond);
// stringHour contains both HH and :SS:MM
strcat(stringHour, stringMinute);
drawText(HRES/2 - 4, VRES - GLYPH_HEIGHT*2, 
stringHour, black);

小时、分钟和秒都是整数。

我想要做的是显示时间如下：HH:MM:SS。

变量保证在 0 到 59 之间（hout 除外，0-24）。

绘制文本是正确的。

这段代码使我的程序崩溃，但我找不到任何问题。你可以吗？

感谢您抽出时间！

Answer 1

这是错误的：

strcat(":", stringSecond);

您不能使用字符串作为第一个参数。请参阅 strcat 文档。

Answer 2

失败的地方很可能是这一部分：

... stuff removed
strcat (":", 
... other stuff removed

您几乎肯定无法将 strcat 填充到您正在使用的平台上的字符串文字上。

strcat 方法也存在缓冲区溢出安全问题。也许使用 snprintf 可能是更好的选择？

Answer 3

另外，最好将数组初始化为零。

char stringHour[50] = {0}, stringMinute[50] = {0}, stringSecond[50] = {0};

然后尝试将猫的结果存储到字符串中，看看你会得到什么。

Answer 4

您不能修改字符串文字；使用 char buffer[] = "MyLiteral" 代替，然后使用 strcat() 。

（这一事实背后的原因是，编译器可以通过将字符串文字的多个实例合并到可执行文件的只读部分中的一个实例来优化代码大小，因此，如果您可以更改一个实例，那么您将更改其他所有实例，这会导致不可预测的行为。）

Answer 5

另外 - 你的 strcat 是相反的顺序。 strcat 实际上是“连接”的 - 所以改变顺序。
strcat(stringHour, ":");
strcat(stringHour, stringMinute);
strcat(stringHour, ":");
strcat(stringHour, stringSecond);

然而，这非常糟糕——为什么不使用 sprintf 呢？另外 - 除了字符串构建之外的其他库 - 可能会查看 BSL。 C 标准库实际上也没有为您进行错误检查 - 所以这是搬起石头砸自己的脚，除非您确切地知道自己在做什么以及 str* 系列函数的期望是什么。