[killermunk1]

I am working on a program, that does various operations to a sorted list. However there seems to be a problem with one of the functions.

RelationType SortedList::ComparedTo(string otherItem) const
{
	if(str < listData ->info)	
		return LESS;

	else if (otherItem > listData ->info)	
		return GREATER;
	
	else
		return EQUAL;
}

There are no errors here but when I tried to use this function in another function I get an error.

	string SortedList::GetItem(string& item, bool& found)
	{
		bool moreToSearch;
		Node* location;

		location = listData;
		found = false;
		moreToSearch = (location != NULL);

		while (moreToSearch && !found)
		{
			switch(item.ComparedTo(location -> info))
			{
			case GREATER:
				location = location -> next;
				moreToSearch = (location != NULL);
				break;

			case EQUAL:
				found = true;
				item = location -> info;
				break;

			case LESS:
				moreToSearch = false;
				break;
			}//end switch
		}//end while

		return item;
	}

I get a red line under item.ComparedTo that says Error: class "std::basic_string<char, std::char_traits<char>,std::allocator<char>>" has no member function Compared to.

Any ideas on how to fix this?

[jimblumberg]

In your GetItem() function item is a std::string not a RelationType SortedList. How is RelationType defined? In order to use it in a switch it must be an integral type.

Jim

This post has been edited by jimblumberg: 13 May 2012 - 06:47 PM

[killermunk1]

Relation type is defined as

enum RelationType {LESS, GREATER, EQUAL};

[jimblumberg]

Okay that should work for the switch but item is a std::string not a SortedList.

Jim

[killermunk1]

What do you think I should do?

[killermunk1]

I changed the code do you think that this would work, it seemed to fix the error.

	string SortedList::GetItem(string& item, bool& found)
	{
		bool moreToSearch;
		Node* location;
		int x = 0;
		location = listData;
		found = false;
		moreToSearch = (location != NULL);

		while (moreToSearch && !found)
		{
			x = item.compare(location -> info);
			if (x > 0)
			{
				location = location -> next;
				moreToSearch = (location != NULL);
				break;
			}
			if (x == 0)
			{
				found = true;
				item = location -> info;
				break;
			}
			if(x <0)
			{
				moreToSearch = false;
				break;
			}

		}//end while

		return item;
	}

edit: after changing all of the functions that used CompareTo to string.compare it compiles and runs, thank you for pointing that out to me.

This post has been edited by killermunk1: 13 May 2012 - 07:20 PM