C ++空指针

Question

pthread_mutex_t mutexREAD;
int main(int argc, char *argv[]){
    pthread_t READERthreads;
    pthread_mutex_init(&mutexREAD, NULL);
    string *fname;
    cin>> *fname;
    pthread_create(&READERthreads, NULL, reader_thread, (void*) fname);
}
void *reader_thread(void *param){
    string fname = *(string *) param;
    cout<<"filename is "<< fname<<endl;
    ifstream myfile(fname.c_str());
    return NULL;
 }

上面的代码抛出分段错误。 我可能弄乱了指针，但是我不知道哪里出了问题以及如何解决？

Answer 1

两个问题：崩溃的第一个原因，很可能是崩溃的原因是，您有一个指针fname但实际上它没有指向任何地方。 当取消引用它时，将导致未定义的行为 。

除非您需要将不同的字符串传递给不同的线程，否则不必是指针，只需在调用pthread_create时使用地址运算符& 。

当然，您可以改用std::thread ，并按值传递字符串，而不必担心指针：

...
std::string fname;
std::cin >> fname;

std::thread READERthread{&reader_thread, fname);
READERthread.join();
...

和

void reader_thread(std::string fname)
{
    ...
}

---

第二个问题是您不等待线程完成再退出进程。 那会杀死线程。 您要么需要加入线程，然后等待线程退出。 或者，您可以分离它，而仅退出主线程（而不退出进程），这将使另一个线程在后台运行。

Answer 2

您声明了一个指向string的指针，只需使用一个string并传递其地址即可。

pthread_mutex_t mutexREAD;
int main(int argc, char *argv[]){
    pthread_t READERthreads;
    pthread_mutex_init(&mutexREAD, NULL);
    string fname;
    cin>> fname;
    pthread_create(&READERthreads, NULL, reader_thread, (void*) &fname);
    pthread_join(&READERthreads,NULL);
}
void *reader_thread(void *param){
    string fname = *(string *) param;
    cout<<"filename is "<< fname<<endl;
    ifstream myfile(fname.c_str());
    return NULL;
 }

另一个问题是您不等待线程终止，那么可能在线程有时间使用它之前释放分配给堆栈的字符串...在生成线程中使用pthread_join 。