Question

我试图实现一个函数来为可变长度的内存区域计算8位Fletcher的校验和，我的想法是我可以传递一个2字节的短字节或2kb的数组并使用相同的功能。我今天只是在研究它，所以我绝对不是校验和算法或指针算法的专家，而且我的代码几乎肯定充满了错误。

基本上我的策略是传递一个指向组中第一个字节地址的指针，以及要扫描的内存区域的大小，只要大小总是sizeof（传递对象），它会有效。我可以在函数中计算sizeof以避免潜在的错误，但我认为我必须将接受的类型限制为特定类型。

根据我的理解，*（ptr + i），应该返回位于ptr地址后面的第i个字节中的值。我之前没有做过这样的事情，所以我可能误解了我的内容。

uint8_t fletcher_8(void *data, uint size){
        data = (uint8_t *)data; // Recast pointer as uint8_t*
        uint8_t sum1 = 0;
        uint8_t sum2 = 0; // Initialise variables for algorithm
        for (int i =0; i < size; i++){
            sum1 += *(data + i);     // get the value of the ith byte after the data pointer's address
            sum2 += sum1;
        }
        sum1 %= 16; // modulo the first sum
        sum1 << 4;  // shift lower four bits to the upper four bits
        sum2 %= 16; // modulo the second sum
        return sum1 + sum2; // add both sums (highest four bits are sum1, lower four bits are sum2
    }

如果我已经完全疯狂了，并且有更简单的方法来实现我想要做的事情，我很乐意听到它！

编辑：

我是专门询问C ++实现，但上面的代码也可能是psudocode。我最想知道我的方法是否正确。

Answer 1

你的第一个版本比第二个版本好。

第二版中的一些错误：

data不应为void *类型，否则*(data + i)是编译时错误。 data = (uint8_t *)data;什么也没做。
sum1 << 4;错过了=做任何事情：sum1 <<= 4;
i和size应该属于同一类型。我更喜欢size_t，因为它是类型sizeof返回。
https://en.wikipedia.org/wiki/Fletcher%27s_checksum表示较高的位应为sum2，而您的位是sum1的位。

我会像那样写

uint8_t fletcher_8(uint8_t *data, size_t size){
    uint8_t sum1 = 0;
    uint8_t sum2 = 0;
    for (size_t i = 0; i < size; i++){
        sum1 += data[i];
        sum2 += sum1;
    }
    return (sum1 & 0xF) | (sum2 << 4);
}

或

uint8_t fletcher_8(uint8_t *data, size_t size){
    uint8_t sum1 = 0;
    uint8_t sum2 = 0;
    while (size--){
        sum1 += *data++;
        sum2 += sum1;
    }
    return (sum1 & 0xF) | (sum2 << 4);
}

Fletcher的校验和用于任意长度的字节组

1 个答案: