пытается реализовать простой ostream singleton class

Question

Я хочу реализовать одноэлементный класс, который получает путь к файлу в качестве параметра. Я попытался написать следующий код. Я знаю, что это не работает, и не хорошо, но я не могу найти, почему ..

class OutputData { 
    std::fstream ofile; 
    std::ostream iout; 
    static OutputData *odata; 
    OutputData(const char* path):iout(std::cout), ofile(path) { 
     if (ofile.is_open()) { 
      iout = ofile; 
     } 
    } 
public: 
    static void print(std::string s) { 
     iout << s; 
    } 
}; 

в .cpp

OutputData *OutputData::odata = nullptr; 

и теперь я хочу, чтобы каждый класс будет иметь способность к напишите в этот поток.

благодарит

Answer 1

Вы не можете принимать любые std::istream или std::ostream экземпляр по копии, ваши переменные-члены должны быть ссылки или указатели:

class OutputData { 
    std::fstream* ofile; 
    std::ostream* iout; 
    static OutputData *odata; 
    OutputData(const char* path):iout(nullptr), ofile(nullptr) { 
     ofile = new fstream(path); 
     if (ofile->is_open()) { 
      iout = ofile; 
     } 
     else { 
      delete ofile; 
      ofile = nullptr; 
      iout = &cout; 
     } 
    } 
    // Take care to provide an appropriate destructor 
    ~OutputData() { 
     delete ofile; 
    } 

}; 

---

Также относительно вашего одноплодной дизайн, я бы скорее рекомендуют одиночную идиому Скотта Мейера:

class OutputData { 
public: 
    static OutputData& instance(const char* path) { 
     static OutputData theInstance(path) 
     return theInstance; 
    } 
    // Make print a non-static member function 
    void print(std::string s) { 
     iout << s; 
    } 
}; 

---

Хотя этот подход просто странно выглядит как противоположное, как то, что считается каноническим решением.