Sonar多维数据集与异常处理机制

Question

我实现了一些允许在应用服务器上处理套接字连接的功能。下面是我代码的摘录：

@Dependent
public class SocketServerService {

  @Resource(name = "DefaultManagedThreadFactory")
  private ManagedThreadFactory threadFactory;

  private ServerSocket serverSocket;
  private Thread socketServerThread;
  private volatile boolean isListening = false;

  public void startListening(int port) {
     socketServerThread = threadFactory.newThread(() ->   listeningOnSocket(port));
     socketServerThread.setName("ServerSocketThread");
     socketServerThread.start();
  }

  private void listeningOnSocket(int port) {
     try {
         serverSocket = new ServerSocket(port);
         waitForRequestAndProcessData();
     } catch (IOException e) {
         logger.error("Could not listen on socket: " + port, e);
     }
  }

 private void waitForRequestAndProcessData() {
     while (isListening) {
         try {
             final Socket socket = serverSocket.accept();
             process(socket);// do something on data received from socket that 
                             // can throw some checked or unchecked exceptions

          } catch (Exception e) {
             logger.error("Error while processing data.", e);
          }
      }
  } 

  private void stopServerSocket() {
      socketServerThread.interrupt();
  }

...

}

如您所见，我使用ManagedThreadFactory创建可管理线程。当我使用Sonar多维数据集验证我的代码时，它显示了以下错误：

捕获特定异常子类型的列表，而不是.。

我知道使用catch (Exception e)处理所有异常都是错误的做法，但我不知道如何修改代码来解决这个问题。当while (isListening) {}方法中出现错误时，我不希望退出process循环(我只希望日志错误并处理套接字中的下一段数据)。但我应该抓住什么样的例外呢？当然，我可以在process方法中声明一些检查过的异常并在waitForRequestAndProcessData方法中捕获它，但是在本例中，我将在process方法中添加catch (Exception e)。在本例中，Sonarcube还将显示捕获特定异常子类型的列表，而不是process方法的消息。也许我该重新设计我的代码了？或者在这种情况下，缓存所有异常不是那么糟糕的做法吗？

Answer 1

ServerSocket.accept()只能引发以下异常：

• IOException及其子类，这是一个检查异常。
• 任意数量的RuntimeException及其子类

因此，你可以把你的渔获物替换为：

  } catch (IOException | RuntimeException e) {
     logger.error("Error while processing data.", e);
  }

而且现在可以肯定的是，所有的例外都会被捕获。

您还可以使RuntimeException显式化，这将导致：

  } catch (IOException | SecurityException | IllegalBlockingModeException e) {
     logger.error("Error while processing data.", e);
  }