quit相关的bug

实验环境

本实验环境为Windows subsystem Linux 2（based on Ubuntu 18.04）+Windows terminal(preview)，编译器为gcc version 7.5.0，未采用VScode，全程在shell中操作。

编译

执行命令gcc -o test linktable.c menu.c，在对应目录生成可执行文件test，./test运行，我们分别对指令进行测试，如下图所示

你也看见了，quit指令被提示为wrong cmd，这显然是不合理的，下面我们来对代码进行分析。

理清代码逻辑

int main()
{
    InitMenuData(&head); 
   /* cmd line begins */
    while(1)
    {
        printf("Input a cmd number > ");
        scanf("%s", cmd);
        tDataNode *p = FindCmd(head, cmd);
        if( p == NULL)
        {
            printf("This is a wrong cmd!\n ");
            continue;
        }
        printf("%s - %s\n", p->cmd, p->desc); 
        if(p->handler != NULL) 
        { 
            p->handler();
        }
   
    }
}

首先我们看main函数，什么时候会打印“wrong cmd”呢？从上述代码中我们可以看出，当p指针为空时会打印出这句，而p指针具体指向什么位置，是由FindCmd函数来决定的。那么我们来看看FindCmd函数做了什么。

tDataNode* FindCmd(tLinkTable * head, char * cmd)
{
    return  (tDataNode*)SearchLinkTableNode(head,SearchCondition);
}

FindCmd函数的返回值又由SearchLinkTableNode来决定。这里多一句嘴，为啥这里要多此一举用这个函数呢？个人分析主要还是因为上课讲过的所谓的“不要和陌生人说话原则”。我们可以把linktable看成对menu提供的一个模块，在menu中直接对linktable做些操作显然是不怎么符合松散耦合的原则的，所以说，我们还是调用linktable提供的接口更合适。
OK我们继续看，SearchLinkTableNode函数长这样、

tLinkTableNode * SearchLinkTableNode(tLinkTable *pLinkTable, int Conditon(tLinkTableNode * pNode))
{
    if(pLinkTable == NULL || Conditon == NULL)
    {
        return NULL;
    }
    tLinkTableNode * pNode = pLinkTable->pHead;
    while(pNode != pLinkTable->pTail)
    {    
        if(Conditon(pNode) == SUCCESS)
        {
            return pNode;                   
        }
        pNode = pNode->pNext;
    }
    return NULL;
}

这段代码老实讲是有点东西的。里面可以引入很重要的回调函数的概念。你也看见了，这个函数传入的第二个参数其实是一个函数指针，当我们使用这个函数指针所指向的函数来让他帮我们做点事情时(和卧底有些类似)，此时它就被称为回调函数。回调函数不是由该函数的实现方直接调用，而是在特定的事件或条件发生时由另外的一方调用的，用于对该事件或条件进行响应。
其实看到这里，bug就已经能被定位出来了。因为最后打印“wrong cmd”的本质原因就是这个函数返回了null。那啥时候这个函数会返回null呢？四种情况，分别为：

• 链表空，函数指针不空
• 链表不空，函数指针空
• 链表空，函数指针空
• 链表遍历到pLinkTable->pTail时，回调函数仍然不满足条件
  很显然前三种情况我们都能排除，那么我们就来分析分析第四种，让我们看看pLinkTable->pTail到底是啥，来，上菜。

int AddLinkTableNode(tLinkTable *pLinkTable,tLinkTableNode * pNode)
{
    if(pLinkTable == NULL || pNode == NULL)
    {
        return FAILURE;
    }
    pNode->pNext = NULL;
    pthread_mutex_lock(&(pLinkTable->mutex));
    if(pLinkTable->pHead == NULL)
    {
        pLinkTable->pHead = pNode;
    }
    if(pLinkTable->pTail == NULL)
    {
        pLinkTable->pTail = pNode;
    }
    else
    {
        pLinkTable->pTail->pNext = pNode;
        pLinkTable->pTail = pNode;
    }
    pLinkTable->SumOfNode += 1 ;
    pthread_mutex_unlock(&(pLinkTable->mutex));
    return SUCCESS;     
}

到了这里bug已经很明显了，最后一个else语句可以看出， pLinkTable->pTail所指向的是链表的最后一个节点，换句话说就是，函数SearchLinkTableNode当遍历到最后一个节点时，根本就不会进行判断，直接就退出循环了。这样一来，如何修改也是显而易见了。上图

最后我们来测试一波

至此，debug阶段结束。

callback接口的设计方法

这里从两个角度来讲。

什么时候使用回调函数

• A类有多种形态，要在B类中实现回调函数。如假设A类是网络请求开源类ASIHttpRequest，它可能请求成功，也可能请求失败。这个时候，B类就要针对以上两个情况，作不同的处理。
• A类的形态由B类决定时，要在B类中实现回调函数。这个有点我们这个链表小程序的情况有点像，确实，linktable像是柜子，它具体放什么，是由menu里的datanode决定的，也确实是在menu里实现的回调函数

• A类需要向B类传递数据时，可以在B类中实现回调函数（A类一般是数据层比较耗时的操作类）。如举的那个发工资的例子。在实际编程中，这样的机制有个好处就是可以提升用户的操作体验。比如用户从X页面跳转到Y页面，需要向网络请求数据，而且比较耗时，那我们怎么办？有三种方案：第一种就是在X页面展示一个旋转指示器，当收到网络传回的数据时，在展现Y页面。第二种就是使用回调函数。用户从X页面直接跳转到Y页面，Y页面需要到数据让数据层去执行，当收到数据时，再在Y页面展现。第三种就是在Y页面中开启多线程。让一个子线程专门到后台去取数据。综合来说，第二种更加简介易懂，而且代码紧凑。

  回调函数的好处

• 可以让实现方，根据回调方的多种形态进行不同的处理和操作。
• 可以让实现方，根据自己的需要定制回调方的不同形态。
• 可以将耗时的操作隐藏在回调方，不影响实现方其它信息的展示。

• 让代码的逻辑更加集中，更加易读。

工程化编程实战callback接口学习笔记

原文：https://www.cnblogs.com/ustcwz/p/12493756.html