我正在尝试制作一个多线程程序。
我可以成功编译它,但是我的程序总是崩溃。

这是我的代码片段:

// in global declaration
typedef struct MyData {
    int nSumber;
    char *szFileName;
} MYDATA, *PMYDATA;

PMYDATA pData[MAX_THREAD];

// in my OpenDialog function
OPENFILENAME ofn;
char szFile[MAX_PATH];
ofn.lpstrFile = szFile;
ofn.lpstrFile[0] = '\0';
if(GetOpenFileName(&ofn))
    // I think this line that makes my program crash
    pData[0]->szFileName = ofn.lpstrFile;

我做错了吗?

提前致谢!

最佳答案

pData数组包含指向MYDATA的指针,这些指针未指向有效内存,因为您尚未分配任何内存。

if(GetOpenFileName(&ofn)) {
  pData[0] = new MYDATA();
  pData[0]->szFileName = ofn.lpstrFile;
}

现在,您需要在每个分配的delete元素上调用pData来释放内存。

因此,请使用std::vector<MYDATA>而不是解决所有麻烦。
std::vector<MYDATA> dataVec;

// ...

if(GetOpenFileName(&ofn)) {
  MYDATA data;
  data.szFileName = ofn.lpstrFile;
  data.nSumber = something; // you probably want to init this also
  dataVec.push_back(data);
}

我不确定szFileNamechar *并只是将其指向ofn.lpstrFile。您可能需要分配内存并将文件名存储在struct中。如果是这种情况,请将szFileName更改为std::string而不是char *

09-26 16:16