从一个BUFFER里取特定特定行的代码，请帮忙DEBUG！急！

wangxiaohu

从大小为size的data里面取出第linenum行的字符串，然后放到target里面去。
行数少的data还行，行数一多就乱七八糟了，能不能帮忙Debug一下？或者帮忙写个有同样功能的CODE出来也行。

1. int gMotionDefGetLine(byte* data, size_t size, int linenum, byte* target){
   
2.         size_t pos, buf_pos = 0;
   
3.         int lineCount = 0;
   
4. 
   
5.         byte buf[MAX_LINE_SIZE];
   
6.         memset(buf, 0, MAX_LINE_SIZE);
   
7.        
   
8.         for (pos = 0; pos < size; pos++){
   
9.                 if (data[pos] == '\n'){
   
10.                         if (lineCount == linenum){
    
11.                                 memcpy(target, buf, buf_pos);
    
12.                                 target[buf_pos] = 0;
    
13.                                 return buf_pos;
    
14.                         }
    
15.                         lineCount++;
    
16.                         pos++;
    
17.                         buf_pos = 0;
    
18.                         memset(buf, 0, MAX_LINE_SIZE);
    
19.                 }
    
20.                 buf[buf_pos] = data[pos];
    
21.                 buf_pos++;
    
22. 
    
23.                 if (buf_pos > MAX_LINE_SIZE)
    
24.                         return -1;
    
25.         }
    
26.         return -1;
    
27. }

复制代码

下面是让这段代码出错的data：

1. 
   
2. 4
   
3. 3
   
4. 8
   
5. 6
   
6. -0.999984 0.000000 25.000060 -10.000013 0.000000 30.000153 0.000000 0.000000 25.000118
   
7. -25.000060 0.000000 110.000312 -80.000216 0.000000 145.000442 -25.000060 0.000000 110.000312 -80.000216 0.000000 145.000442
   
8. -25.000060 0.000000 110.000312 -80.000216 0.000000 145.000442 -25.000060 0.000000 110.000312 -80.000216 0.000000 145.000442
   
9. 25.000060 0.000000 60.000191 -105.000334 0.999984 147.000468 25.000060 0.000000 60.000191 -105.000334 0.999984 147.000468
   
10. 25.000060 0.000000 60.000191 -105.000334 0.999984 147.000468 25.000060 0.000000 60.000191 -105.000334 0.999984 147.000468
    
11. 0.000000 0.000000 9.999955 -90.000115 0.000000 135.000201 -0.599945 0.000000 10.600072 -90.000344 0.000000 135.000716
    
12. 0.000000 0.000000 9.999955 -90.000115 0.000000 135.000201 -0.599945 0.000000 10.600072 -90.000344 0.000000 135.000716
    
13. 0.000000 0.000000 9.999955 -90.000115 0.000000 135.000201 -0.599945 0.000000 10.600072 -90.000344 0.000000 135.000716
    

复制代码

谢谢了！

wangxiaohu

因为是给嵌入系统用的，不能用<string.h>，只能用<stdlib.h>和<stdio.h>

kj501

能不能帮忙Debug一下？或者帮忙写个有同样功能的CODE出来也行。

如果是想让我们给你写代码或者做测试，出个价吧。

doubleelec

这几行有点问题吧(当遇到'\n'并且lineCount != linenum时)
lineCount++;
pos++;
buf_pos = 0;
memset(buf, 0, MAX_LINE_SIZE);
这时后面的内容不需要做了,应该continue,于是pos++会自动执行
不需要自己再写一个pos++,所以我认为应该改成
lineCount++;
buf_pos = 0;
memset(buf, 0, MAX_LINE_SIZE);
continue;

另外,不管这一行要不要,你都做了buf[buf_pos] = data[pos];
这也太浪费资源了吧,只需要把位置记下来,到了需要的那一行
来个memcpy就行了

doubleelec

呵呵, kj501版主言之有理

stoner==swkcl

人家贴出来了一段代码，自己写个驱动程序，不是也可以锻炼自己的调试技术？(比要钱，要值吧，我知道版主说的是笑话，呵呵。)

应该是有缓冲区溢出的问题。你用的那个常量MAX_LINE_SIZE是多少呢？是一个定值吧。很危险喔。代码的结构也还清晰，可以从下面的测试数据中，可以看出后面的行很长唉。

而且，代码里并没有输入数据行的长度。

wangxiaohu

stoner==swkcl猜对了，就是缓冲区问题。这段代码没有问题，而是我另外的地方定义MAX_LINE_SIZE重复了。。。。