Question

我正在尝试为C中的学校分配创建一个火车站模拟。这是一个理解线程编程的练习。这是我目前的代码：

#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
#include <string.h>
#include <readline/readline.h>

void *train_function(void *args) {
    printf("Train created!\n");
    pthread_exit(0);
}

int main() {
    FILE *ptr_file;
    char buff[10];
    int ch;
    int train_count = 0;

    ptr_file = fopen("./trains.txt", "r");

    if (!ptr_file)
        return 0;

    /* Count number of trains */
    while(!feof(ptr_file))
    {
        ch = fgetc(ptr_file);
        if(ch == '\n')
        {
            train_count++;
        }
    }
    pthread_t trains[train_count];

    /* Create train for each line of file */
    int train_index = 0;
    while (fgets(buff,10, ptr_file)!=NULL) {
        pthread_create(&trains[train_index], NULL, train_function, NULL);
        train_index++;
    }
    fclose(ptr_file);

    /* Wait for all trains to leave the station */
    for (int x = 0; x < train_count; x++) {
        pthread_join(trains[x], NULL);
    }
    exit(0);
}

代码从文件train.txt中读取有关列车的信息，并为文件的每一行（每列火车）创建一个新线程。

e:10,6
W:5,7
E:3,10

我希望输出为＃34; Train Created！＆＃34;三次。编译代码会产生分段错误。我错过了什么？

Answer 1

 while (fgets(buff,10, ptr_file)!=NULL) {
        pthread_create(&trains[train_index], NULL, train_function, NULL);
        train_index++;
    }

在这里，您正在读取文件但文件指针已经在文件的末尾，因为您在此while循环之前已经读取了整个文件。因此，您根本不创建任何线程，但稍后尝试使用不存在的线程加入（pthread_join调用）。这是未定义的行为。您只需使用train_count并在for循环中创建线程：

 for (int x = 0; x < train_count; x++) {
    pthread_create(&trains[x], NULL, train_function, NULL);
 }

另外，请注意文件读取部分存在缺陷：

while(!feof(ptr_file)) {...}

参见Why is “while ( !feof (file) )” always wrong? 修理它。

Answer 2

代码有几个小问题。

以下代码消除了已发布代码中未使用的语句。

不要使用feof（）作为循环控件。由于多种原因，它作为循环控制失败。

必要的错误检查会添加到代码中。

请注意向stderr正确输出正确的错误消息通过使用perror（）函数

注意通过使用exit（）函数退出正确的错误

注意最后通过使用return（）函数

正确退出main

以下代码经过测试且运行正常。

#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
#include <string.h>
//#include <readline/readline.h>

void *train_function(void *args __attribute__((unused)) )
{
    printf("Train created!\n");
    pthread_exit(0);
}

int main()
{
    FILE *ptr_file;
    int ch;
    int train_count = 0;

    ptr_file = fopen("trains.txt", "r");

    if (!ptr_file)
    {
        perror( "fopen for trains.txt failed" );
        exit( EXIT_FAILURE);
    }

    // implied else, fopen successful

    /* Count number of trains */
    while( (ch = fgetc(ptr_file) ) != EOF)
    {
        if(ch == '\n')
        {
            train_count++;
        }
    }
    pthread_t trains[train_count];

    /* Create train for each line of file */
    int train_index = 0;
    for ( train_index = 0; train_index < train_count; train_index++ )
    {
        if( pthread_create(&trains[train_index], NULL, train_function, NULL) )
        { // then pthread_create failed
            perror( "pthread_create failed" );
            exit( EXIT_FAILURE ); // this will cause all threads to also exit
        }

        // implied else, pthread_create successful
    }

    fclose(ptr_file);

    /* Wait for all trains to leave the station */
    for (int x = 0; x < train_count; x++)
    {
        pthread_join(trains[x], NULL);
    }
    return(0);
} // end function: main

pthread_create启动例程没有执行

2 个答案: