0

我的代码有问题。我已经运行和调试了好几次。如果我不在 getEntry 函数中抛出异常,它似乎工作正常。但是当我确实抛出异常时,我的程序在那之后出现了分段错误。当我通过程序调试时,似乎getEntryHelper中的nextNodePtr不是0x0。所以它在抛出异常后会以某种方式改变,我不知道为什么。

我的主要:

#include <iostream>
#include "BinarySearchTree.h"

int main {
BinarySearchTree<std::string,std::string> myTree;
    myTree.add("book");
        myTree.add("encyclopedia");
    myTree.add("automobile");
    myTree.add("zebra");
        myTree.getEntry(zebra);
        myTree.getEntry(xylophone);
        myTree.getEntry(tree); // Does not get to here

}

这是我的 add an 和 getEntry 方法(尽管看起来我的 getEntry 是问题所在:

template<typename KeyType, typename ItemType>
void BinarySearchTree<KeyType,ItemType>::add(const ItemType& newEntry) {
    if(rootPtr == NULL)
        rootPtr = new BinaryNode<ItemType>(newEntry);
    else {
        addHelper(rootPtr,rootPtr,newEntry);
    }
}

template<typename KeyType, typename ItemType>
ItemType BinarySearchTree<KeyType,ItemType>::getEntry(const KeyType& aKey) const
        throw(NotFoundException) {
    try {
        BinaryNode<ItemType>* temp = getEntryHelper(rootPtr,aKey);
        std::cout << temp->getItem() << "\n";
        return temp->getItem();
    }
    catch(NotFoundException& nf) {
        std::cout << nf.what();
    }
}

template<typename KeyType, typename ItemType>
void BinarySearchTree<KeyType,ItemType>::addHelper(BinaryNode<ItemType>* prevNodePtr,
                                                    BinaryNode<ItemType>* nextNodePtr,
                                                    const ItemType& newEntry) {

    if(nextNodePtr == NULL) { // Base Case
        nextNodePtr = new BinaryNode<ItemType>(newEntry,NULL,NULL);
        if(newEntry < prevNodePtr->getItem())
            prevNodePtr->setLeftChildPtr(nextNodePtr);
        else
            prevNodePtr->setRightChildPtr(nextNodePtr);
        return;
    }
    if(newEntry < nextNodePtr->getItem()) {
        prevNodePtr = nextNodePtr;
        nextNodePtr = nextNodePtr->getLeftChildPtr();
        addHelper(prevNodePtr,nextNodePtr,newEntry);
    }
    else {
        prevNodePtr = nextNodePtr;
        nextNodePtr = nextNodePtr->getRightChildPtr();
        addHelper(prevNodePtr,nextNodePtr,newEntry);
    }
}

template<typename KeyType, typename ItemType>
BinaryNode<ItemType>* BinarySearchTree<KeyType,ItemType>::getEntryHelper(BinaryNode<ItemType>* nextNodePtr,const KeyType& aKey) const {
    if(nextNodePtr == NULL) {
        throw NotFoundException("does not exist in tree.\n");
    }
    else if(nextNodePtr->getItem() == aKey)
        return nextNodePtr;
    else if(aKey < nextNodePtr->getItem()) {
        getEntryHelper(nextNodePtr->getLeftChildPtr(),aKey);
    }
    else {
        getEntryHelper(nextNodePtr->getRightChildPtr(),aKey);
    }
}

输出:汽车书籍百科全书斑马斑马先决条件违反异常:木琴在树中不存在。分段错误(核心转储)

4

1 回答 1

1

快速浏览一下,我可以看到很少有异常处理不当的问题。

  1. 函数 getEntry() 表示它可以抛出 NotFoundException,但在 main() 中我看不到任何异常处理程序。因此,在 main() 函数中放置一个基本的 try catch 可以处理任何异常。

    int main()
    {
     try
     {
      //some code
     }
     catch(..)
     {
      cout << "Unkown Exception";
     }
     return 0;
    }
    
  2. 函数 getEntry() 表示它可以抛出 NotFoundException,但您有一个 try catch 块,您可以在其中处理异常但从未重新抛出或抛出任何新的\修改的 NotFoundException。如果您不想抛出它,请在函数声明中注释 throw(NotFoundException)。

        ItemType BinarySearchTree<KeyType,ItemType>::getEntry(const KeyType& aKey) //const throw (NotFoundException) -> Comment this
    

如果在处理 NotFoundException 后没有,则重新抛出它。

catch(NotFoundException& nf) {
    std::cout << nf.what();
    rethrow;
}

但我仍然不确定您的二进制代码插入逻辑是否工作正常。如果您遇到任何逻辑问题,请发布整个头文件和 cpp 文件。

于 2013-04-29T18:36:33.767 回答