Question

这段代码有什么问题：

std::string StringPrintf(const char* fmt, ...) {
  int size = 512;
  char* buffer = new char[size];
  va_list vl; 
  va_start(vl, fmt);
  int nsize = vsnprintf(buffer, size, fmt, vl);

  if (size <= nsize) {  //fail delete buffer and try again
    delete[] buffer;
    buffer = 0;
    buffer = new char[nsize + 1];  //+1 for /0
    nsize = vsnprintf(buffer, size, fmt, vl);
  }
  std::string ret(buffer);
  va_end(vl);
  delete[] buffer;
  return ret;
}

如果我使用这样的功能，它会崩溃：

string a = "x^x-SIL+zh=ang@x_x/A:x_x_x/B:x-x-x@x-x&x-x|x/C:2+x+2/D:x_x/E:x+x@x+x&x+x#x+x/F:uj_1/G:x_x/H:x=x^x=x|x/I:1=1/J:7+6-6x^x-SIL+zh=ang@x_x/A:x_x_x/B:x-x-x@x-x&x-x|x/C:2+x+2/D:x_x/E:x+x@x+x&x+x#x+x/F:uj_1/G:x_x/H:x=x^x=x|x/I:1=1/J:7+6-6 x^SIL-zh+ang=ch@2_2/A:x_0_x/B:1-x-3@1-2&1-2|x/C:2+x+2/D:x_x/E:nr+2@1+1&x+x#x+x/F:uj_1/G:x_x/H:2=1^1=6|0/I:1=1/J:7+6-6 SIL^zh-ang+ch=i@3_1/A:x_0_x/B:1-x-3@1-2&1-2|x/C:2+x+2/D:x_x/E:nr+2@1+1&x+x#x+x/F:uj_1/G:x_x/H:2=1^1=6|0/I:1=1/J:7+6-6 zh^ang-ch+i=d@1_2/A:1_0_3/B:2-x-2@2-1&2-1|i/";
string rs = StringPrintf("result=>\n%s;", a.c_str());

这是什么问题？我该如何解决？

非常感谢!!

Answer 1

在这里用完了va_list：

int nsize = vsnprintf(buffer, size, fmt, vl);

但稍后您尝试在结束时再次使用相同的列表vl：

nsize = vsnprintf(buffer, size, fmt, vl)

在进行另一次vsnprintf通话之前，您需要开始一个新列表：

va_end(vl);
va_start(vl, fmt);

另外，为了避免内存错误，请使用vector代替new char。实际上你可以调整输出字符串的大小并直接写入。

Answer 2

您的字符串长度为512个字符，并且您正在为函数中的buffer分配512个字节，因此出错。增加缓冲区的大小。改变这个陈述

char* buffer = new char[size];

vsnprintf()及其变体在缓冲区太小时会静默截断字符串，但它们不会返回字符串的实际长度，因此您无法再使用较长的缓冲区再次尝试它。他们返回-1。

使用_vscprintf。这将返回所需的字符数，而不实际格式化任何内容。

处理长字符串（或奇怪）时出现c ++ vsnprintf错误

2 个答案: