0

当我编译并运行我的程序时，除了我收到“运行时检查失败 #2”错误之外，似乎一切都可以正常处理手头的任务。顺便说一句，这是我的硬件任务，这是我第一次尝试使用任何 cstring 函数，所以我确定这是我出错的地方。基本上我将 2 个字符串附加在一起，我几乎 100% 确定它与我的结果参数溢出有关。只是不知道如何解决它。

#include <iostream>
#include <cstring>
using namespace std;

void concat(const char a[ ], const char b[ ], char result[ ], int result_maxlength);
int main()
{
  char a[] = "Woozle";
  char b[] = "Heffalump";
  char c[5];
  char d[10];
  char e[20];
  concat(a, b, c, 5);
  concat(a, b, d, 10);
  concat(a, b, e, 20);
  cout << c << "\n";
  cout << d << "\n";
  cout << e << "\n";
  return 0;
}
void concat(const char a[ ], const char b[ ], char result[ ], int result_maxlength)
{
strncpy (result,a, result_maxlength);
strncat (result, b, result_maxlength);
result[result_maxlength-1] = '\0';
}
4

2 回答 2

1

strncat非常没用，因为num参数不是目标的总大小，而是要复制的字节数。

于 2012-09-01T04:20:27.913 回答
1

到目前为止，有几个人已经指出了这段代码的几个问题。似乎很少有人谈论的一个非常微妙的问题是：根据 POSIX 规范，strncpy如果移动的非空字符数达到指定的限制（'n'），则不会在字符串上设置空终止符的行为. 但是，它会尾部填充空值以达到“n”。因此：

char ar[3];
strncpy(ar, "food", sizeof(ar)/sizeof(ar[0]));

这将以 a(0) = 'f'、a(1) = 'o' 和 ar(2) = 'o' 结束。不会附加空终止符。忽略最后一个字符正是因为这个原因，人们对调用这个运行时库函数感到如此头晕目眩。考虑到这一点很重要。因此，您会经常看到这样的代码。

char ar[3];
strncpy(ar, "food", sizeof(ar)/sizeof(ar[0])-1);
ar[sizeof(ar)/sizeof(ar[0])-1] = 0;

这可能会得到你真正想要的东西，即 ar = "fo"。充足的空间情况逆转了这一点。如果您有一个有足够空间的缓冲区，如果副本的源字符串在到达“n”之前到达它自己的终止符，则“n”将用空值填充直到到达“n”。因此：

char ar[30];
strncpy(ar, "food", sizeof(ar)/sizeof(ar[0])-1);
ar[sizeof(ar)/sizeof(ar[0])-1] = 0;

将导致 ar = "food" 在 'd' 后面有 26 个零字符。让人们保持警惕，不那么明显的 memset-to-zero 调用：

char ar[30];
strncpy(ar, "", sizeof(ar)/sizeof(ar[0]));

是的，这是错误的，我知道。

话虽如此，显然 strncat() 的行为与您的代码似乎也反映了您目前可能认为的不同，这不足为奇。例如，最后一个参数未指定目标中要限制的字符数。相反，它描述了限制从源复制的字符数。换句话说，随着您继续添加新琴弦，尾部空间限制器应该不断变短。当然，要知道多少涉及跟踪每个步骤复制的字符，正如有人指出的那样，这使得 strncat 边界线在许多情况下无用，提供的混乱多于实用程序。

作为参考，可以在此处查看具有已定义行为的精确定义strncpy和strncat。在继续你的作业之前，我强烈建议你清楚地阅读两者。

于 2012-09-01T08:06:03.750 回答