Mutex-使用线程计算文件中char的出现

Question

嗨，我正在编写一个代码，用于计算使用线程在一个或多个文件中某个字母的出现次数。 我必须为每个文件使用一个线程，然后使用互斥量修改全局总数。

这是我的代码：

typedef struct _CharFile{

    char c;
    char *fileName;

} CharFile;

pthread_mutex_t count = PTHREAD_MUTEX_INITIALIZER;
int sum = 0;

void *CountFile(void *threadarg);

int main(int argc, const char * argv[]) {

    pthread_t threads[argc-2];
    int chck, t;
    CharFile cf;

    if (argc <= 2){

        perror("Wrong inputs: need to select a letter and one or more files\n");
        exit(EXIT_FAILURE);

    } else if (argc > 51) {

        perror("Too many files\n");
        exit(EXIT_FAILURE);

    }

    for ( t=0 ; t<argc-2 ; t++ ){

        cf.c = argv[1][0];
        cf.fileName = (char *)argv[t + 2];

        chck = pthread_create(&threads[t], NULL, CountFile, (void *) &cf);
        if (chck){
            printf("ERROR; return code from pthread_create() is %d\n", chck);
            exit(EXIT_FAILURE);
        }

    }

    for (int i = 0; i < 2; i++)
        pthread_join(threads[i], NULL);

    printf("%lld occurrences of the letter %c in %lld threads\n", (long long)sum, argv[1][0], (long long)argc-2);

    return 0;
}

void *CountFile(void *threadarg){

    FILE *in;
    CharFile *cf;
    char c;
    int counter = 0;

    cf = (CharFile *) threadarg;
    in = fopen(cf->fileName, "r");

    if (in == NULL){

        perror("Error opening the file!\n");
        pthread_exit(NULL);

    }

    while (fscanf(in, "%c", &c) != EOF){

        if(c == cf->c){

            counter ++;

        }

    }

    fclose(in);

    pthread_mutex_lock(&count);
    sum += counter;
    printf("%d sum value, %d counter value \n", sum, counter);
    pthread_mutex_unlock(&count);

    pthread_exit(NULL);
}

我不知道哪里出了问题，但是使用一些带有字母束的文件，结果却很奇怪。

运行此行./a.out f file1.in lorem.txt notes.txt两次，我得到以下结果：

4 sum value, 4 counter value 
9 sum value, 5 counter value 
13 sum value, 4 counter value 
13 occurrences of the letter f in 3 threads

4 sum value, 4 counter value 
8 sum value, 4 counter value 
12 sum value, 4 counter value 
12 occurrences of the letter f in 3 threads

如果我只用一个文件运行一个简单的命令行，则计数是正确的，即./a.out f file1.in

5 sum value, 5 counter value 
5 occurrences of the letter f in 1 threads

谢谢您的帮助。

Answer 1

我认为您的问题可能在这里：

for ( t=0 ; t<argc-2 ; t++ ){

    cf.c = argv[1][0];
    cf.fileName = (char *)argv[t + 2];

    chck = pthread_create(&threads[t], NULL, CountFile, (void *) &cf);
    if (chck){
        printf("ERROR; return code from pthread_create() is %d\n", chck);
        exit(EXIT_FAILURE);
    }

}

您将所有线程的指针传递给同一变量（即cf ）。 将指针传递到第一个线程之后，您可以更改cf的值，并将指针传递给第二个线程，依此类推。

因此，可能会发生这样的情况，即在“正好被盯着看”的线程读取其值之前，主循环会更改cf ，即该线程可能打开了另一个未预期的文件。

通过将cf更改为数组（即每个线程一个元素），确保每个线程获得自己的cf