Question

我在我的snippets文件夹中找到了这个旧的颜色淡化功能，并希望将其实现到我的一个项目中。它可以用来将一种颜色淡化为另一种颜色。这是一个非常长的单行：

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

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

Answer 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_;
}

褪色功能出错

1 个答案: