在构造函数c ++中打开文件答案

【问题标题】：open file in constructor c++在构造函数c ++中打开文件
【发布时间】：2015-11-06 09:50:14
【问题描述】：

我正在尝试在构造函数中打开一个文件并从其他函数写入该文件，但未创建文件。

#include<iostream>
#include<fstream>
#include<cstring>

#define BAD_FILE_NAME_PATTERN "MYFILE_"
#define BAD_FILE_PATH "$HOME/"


class A
{
        private:
        std::ofstream m_badReprtLogFilePtr;
        char *m_badReprtLogFilePtrName;

        public:
        void rechargeEventBadFileHandler();
        void writeToFile();
        A();
};

A::A()
{
        time_t     now = time(0);
        struct tm  tstruct;
        char dateTime[80];
        tstruct = *localtime(&now);
        strftime(dateTime, sizeof(dateTime), "%F_%H%M%S", &tstruct);
        m_badReprtLogFilePtrName = new char[strlen(BAD_FILE_NAME_PATTERN) + sizeof(dateTime)];
        strcpy(m_badReprtLogFilePtrName,BAD_FILE_PATH);
        strcat(m_badReprtLogFilePtrName,BAD_FILE_NAME_PATTERN);
        strncat(m_badReprtLogFilePtrName,dateTime,sizeof(dateTime));
        strcat(m_badReprtLogFilePtrName,".bad");
        std::cout<< "Bad File Name " << m_badReprtLogFilePtrName << std::endl;
        m_badReprtLogFilePtr.open(m_badReprtLogFilePtrName, std::ios::out | std::ios::app);
        if(!m_badReprtLogFilePtr.is_open())
        {
           std::cout<<"Error opening Bad file"<<std::endl;
        }
}

void A::writeToFile()
{
        m_badReprtLogFilePtr << "Writting to BAD FILE"<<std::endl;
}

/*
A::~A()
{
        m_badReprtLogFilePtr.close();
}
*/

int main()
{
        A obj;
        obj.writeToFile();
        return 0;
}

我遇到了问题，问题出在路径上，但是如何解决这个问题

【问题讨论】：

里面的 C 比 C++ 多！！
帮我一个忙，运行一次程序，然后修改程序并在打开文件时添加以下标志。标准::ios::noreplace。文件是否在您第二次运行程序时打开？
$HOME 等环境变量不是文件系统的一部分。除非您有一个字面上称为“$HOME”的目录，否则您需要自己扩展它（参见例如getenv）。
我无法重现该问题：coliru.stacked-crooked.com/a/06cc26d6e68e0eee 我的猜测：您忘记创建目录了。
Offtopic：您可以将此 strcpy / strcat 部分缩短为：snprintf(m_badReprtLogFilePtrName, ..., "%s%s%s.bad", BAD_FILE_PATH, BAD_FILE_NAME_PATTERN, dateTime)

标签： c++ file-handling

【解决方案1】：

问题在于：

#define BAD_FILE_PATH "$HOME/"

您可以在 Unix 上使用 getenv("HOME") 或在 Windows 上连接 getenv("HOMEDRIVE") 和 getenv("HOMEPATH") 的结果。

例如在 Unix 上

strcpy(m_badReprtLogFilePtrName,getenv("HOME"));
strcat(m_badReprtLogFilePtrName,"/");

【讨论】：

感谢您的信息:)

【解决方案2】：

要解决的问题：

m_badReprtLogFilePtrName 数组的大小错误。只有名称和日期的空间（不用于路径、扩展名和空终止字符）。
strncat(m_badReprtLogFilePtrName,dateTime,sizeof(dateTime));.

strncat 的大小参数是在目标中已经存在的字符串末尾之后剩余的空间量。 您正在使用源尺寸。
存在内存泄漏（析构函数应删除m_badReprtLogFilePtrName）。
您应该展开 $HOME 路径 (std::getenv)。要让getenv 工作，BAD_FILE_PATH 应定义为：
```
#define BAD_FILE_PATH "HOME"
```
（看看getenv on cppreference.com）。
你应该使用std::string

比如：

#include <cstdlib>
#include <cstring>
#include <iostream>
#include <fstream>
#include <string>

#define BAD_FILE_NAME_PATTERN "MYFILE_"
#define BAD_FILE_PATH "HOME"

class A
{
public:
  A();

  // ...

private:
  std::ofstream m_badReprtLogFilePtr;
  std::string m_badReprtLogFilePtrName;
};

A::A()
{
  time_t now = time(0);

  struct tm tstruct;
  char dateTime[80];
  tstruct = *localtime(&now);

  strftime(dateTime, sizeof(dateTime), "%F_%H%M%S", &tstruct);

  char *path = std::getenv(BAD_FILE_PATH);
  if (path)
    m_badReprtLogFilePtrName = std::string(path) + "/";

  m_badReprtLogFilePtrName += BAD_FILE_NAME_PATTERN;
  m_badReprtLogFilePtrName += dateTime;
  m_badReprtLogFilePtrName += ".bad";

  std::cout << "Bad File Name " << m_badReprtLogFilePtrName << "\n";

  m_badReprtLogFilePtr.open(m_badReprtLogFilePtrName.c_str(),
                            std::ios::out | std::ios::app);

  if (!m_badReprtLogFilePtr.is_open())
    std::cout << "Error opening Bad file\n";
}

可能有效（当然还有很多其他需要改进的地方）。

【讨论】：

【解决方案3】：

有几个地方出错了。

C 或 C++ 程序（您的大部分代码都使用 C 技术，即使您使用 C++ 编译器构建它）不能解释 $HOME 的含义。这取决于 shell（在 unix 下），需要以其他方式扩展（例如使用 getenv()）。

分配给m_badReprtLogFilePtrName 的字符数等于strlen(BAD_FILE_NAME_PATTERN) + sizeof(dateTime)。 strcpy()、strcat() 和 strncat() 调用可能会将 strlen(BAD_FILE_PATH) + strlen(BAD_FILE_NAME_PATTERN) + min(strlen(dateTime), sizeof dateTime) + strlen(".bad") + 1 字符写入该缓冲区。奇怪的是，正在写入的字符数超过了分配的长度，因此结果是未定义的行为。

不要使用运算符new 手动分配字符数组，而是尝试使用标准std::string 类来管理字符串。然后当对象被销毁时，字符串将被很好地清理，而不是像现在这样泄漏。

哦：“写作”用一个“t”拼写。

【讨论】：