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

Question

简而言之，我的上下文是：调用一个 API，如果我找到某个标头变量，我将其子字符串化以返回一个值。如果不是，我必须返回 Response.status。我已经用这段代码成功地达到了这样的要求：

...

import com.mashape.unirest.*

...

@Controller

public 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() {&nbsp; &nbsp; Optional<String> optionalLocation = null;&nbsp; &nbsp; HttpResponse<String> response = null;&nbsp; &nbsp; try {&nbsp; &nbsp; &nbsp; &nbsp; response = Unirest&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .post("https://skyscanner-skyscanner-flight-search-v1.p.rapidapi.com/apiservices/pricing/v1.0")&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .header("x-rapidapi-host", "skyscanner-skyscanner-flight-search-v1.p.rapidapi.com")&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .header("x-rapidapi-key", gmailKey).header("Content-Type", "application/x-www-form-urlencoded")&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .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")&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .asString();&nbsp; &nbsp; &nbsp; &nbsp; optionalLocation = Optional.ofNullable(response.getHeaders().getFirst("Location"));&nbsp; &nbsp; } catch (Exception e) {&nbsp; &nbsp; &nbsp; &nbsp; e.printStackTrace();&nbsp; &nbsp; }&nbsp; &nbsp; return optionalLocation.map(l -> l.substring(l.lastIndexOf("/") + 1))&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .orElse(String.valueOf(response.getStatus()));}

Answer 2

我正在做一些要避免的事情吗？是的。将变量拉出与其相关的唯一方法之外是一种糟糕的形式。此外，实际上，它会为您带来线程安全问题——正如目前所写的那样，使用您的类并允许调用两个不同线程的程序可能因此无法正确同步MainController.getSessionKey()。如果它是方法的局部变量，情况就不一样了。如果它是字符串的可变容器（例如数组、List或）&nbsp;，则可以使用该方法的本地变量。AtomicReference然后容器可以是最终的或实际上是最终的，而不会阻止您更改其内容。但是使用map()和的组合会更干净orElse()，这样您就不需要首先使用 lambda 设置字符串引用：stringRetorno&nbsp;=&nbsp;optionalLocation.map(l&nbsp;->&nbsp;l.substring(l.lastIndexOf("/")&nbsp;+&nbsp;1)) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;.orElse(String.valueOf(response.getStatus()));我是否应该更喜欢其他 Java HTTP 客户端而不是 com.mashape.unirest ，它可能已经提供了 Optional 这样我就可以编写不那么冗长的代码？也许我误解了，但我从中看到的唯一代码节省是删除调用Optional.ofNullable()（但保留其参数的等价物）。这并没有减少多少冗长，也没有多少（如果有的话）清晰度的提高。我不会认为这样做的可能性是寻找不同客户的任何理由。（但如果您确实在寻找不同的客户，请不要看这里。对图书馆和其他场外资源的推荐在这里是题外话。）在 Try/Catch 块中写 Optional 不是很费劲吗？不？也许“压倒性”不是您要查找的术语，但即使您的意思真的是“矫枉过正”或“过度”或类似的意思，也仍然不是。我什至不明白为什么你认为它可能是这样。

Answer 3

ifPresentOrElse可能不是您要使用的方法，而是.orElse.return&nbsp;optionalLocation.orElse(String.valueOf(response.getStatus()));ifPresentOrElse当可选项为空时，如果您想执行某些操作（例如日志记录），您会想要使用。