SonarLint提示‘orElseThrow返回值必须被使用’问题如何解决？

I get it, this is a common gotcha with SonarLint's squid:S2201 rule—it flags cases where you call a method that returns a value but don't use that return value. In your case, you're using orElseThrow purely to validate that an item exists (throwing an exception if it doesn't), so you don't actually need the returned item. Here are a few clean ways to resolve this:

Option 1: Rewrite with anyMatch (most readable) #

Instead of using findAny().orElseThrow(), use anyMatch to explicitly check for the item's existence, then throw the exception if it's missing. This makes your intent crystal clear to both other developers and SonarLint:

if (!itemList.stream()
    .anyMatch(item -> orderItemId.equals(item.getId()))) {
    throw new BadRequestException("12345","Item Not Found");
}

This approach avoids the unused return value entirely, so SonarLint won't flag it at all.

Option 2: Suppress the specific rule warning #

If you prefer to keep your original orElseThrow syntax, you can tell SonarLint that you're intentionally ignoring the return value by adding a targeted suppression. You can do this either on the line itself or on the enclosing method:

• Line-level suppression:

  itemList.stream()
      .filter(item -> orderItemId.equals(item.getId()))
      .findAny()
      .orElseThrow(() -> new BadRequestException("12345","Item Not Found")); // NOSONAR squid:S2201 - Used solely for existence validation
  

• Annotation-based suppression (for Java 8+):

  @SuppressWarnings("squid:S2201")
  public void yourMethodName() {
      // Your existing orElseThrow code here
  }
  

Just make sure to add a comment explaining why you're suppressing the rule—this helps future maintainers understand your intent.

Option 3: Assign to an unused variable (last resort) #

You can assign the returned item to a variable and mark it as unused. While this works, it's less readable than the first option since it introduces a variable that serves no functional purpose:

// Variable intentionally unused—only validating item existence
@SuppressWarnings("unused")
Item ignoredItem = itemList.stream()
    .filter(item -> orderItemId.equals(item.getId()))
    .findAny()
    .orElseThrow(() -> new BadRequestException("12345","Item Not Found"));

My recommendation is Option 1—it's the most straightforward and aligns perfectly with your intent of validating existence without needing the actual item.

内容的提问来源于stack exchange，提问作者Baji Shaik