访问/分配列表中结构中的值的问题 - C ++

时间:2010-02-06 07:58:37

标签: c++ pointers struct

我正在尝试创建一个Trie数据结构(学校工作),我正在使用一个我自己也做过的列表并且工作正常(经过测试)将N节点存储在Trie中。

无论如何,问题是,每个节点都应该存储一个节点列表,这样我就可以制作我的N-ary树/特里但我遇到了麻烦...

当我调试并通过 for 循环时,我可以看到currentNode正在深入到我的树中。但是,当我从根的角度来看它时,我的根只有一个链表,其中包含在循环的第一次迭代中生成的第一个节点。连续迭代不在ROOT NODE的BRANCH中的NODE中注册...但是它在currentNode中工作,就好像它们是单独的副本一样,即使currentNode是指向正确(我希望)节点的指针。 / p>

我的代码有问题吗?!?我误解指针是如何工作的?救命!感谢。

我的节点结构。

struct Node {
    char         letter;
    ItemType     item;
    List<Node> * branches;
};

Node * root;
int    size;

我的推送功能。

ItemType put(KeyType newKey, ItemType newItem) {
    Node * currentNode = root;
    if (isEmpty()) {

        //So build a new key
        for (int levelIndex = 1; ((int) (newKey.length()) - levelIndex) >= 0; ++levelIndex) {
            currentNode->branches = new List<Node>;
            Node * tempNode = new Node;
            tempNode->letter = newKey.at(levelIndex - 1);
            currentNode->branches->add(*tempNode);
            currentNode = tempNode;
        }
        //Store
        currentNode->item = newItem;
        ++size;
        return NULL; //The former item.

    } else {
        //Begin
        return puttanesca(newKey, newItem, *(currentNode->branches), 1, 1);
    }
}

编辑:哦,谢天谢地,我忘了说puttanesca是一个递归函数,我用来遍历和放置节点和东西。但是我还没有真正测试过,我只是试图将第一个键添加到我的空Trie中,因为这个问题......

更多编辑:

这是puttanesca,我认为这与问题无关但是......无论如何都是这样。

我正在将Node结构中的List指针更改为一个对象,所以这些东西可能看起来有些错误,或者它可能只是错误,因为我对C ++和我不是很好这里仍然有问题,但可以看到一般的概念/算法......哦,我正在使用 typedef string KeyType 作为我的密钥,仅用于未来的验证和模板 for ItemType

ItemType puttanesca(KeyType newKey, ItemType newItem, List<Node> & tempList, int listIndex, int levelIndex) {
    Node currentNode = tempList.get(listIndex);

    //Am I at the right node? (searching)
    if (newKey.at(levelIndex - 1) == currentNode.letter) { //Yes, I am.
        //Is this a leaf node?
        if (currentNode.branches == NULL) {

            //Key does not already exist
            if (newKey.length() != levelIndex) {
                //So build a new key
                for (; ((int) (newKey.length()) - levelIndex) >= 0; ++levelIndex) {
                    currentNode.branches = new List<Node>;
                    Node * tempNode = new Node;
                    tempNode->letter = newKey.at(levelIndex - 1);
                    currentNode.branches.add(*tempNode);
                    currentNode = *tempNode;
                }
                //Store
                currentNode.item = newItem;
                ++size;
                return NULL; //The former item.

            } else { //Key matched!
                //Replace with new item
                ItemType currentItem = currentNode.item;
                currentNode.item = newItem;
                return currentItem; //which is actually the now old item after the previous statement
            }
        } else { //Not a leaf, keep going
            //Go to the next level and start from the first index
            ItemType currentItem = puttanesca(newKey, newItem, currentNode.branches, 1, levelIndex + 1);
            if (currentItem == NULL) {
                //Key found to be inexistant
                //So build a new key - create new sibling
                Node * tempNode = new Node;
                tempNode->letter = newKey.at(levelIndex - 1);
                currentNode.branches.add(*tempNode);
                currentNode = *tempNode;

                //Continue building key - extend sibling
                for (++levelIndex; ((int) (newKey.length()) - levelIndex) >= 0; ++levelIndex) {
                    currentNode.branches = new List<Node>;
                    Node * tempNode = new Node;
                    tempNode->letter = newKey.at(levelIndex - 1);
                    currentNode.branches.add(*tempNode);
                    currentNode = *tempNode;
                }
                //Store
                currentNode.item = newItem;
                ++size;
                return NULL; //The former item

            } else {
                return currentItem; //The former item;
            }
        }
    } else { //Wrong node
        if (tempList.getLength() > listIndex) {
            return puttanesca(newKey, newItem, tempList, ++listIndex, levelIndex);

        } else {//End of the line, chump
            return NULL; //Tell parent you failed
        }
    }
}

1 个答案:

答案 0 :(得分:1)

您的问题出在此处: currentNode->branches->add(*tempNode);

您正在插入tempNode的副本,而不是tempNode本身。您可能需要考虑使用List<Node *> branches而不是List<Node> branches;