einfügen auf den ersten BST geben Ende ohne Blasenfunktion errror?

Question

einfügen auf den ersten BST geben Ende ohne Blasenfunktion errror?

stimmen

1

So bin ich versucht zu lernen, wie mein erstes BST codieren, und es schwer ist .... Ich bin schon Probleme, mit nur wenigen Zeilen Code. das Problem ist im Einsatz, aber ich habe alles enthalten, so dass ich ein Feedback auf meine Art / andere Fehler zu bekommen. Ich habe einen Zeiger auf Zeiger-Implementierung verwenden vorgeschlagen, aber wir haben es noch gelernt, so dass ich nicht das Gefühl, Komfort / wissen, wie es noch zu codieren. das

Fehler

cc1plus: warnings being treated as errors
tree.cpp: In member function âbool Tree::insert(Tree::Node*&, int, std::string)â:
tree.cpp:34: warning: control reaches end of non-void function

die tree.h Datei

#ifndef TREE_H
#define TREE_H

#include <iostream>
#include <string>
using namespace std;

class Tree
{
 public:
  Tree();
  bool insert(int k, string s);

 private:
  struct Node
  {
    int key;
    string data;
    Node* left;
    Node* right;
  };
  Node* root;
  bool insert(Node*& root, int k, string s);
};

#endif

tree.cpp

#include <iostream>
#include tree.h
#include <stack>
#include <queue>
#include <string>
using namespace std;

Tree::Tree()
{
  root = NULL;
}

bool Tree::insert(int k, string s)
{
  return insert(root, k, s);
}

bool Tree::insert(Node*& currentRoot, int k, string s)
{
  if(currentRoot == NULL){
    currentRoot = new Node;
    currentRoot->key = k;
    currentRoot->data = s;
    currentRoot->left = NULL;
    currentRoot->right = NULL;
    return true;
  }
  else if (currentRoot->key == k)
    return false;
  else if (currentRoot->key > k)
    insert(currentRoot->left, k, s);
  else
    insert (currentRoot->right,k, s);
}

movieList.cpp

#include <iostream>
#include <stack>
#include <queue>
#include <string>
#include tree.h


using namespace std;

int main()
{
  Tree test;
  test.insert(100, blah);
  return 0;
}

c++linux g++binary-search-tree

Veröffentlicht am 27/04/2011 um 05:26
quelle vom benutzer kingcong3

In anderen Sprachen...

2 antworten

stimmen

1

Wie wäre es mit:

bool Tree::insert(Node*& currentRoot, int k, string s)
{
  if(currentRoot == NULL){
    currentRoot = new Node;
    currentRoot->key = k;
    currentRoot->data = s;
    currentRoot->left = NULL;
    currentRoot->right = NULL;
    return true;
  }
  else if (currentRoot->key == k)
    return false;
  else if (currentRoot->key > k)
    insert(currentRoot->left, k, s);
  else
    insert (currentRoot->right,k, s);
  return true;
}

Alle Branchen brauchen einen Wert (boolean in diesem Fall) zurückzukehren.

Beantwortet am 27/04/2011 um 05:29
quelle vom benutzer Pete

Xeo · Accepted Answer · 2011-04-27 05:29

cc1plus: warnings being treated as errors
tree.cpp: In member function âbool Tree::insert(Tree::Node*&, int, std::string)â:
tree.cpp:34: warning: control reaches end of non-void function

Das sagt nur, dass Sie nicht etwas auf jedem möglichen Weg zurück. Versuche dies:

bool Tree::insert(Node*& currentRoot, int k, string s)
{
  if(currentRoot == NULL){
    currentRoot = new Node;
    currentRoot->key = k;
    currentRoot->data = s;
    currentRoot->left = NULL;
    currentRoot->right = NULL;
    return true;
  }
  else if (currentRoot->key == k)
    return false;
  else if (currentRoot->key > k)
    return insert(currentRoot->left, k, s);
//  ^^^^^^
  else
    return insert (currentRoot->right,k, s);
//  ^^^^^^
}