20130712--细节问题

代码效率问题！

实现一个功能，插入一个数组成员。

我写的：

int gw_cfm_md_set(const char *name, int level)
{
    int index = -1;
    int i = 0;
                                           
    for(i = 0; i < GW_CFM_MD_INSTANCES_NUM; ++i)
    {
        /* Check old */
        if(GW_CFM_MD_ACTIVE == gw_cfm_md[i].active)
        {
            if( 0 == strcmp(gw_cfm_md[i].md_name, name) )
            {
                index = i;
                break
            }
        }
        /* Set new */
        else
        {         
            gw_cfm_md[i].active = GW_CFM_MD_ACTIVE;
            if(name)
            {
                memcpy(gw_cfm_md[i].md_name, name, GW_CFM_MD_NAME_LEN);
            }
            gw_cfm_md[i].level = level;
            index = i;
            break;
        }
    }
    return index;
}

王荻写的：

int gw_cfm_md_set(const char *name, int level)
{
    int index = -1;
    int i = 0;
    /* Check old */
    for(i = 0; i < GW_CFM_MD_INSTANCES_NUM; ++i)
    {
        if(!strncmp(name, gw_cfm_md[i].md_name, GW_CFM_MD_NAME_LEN))
        {
            return i;
        }
    }
    /* Set new */
    for(i = 0; i < GW_CFM_MD_INSTANCES_NUM; ++i)
    {
        if(GW_CFM_MD_NON_ACTIVE == gw_cfm_md[i].active)
        {
            index = i;
            gw_cfm_md[i].active = GW_CFM_MD_ACTIVE;
            memcpy(gw_cfm_md[i].md_name, name, GW_CFM_MD_NAME_LEN);          
            gw_cfm_md[i].level = level;
            break;
        }
    }
    return index;
}

   我只用了一个for，效率高吧。

   等到用的时候就错了！如果数组下标用了0、1、2、3、4，删除了3之后，我想查找4。用我的代码，将会发现数组3没用，就会添加在数组下标3里，造成代码错误。

   所以，代码写完后，把自己当做使用人员，从各个角度考虑一下代码的可靠性，这样能省去很多的修改BUG时间，这才是真正的节约时间。