java - 安卓工作室说 'Local variable is redundant'

<分区>

我在许多方法上收到警告，说局部变量是多余的。

这是一个示例方法:

public MyObject getMSListItem(int pos) {
    MyObject li = getItem(pos);
    return li;
}

现在看来，我想，我可以这样做来修复它:

public MyObject getMSListItem(int pos) {
    return  getItem(pos);
}

另一个例子:

public String getTeacher(int pos) {
    ffTeacherListItem t = getItem(pos);
    String teacher = t.teacher;
    return teacher;
}

似乎这可能是:

public String getTeacher(int pos) {
    ffTeacherListItem t = getItem(pos);
    return t.teacher;
}

或者按照下面的建议，甚至更好!

public String getTeacher(int pos) {
    return  getItem(pos).teacher;
}

真的有这方面的“最佳实践”吗？一种方法比另一种更好吗？还是仅与代码可读性有关？

最佳答案

Is there really a "best practice" for this? Is one way better than the other? Or is it just about code readability and nothing more?

简体说:在你的场景中没用。这没有错，但你为什么要这样做:

ffTeacherListItem t = getItem(pos);
String teacher = t.teacher;
return teacher;

当你可以做同样的事情时:

ffTeacherListItem t = getItem(pos);
return t.teacher;

或者你也可以这样做:

return getItem(pos).teacher;

以上所有内容都相同，但第二个和第三个代码更清晰，您应该始终尝试编写没有无用行和引用的干净代码¹。还有一条不成文的规定 - 代码越少，错误越少。

¹_{这是像 C++ 这样没有垃圾收集器的语言的“优势”，你要对你创建的所有对象和实例负责(它们从内存中释放等)。 ).因此，在决定创建某个对象的新实例之前，您需要考虑更多。}

关于java - 安卓工作室说 'Local variable is redundant'，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/23159351/