Class has no member named function

[biamigueis]

Hi guys,
This error is really annoying me but its late and i've been programming all day so I might be missing something stupid.
I keep getting an error saying ui.h:30: error: 'class BTree<Word>' has no member named 'prntInOrder'

here is my ui.h

#pragma once

#include "btree.h"

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

class UI
{
	public:
		void go (string filename);
		void help ();
	private:
		BTree<Word> tree;
		
		bool load(string filename);
};

ui.cpp

#include "ui.h"

void UI::help()
{
	cout << "::Binary Search Tree::" << endl;
	cout << endl;
	cout << "Usage: Tree <filename>" << endl;
}

void UI::go(string filename)
{
	if (!load(filename))
		return;
	tree.printInOrder();
	
}

and part of my btree.h

#pragma once

#include "node.h"

#include <iostream>
using namespace std;

template<class T>
class BTree
{
	public:
		
		BTree (): root(NULL){};
		~BTree ()
		{
			clear(root);
		}
		void printInOrder()
		{
			count = 1;
			printInOrder(root);
		}
		
	private:
		
		Node<T>* root;
		
		void printInOrder(Node<T>* node)
		{
			printInOrder(node->getLeft());
			cout << node->getData() << endl;
			printInOrder(node->getRight());
		}
};

As you can see the printInOrder() function is there so would it not see it?

[Disch]

You are calling:
tree.printInOrder(); (no parameters)

Your function is:
void printInOrder(Node<T>* node) (1 parameter)


You need to give it a parameter.

EDIT: crap, just saw the other overload, nevermind...

[vlad from moscow]

You should show the whole error message and the place in your code where the function is called.

[biamigueis]

ui.h: In member function 'void UI::go(std::string)':
ui.h:30: error: 'class BTree<Word>' has no member named 'printInOrder'

ui.h:30 doesnt exist but the place that im calling printInOrder is line 14 in ui.cpp

[vlad from moscow]

What is tree in this code?

void UI::go(string filename)
{
if (!load(filename))
return;
tree.printInOrder();

}

[biamigueis]

BTree<Word> tree;
as shown in line 15 of ui.h

[vlad from moscow]

I do not see the definition of tree. So your presented code is not adequate to the proiblem.

[JLBorges]

> ui.h: In member function 'void UI::go(std::string)':
> ui.h:30: error: 'class BTree<Word>' has no member named 'printInOrder'
> ui.h:30 doesnt exist ...

Do uou have more than one file ui.h?

Are you using an IDE of some kind, and compiling the code from within the IDE?

If so, perhaps the files that you think are being compiled are not the files that are actually compiled.

[biamigueis]

Im compiling with g++ on the console using a makefile and I only have 1 ui.h

Makefile

Tree.exe: word.o ui.o CinReader.o node.h btree.h driver.cpp
	g++ ui.o word.o CinReader.o driver.cpp -o Tree.exe
	
word.o: word.h word.cpp
	g++ -c word.cpp

ui.o: ui.h ui.cpp
	g++ -c ui.cpp 
	
CinReader.o: CinReader.h CinReader.cpp
	g++ -c CinReader.cpp

[3TXyhbRD]

The line 30 (which doesn't exist) may come from your template class, the compiler generates the class after the template and in this case the generated class is probably placed into your ui.h file (since there is where you use the template, line 15). Have you tested the template separately from ui.h?
Can we also see your node specification?

[biamigueis]

node.h

/*
 * Node 
 * Binary tree node
 * 
 * Author: Bianca Migueis
 * Created: 05/07/12
 * Edited: 5/18/12
 *
 */
#pragma once

#include "word.h"

template<class T>
class Node
{
	public:
		/**
		 * Overloaded constructor
		 * sets the data to newData and left and right children to NULL
		 * @param T* new data to create the object
		 */
		Node<T> (T* newData):data(newData), leftChild(NULL), rightChild(NULL){}
		
		/**
		 * Gets the data
		 * @returns T* pointer to data inside the object
		 */
		T* getData ()
		{
			return data;
		}
		
		/**
		 * Set left child
		 * @param Node* new left node
		 */
		void setLeft (Node* newLeft)
		{
			leftChild = newLeft;
		}
		
		/**
		 * Set right child
		 * @param Node* new right node
		 */
		void setRight (Node* newRight)
		{
			rightChild = newRight;
		}
		
		/**
		 * Get Left child
		 * @returns Node<T>*& the left child
		 */
		Node<T>* getLeft () const
		{
			return leftChild;
		}
		
		/**
		 * Get Left child
		 * @returns Node<T>*& the left child
		 */
		Node<T>*& getLeft ()
		{
			return leftChild;
		}
		
		/**
		 * Get Right child
		 * @returns Node<T>*& the right child
		 */
		Node<T>* getRight () const
		{
			return rightChild;
		}
		
		/**
		 * Get Right child
		 * @returns Node<T>*& the right child
		 */
		Node<T>*& getRight ()
		{
			return rightChild;
		}
		
		/**
		 * Overloaded operator >
		 * @returns true if this->data is bigger than src.data, false if not
		 */
		bool operator> (const Node<T>& src)
		{
			return (data > src.data);
		}
		
		/**
		 * Overloaded operator <
		 * @returns true if this->data is smaller than src.data, false if not
		 */
		bool operator< (const Node<T>& src)
		{
			return (data < src.data);
		}
		
		/**
		 * Overloaded operator ==
		 * @returns true if this->data is equal to src.data, false if not
		 */
		bool operator== (const Node<T> src)
		{
			return (data == src.data);
		}
		
		/**
		 * Overloaded operator <<
		 * @returns ostream& formatted to print the node contents
		 */
		friend ostream& operator<< (ostream& outs, const Node<T>& src) 
		{
			outs << src.data;
			
			return outs;
		};
		
	private:
		
		T* data;
		Node* leftChild;
		Node* rightChild;
};

[3TXyhbRD]

btree.h line 13. I don't think that's the issue, however there is a ; after the { } of the constructor.