apue 线程的代码是不是有一点点瑕疵啊？

xiantaozhaowei

1. #Include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <sys/types.h>
   
4. #include <pthread.h>
   
5. 
   
6. struct foo {
   
7.   int a, b, c, d;
   
8. };
   
9. 
   
10. void
    
11. printfoo(const char *s, const struct foo *fp)
    
12. {
    
13.   printf(s);
    
14.   printf("  structure at 0x%x\n", (unsigned)fp);
    
15.   printf("  foo.a = %d\n", fp->a);
    
16.   printf("  foo.b = %d\n", fp->b);
    
17.   printf("  foo.c = %d\n", fp->c);
    
18.   printf("  foo.d = %d\n", fp->d);
    
19. }
    
20. 
    
21. void *
    
22. thr_fn1(void *arg)
    
23. {
    
24.   struct foo foo = {1, 2, 3, 4};
    
25. 
    
26.   printfoo("thread 1:\n", &foo);
    
27.   pthread_exit((void *)&foo);
    
28. }
    
29. 
    
30. void *
    
31. thr_fn2(void *arg)
    
32. {
    
33.   printf("thread 2: ID is %d\n", pthread_self());
    
34.   pthread_exit((void *)0);
    
35. }
    
36. 
    
37. int
    
38. main(void)
    
39. {
    
40.   int err;
    
41.   pthread_t tid1, tid2;
    
42.   struct foo*fp;
    
43. 
    
44.   err = pthread_create(&tid1, NULL, thr_fn1, NULL);
    
45.   if (err != 0)
    
46.     printf("can't create thread 1: %s\n", strerror(err));
    
47.   err = pthread_join(tid1, (void *)&fp);
    
48.   /*
    
49.   err = pthread_join(tid1, (void **)&fp);
    
50.   和原型不一样啊
    
51.   int pthread_join(pthread_t th, void **thread_return);
    
52.   */
    
53.   if (err != 0)
    
54.     printf("can't join with thread 1: %s\n", strerror(err));
    
55.   sleep(1);
    
56.   printf("parent starting second thread\n");
    
57.   err = pthread_create(&tid2, NULL, thr_fn2, NULL);
    
58.   if (err != 0)
    
59.     printf("can't create thread 2: %s\n", strerror(err));
    
60.   sleep(1);
    
61.   printfoo("parent:\n", fp);
    
62.   exit(0);
    
63. }
    

复制代码

SYNOPSIS
       #include <pthread.h>

       int pthread_join(pthread_t th, void **thread_return);

OwnWaterloo

apue 也就这水平？


1. printf(s);

这代码一看就惊悚， 好歹也：
printf("%s", s); 好吗？
puts(s); 好吗？

当然， 如果整个代码都是受自己控制的， 这也算不上错误。
但作为经典书籍， 这样的例子难辞其咎……

2. printf("  structure at 0x%x\n", (unsigned)fp);
printf("  structure at 0x%p\n", fp);

3. pthread_exit((void *)&foo);

3.1 强转是不必要的
3.2 pthread_exit 不需要显示调用， return &foo。
3.3 这是局部变量啊……  后面的代码最多只能用其地址， 居然敢用其值……

4. err = pthread_join(tid1, (void *)&fp);
也就是楼主的问题。


int err;
struct foo*fp;
{
    void* ret;
    err = pthread_join(tid1, &ret);
    fp = ret;
}


5. exit(0)

不多写点代码不舒服斯基……

OwnWaterloo

回复 2# OwnWaterloo

2. printf("  structure at 0x%x\n", (unsigned)fp);
printf("  structure at 0x%p\n", fp);
改正为： printf("  structure at 0x%p\n", (void*)fp);

xiantaozhaowei

3.  pthread_exit  5. exit(0)  他是显似的教别人怎么搞。。

雨过白鹭洲

回复 2# OwnWaterloo


    3. pthread_exit((void *)&foo);

3.1 强转是不必要的


这个在C++可以不转，但C语言是要求转的吧。。

OwnWaterloo

回复 4# xiantaozhaowei

exit和pthread_exit 只有在非toplevel里打算结束才需要显示调用。
toplevel里直接return 即可， 效率可能稍欠。

这书没细看， 不知道有没有提这点。

如果没有， 那这么多代码里面不必要的exit调用， 它又难辞其咎了……

OwnWaterloo

回复 5# 雨过白鹭洲

T* -> void* 两种语言中都是隐式的。
void* -> T* 在C里面依然是隐式的， 在C++里面才需要static_cast


int err;
struct foo*fp;
{
    void* ret;
    err = pthread_join(tid1, &ret);
    fp = ret;
}


C++的话：
int err;
foo*fp;
{
    void* ret;
    err = pthread_join(tid1, &ret);
    fp = static_cast<foo*>(ret);
}


兼容的写法：
int err;
struct foo*fp;
{
    void* ret;
    err = pthread_join(tid1, &ret);
    fp = (struct foo*)ret;
}

MaxWu

那一行就是printf(s) ... 汗.. LZ还是仔细读一下诶. {:2_168:}

1. printf(s);

这代码一看就惊悚， 好歹也：
printf("%s", s); 好吗？
puts(s); 好吗

davelv

其实这是一个错误的示范  --!

OwnWaterloo

回复 9# davelv

这……  这样啊……