Visual Studio строки

[CodeNOT]

Доброго всем времени, не мог ли бы мне сказать, в чем я "ДЕБИЛ", в общем пытаюсь вернуть путь до элемента, при проходе по дереву, но на такое выражение он у меня ругается: Path+"->Left"; например, т.е. говорит что у класса строки нет как бы выражения +. Что тогда сделать?

[Ru_DoLF]

Код и лог компилятора в студию!

[CodeNOT]

	Tree* SearchTree(Tree *Root,int SearchElem,string Path)
	{
		if(Root==NULL)
		{
			return NULL;
		}else
		{
			if(Root->Key=SearchElem)
			{
				cout<<"\n Element found! ";
				cout<<"\n Path: "<<Path.c_str();
				return Root;
			}else
			{
				if(Root->Key<SearchElem)
				{
					SearchTree(Root->Right,SearchElem,Path+"->Right");
				}
				if(Root->Key>SearchElem)
				{
					SearchTree(Root->Left,SearchElem,Path+"->Left");
				}
			}
		}
	}

Сама ошибка: бинарный "+": "std::string" не определяет этот оператор или преобразование к типу приемлемо к встроенному оператору

[-LeV-]

вроде есть функция strcat называется

[optimator]

Еще можно

Path.append("->Right");

[Obey-Kun]

cout<<"\n Path: "<<Path.c_str();

лишняя работа же... и лишний код.
и вот тут не косяк ли?

if(Root->Key=SearchElem)

[CodeNOT]

ой, косяк, извините, с спасибо за ответ)! пс: Извините за быдлокод, но я исправлюсь)!

[Obey-Kun]

Тогда заодно именуй всё так:
MyClass — классы
myFunction() — функции, в т.ч. методы
my_variable — локальные переменные, в т.ч. аргументы
m_my_variable — поля классов (то есть так же, как и локальные переменные, только начиная с "m_")

И не делай открытые поля, то есть вот такой фигни быть не должно: Root->Key. Надо делать приватное поле m_key, геттер key() (если он нужен) и сеттер setKey() (если он нужен).

И просто полюбопытствую... Root->Left,SearchElem,Path — это что? Запятую перегрузил?

И насчёт переносов строк и пробелов. Правда лучше выглядит?

	Tree *SearchTree(Tree *Root, int SearchElem, string Path)
	{
		if(Root == NULL) {
			return NULL;
		} else {
			if(Root->Key == SearchElem) {
				cout << "\n Element found!";
				cout << "\n Path: " << Path;
				return Root;
			} else {
				if(Root->Key < SearchElem) {
					SearchTree(Root->Right, SearchElem, Path + "->Right");
				}
				if(Root->Key > SearchElem) {
					SearchTree(Root->Left, SearchElem, Path + "->Left");
				}
			}
		}
	}

То есть хорошей идеей является пристыковывать "{" ко всему, кроме описаний классов и функций и пристыковывать else к "}". А также обивать пробелами почти все операторы (+, -, <, ==, = и т.п.) с обоих сторон и к запятым слева.

Также, * в указателях и & в ссылках лучше пристыковывать к названию. То есть T *myFunc(), а не T* myFunc().

Да и, по хорошему, лучше перенос вывода делать не в начале, а в конце:

cout << "Element found!\n";
cout << "Path: " << Path << "\n";

Но, вообще сказать, хотя бы иногдя используй endl. А именно тогда, когда может понадобиться flush-ить поток (в случае cout — после каждого куска текста, который юзеру надо видеть за раз).