我需要编写一个以单个链接的整数列表和一个称为拆分值的特殊值开头的方法。列表的元素没有特定的顺序。该方法将节点划分为两个链接列表:一个包含所有包含小于拆分值的元素的节点,另一个包含所有其他节点。如果原始链表具有任何重复的整数(即,其中具有相同元素的任何两个或更多个节点),则具有该元素的新链表应具有重复该元素的相同数量的节点。该方法返回两个头引用 - 每个引用创建一个链接列表。
我已经花了无数个小时试图做到这一点,并认为这是最接近但我在编译时可能没有初始化我的copyTail * IntNodes时出错。我的代码也可能完全错了.... 任何帮助指向我正确的方向??
public static IntNode[ ] listSplitLessGreater(IntNode source, int splitter)
{
IntNode copyHeadLess;
IntNode copyTailLess;
IntNode copyHeadGreater;
IntNode copyTailGreater;
IntNode[ ] answer = new IntNode[2];
boolean less = true;
boolean greater = true;
// Handle the special case of the empty list.
if (source == null)
return answer; // The answer has two null references .
//Split list into two lists less and greater/equal than splitter.
while (source.link != null)
{
if (splitter < source.data)
{
if (less)
{
copyHeadLess = new IntNode(source.data, null);
copyTailLess = copyHeadLess;
less=false;
}
else
{
source = source.link;
copyTailLess.addNodeAfter(source.data);
copyTailLess = copyTailLess.link;
}
}
else
{
if (greater)
{
copyHeadGreater = new IntNode(source.data, null);
copyTailGreater = copyHeadGreater;
greater=false;
}
else
{
source = source.link;
copyTailGreater.addNodeAfter(source.data);
copyTailGreater = copyTailGreater.link;
}
}
}
//Return Head References
answer[0] = copyHeadLess;
answer[1] = copyHeadGreater;
return answer;
}
答案 0 :(得分:3)
我认为通过使用单个类(IntNode
)建模列表,使其变得比它需要的更复杂。如果您将其建模为“列表”和“列表中的节点”,那么很容易有一个空列表。你也不需要跟踪头部和尾部 - 列表可以做到这一点。那时,它非常简单:
请注意,即使没有它,只需使用null
表示“我还没有这个列表”,您就可以简化代码。目前您的代码无法编译,因为copyHeadLess
等在使用时未明确分配。 您知道在分配之前不会尝试使用它们,但编译器不会。我仍然建议重塑方法:)
答案 1 :(得分:2)
如果source不为null,但source.link为null(list只由一个元素组成),那么您永远不会为copyHeadLess等分配变量。尝试将它们初始化为null或默认值为:
IntNode copyHeadLess = null;
IntNode copyTailLess = null;
IntNode copyHeadGreater = null;
IntNode copyTailGreater = null;
IntNode[ ] answer = new IntNode[2];
boolean less = true;
boolean greater = true;
// Handle the special case of the empty list.
if (source == null)
return answer; // The answer has two null references .
//Split list into two lists less and greater/equal than splitter.
while (source.link != null)
{
// what about case where source isn't null but source.link is null?
}
//Return Head References
answer[0] = copyHeadLess; // this may have never been assigned in your original code
answer[1] = copyHeadGreater; // this may have never been assigned in your original code
return answer;
}