0

我在我的片段文件夹中找到了这个旧的褪色功能，并希望将它实现到我的一个项目中。它可用于将一种颜色淡化为另一种颜色。这是一个很长的单行：

D3DCOLOR GetFadedColor(D3DCOLOR from, D3DCOLOR to, float factor)
{
    return (factor<0.0f)?from:((factor>1.0f)?to:((((from>>24)>(to>>24))?((from>>24)-(D3DCOLOR)(factor*(float)((from>>24)-(to>>24)))):((from>>24)+(D3DCOLOR)(factor*(float)((to>>24)-(from>>24))))<<24)|((((from<<8)>>24)>((to<<8)>>24))?(((from<<8)>>24)-(D3DCOLOR)(factor*(float)(((from<<8)>>24)-((to<<8)>>24)))):(((from<<8)>>24)+(D3DCOLOR)(factor*(float)(((to<<8)>>24)-((from<<8)>>24))))<<16)|((((from<<16)>>24)>((to<<16)>>24))?(((from<<16)>>24)-(D3DCOLOR)(factor*(float)(((from<<16)>>24)-((to<<16)>>24)))):(((from<<16)>>24)+(D3DCOLOR)(factor*(float)(((to<<16)>>24)-((from<<16)>>24))))<<8)|((((from<<24)>>24)>((to<<24)>>24))?(((from<<24)>>24)-(D3DCOLOR)(factor*(float)(((from<<24)>>24)-((to<<24)>>24)))):(((from<<24)>>24)+(D3DCOLOR)(factor*(float)(((to<<24)>>24)-((from<<24)>>24)))))));
}

D3DCOLOR只是一个DWORD( unsigned long)。例如，颜色可以是 0xAARRGGBB（A-alpha、R-red、G-green、B-blue），但也适用于其他组合。

显然这是一团糟，但这正是我所需要的。

问题是它不能按预期工作：

GetFadedColor(0x00000000, 0xff33cccc, 0.3f)
// = 0x4c0f3d3d - working as intended
GetFadedColor(0xff33cccc, 0x00000000, 0.3f)
// = 0x000000bf - pretty wrong
GetFadedColor(0xff00ff00, 0x00ff00ff, 0.3f)
// = 0x004c00ff - second color value is correct, everything else wrong

我实际上不知道它是如何工作的，也不记得我从哪里得到它，所以我在这里寻求帮助。要么帮助我找到错误，要么找到一个替代函数来完成这个。

4

1 回答 1

1

你现在应该做的是，首先你应该花 5 分钟的时间来写下一些非常基本的测试，其中包含你知道你期望什么的情况。你甚至不需要使用任何测试框架，因为你可以使用assert：

// basicTests.c
#include <assert.h>

int getFadedColor_basicTests()
{
    assert(GetFadedColor(0x00000000, 0xff33cccc, 0.3f) == 0x4c0f3d3d  && "30% from black to light blue should be greenish");
    assert(GetFadedColor(0xff33cccc, 0x00000000, 0.3f) == something   && "30% from one color to another should be...");

    // if you're not sure what the exact value should be, you should write a helper function
    // that returns true/false for if each of the four components of the actual color
    // are in a sensible expected range
    ...
}


int main()
{
    getFadedColor_basicTests();
    return 0;
}

一旦你对测试的覆盖率感到满意，无论是总共只有 3 个断言，或者如果你愿意的话，可能是 50 个断言，你应该开始重新格式化单行，打破行，添加有意义的缩进和注释。开始重构，提取常用表达式，为他们做什么或应该做什么添加注释，同时在更改之间运行测试并在设计新测试时添加测试。

编辑：

难道不应该分别线性外推每个组件吗？

int fade(int from_, int to_, float factor)
{   
    unsigned char *from = (unsigned char*)&from_;
    unsigned char *to = (unsigned char*)&to_;
    int result_;
    unsigned char *result = (unsigned char*)&result_;

    for (int i = 0 ; i < 4; ++i)
    {
        result[i] = factor * ((int)to[i] - (int)from[i]) + from[i];
    }   

    return result_;
}
于 2013-05-29T16:46:29.667 回答