Java switch/case方法建议的长度

Question

我是一名学生，正在写一篇关于SSL/TLS和JSSE的论文。在oracle提供的实施示例中（可在此处找到：http://docs.oracle.com/javase/7/docs/technotes/guides/security/jsse/JSSERefGuide.html#AppA)我找到了实施的例子：

   public void doHandshake(SocketChannel socketChannel,
                        SSLEngine     engine,
                        ByteBuffer    myNetData, 
                        ByteBuffer    peerNetData) throws Exception {

    // 2)
    int appBufferSize = engine.getSession().getApplicationBufferSize();
    // 3)
    ByteBuffer myAppData = ByteBuffer.allocate(appBufferSize);
    ByteBuffer peerAppData = ByteBuffer.allocate(appBufferSize);

    // 4)
    engine.beginHandshake();
    SSLEngineResult.HandshakeStatus hs = engine.getHandshakeStatus();

    // 5)
    while (hs != SSLEngineResult.HandshakeStatus.FINISHED
            && hs != SSLEngineResult.HandshakeStatus.NOT_HANDSHAKING) {

        switch (hs) {

            case NEED_UNWRAP:
                // Receive handshaking data from peer
                if (socketChannel.read(peerNetData) < 0) {
                    // The channel has reached end-of-stream
                }

                // Process incoming handshaking data
                peerNetData.flip();
                SSLEngineResult res = engine.unwrap(peerNetData, peerAppData);
                peerNetData.compact();
                hs = res.getHandshakeStatus();

                // Check status
                switch (res.getStatus()) {
                    case OK:
                        // Handle OK status
                        break;

        // Handle other status: BUFFER_UNDERFLOW, BUFFER_OVERFLOW, CLOSED
                    //...
                }
                break;

            case NEED_WRAP:
                // Empty the local network packet buffer.
                myNetData.clear();

                // Generate handshaking data
                res = engine.wrap(myAppData, myNetData);
                hs = res.getHandshakeStatus();

                // Check status
                switch (res.getStatus()) {
                    case OK:
                        myNetData.flip();

                        // Send the handshaking data to peer
                        while (myNetData.hasRemaining()) {
                            socketChannel.write(myNetData);
                        }
                        break;

        // Handle other status:  BUFFER_OVERFLOW, BUFFER_UNDERFLOW, CLOSED
                    //...
                }
                break;

            case NEED_TASK:
                // Handle blocking tasks
                break;

    // Handle other status:  // FINISHED or NOT_HANDSHAKING
            //...
        }
    }

// Processes after handshaking
    //   ...
}

现在，根据我对正确的OOP体系结构的了解，这个方法不应该超过20行，但我发现Oracle会犯这样的基本体系结构错误是很奇怪的。任何超过20行的都应该被分割成不同的方法，但也有例外。我想知道，这是例外吗？如果没有，将switch/case方法拆分为更小的方法的最佳方法是什么。

Answer 1

你在拼凑术语。。。那不一定属于一起。

您可以创建一个伟大的OOP设计...并且在每个方法中仍然有很多代码。

您可以遵循干净的代码规则，例如，避免任何包含多个抽象级别的方法；最终的设计可能很糟糕，“不是OOP”（但我认为：这很难——如果你遵循所有干净的代码规则，你会自动创建更好的OOP设计）。

但是，要回答最初的问题（这实际上是一个糟糕的问题；因为它需要基于意见的答案，比如我的答案）：上述代码至少“一点也不干净”。事实上，我觉得这很可怕，而将“更多”塞进enum案例的想法对我来说听起来非常糟糕。除此之外，您将不会以这种方式使用枚举-这种“开关”应该对html" target="_blank">客户端代码不可见；相反，你会依赖多态性；然后简单地告诉你在某个对象上调用某个方法。。。

到现在为止，我已经练习了多年干净的代码，发现生成的代码更易于阅读和维护。在“干净代码”中；如前所述：您不会在一个方法中放入多个switch语句。每个循环；而每一个破绽案例。。。将被放在自己的方法中。

许多人认为这很疯狂，但我发现这确实是“更好”的方法，在实践了几个月之后。它确实阻止了我创建像您的示例那样不可测试的代码。

回答你的第二个问题：这真的很难。人们必须理解整个设计，然后一点一点地把它拆开，重构它的元素。但是这当然需要你有一套完整的单元/功能测试（所以你要证明你的改变不会破坏整个事情）。因此，“改进”此类代码的通用方法是：

1. 了解更改现有软件的真正成本。你不应该仅仅因为“坏代码”而改变东西。您需要一个好的（业务）理由来证明更改代码的费用。