这个函数逻辑上是不是有问题？

[id27145]

// 函数本意应该是找出 data 中出现次数最多的值给 bg，出现第二多的值给 fg
// 如 data[] = { 1, 2, 3, 4, 1, 1, 2 } --> bg = 1, fg = 2
        
int func_1(BYTE* data, int w, int h, BYTE* bg, BYTE* fg)
{
    int tileType = 0;
    BYTE pix1 = *data, pix2 = 0;
    int count1 = 0, count2 = 0;
    BYTE end = data + w*h;
    
    for (BYTE* ptr = data; ptr < end; ptr++)
    {
        if (*ptr == pix1)
        {                       // 这里在 loop 1 时恒成立
            count1++;
            continue;
        }
    
        if (count2 == 0)
        {                       // 这里成立时，表示碰到第一个不同的值(与data[0]值不同的第一个值)
            tileType |= 0x01;   // 含有两种以上的值
            pix2 = *ptr;        // 这里把第一个不同的值赋于 pix2
        }
    
        if (*data == pix2)
        {                       // 所以这里第一次执行时，恒不成立，count2不会变
            count2++;
            continue;
        }
        
        tileType |= 0x02;
        break;                  // 接下来就 break 了，count2 恒为0
    }
    
    if (count1 >= count2)
    {
        *bg = pix1; *fg = pix2;
    }
    else
    {
        *bg = pix2; *fg = pix1;
    }
    
    return tileType;
}

count2 恒为0的话，这个函数岂不是失去意义？

[healer_kx]

逻辑是不是有问题要看需求是什么

[hustlaofan]

        if (count2 == 0)
        {                       // 这里成立时，表示碰到第一个不同的值(与data[0]值不同的第一个值)
            tileType |= 0x01;   // 含有两种以上的值
            pix2 = *ptr; 
        }
    
        if (*ptr == pix2)
        {                       // 所以这里第一次执行时，恒不成立，count2不会变
            count2++;
            continue;
        }

[hustlaofan]

还有一个问题，这样得到的两个值不一定是出现最多的和第二多的
可以搞个哈希表来存相应数出现的次数，在来两次冒泡就出来最大和第二大的了

[sunlin7]

楼主列出的代码应该是有问题的。试用
data[] = {0, 1, 2, 3, 2, 2, 1, 1, 1}
试验

[hustlaofan]

就是计算数组一个数和第一个不同于第一个数出现的次数，那个不会执行的是个错误，我不是改了吗

[id27145]

[Quote=引用 1 楼 healer_kx 的回复:]
逻辑是不是有问题要看需求是什么
[/Quote]
抛开需求，只看代码，

            count2++;
            continue;

这两句永远不会被执行，岂不是说明代码自身就有问题？

[Quote=引用 3 楼 hustlaofan 的回复:]
还有一个问题，这样得到的两个值不一定是出现最多的和第二多的
可以搞个哈希表来存相应数出现的次数，在来两次冒泡就出来最大和第二大的了
[/Quote]
从调用这个函数的上下文来看是应该实现“取第一第二多”的，但从函数本身的实现来看，我也没明白它到底要做什么。不然也不至于怀疑它的逻辑有问题了。