4

我希望在我的项目中使用 41 个输出文件来在它们上面写文本。首先创建一个字符串数组list来命名这些输出文件,然后我尝试定义一个 ofstream 对象数组并使用list它们来命名它们,但是我得到了这个错误'outfile' cannot be used as a function。下面是我的代码:

#include <sstream>
#include <string>
#include <iostream>
#include <fstream>
using namespace std ;
int main ()
{
  string list [41];
  int i=1;
  ofstream *outFile = new ofstream [41];

  for (i=1;i<=41 ;i++)
  {
    stringstream sstm;
    sstm << "subnode" << i;
    list[i] = sstm.str();
  }

  for (i=0;i<=41;i++)
    outFile[i] (list[i].c_str());

  i=1;
  for (i=1;i<=41;i++)
    cout << list[i] << endl;

  return 0; 
}
4

2 回答 2

7

有关以下修复,请参见下文:

  1. 除非必须,否则不要使用new(您泄漏了所有文件并且没有正确破坏它们将导致数据丢失;如果您没有正确关闭它们,ofstreams 可能不会被刷新,并且挂起的输出缓冲区将丢失)
  2. 使用正确的数组索引(从 0 开始!)
  3. 调用默认构造.open(...)来打开文件 ofstream
  4. 建议:
    • 我建议反对using namespace std;(以下未更改)
    • 我建议重复使用stringstream. 这是一个好习惯
    • 更喜欢使用 C++ 风格的循环索引变量 ( for (int i = ....)。这可以防止意外的i范围过大。
    • 事实上,与时俱进,使用 ranged for


#include <sstream>
#include <string>
#include <iostream>
#include <fstream>
using namespace std;
int main ()
{
    ofstream outFile[41];

    stringstream sstm;
    for (int i=0;i<41 ;i++)
    {
        sstm.str("");
        sstm << "subnode" << i;
        outFile[i].open(sstm.str());
    }

    for (auto& o:outFile)
        cout << std::boolalpha << o.good() << endl;
}
于 2012-10-11T08:57:33.090 回答
1

你不能像你一样调用构造函数。尝试调用outFile[i].open(list[i].c_str()). 注意“开放”。

于 2012-10-11T08:42:47.797 回答