大家帮忙看下，我这个函数有问题么

pthread

void* Search_Data(void *pData, u16 size,void* searchedDat,u16 len)
{
        if(pData == NULL || size < len || searchedDat == NULL) return NULL;
        size = size - len + 1;
        while(size--)
        {
                switch(len)
                {
                        case 1: if(*(u8*)pData++ == *(u8*)searchedDat)  return  pData;break;
                        case 2: if(*(u16*)pData  == *(u16*)searchedDat)        return  pData;
                        case 4: if(*(u32*)pData  == *(u32*)searchedDat) return  pData;
                                        (u8*)pData++;
                                        break;
                }
        }
        return NULL;               
}

xuande

感觉LZ自学c语言？

只看格式问题，
函数声明，void后面加*，我怎么没见过。。。
case 1内容比较多，应当加大括号，否则执行顺序就会有问题，
case 2没有加break。
最后还缺default

估计编译都通不过。

憨厚诚实大叔

LZ厉害，下面的解释很不错

pthread

xuande 发表于 2016-4-9 11:30
感觉LZ自学c语言？

只看格式问题，

你的C语言该好好学学了哦，老兄。      
1，之所以形参类型为void*型，是为了提高该函数的通用性，你可以传int*，char*进来都可以的。
2，这个可加可不加的啊。
3，我之所以没加break，是让它接着往下执行啊，在4里跳出啊。当然，它会将4的判断再执行一遍，这是个问题。
4，switch语句的default分支可要可不要的啊，当然加上可监控一些异常情况的产生。
5，KEIL编译确实通不过，但在GCC可以通过的。它报的错误是我不能*（u8*）pData++有问题，该成*(（（u8*）pData）++)就只有警告了，如果再加一个确定的指针类型，u8* p = （u8*）pData; *p++就无警告无错误了。

pthread

pthread 发表于 2016-4-9 11:54
你的C语言该好好学学了哦，老兄。      
1，之所以形参类型为void*型，是为了提高该函数的通用性，你可 ...

KEIL编译器根据ANSI标准是不能让我对void*进行自加的，即使强制类型转换了。只能通过间接办法。

xuande

哦