创建一个类变量只是为了在 Optional.ifPresentOrElse 中使用是一种应该避免

Question

简而言之，我的上下文是：调用一个 API，如果我找到某个标头变量，我将其子字符串化以返回一个值。如果不是，我必须返回 Response.status。我已经用这段代码成功地达到了这样的要求：...import com.mashape.unirest.*...@Controllerpublic class MainController {    private final String gmailKey = "XXX";    private String stringRetorno = "0";    @ResponseBody    @GetMapping("/getsessionkey")    public String getSessionKey() {        try {            HttpResponse<String> response = Unirest                    .post("https://skyscanner-skyscanner-flight-search-v1.p.rapidapi.com/apiservices/pricing/v1.0")                    .header("x-rapidapi-host", "skyscanner-skyscanner-flight-search-v1.p.rapidapi.com")                    .header("x-rapidapi-key", gmailKey).header("Content-Type", "application/x-www-form-urlencoded")                    .body("inboundDate=2019-11-25&cabinClass=economy&children=0&infants=0&Country=BR&Currency=BRL&locale=pt-BR&originPlace=GRU-sky&destinationPlace=MCZ-sky&outboundDate=2019-11-19&adults=2")                    .asString();            Optional<String> optionalLocation = Optional.ofNullable(response.getHeaders().getFirst("Location"));            optionalLocation.ifPresentOrElse(l -> stringRetorno = l.substring(l.lastIndexOf("/") + 1),                    () -> stringRetorno = String.valueOf(response.getStatus()));        } catch (Exception e) {            e.printStackTrace();        }        return stringRetorno;    }我的怀疑取决于我编码的正确程度。我没有编写 if&null 链，而是决定使用 Optional 的更优雅和更易读的方式。但是如果我在方法内部创建 stringRetorno，我会面临“在封闭范围内定义的局部变量 stringRetorno 必须是最终的或有效的最终”。四处搜索我发现作为解决方案创建 stringRetorno 作为类变量。所以我的直截了当的问题是：我是不是在做一些要避免的事情？一些可能无法回答我的问题但非常有用的问题是：我是否应该更喜欢其他 Java HTTP 客户端而不是 com.mashape.unirest ，它可能已经提供了 Optional 这样我就可以编写不那么冗长的代码？我根本不想比较图书馆。我的重点是与接收缺少特定键（位置）的响应标头的可能性相结合。而且，如果 API 提供者建议的轻量级 unirest 迫使我采取变通办法，那么另一个提供 Optional 作为答案的库的建议应该对我有所帮助。在 Try/Catch 块中写 Optional 不是很费劲吗？

Answer 1

@ResponseBody

@GetMapping("/getsessionkey")

public String getSessionKey() {

    Optional<String> optionalLocation = null;

    HttpResponse<String> response = null;

    try {

        response = Unirest

                .post("https://skyscanner-skyscanner-flight-search-v1.p.rapidapi.com/apiservices/pricing/v1.0")

                .header("x-rapidapi-host", "skyscanner-skyscanner-flight-search-v1.p.rapidapi.com")

                .header("x-rapidapi-key", gmailKey).header("Content-Type", "application/x-www-form-urlencoded")

                .body("inboundDate=2019-11-25&cabinClass=economy&children=0&infants=0&Country=BR&Currency=BRL&locale=pt-BR&originPlace=GRU-sky&destinationPlace=MCZ-sky&outboundDate=2019-11-19&adults=2")

                .asString();

        optionalLocation = Optional.ofNullable(response.getHeaders().getFirst("Location"));

    } catch (Exception e) {

        e.printStackTrace();

    }

    return optionalLocation.map(l -> l.substring(l.lastIndexOf("/") + 1))

            .orElse(String.valueOf(response.getStatus()));

}

Answer 2

我正在做一些要避免的事情吗？

是的。将变量拉出与其相关的唯一方法之外是一种糟糕的形式。此外，实际上，它会为您带来线程安全问题——正如目前所写的那样，使用您的类并允许调用两个不同线程的程序可能因此无法正确同步MainController.getSessionKey()。如果它是方法的局部变量，情况就不一样了。

如果它是字符串的可变容器（例如数组、List或） ，则可以使用该方法的本地变量。AtomicReference然后容器可以是最终的或实际上是最终的，而不会阻止您更改其内容。

但是使用map()和的组合会更干净orElse()，这样您就不需要首先使用 lambda 设置字符串引用：

stringRetorno = optionalLocation.map(l -> l.substring(l.lastIndexOf("/") + 1))
        .orElse(String.valueOf(response.getStatus()));

• 我是否应该更喜欢其他 Java HTTP 客户端而不是 com.mashape.unirest ，它可能已经提供了 Optional 这样我就可以编写不那么冗长的代码？

也许我误解了，但我从中看到的唯一代码节省是删除调用Optional.ofNullable()（但保留其参数的等价物）。这并没有减少多少冗长，也没有多少（如果有的话）清晰度的提高。我不会认为这样做的可能性是寻找不同客户的任何理由。

（但如果您确实在寻找不同的客户，请不要看这里。对图书馆和其他场外资源的推荐在这里是题外话。）

• 在 Try/Catch 块中写 Optional 不是很费劲吗？

不？也许“压倒性”不是您要查找的术语，但即使您的意思真的是“矫枉过正”或“过度”或类似的意思，也仍然不是。我什至不明白为什么你认为它可能是这样。

Answer 3

ifPresentOrElse可能不是您要使用的方法，而是.orElse.

return optionalLocation.orElse(String.valueOf(response.getStatus()));

ifPresentOrElse当可选项为空时，如果您想执行某些操作（例如日志记录），您会想要使用。