Question

我收到了此代码的死代码警告：

Topic topic = findTopicByID(point.getIDTopic());

if (topic != null)
{
   ...
}
else if (topic == null)
{
    // I get Dead code warning for every thing I write here!
}

有时当我收到警告但一切看起来都没问题时，我重新启动IDE并且我不再收到警告了！但这次......

修改

public Topic findTopicByID(int IDTopic) {
    for (Topic topic : topics)
        if (topic.getID() == IDTopic)
            return topic;
    return null;
}

编辑：完整代码：

Topic topic = Res.getSchedule().getTopicBox().findTopicByID(point.getIDTopic());
            Section section =     topic.findSectionByID(point.getIDSection());

            if (topic != null)
            {
                View rowView = inflater.inflate(R.layout.daily_day_day_row, lLDoneWorks, false);

                TextView tVLabel = (TextView) rowView.findViewById(R.id.daily_day_day_row_label);
                TextView tVNum = (TextView) rowView.findViewById(R.id.daily_day_day_row_num);
                TextView tVTopic = (TextView) rowView.findViewById(R.id.daily_day_day_row_topic);
                TextView tVSection = (TextView) rowView.findViewById(R.id.daily_day_day_row_section);

                int color = topic.getColor();
                tVLabel.setBackgroundColor(color);
                tVNum.setTextColor(color);
                tVTopic.setTextColor(color);
                tVSection.setTextColor(color);

                tVLabel.setText(topic.getName().substring(0,1).toUpperCase(Locale.US));
                tVNum.setText(Integer.toString(point.getCount()));
                tVTopic.setText(topic.getName());
                if (point.getIDSection() != Point.DEFAULT_SECTION)
                    tVSection.setText(section.getName());

                lLDoneWorks.addView(rowView);
            }
            else if (topic == null)
            {
                TopicArchived archivedTopic = Res.getSchedule().getTopicBox()
                            .findArchivedTopicByID(point.getIDTopic());
                    if (archivedTopic == null)
                        removedTopicsPoint += point.getCount();
                    else
                        archivedTopicsPoint += point.getCount();
            }

Answer 1

你可以编码

if (topic != null)
{
   ...
}
else
{
   ...
}

代替。

Answer 2

编译器可能误解了条件的目标。将其改写为：

if (topic != null) {
   ...
}
else {
    // Code
}

在else条件中，topic在逻辑上保证为null。如果它不为空，则会if阻止，else不会。

Answer 3

查看您的代码。

if(topic != null) {
} else { // here topic definitely not null, so the your if is redundant.
}

Answer 4

我认为你所拥有的是过度杀伤力。为什么不呢：

if (topic != null)
{
    ...
}
else
{
    // It must be null if it gets here
}

Answer 5

这是因为这一行：

Section section =     topic.findSectionByID(point.getIDSection());

如果topic为null，则表示存在NullPointerException，并且未到达其余代码。因此，对topic之后出现的空值的每次检查都是无关紧要的：编译器知道topic在该行之后不为空。

您应该将该行放在if语句的第一个分支中。

尝试编写if条件时，我收到死代码警告

5 个答案: