3

我有一个练习，我必须在链表中插入一个字符串。假设字符串如下：

"Java Coding Is Great"

合并排序之后，链表应该是这样的：

coding >>> great >>> is >>> java.

问题是在我的合并排序代码中，我收到以下内容：

 great >> is >> java >> coding

所有单词都已排序，但第一个单词（原始列表的开头）不是。

我有两个类：TextList 和 WordNode。

WordNode 类有两个属性：

String _word; WordNode _next; //an address to the next link

TextList 类只有一个属性：链表头的地址：

WordNode _head;

我有一个构造函数，我在其中将字符串随机插入到链表中。最后它开始合并排序列表。该算法适用于本练习。

public TextList(String text){
     String s=""; int index=text.length();
    //we will stop at the end of the String.
    for (int i=text.length()-1; i>=0; i--){
        //if we reached a space, insert each string in appropriate order, 
        //the first word is the head of the string and last word points to null.
        if (!(text.charAt(i)>='a' && text.charAt(i)<='z')){ 
            s=text.substring(i,index);
            _head=new WordNode(s,_head);
            s="";
            index=i;
        }
        if (i==1){
            s=text.substring(i-1,index);
            _head=new WordNode(s,_head);
        }
    }

//start merge sotring the list.
    this._head=this._head.mergeSort();
}

合并排序方法：mergeSort、merge 和 split：（这些在 WordNode 类中）：

合并排序方法 

public WordNode mergeSort(){
    return mergeSort(this);
}
private WordNode mergeSort(WordNode h){
    // Sort h by recursively splitting and merging
    if (h==null || h._next==null)
        return h;
    else{
        WordNode evens=h.splitOdds();
        WordNode odds=h.splitEvens();
        return mergeSort(odds).merge(mergeSort(evens)); 
    }
}

合并方法

private WordNode merge(WordNode h){
        //method merges this's list with h's list

        //if h is null, just return this.
        if (h==null){
            return this;
        }
        if (this._word.compareTo(h._word)<0){
            if (this._next==null)
                return new WordNode(this._word,h);
            else
                return new WordNode(this._word,this._next.merge(h));
        }
        else
            return new WordNode (h._word, merge(h._next));

    }

拆分方法：一种用于偶数位置，一种用于奇数位置。 

private WordNode splitOdds(){
    boolean flag=true;
    WordNode odds=null;
    WordNode ptr=this;
    while (ptr!=null){  
        if(flag)
        odds=new WordNode(ptr._word,odds);
        ptr=ptr.getNext();
        flag=!flag;
    }
    return odds;
}
//MUST BE INITILIZED ON HEAD
    private WordNode splitEvens(){
        boolean flag=true;
        WordNode evens=null;
        WordNode ptr=this._next;
        while (ptr!=null){
            if (flag)
                evens=new WordNode(ptr._word,evens);
                ptr=ptr.getNext();
                flag=!flag;
            }



        return evens;
    }

请帮我找出问题所在。不幸的是，我不能使用第三类，也不能使用指向列表开头或结尾的指针。

4

3 回答 3

1

这里的问题有点好笑。

在我的构造函数中，我在列表中插入的每个单词都带有一个空格。

我通过这段代码解决了这个问题：

            s=text.substring(i+1,index);

代替：

            s=text.substring(i,index);

归功于 DevForum 的 NormR 的答案。

于 2013-06-08T18:29:29.610 回答
1

您可以使用调试器单步执行您的代码吗？这将帮助您查明问题。即使是一些明智放置的断点也会有所帮助。

从只包含一个条目的列表开始：“Java”。走着瞧吧。

然后尝试一个包含两个条目的列表：“Java Coding”。看看在这种情况下会发生什么。

弄清楚在简单的情况下发生了什么，然后再处理更复杂的情况。

于 2013-06-08T14:34:16.020 回答
0

很好，但我不喜欢你在 WordNode 中的 merge() 解决方案。一方面，您可以比较您喜欢的每个节点：

this._word.compareTo(h._word);

但在那种情况下，merge() 和 split() 都是私有的，所以我认为更好的方法是将它们放入 TextList 和 mergeSort() 而不重载。每当您添加一个节点时，您都​​需要对链表中的所有节点进行排序，而不仅仅是它的一部分。这就是为什么

this._head = this._head.mergeSort();

和这个

public WordNode mergeSort(){
    return mergeSort(this);
}

在 WordNode 中看起来没用。
另一方面，如果您像这样将mergeSort调用到TextList中

this._head = this.mergeSort(this._head);

并在 TextList 中合并排序到这个

 public WordNode mergeSort(WordNode n){

 }

它已经更好了，但是您可以通过减少将列表拆分为这样的赔率和偶数的方式的更多时间和空间复杂性来做得更好

int counter = 1;    //  nodes counter helps to know if the current node is odd or even
WordNode L = null,  //  odd nodes
         R = null;  //  even nodes
while(h != null)
{
    if(counter%2 == 0)
        R = new WordNode(h.getWord(), R, h.getWordCounter());
    else
        L = new WordNode(h.getWord(), L, h.getWordCounter());
    //  
    h = h.getNext();
    counter++;
}

当你把它放在一起时，你会得到这样的东西（不要忘记单词计数器）

public WordNode mergeSort(WordNode h){
    int counter = 1;    //  nodes counter helps to know if the current node is odd or even
    WordNode L = null,  //  odd nodes
             R = null;  //  even nodes
    while(h != null)
    {
        if(counter%2 == 0)
            R = new WordNode(h.getWord(), R, h.getWordCounter());
        else
            L = new WordNode(h.getWord(), L, h.getWordCounter());
        //  
        h = h.getNext();
        counter++;
    }
    return merge(mergeSort(L), (mergeSort(R)));
}

现在剩下的就是像这样完成 merge() 并且再次不要忘记单词计数器

private WordNode merge(WordNode L, WordNode R)
{   
    while(L != null || R != null)
    {
        if(L != null && R != null)
            if(L.getWord().compareTo(R.getWord()) <= 0)
                return new WordNode(L.getWord(), merge(L.getNext(), R), L.getWordCounter());
            else
                return new WordNode(R.getWord(), merge(L, R.getNext()), R.getWordCounter());
        else if(L != null)
            return new WordNode(L.getWord(), merge(L.getNext(), R), L.getWordCounter());
        else if(R != null)
            return new WordNode(R.getWord(), merge(L, R.getNext()), R.getWordCounter());
    }
    return null;
}

向罗森斯坦教授问好 ;-)

于 2013-06-18T09:47:31.627 回答