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

Question

简而言之，我的上下文是:调用 API，如果我找到某个 header 变量，我会将其子字符串化以返回一个值。如果不是，我必须返回 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;

    }

我的怀疑取决于我编码的正确程度。我决定使用更优雅和可读的方式与Optional，而不是编写一系列if&null。但如果我在方法内部创建 stringRetorno ，我会遇到“在封闭范围中定义的局部变量 stringRetorno 必须是最终的或有效的最终”。

搜索周围我发现解决方案创建 stringRetorno 作为类变量。

所以我直接的问题是:我是否在做一些需要避免的事情？一些可能无法回答我的问题但非常有用的问题是:

• 我是否应该更喜欢其他 Java HTTP 客户端而不是 com.mashape.unirest，因为它可能已经提供了Optional，这样我就可以编写更少冗长的代码？我根本不想比较库。我的重点是与接收缺少特定键(位置)的响应 header 的可能性相结合。如果 API 提供者建议的轻量级 unirest 迫使我采取变通办法，也许另一个提供可选作为答案的库的建议应该对我有帮助。

• 在 Try/Catch block 中编写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

am I doing something to be avoid?

是的。将变量拉到与其相关的唯一方法之外是一种糟糕的形式。此外，作为一个实际问题，它会为您带来线程安全问题 - 正如当前编写的那样，使用您的类并允许两个不同线程调用 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?

嗯，不是吗？也许“压倒性”不是您要寻找的术语，但即使您的真正意思是“过度杀戮”或“过度”或类似的意思，那么仍然不是。我什至不明白为什么你认为会这样。