Cucumber行内数据表转换迭代与Java 8 Lambda版Step Definition优化问题

原代码存在的问题 #

• 变量命名冲突：方法入参已命名为reportData，方法内重复声明同名字符串变量，触发编译错误
• 语法结构错误：forEach的lambda闭包括号匹配错误，sendReportGrades调用被放置在闭包右括号外侧，因此IDE抛出「Unexpected token」错误
• 逻辑不符合需求：需要为每一条成绩单独调用上报接口，原写法即使修正语法也只会上报最后一条成绩
• 方法调用错误：String.join第一个参数为分隔符，原写法参数顺序错误，无法生成符合要求的科目:成绩格式
• 变量未声明：report变量没有提前声明，直接赋值会触发编译错误

最终优化代码 #

@When("they have submitted the following grades")
public void theyHaveSubmittedTheFollowingGrades(Map<String, String> reportData) {
    reportData.forEach((subject, grade) -> {
        String report = String.join(":", subject, grade);
        sendReportGrades(report);
    });
}

方案说明 #

• 直接在lambda遍历的单次循环内完成单条成绩的拼接与上报，不需要额外声明外部变量，避免了lambda作用域变量修改限制问题
• 修正了String.join的参数顺序，拼接结果完全符合接口要求的格式
• 每遍历一条成绩就调用一次上报接口，满足所有成绩条目都需要上报的需求
• 移除了冗余的冲突变量声明，修正了括号匹配问题，无语法错误

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