老话题了，指针作为参数被sizeof

大家都已经知道当数组作为函数参数传递的时候会退化为指针。

但是在使用的时候还是经常会犯错，在函数

int get_pexEeprom(pPexIns_t *pPexIns, char *pexEeprom, int bufLen)

{

        ...

        int len = sizeof(pexEeprom); //这里又犯错了

}

 

一般写代码的时候是不会出现在这种低级错误的，但是一般我们会拷贝一些代码。比如把某些代码行组织成一个函数，这些代码行单独跑的时候不会出问题，但是当作为函数跑的时候，问题来了，往往拷贝代码只注重参数传递，而不关心里面的内容。

 

比如

int main(..)

{

      char string[64];

      ....

      readPex(pPexIns, string, sizeof(string));

}

 

然后把里面东西拿出来作为一个函数

int get_pexEeprom(pPexIns_t *pPexIns, char *pexEeprom, int bufLen)

{

        ...

        //readPex(pPexIns, string, sizeof(string));

        readPex(pPexIns, pexEeprom, sizeof(pexEeprom)); //正是这里出了问题，应该用bufLen

}

 

当数组不是函数函数参数，也就是说你得到该数组（比如pexEeprom）的方式不是通过函数参数传递进来，那么sizeof的得到的是数组的大小，比如现在main 函数中的readPex(pPexIns, string, sizeof(string))，相当于readPex(pPexIns, string, 64);

 

但是一旦进入函数内部，数组已经变为了一个指针。其实是在参数传递的时候进行了浅拷贝，编译器会声明一个指针指向该数组，在函数内部所有的操作都是对该临时指针的操作。

 

 

 

 

 

 

文章写得比较乱，虽然大部分人都已经知道了这个知识点，但是还是想提醒大家，也是提醒我自己，在拷贝代码的时候要小心（谁敢说自己从来不拷贝代码，即使是自己写得，有时候也要组织一下嚒）。

另外，对于函数的功能，我们的工程中写得很差，主要是函数功能太过复杂，一眼看过去，感觉这个函数要完成所有的功能，而没有把功能细分，每个函数去实现一个小功能。函数写得太大，当需要做修改的时候太麻烦。另外代码耦合太高，稍微修改有点就有种牵一发而动全身的感觉，主要是几乎所有的函数都有一个相同的参数，一个大的结构体指针，当你需要打乱代码结构的时候，需要注意的东西太多，比如这个pPexIns_t *指针，当你需要一个修改其中的成员时，还要先保存它，用完了还要恢复，太麻烦，容易出错。不过目前还没找到好的解决办法，有时间一定要好好整理整理代码。