[apue] 书中关于打印服务代码的一个错误

在看 apue 第 21 章 与网络打印机通信一章时,发现一段关于链表操作的代码有问题,现在摘出来让大家 review 一下。先上代码:

printd.c

 

这是打印服务的源代码,在打印时,用户通过  print 命令提交待打印的文件,print 命令通过 tcp 与 printd 服务通讯,

将文件及打印相关的参数传递给后者;对于每个客户,printd 服务会创建一个 worker 结构节点,

放在一个由 workers 变量指定了头的双向链表中。所以这段代码本质上就是简单的双向链接操作:

复制代码
 1 void add_worker (pthread_t tid, int sockfd)
 2 {
 3     struct worker_thread *wtp; 
 4     if ((wtp = malloc (sizeof (struct worker_thread))) == NULL) { 
 5         log_ret ("add_worker: can't malloc"); 
 6         pthread_exit ((void *)1); 
 7     }
 8 
 9     wtp->tid = tid; 
10     wtp->sockfd = sockfd; 
11 
12     log_msg ("prepare to add worker"); 
13     pthread_mutex_lock (&workerlock); 
14 
15     wtp->prev = NULL; 
16     wtp->next = workers; 
17     if (workers == NULL)
18         workers = wtp; 
19     else 
20         workers->prev = wtp; 
21 
22     pthread_mutex_unlock (&workerlock); 
23 }
复制代码

 

重点就是 15-20 这 6 行啦(原文p633,代码499-504行),当第一次加入节点时, workers 为 NULL,所以走第一个条件分支,这没有问题;

但是再加入节点时, workers 不为 NULL,此时走 else 分支,将当前头的上一个节点设置为待插入的新节点  wtp,

到现在还好,可是等等,怎么就没下文了?!这个节点还没完全加入链表呢……

正确的做法应该是在结尾前再加一句:

    else 
    {
        workers->prev = wtp; 
        workers = wtp; 
    }

 

这样才能算完嘛。道理就不多说了,不信自己画个链表看看。下面给出优化后的完整代码:

复制代码
 1 void add_worker (pthread_t tid, int sockfd)
 2 {
 3     struct worker_thread *wtp; 
 4     if ((wtp = malloc (sizeof (struct worker_thread))) == NULL) { 
 5         log_ret ("add_worker: can't malloc"); 
 6         pthread_exit ((void *)1); 
 7     }
 8 
 9     wtp->tid = tid; 
10     wtp->sockfd = sockfd; 
11     pthread_mutex_lock (&workerlock); 
12 
13     wtp->prev = NULL; 
14     wtp->next = workers; 
15     if (workers != NULL)
16         workers->prev = wtp; 
17 
18     workers = wtp; 
19 
20     pthread_mutex_unlock (&workerlock); 
21 }
复制代码

 

好吧,我承认作为经典著作也会有这种低级错误。

今天的吹毛求疵就到这里,作为一个有职业素养的程序员,不在鸡蛋里挑出骨头来不罢休,嘿嘿……

 

posted @   goodcitizen  阅读(410)  评论(0编辑  收藏  举报
编辑推荐:
· 一个奇形怪状的面试题:Bean中的CHM要不要加volatile?
· [.NET]调用本地 Deepseek 模型
· 一个费力不讨好的项目,让我损失了近一半的绩效!
· .NET Core 托管堆内存泄露/CPU异常的常见思路
· PostgreSQL 和 SQL Server 在统计信息维护中的关键差异
阅读排行:
· DeepSeek “源神”启动!「GitHub 热点速览」
· 我与微信审核的“相爱相杀”看个人小程序副业
· 微软正式发布.NET 10 Preview 1:开启下一代开发框架新篇章
· 如何使用 Uni-app 实现视频聊天(源码,支持安卓、iOS)
· C# 集成 DeepSeek 模型实现 AI 私有化(本地部署与 API 调用教程)
点击右上角即可分享
微信分享提示