java - Groovy - 压缩多个 if 语句

Question

我是一个出色的新手，所以请耐心等待。我喜欢 groovy 编写更少且通常更干净的代码的能力，但我试图找出是否有更好或更易读的方法来压缩这些多个 if 语句。这是一个相当简单的代码片段，但必须有更好的方法来做到这一点。我是新手，因此非常感谢任何代码片段。

if (!textOverlay) {
  textType = "" 
  if(url != null){
    Page getPage = resource.getResourceResolver().getResource(url).adaptTo(Page.class)
    if (getPage != null) {
      showLink = showLink + ".html"
        if (fragment.length() > 0) {
          url += "#"+fragment;
        }
    }
  }             
}  else { 
    //do something else
}

预先感谢您的帮助!

Answer 1

这对嵌套没有帮助，但在某些地方您可以利用 Groovy 使代码更加紧凑。我添加了一些解释性评论

if (!textOverlay) {
  textType = "" 

  // null is considered false, so no need to explicitly check for null
  if (url) {

    // getResourceResolver() replaced by resourceResolver
    // Page and Page.class are the same thing
    Page getPage = resource.resourceResolver.getResource(url).adaptTo(Page)

    // Groovy truth
    if (getPage) {

      // use String concatenation operator (also works in Java)
      showLink += ".html"

      // non-empty strings evaluate to true
      if (fragment) {
          // GString instead of string concatenation
          url += "#$fragment"
      }
    }
  }             
}  else { 
    //do something else
}