SonarLint：让这个匿名的内部类成为lambda

Question

以下代码收到Sonarint的通知：

handler.setFileNameGenerator((new FileNameGenerator() {
    @Override
    public String generateFileName(Message<?> message) {
        if (message.getPayload() instanceof File) {
            return ((File) message.getPayload()).getName();
        } else {
            throw new IllegalArgumentException("File expected as payload.");
        }
    }
}));

我尝试了以下内容，但在抛出新的IllegalArgumentException时遇到了问题：

handler.setFileNameGenerator(message -> 
       (message.getPayload() instanceof File) ? 
       ((File) message.getPayload()).getName() : 
       throw new IllegalArgumentException("File expected as payload."));

你能建议一下吗？

Answer 1

您使用的是条件表达式：不能将throw语句作为条件表达式的一部分。这是第一个问题。

第二个挑战是，您需要引发异常，因此当前使用条件表达式的表达式无法工作，除非您使用块：

handler.setFileNameGenerator(message -> {
    if (message.getPayload() instanceof File) {
        return ((File) message.getPayload()).getName();
    } else {
        throw new IllegalArgumentException("File expected as payload.");
    }
});

看起来更像是匿名类的复制/粘贴。。。

或者，您可以使用选件进行清理：

handler.setFileNameGenerator(message -> Optional.of(message.getPayload())
        .filter(p -> p instanceof File)
        .map(f -> (File) f)
        .map(File::getName)
        .orElseThrow(() -> new IllegalArgumentException("File expected as payload.")));