c - C多路复用器功能

我有一个多路复用器的功能。

// Enumerated type for a single bit.
typedef enum { FALSE=0, TRUE=1 } BIT;

BIT multi(BIT A, BIT B, BIT C, BIT D, BIT S1, BIT S0)
{
    if(S1== FALSE && S0 ==FALSE)
        return A;

    else if(S1==FALSE && S0==TRUE)
        return B;

    else if (S1== TRUE && S0== FALSE)
        return C;

    else
        return D;
}

对于多路复用器，S1，S0作为二位二进制数，按A、B、C、D索引，按给定的顺序排列。
如S1==0&S0==0表示A，S1==0&S0==1表示B等。
我觉得我的代码是接近或远离或它是正确的，我只需要修复我是如何测试它在主要我有。。。

 assert(multiplexer(FALSE, FALSE, FALSE, FALSE, FALSE, FALSE) == A);
 assert(multiplexer(TRUE, TRUE, TRUE, TRUE, TRUE, TRUE) == D);

最佳答案

这两个测试设计得不是很好。让我们以第一个为例（更改为multi的实际函数名）：

assert(multi(FALSE, FALSE, FALSE, FALSE, FALSE, FALSE) == A);

在这种情况下，multi()应该返回第一个参数，但是如果返回了其他三个参数中的一个，您如何知道它们是否都是false。此外，我不确定D变量是否在函数之外可用，所以我要切换到一个更具体的值。
因此，一个更好的测试集是：

assert(multi(FALSE, TRUE,  TRUE,  TRUE,  FALSE, FALSE) == FALSE);
assert(multi(TRUE,  FALSE, FALSE, FALSE, FALSE, FALSE) == TRUE);

这是为了测试A的返回。对于B，C和D只需将唯一值移动到正确的位置（分别为2、3或4）并更改作为选择器传递的内容（位置5和6）。
同样，对于第二个测试，您需要重新编写测试，以便区分成功与失败：

assert(multi(FALSE, FALSE, FALSE, TRUE, TRUE, TRUE) == TRUE);
assert(multi(TRUE,  TRUE,  TRUE, FALSE, TRUE, TRUE) == FALSE);

顺便说一下，我不确定为一个类型创建一个特殊的枚举是否有用，这个类型C已经能够很好地单独处理了（boolean作为int）。
在我看来，代码可以大大简化为：

int multi (int r0, int r1, int r2, int r3, int s0, int s1) {
    if (s0) {
        // Must be r2 or r3, depending on s1.
        if (s1) return r3;
        return r2;
    }

    // s0 is false, must be r0 or r1, depending on s1.
    if (s1) return r1;
    return r0;
}

或者，一旦你真正理解了C:-）

// Returns rX based on sX:
//   s0 false, s1 false, return r0.
//   s0 false, s1 true,  return r1.
//   s0 true,  s1 false, return r2.
//   s0 true,  s1 true,  return r3.

int multi (int r0, int r1, int r2, int r3, int s0, int s1) {
    return ((s0) ? ((s1) ? r3 : r2) : ((s1) ? r1 : r0));
}

关于c - C多路复用器功能，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/28800922/