0

我正在尝试通过串行通信与 USB 加密狗进行通信。我的通信正常,但是我无法让设备正确解析通信。我的设备读取消息并将其与硬编码的 c 字符串进行比较。它解析并识别出它是正确的字符串,但是当我尝试解析 : 字符之后的值时,它返回 0x00000000 我不知道为什么。我尝试使用 char cast 并使用 atoi,我尝试使用简单的 ascii 翻译,甚至进行按位加法运算,如下所示:将 vector<uint8_t> 的子集转换为 int

例如:
我发送“心率:55”它解析并识别出“心率:”但是当我告诉它去寻找 55 并将它带回来做一些事情时,它给了我一个 0x00000000 这
是一个片段:

const uint8_t hrmSet[] = "Heart Rate:";

/** Find the : character in the string and break it apart to find if it matches, 
and determine the value of the value of the desired heart rate. **/
int parse(uint8_t *input, uint8_t size)
{
    for (uint8_t i = 0; i < size; i++)
    {
        if (input[i] == ':')
        {
            if (compare_string(input, hrmSet, i) == 0)
            {
                int val = 0;
                for (int j = i+1; j < size; j++)
                {
                    if (!isdigit(input[j]))
                    {
                        for (int k = i; k < j; k++)
                        {
                            val <<= 8;
                            val |= input[k];
                        }
                    }   
                }
                return val;
            }
            return -1;
        }
    }
    return -1;
}

比较字符串函数

/** Compare the input with the const values byte by byte to determine if they are equal.**/
int compare_string(uint8_t *first, const uint8_t *second, int total)
{
    for (int i = 0; i < total; i++)
    {
        if (*first != *second)
        {
            break;
        }

        if (*first == '\0' || *second == '\0')
        {
            break;
        }

        first++;
        second++;
    }

    if (*first == ':' && *second == ':')
    {
        return 0;
    }
    else
    {
       return -1;
    }
}
4

2 回答 2

1

这里的问题是您正在使用嵌套循环来执行应该使用顺序循环完成的任务。

例如,i循环搜索冒号,然后循环 incompare_string再次搜索冒号。您可以先运行i循环,然后compare_stringi循环完成后调用。但更好的设计是compare_string在比较时搜索冒号,然后返回冒号后面字符的索引(如果没有找到冒号,则返回-1)。

jk嵌套循环也是如此。j循环正在搜索数字的结尾。循环只在循环完成k后运行一次j,因此k循环应该在j循环之后,而不是嵌套。但更好的设计是在搜索数字末尾时转换数字的单个循环。

下面的代码演示了一种使用我所描述的技术的可能实现。

const uint8_t hrmSet[] = "Heart Rate:";

int compare_string( uint8_t *input, const uint8_t *expected, int size )
{
    for ( int i = 0; i < size; i++ )
    {
        if ( *input != *expected || *expected == '\0' )
            return( -1 );

        if ( *input == ':' && *expected == ':' )
            return( i + 1 );

        input++;
        expected++;
    }

    return( -1 );
}

int parse( uint8_t *input, uint8_t size )
{
    int i, val;

    if ( (i = compare_string( input, hrmSet, size )) < 0 )
        return( -1 );

    val = 0;
    for ( ; i < size && isdigit( input[i] ); i++ )
        val = val * 10 + input[i] - '0';

    return( val );
}

int main( void )
{
    uint8_t input[] = "Heart Rate:75";
    int rate = parse( input, sizeof(input) - 1 );
    printf( "%d\n", rate );
}
于 2014-10-22T23:58:18.567 回答
1
int val = 0;
for(int j = i+1; j < size; j++){
    if(isdigit(input[j] )){
        val = val * 10 + input[j]-'0';// val = (val << 8) | input[j];
    }   
}
return val;
于 2014-10-22T23:45:48.770 回答