0

我在使用函数 vsprintf 时遇到问题。

我有 3 个函数来打开、关闭和写入 XML 文件。open 函数将输入文本的第一个单词存储在一个数组中，close 函数关闭带有该单词的标签。问题是我存储要使用的关闭标记的数组在每次调用打开或写入函数时都会被覆盖（即使写入函数没有对用于存储关闭标记的数组的任何引用）。

int xml_level = 0;
char *xml_header[64];
FILE *xml_out;

void xmlopen(const char *format, ...){
   char buffer[256];
   va_list arglist;
   va_start(arglist,format);
   vsprintf(buffer,format,arglist);
   va_end(arglist);
   int i;
   for(i=0; i<xml_level; i++){
      fprintf(xml_out,"\t");  
   }
   fprintf(xml_out,"<%s>\n",buffer);
   xml_header[xml_level] = strtok (buffer, " ");
   xml_level++;
}

void xmlclose(){
   xml_level--;
   int i;
   for(i=0; i<xml_level; i++){
      fprintf(xml_out,"\t");  
   }
   fprintf(xml_out,"</%s>\n",xml_header[xml_level]);
}

void xmlwrite(const char *format, ...){
   char buffer[256];
   va_list arglist;
   va_start(arglist,format);
   vsprintf(buffer,format,arglist);
   va_end(arglist);
   int i;
   for(i=0; i<xml_level; i++){
      fprintf(xml_out,"\t");  
   }
   fprintf(xml_out,"<%s/>\n",buffer);
}

使用示例：

xmlopen("Hello Word");
xmlopen("Foo Bar");
xmlwrite("Potato");
xmlwrite("Sentence longer than the other ones");
xmlclose();
xmlclose();

输出示例：

<Hello Word>
        <Foo Bar>
                <Potato/>
                <Sentence longer than the other ones/>
        </Sentence longer than the>
</Sentence longer than the>

它应该在哪里：

<Hello Word>
        <Foo Bar>
                <Potato/>
            <Sentence longer than the other ones/>
        </Foo>
</Hello>

谢谢你。

4

2 回答 2

1

你的问题是你只有一个局部变量被调用buffer，xmlopen你（a）存储指向它的指针以在函数外部使用（未定义的行为）和（b）试图在多个调用中使用它（逻辑错误）。

您需要为返回的字符串分配存储空间strtok，并确保稍后处理它们，例如更改：

xml_header[xml_level] = strtok (buffer, " ");

到：

char * s = strtok(buffer, " ");
if (s != NULL)
{
    xml_header[xml_level] = strdup(s);
}

（稍后在不再需要这些字符串时处理这些字符串，留给读者作为练习。）

于 2012-05-21T10:16:09.537 回答
1

解决方案

更改此行：

xml_header[xml_level] = strtok (buffer, " ");

到

xml_header[xml_level] = strdup (strtok(buffer, " "));

并记住在程序退出时释放 xml_headers。

[更新] 当然，您还需要检查可能的极端情况，例如strtok退货NULL等...

解释

strtok不会为返回的令牌分配额外的存储空间。我个人的怀疑是，它会用\0 in place替换分隔符，并每次都将指针返回到下一个标记的开头。

请注意，在您的代码中，您在开头和开头都分配了 256 字节的缓冲区xmlopen，xmlwrite并记得该缓冲区将在堆栈中分配。xmlopen所以在你对or的调用中xmlwrite，buffer实际上会指向同一个地址（你可以打印它的值来验证这个printf("buffer is %p\n", buffer)，它0xbff2481c在我的机器上）。

首先，您调用xmlopen("Hello World")，xml_header[0]并将指向"Hello"，这也是 的开始buffer。然后你调用xmlopen("Foo Bar")，并且xml_header[1]会指向"Foo"，这也是开始的buffer。然后你打电话xmlwrite("Portato")和xmlwrite("Sentence longer than the other ones")。请注意，此时，xml_header[1]仍指向 的开始buffer，现在是"Sentence longer than the other ones"。因此，当您调用 时xmlclose()，它将打印出该句子而不是您预期的令牌，该令牌将被您以后的缓冲区覆盖。

有趣的是，如果您在 和 中分配不同的缓冲区大小xmlopen，xmlwrite例如 256 字节xmlopen和 128 字节xmlwrite，那么您会看到xmlclose会打印出一些不可读的混乱代码。

xml_header[0], xml_header[1]您可以通过检查（例如，打印）和的值来验证所有这些buffer。

于 2012-05-21T11:04:15.130 回答