Question

我可能会得到'重复发布'，但无论如何。

自从我开始编程以来，我了解到你应该避免嵌套循环，if / else，try / catch等因为代码难以阅读并且理解上下文非常困难。您可以采取一些措施来避免它们，如ENUM和接口。

在寻找答案时，我发现帖子说，嵌套循环并不总是坏的，只要它可读。

现在，在处理项目时，我遇到了java.util.Properties类，其中包含一些非常复杂的方法，并且有很多嵌套方式：

 /*
 * Converts encoded &#92;uxxxx to unicode chars
 * and changes special saved chars to their original forms
 */
private String loadConvert (char[] in, int off, int len, char[] convtBuf) {
    if (convtBuf.length < len) {
        int newLen = len * 2;
        if (newLen < 0) {
            newLen = Integer.MAX_VALUE;
        }
        convtBuf = new char[newLen];
    }
    char aChar;
    char[] out = convtBuf;
    int outLen = 0;
    int end = off + len;

    while (off < end) {
        aChar = in[off++];
        if (aChar == '\\') {
            aChar = in[off++];
            if(aChar == 'u') {
                // Read the xxxx
                int value=0;
                for (int i=0; i<4; i++) {
                    aChar = in[off++];
                    switch (aChar) {
                      case '0': case '1': case '2': case '3': case '4':
                      case '5': case '6': case '7': case '8': case '9':
                         value = (value << 4) + aChar - '0';
                         break;
                      case 'a': case 'b': case 'c':
                      case 'd': case 'e': case 'f':
                         value = (value << 4) + 10 + aChar - 'a';
                         break;
                      case 'A': case 'B': case 'C':
                      case 'D': case 'E': case 'F':
                         value = (value << 4) + 10 + aChar - 'A';
                         break;
                      default:
                          throw new IllegalArgumentException(
                                       "Malformed \\uxxxx encoding.");
                    }
                 }
                out[outLen++] = (char)value;
            } else {
                if (aChar == 't') aChar = '\t';
                else if (aChar == 'r') aChar = '\r';
                else if (aChar == 'n') aChar = '\n';
                else if (aChar == 'f') aChar = '\f';
                out[outLen++] = aChar;
            }
        } else {
            out[outLen++] = aChar;
        }
    }
    return new String (out, 0, outLen);
}

对我而言，这是可读的。所以我一般很好奇，如果这是最佳或唯一的方法来做到这一点。因为这和JDK1.0一样久不应该被“清理”或更改为可读性？而且有许多方法需要复杂而复杂的代码。

感谢

Answer 1

让我们专注于事实，而不是这里的意见：

虽然＆＃34;清洁守则倡导者＆＃34;不喜欢听到 - 但是＆＃34;清洁代码＆＃34;规则仍然是主观的。有些人认为他们是圣杯＆＃34;其他人不同意。
更重要的是，您在此处展示的代码可能已超过15年。最有可能的是，现在它的写法也不同。

是的，你绝对不是一个人在思考＆＃34;这段代码看起来不可读＆＃34;。在某种程度上，这是因为我们不再习惯这样的代码，并且在某种程度上是因为这些代码难以阅读/掌握/理解。

Answer 2

它是低级别的（如后增量），为速度 /生产使用，两个嵌套循环和方面编写，对我来说没问题。一些空格，也许是评论缺失。

这种低级角色在不使用其他功能时特别明显。

但是，如果不调用其他新方法（因为函数解压缩会减慢速度），我不会看到更好的样式是否可行。

嵌入java.util类和方法的可读性

2 个答案: