0

可能重复：
c ++ - 字符串上的 printf 打印乱码

我想将几个字符串写入文件。字符串是

37 1 0 0 0 0
15 1 0 0 0 0
33 1 0 0 0 0
29 1 0 0 0 0
18 1 0 0 0 0
25 1 0 0 0 0

我首先想将每一行存储为字符串数组的元素，然后调用相同的字符串数组并将其元素写入文件。

#include <stdio.h>
#include <vector>
#include <string>
using namespace std;

int writeFile() {

  char line[100];
  char* fname_r = "someFile_r.txt"
  char* fname_w = "someFile_w.txt"; 
  vector<string> vec;

  FILE fp_r = fopen(fname_r, "r");
  if(fgets(line, 256,fp_r) != NULL)   {
     vec.push_back(line);
  }

  FILE fp_w = fopen(fname_w, "w");
  for(int j = 0; j< vec.size(); j++) {
    fprintf(fp_w, "%s", vec[j]); // What did I miss? I get funny symbols here. I am expecting an ASCII
  }

  fclose(fp_w);
  fclose(fp_r);
  return 0;
}
4

2 回答 2

7

格式说明符"%s"需要一个 C 风格的以空字符结尾的字符串，而不是一个std::string. 改成：

fprintf(fp_w, "%s", vec[j].c_str());

由于这是 C++，您应该考虑使用ofstream类型安全并接受std::string作为输入的替代：

std::ofstream out(fname_w);
if (out.is_open())
{
    // There are several other ways to code this loop.
    for(int j = 0; j< vec.size(); j++)
        out << vec[j];
}

同样，ifstream用于输入。发布的代码有潜在的缓冲区溢出：

char line[100];
...
if(fgets(line, 256,fp_r) != NULL)

line最多可以存储100字符，但fgets()说明它可以容纳256. Usingstd::getline()在填充 a 时消除了这种潜在的危险std::string：

std::ifstream in(fname_r);
std::string line;
while (std::getline(in, line)) vec.push_back(line);
于 2012-08-29T09:52:54.353 回答
0

在这种情况下 vec[j] 是 std::string 对象。但是需要 c 风格的fprintf以s空字符结尾的字符串。

for(int j = 0; j< vec.size(); j++) {
    fprintf(fp_w, "%s", vec[j]); 
}

您只需要从 std::string 获取指向 c 样式字符串的指针。可以使用c_str方法：

for(int j = 0; j< vec.size(); j++) {
    fprintf(fp_w, "%s", vec[j].c_str()); 
}

在任何情况下，您都混合了 C++ 和 C 代码。它很丑。使用 std::fstream 更好。

于 2012-08-29T09:56:26.500 回答