java - 如何减少自定义解串器中 if 的数量

Question

我使用 SonarLint 分析了我的自定义 JSON 反序列化器，它表明我应该将认知复杂度从 21 降低到至少 15。换句话说，这意味着我必须有很多 if 语句。我的解串器如下所示:

   @Override
    public Movie deserialize(JsonParser p, DeserializationContext ctxt) throws IOException {
        ObjectCodec oc = p.getCodec();
        JsonNode node = oc.readTree(p);

        Movie movie = new Movie();
        movie.setId(node.get("id").textValue());
        movie.setTitle(node.get("movieInfo").get("title").textValue());

        int identifiersSize = node.get("movieInfo").get("identifiers").size();
        String IMSC="";
        for(int i=0;i<identifiersSize;i++){
            if(node.get("movieInfo").get("identifiers").get(i).get("type").textValue().equals("IMSC")){
                IMSC = node.get("movieInfo").get("identifiers").get(i).get("identifier").textValue();
            }
        }
        if(IMSC.isEmpty()){
            IMSC = node.get("id").textValue();
        }
        movie.setIMSC(IMSC);

        if(node.get("movieInfo").has("subtitle")){
            movie.setSubtitle(node.get("movieInfo").get("subtitle").textValue());
        }

        if(node.get("movieInfo").has("publishedDate")){
            DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd");
            LocalDate date;
            try {
                date = LocalDate.parse(node.get("movieInfo").get("publishedDate").asText(), formatter);
            } catch (DateTimeParseException e){
                date = LocalDate.of(node.get("movieInfo").get("publishedDate").asInt(),1,1);
            }
            movie.setPublishedDate(date);
        }


        if(node.get("movieInfo").has("publisher")){
            movie.setPublisher(node.get("movieInfo").get("publisher").textValue());
        }

        if(node.get("movieInfo").has("description")){
            movie.setDescription(node.get("movieInfo").get("description").textValue());
        }

        if(node.get("movieInfo").has("length")){
            movie.setLength(node.get("movieInfo").get("length").asInt());
        }

        if(node.get("movieInfo").has("imageLinks")){
            movie.setThumbnailUrl(node.get("movieInfo").get("imageLinks").get("thumbnail").asText());
        }

        if(node.get("movieInfo").has("language")){
            movie.setLanguage(node.get("movieInfo").get("language").asText());
        }

        if(node.get("movieInfo").has("previewLink")){
            movie.setPreviewLink(node.get("movieInfo").get("previewLink").asText());
        }

        if(node.get("movieInfo").has("ratingsCount")) {
            movie.setRatingsCount(node.get("movieInfo").get("ratingsCount").asInt());
            movie.setAverageRating(node.get("movieInfo").get("averageRating").doubleValue());
        }


        if(node.get("movieInfo").has("authors")) {
            List<String> authors= new ArrayList<>();
            int size = node.get("movieInfo").get("authors").size();
            for(int i =0;i<size;i++){
               authors.add(node.get("movieInfo").get("authors").get(i).asText());
            }
            movie.setAuthors(authors);
        }

        if(node.get("movieInfo").has("categories")) {
            List<String> categories= new ArrayList<>();
            int size = node.get("movieInfo").get("categories").size();
            for(int i =0;i<size;i++){
            categories.add(node.get("movieInfo").get("categories").get(i).asText());
            }
            movie.setCategories(categories);
        }

        return movie;
    }

我的问题是:有没有更好的方法？我希望 SonarLint 的错误为零。

Answer 1

您似乎多次调用 node.get("movieInfo") 和 node.get("id") 。您可以通过将这两个调用的值缓存在局部变量中来大大简化代码。

但我认为没有好的方法可以大幅减少 if 语句。 (您可以通过创建 lambda 表然后迭代节点的“键”、查找并调用相应的 lambda 来重新计算 if 语句。但我怀疑这会让代码更容易阅读。)

您可以将方法拆分为子方法，并减少每个方法的 if 语句数量。这可能会让 SonarLint 更高兴，但它是否使代码更具可读性是有争议的。

<小时/>

I would like to have zero errors from SonarLint.

这不是你的目标。像 SonarLint 这样的工具表面上的目的是通过人工测量代码复杂性来帮助您使代码更具可读性和可维护性。但是:

• 程序员通常可以处理此处所示的复杂性。此代码重复性大于复杂性。
• 有时，复杂性是您尝试执行的任务所固有的。