2014-12-03 52 views
1

我在执行我的跳过列表时出现此问题,当我在int main()中出现return 0;时,出现堆损坏。这是我调试到的地方,直到它崩溃。我的跳过列表中存在堆损坏

Debug error

enter image description here

这是我的代码:

skipnode.h

template <typename T> 
class SkipNode 
{ 
public: 
    T data; 
    SkipNode<T> **next; 
    SkipNode(T d, int level); 
    ~SkipNode(); 
}; 

skipnode.cpp

#include "skipnode.h" 

template<typename T> 
SkipNode<T>::SkipNode(T d, int level) 
{ 
    data = d; 
    next = new SkipNode<T>*[level]; 

    for (int i = 0; i < level; i++) 
     next[i] = 0; 
} 

template<typename T> 
SkipNode<T>::~SkipNode() 
{ 
    delete [] next; 
} 

skiplist.h

#include "skipnode.cpp" 

#define MAXLEVEL 4 

template<typename T> 
class SkipList 
{ 
public: 
    SkipList(); 
    ~SkipList(); 
    int randLvl(int max); 
    T search(T); 
    void insert(T); 
private: 
    SkipNode<T> *root; 
}; 

skiplist.cpp

#include "skiplist.h" 
#include <stdlib.h> 
#include <time.h> 

template<typename T> 
SkipList<T>::SkipList() 
{ 
    root = new SkipNode<T>(0,MAXLEVEL); 
} 

template<typename T> 
SkipList<T>::~SkipList() 
{ 
    delete root; 
} 

template<typename T> 
int SkipList<T>::randLvl(int max) 
{ 
    srand(time(NULL)); 
    return rand() % max + 1; //+ 1, så værdien ikke bliver 0 
} 

template<typename T> 
void SkipList<T>::insert(T value) 
{ 
    int level = randLvl(MAXLEVEL); 

    SkipNode<T> *insertNode = new SkipNode<T>(value,level); 
    SkipNode<T> *currentNode = root; 

    for (int i = level; i > 0; i--) 
    { 
     for (; currentNode->next[i] != 0; currentNode = currentNode->next[i]) 
     { 
      if (currentNode->next[i]->data > value) 
       break; 
     } 
     if (i <= level) 
     { 
      insertNode->next[i] = currentNode->next[i]; 
      currentNode->next[i] = insertNode; 
     } 
    } 
} 

的main.cpp

#include "skiplist.cpp" 

int main() 
{ 
    SkipList<int> SList; 
    SList.insert(3); 
    return 0; 
} 

我有一个理论,该错误可能会在此行中skiplist.cpp来发生的事情:

if (currentNode->next[i]->data > value) 

因为它似乎就像它不能访问next-> data,但我不知道为什么。

任何人都可以帮我吗?对不起,如果我问这个问题是错误的,我对堆栈溢出是一个新东西。提前致谢!

+1

不能做的崩溃,而是不断播种用的时候你的RNG将意味着它一遍又一遍地返回相同的数,直到时间的变化,这可能不是你的意图。 – 2014-12-03 15:59:49

+0

嗯,它会产生不同的输出。也许我不太明白你的意思? – Thisen 2014-12-03 16:03:04

+1

你确定吗? 'time'返回一个'time_t',在大多数平台上是自纪元以来的秒数。你正在播种一个伪随机数发生器,所以它应该返回相同的数字,直到时钟改变到下一秒。 – 2014-12-03 16:16:28

回答

3

你在insert(T)的for循环中有off-by-one error,它可能应该是

for (int i = level-1; i >= 0; i--)

而且插入()泄漏内存,删除insertNode