1. u8 j=0;
   
2. u8 sun2=0; // 数据包数
   
3. u8 buf2[CAN_BUF_LENGTH]={0}; //记录can2接收到的原始数据
   
4. u8 temp2[CAN_BUF_LENGTH]={0}; //记录can2重组后的有效数据
   
5. u8 k=0;
   
6. int count2=0; //有效数据个数
   
7. void CAN2_RX0_IRQHandler(void)
   
8. {
   
9. CanRxMsg RxMessage;
   
10. int i=0;
    
11. 
    
12. 
    
13. CAN_ClearITPendingBit(CAN2,CAN_IT_FMP0);
    
14. CAN_Receive(CAN2, 0, &RxMessage);
    
15. 
    
16. if(RxMessage.Data[1]==0x01)
    
17. {
    
18. sun2= RxMessage.Data[2] ;        
    
19. }
    
20. 
    
21. 
    
22. //接收can的原始数据
    
23. for(i=0;i<8;i++)
    
24. {
    
25. buf2[j] =RxMessage.Data[i];        
    
26. //        printf("rxbuf2[%d]:%0x\r\n",j,        buf2[j]);        
    
27. j++;        
    
28. }
    
29. 
    
30. //重新组包数据        
    
31. if(j>7)
    
32. {
    
33. memTomem(temp2+k,buf2+j,8);
    
34. 
    
35. for(i=0;temp2[k]!=0&&i<8;i++)
    
36. {        
    
37. printf("rxbuf3[%d]:%0x\r\n",k,        temp2[k]);     //开发板断电在上电，这里每次第一次打印temp2时都是空，就是什么都不输出，是什么原因呢？ </font>
    
38. k++;
    
39. count2++;
    
40. }        
    
41. 
    
42. }
    
43. 
    
44. if(sun2==0)//是否接收完当前报文帧数
    
45. {        
    
46. 
    
47. j=0;//将数组下标还原为0
    
48. k=0;        
    
49. count2=0;        
    
50. }
    
51. sun2--;        
    
52. 
    
53. }
    

复制代码

问题在37行，请大佬们帮忙看下，是不是我哪里写的不对？

memTomem(temp2+k,buf2+j,8);
这里写的是不是有问题？总是把buf2中最后的地址copy到temp2.
从代码上看比较乱，如果只是简单的数据入队，没必要这么麻烦，可以参考ring.c这种循环队列。
也可以直接用memcpy（temp2+k,RxMessage.Data,8）;没必要在中间再倒一次手吧。

安 发表于 2020-12-26 14:10
memTomem(temp2+k,buf2+j,8);
这里写的是不是有问题？总是把buf2中最后的地址copy到temp2.
从代码上看比较 ...

大佬，这里是做了一个把接收到的原始数据重新做了组包，因为要筛选出里面需要的数据，

楼主，其他的我只是建议，终端中尽量少做这些处理，交由中断外处理。
memTomem(temp2+k,buf2+j,8);检查这个函数，j时增加后的数。