這是一個過於複雜的pthreads代碼？

這應該是創建兩個線程，讓他們宣佈他們的ID和他們分配給PID。一些基本的錯誤檢查也被執行。這是一個過於複雜的pthreads代碼？

有沒有更簡單的方法來做到這一點而不犧牲錯誤檢查？

#include<stdio.h> 
#include<string.h> 
#include<pthread.h> 
#include<stdlib.h> 
#include<unistd.h> 


pthread_t tid[1]; 

void* doSomething(void *arg) 
{ 
    int i = 0; 
    pthread_t id = pthread_self(); 
    const char* a[2]; 
    a[0]="Client(1)"; 
    a[1]="Server(2)"; 


    while (i<2) 
    { 

     if (pthread_equal(id,tid[i])) 
     printf("\n I'm the %s! My ID is: %ld. Our PID is= %d\n",a[i], (long int)&(tid[i]) , getpid()); 
     i++; 


    } 
    pthread_exit(0); 
} 

int main(void) 
{ 
    int i = 0; 
    int error; 

    while(i < 2) 
    { 
     error = pthread_create(&(tid[i]), NULL, &doSomething, NULL); 
     if (error != 0){ 
      printf("\n Error creating thread %d:[%s]",i+1, strerror(error)); 
      } 
    else{ 
     if(i==0){ 
      printf("\n Principal thread: Client thread (%i) created! Thread ID: %ld \n", i+1, (long int)&(tid[0])); 
      } 
     if(i==1){ 
      printf("\n Principal thread: Server thread (%i) created! Thread ID: %ldn", i+1, (long int)&(tid[1]));  
      } 

      i++; 
     } 
    } 
    if (pthread_join((tid[0]), NULL) == 0){ 
    printf ("\n Client has closed \n"); 
    } else {  
    printf ("\n Client closed with an error \n"); 
    } 
    if (pthread_join((tid[1]), NULL) == 0){ 
    printf ("\n Server has closed \n"); 
    }else{ 
    printf ("\nClient closed with an error \n"); 
    } 

    return 0; 
}

來源

2015-11-27 Vernon Albayeros

不太複雜;只是調用未定義的行爲。因爲你已經定義了'pthread_t tid [1]'，所以你有'if（pthread_equal（id，tid [i]））''它將訪問'tid [0]'（OK）和'tid [1]'（不是OK） ;'所以你正在訪問數組越界;誰知道發生了什麼。你在main（）中的代碼應該使用'for'循環而不是'while'循環，所有的備件都分散在代碼中。它不應該訪問'tid'數組越界。 –

對於codereview，請在[codereview.se]上詢問，而不是在這裏。 – fuz

我投票結束這個問題作爲題外話，因爲它是關於代碼審查屬於[codereview.se]。 – fuz

除了在評論中提到的未定義行爲，你也可以重寫代碼不使用while循環，簡化線程功能，如：

#include<pthread.h> 
#include<stdlib.h> 
#include<unistd.h> 

pthread_t tid[2]; 

void* doSomething(void *arg) 
{ 
    char *str = arg; 
    printf("\n I'm the %s! My ID is: %ld. Our PID is= %d\n", str, (long) pthread_self() , getpid()); 
    pthread_exit(0); 
} 

int main(void) 
{ 
    int i = 0; 
    int error; 
    const char *a[2] = {"Client (1)", "Client (2)" }; 

    for(i=0; i<2; i++) 
    { 
     error = pthread_create(&(tid[i]), NULL, &doSomething, (void*)a[i]); 
     if (error != 0) 
      printf("\n Error creating thread %d:[%s]",i+1, strerror(error)); 
     else 
      printf("\n Principal thread: Client thread (%i) created! Thread ID: %ld \n", i+1, (long int)&(tid[i])); 
    } 

    if (pthread_join((tid[0]), NULL) == 0){ 
    printf ("\n Client has closed \n"); 
    } else { 
    printf ("\n Client closed with an error \n"); 
    } 
    if (pthread_join((tid[1]), NULL) == 0){ 
    printf ("\n Server has closed \n"); 
    }else{ 
    printf ("\nClient closed with an error \n"); 
    } 

    return 0; 
}

你可以也跳過pthread_join()上的錯誤檢查。無論如何，如果它失敗了，你就無能爲力了。

另外，註釋鑄件pthread_t到long不保證能正常工作。沒有標準格式說明符可以輕鬆打印。如果您真的關心它，請將其轉換爲unsinged char*並打印字節。

來源

2015-11-27 10:28:53

這是一個過於複雜的pthreads代碼？

回答

相關問題