IntelliJ提示方法返回值未使用，Builder模式代码排查与优化咨询

首先明确说：你的代码没有实例化错误，build()方法和StreamParser的私有构造函数实现完全正确，能正常创建目标实例。不过有个容易忽略的拼写bug要留意——Builder里的字段isLastSteam（末尾是Steam）和方法名isLastStream、StreamParser里的字段名isLastStream不一致，虽然当前不影响运行，但后续维护很容易踩坑，建议统一改成isLastStream。

警告产生的原因 #

IntelliJ IDEA提示的“Return value of the method is never used”，本质是因为你的Builder方法设计成了返回自身（return this;），目的是支持链式调用（比如builder.withA().withB().build()）。但你调用这些方法时，只是直接写了streamBuilder.withSpanLimit1(spanLimit1);，没有使用返回值——IDEA就会认为“你返回了一个对象但没用到，是不是逻辑写错了？”，因此弹出警告。

这里要强调：你的代码逻辑是完全正确的！因为方法内部已经修改了streamBuilder本身的状态，只是不符合IDEA对返回值使用的检测规则而已。

更优的实现方式 #

针对这个警告和Builder模式的规范使用，给你几个可选方案：

方案1：复用返回值（最简消警方式） #

直接把调用后的返回值赋值回原变量，让IDEA知道你用到了返回值：

StreamParserBuilder streamBuilder = new StreamParserBuilder(args, inputFile, stream, isPaired);
if (isSpanOneReplaced) streamBuilder = streamBuilder.withSpanLimit1(spanLimit1);
if (isSpanTwoReplaced) streamBuilder = streamBuilder.withSpanLimit2(spanLimit2);
if (isSpanThreeReplaced) streamBuilder = streamBuilder.withSpanLimit3(spanLimit3);
if (areCoordinatesReplaced) streamBuilder = streamBuilder.withCoordinates(coordinates);
if (isOutputDirectoryReplaced) streamBuilder = streamBuilder.withOutputDirectory(outputDirectory);
if (streamCount == streamData.size()) streamBuilder = streamBuilder.isLastStream(true);
StreamParser streamParser = streamBuilder.build();

方案2：使用链式调用（Builder模式的标准写法） #

把所有参数配置整合到链式调用中，既符合Builder模式“流畅API”的设计初衷，代码也更简洁：

StreamParser streamParser = new StreamParserBuilder(args, inputFile, stream, isPaired)
    .withSpanLimit1(isSpanOneReplaced ? spanLimit1 : 2000L)
    .withSpanLimit2(isSpanTwoReplaced ? spanLimit2 : 100000L)
    .withSpanLimit3(isSpanThreeReplaced ? spanLimit3 : 3000000L)
    .withCoordinates(areCoordinatesReplaced ? coordinates : new String[]{"L1", "R2"})
    .withOutputDirectory(isOutputDirectoryReplaced ? outputDirectory : System.getProperty("user.dir"))
    .isLastStream(streamCount == streamData.size())
    .build();

方案3：修改为无返回值方法（放弃链式调用） #

如果完全不需要链式调用的特性，可以把所有withXXX()和isLastStream()方法的返回类型改成void，移除return this;语句，这样就不会有返回值未使用的警告了：

public void withSpanLimit1(long spanLimit1){
    this.spanLimit1 = spanLimit1;
}

但这种方式会丢失Builder模式的链式优势，适合参数配置极少的场景。

方案4：抑制警告（不推荐） #

如果不想修改代码结构，可以在调用处或Builder方法上添加@SuppressWarnings("unused")注解来抑制警告，但这只是掩盖问题而非根本解决，不建议长期使用。

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