指令分配一个值，但该值不会在任何后续指令中读取或使用

Question

我有一段代码

for(int i = 0; i < num_of_random; i++){

    String str = in.readLine();
    if(str != null){
        String[] randoms = new String[4];
        randoms = str.split(",");

        dateRanges[i] = Integer.parseInt(randoms[0]);
        id[i] = Integer.parseInt(randoms[1]);
        flag[i] = Integer.parseInt(randoms[2]);
        system[i] = Integer.parseInt(randoms[3]);
    }
}

的建议

当我针对 findBugs 运行这段代码时，我得到了“String[] randoms = new String[4];”

该指令将一个值分配给局部变量，但该值不会在任何后续指令中读取或使用。通常，这表示错误，因为计算出的值从未被使用过。

为什么我会得到这个？

多谢

Answer 1

因为您将变量初始化为一个值 (new String[4])，然后将变量值替换为另一个值（str.split(",") 的结果>) 就在之后。因此初始化是不必要的。

您的代码在功能上等同于，

String[] randoms = str.split(",");

只不过它分配了一个新的 String 数组，该数组立即被丢弃。

Answer 2

因为你可以简单地这样做：

String[] randoms = str.split(",");

Answer 3

直接尝试：

String[] randoms = str.split(",");

您不需要实例化 String[]，split 方法已经做到了。

Answer 4

这是我称之为“我很笨”技术的一个例子。

通常，这是程序员编写代码但不理解他们在做什么的结果。在某些时候，程序员读到或听到这样的话：“你必须初始化所有局部变量！”。当他们编写代码时，String[] randoms 突然出现在他们的脑海中，因此他们添加了 = new String[4]。

经验丰富的程序员可能会认为我曾经很愚蠢，但现在不再了！让我们将变量声明移出循环并生成如下内容：

String str;
String[] randoms;
for(int index = 0; index < num_of_random; ++index)
{
    str = in.readLine();
    if (str != null)
    {
        randoms = str.split(",");
        dateRanges[index] = Integer.parseInt(randoms[0]);
        id[index] = Integer.parseInt(randoms[1]);
        flag[index] = Integer.parseInt(randoms[2]);
        system[index] = Integer.parseInt(randoms[3]);
    }
}