试图缩短一些代码，获得-1返回值

Question

所以现在我有了这段代码-

private int getPinForComponent(final int componentId,
        final boolean lastDigit) {
    if (lastDigit) {
        switch (componentId) {
        case 6:
            return 0;
        case 7:
            return 1;
        case 8:
            return 2;
        case 9:
            return 3;
        case 10:
            return 4;
        case 11:
            return 5;
        case 12:
            return 6;
        case 13:
            return 7;
        case 14:
            return 8;
        case 15:
            return 9;
        }
    } else {
        switch (componentId) {
        case 4:
            return 0;
        case 8:
            return 1;
        case 12:
            return 2;
        case 16:
            return 3;
        case 20:
            return 4;
        case 24:
            return 5;
        case 28:
            return 6;
        case 32:
            return 7;
        case 36:
            return 8;
        case 40:
            return 9;
        }
    }
    return -1;
}

看起来一切正常，它只是返回0-9的值，具体取决于单击哪个组件。 我一直试图缩短它，我想到了这个。

private int getPinForComponent(int componentId, boolean lastDigit) {
    if (lastDigit) {
        return componentId - 6;
    } else {
        return ((componentId / 12) * 3) - 1;
    }
}

这将返回-1个值，或者返回错误的值。

有任何想法吗？ 谢谢。

Answer 1

尝试这个

if(lastDigit) {
    if (componentId >= 6 && componentId <= 15) {
        return componentId - 6;               
    }
} else {
    if (componentId % 4 == 0 && componentId >= 4 && componentId <= 40) {
        return (componentId / 4) - 1;
    }
}

Answer 2

使用模运算符。

if(lastDigit) {
    return (componentId+4) % 10;
}

Answer 3

有时将((componentId / 12) * 3) - 1 -1评估为-1是有道理的。 在大多数语言中，如果componentId是整数类型，并且其值小于12，则(componentId / 12)会四舍五入为0，然后(0 * 3) - 1等于-1 。

在纯数学中使用的实数中，除以12并乘以3等于除以4。但是，当涉及到舍入时，这两个运算序列是不同的。 这就是为什么sujithvm的答案行之有效的理论依据，即使乍看之下它似乎与问题中的错误代码具有相同的作用。

顺便说一句，作为样式问题...为什么必须完全缩短原始的开关案例代码？ 它正在工作，而且非常清晰。 缩短的版本更难理解，更不用说在引脚改变时进行更新了。