Sonarqube 声纳问题分配不应重复（squid:S4165）

由于连接的复杂性，我制作了一些函数。上面的一个只是示例，代码更大

删除这个无用的任务；“emp”已沿所有执行路径保存指定的值

在updateEmployee（）的第2行

声纳问题详情

private void updateEmployee(String employeeCode, UpdateUserDto updateUserDto){
    Employee emp = getEmpDetails(employeeCode);
    emp = updateMobile&Email(emp, updateUserDto.getMobile(), updateUserDto.getMail());
    // Remove this useless assignment; "emp" already holds the assigned value along all execution paths.

    ...
    ...
    emp.setisActive(updateUserDto.getIsActive());
    empRepo.save(emp);
}

private Employee updateMobile&Email(Employee emp, String mobile, String mail){
    if(emp.getMobile() == null && (mobile != null || mobile.isBlank())){
        emp.setMobile(mobile);
    }
    if(emp.getMail() == null && (getMail != null || getMail.isBlank())){
        emp.setMail(mail);
    }
    return emp;
}

---

emp参数将通过引用传递到updateMobile&Email方法中，这意味着不需要第2行赋值

---

我建议更新updateMobile&Email，使其具有无效返回类型。

emp参数将通过引用传递到updateMobile&Email方法中，这意味着不需要第2行赋值

我建议更新updateMobile&Email，使其具有无效返回类型

Assignments should not be redundant (squid:S4165)

Noncompliant
a = b;
c = a;
b = c; // Noncompliant: c and b are already the same

Compilant
a = b;
c = a;