Question

我有这个功能，它是一个多路复用器。

// 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＆amp; S0 == 0指A和S1 == 0＆amp; S0 == 1指B等等。

我觉得我的代码很接近或关闭或者它是正确的我只需要修复我在主要测试它的方式...

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

Answer 1

他们不是真正设计良好的测试，那两个。让我们看第一个作为示例（更改为{em>实际函数名称multi）：

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

在这种情况下multi()应该返回第一个参数，但是如果它们 all 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本身已经处理得很好的类型创建特殊枚举的有用性（布尔值为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多路复用器功能

1 个答案: