创建一个 class 变量只是为了在 Optional.ifPresentOrElse 中使用是一种应该避免的做法吗？

Question

简而言之，我的上下文是：调用一个 API，如果我找到某个 header 变量，我将它子串到 return 一个值。如果不是我必须 return 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，我将面临 "Local variable stringRetorno defined in an enclosing scope must be final or effectively final"。

四处搜索，我发现解决方案是将 stringRetorno 创建为 class 变量。

所以我的直截了当的问题是：我是否正在做一些应该避免的事情？一些可能无法回答我的问题但非常有用的问题是：

我是否应该比 com.mashape.unirest 更喜欢其他 Java HTTP 客户端，后者可能已经提供了 Optional，这样我就可以编写不那么冗长的代码？我根本不想比较图书馆。我的重点是接收缺少特定键（位置）的响应 header 的可能性。如果 API 提供者建议的轻量级 unirest 迫使我变通，也许另一个提供 Optional 作为答案的库的建议应该对我有帮助。
在 Try/Catch 块内写 Optional 是不是太过分了？

*** 解决方案

感谢提供的答案

@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 1

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

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

如果在可选项为空时需要执行某些操作（如日志记录），您可能希望使用 ifPresentOrElse。

Answer 2

am I doing something to be avoid?

是的。将变量拉出与其相关的唯一方法之外是一种糟糕的形式。此外，作为一个实际问题，它为您创建了一个线程安全问题——正如目前所写的那样，一个使用您的 class 并允许两个不同线程调用 MainController.getSessionKey() 的程序可能出于这个原因没有被正确同步。如果是方法的局部变量就不一样了。

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

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

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

should I prefer other Java HTTP Client than com.mashape.unirest which may already provide Optional so I would be able to write less verbose code?

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

（但如果您确实在寻找不同的客户，请不要在 此处查看 。有关图书馆和其他场外资源的建议在这里是题外话。）

isn't overwhelming writing Optional inside of Try/Catch block?

嗯，不是吗？也许 "overwhelming" 不是您要查找的术语，但即使您的意思确实是 "overkill" 或 "excessive" 或类似的术语，也仍然不是。我什至不明白你为什么认为它可能是这样。

创建一个 class 变量只是为了在 Optional.ifPresentOrElse 中使用是一种应该避免的做法吗？

is create a class variable just to be used in Optional.ifPresentOrElse a practice to be avoid?

java

http

apache-httpclient-4.x

okhttp3

unirest