用strcmp比较字符串有时候返回不对

zhiyuan3635

比较部分的代码如下：
int strcmp(const char *s1,const char *s2)
{
        //区分大小写
        for(;*s1==*s2;++s1,++s2)
        {
            if(*s1=='\0')
            {
                return(0);
            }
        }
        return((*(unsigned char*)s1<*(unsigned char*)s2)?-1:+1);
}

需要比较的字符串我是从串口中接收的，代码如下：
void getSerialData(UINT8 sdata)
{
    *p = sdata;
    ++p;
}
void USART1_IRQHandler(void)
{
    volatile char res;
    if(USART1->SR&(1<<5))
    {
        res = USART1->DR;
        getSerialData(res);
    }
}

调用的时候就是声明一个字符数组和一个指针，然后把数组的地址赋给指针。
类似如下的代码写在主函数里面：
char str[100], *p;
p=str;
strcmp(str, "abcdefg");
串口精灵里面写到串口接收的字符设定死了为“abcdefg”
但是有一定机率比较的时候是不同的。
不知道为什么，有没有坛友遇到过这种情况，怎么解决的。

jkd405

比较代码是基于'\0'作为字符串结束符，你串口接收的时候显然没有专门处理这块

正点原子

对比之前的两个字符串printf出来，可以辅助分析原因。

zhiyuan3635

正点原子 发表于 2018-4-18 01:26
对比之前的两个字符串printf出来，可以辅助分析原因。

基准字符串我是用#define BASE "abcdefg"，这样定义的。
也有把待对比的字符串printf出来，字面上看没错。
比较的时候我是把结果放在if里面做判断，if(!strcmp(str, BASE ))这样，不知道会不会影响。
而且加大串口发送的速度的时候出现的频率会升高

zhiyuan3635

jkd405 发表于 2018-4-18 10:17
比较代码是基于'\0'作为字符串结束符，你串口接收的时候显然没有专门处理这块

那用strncmp就可以不需要\0了，但是我测试出来的结果是一样的，还是会对比不准确。
如果在串口里加上\0的处理我应该怎么做

jkd405

自己写比较函数，加一个长度参数，这个最简单，也最不容易出问题，因为不知道你具体要干什么不太好给意见

aozima

str都没有赋值。。。
里面是什么东西随缘吧！

zhiyuan3635

aozima 发表于 2018-4-18 10:47
str都没有赋值。。。
里面是什么东西随缘吧！

你是说初始化么

bootblack

以下为个人猜测：
楼主应该没有搞明白字符数组和指向字符串的指针之间的区别。
int strcmp(const char *s1,const char *s2)中++s2是建立在s2是一个字符数组的基础之上的，因为字符数组是存放在ram中的，可以对其中每个字符进行指针操作。
而“abcdefg”是一个指向字符串的指针，这个指针是不能移动的，因为这个字符串是放在flash（或者说是rom）中一个叫做string pool的地方，“abcdefg”是一个编译时期确定的const指针。

林光华ing

一定要连同结束符‘\0’发送或者接受后添加进去。。。否则当然会随机不行。
或者建议使用更安全的strncmp()；带长度的比较。

zhiyuan3635

bootblack 发表于 2018-4-18 11:27
以下为个人猜测：
楼主应该没有搞明白字符数组和指向字符串的指针之间的区别。
int strcmp(const char *s ...

我先前有把这些基准的字符串都放在一个二维数组里面，但是效果也不是很好。
另外，const char *s2 中const修饰的不是指针，是指针指向的数据。如果是要修饰指针，应该写做char * const s2

Acuity

str没赋值，代码有指针的加“空语句”检测啊，养成好习惯！

zhiyuan3635

Acuity 发表于 2018-4-18 21:57
str没赋值，代码有指针的加“空语句”检测啊，养成好习惯！

嗯，为了简明扼要，部分代码没有贴出来。
进行处理的时候有进行空语句判断的。
谢谢

zhiyuan3635

zhiyuan3635 发表于 2018-4-18 23:26
嗯，为了简明扼要，部分代码没有贴出来。
进行处理的时候有进行空语句判断的。
谢谢

经过调试，应该是串口接收结束判断的问题。
处理方式：
开一个定时器做一个超时判断，超过一段时间没有数据接收，判断接收结束，然后进行数据处理。
连续测试1000个相同数据，发送间隔200ms，处理成功率100%。
定时器中断：
//定时器5中断服务程序         
void TIM5_IRQHandler(void)
{
    static uint16 timeOut=0;
        if(TIM5->SR&TIM_IT_Update)//溢出中断
        {
                TIM5->SR &= ~TIM_IT_Update ;//清除中断标志位
        if(fRcvOver==1)
        {
            if(++timeOut>25)
            {//250us时间判断
                timeOut = 0;
                fRcvOver= 0;
            }
        }
        }                                               
}
接收数据段
void getSerialData(char sdata)
{
    *p = sdata;
    ++p;
    fRcvOver = 1;
}

主循环中进行数据处理
if(!fRcvOver)
{
     StrRUN();
}