Почему моя функция возвращает мусор?

Question

[danij_com]

Должна быть функция, которая возвращает значение s (чтобы использовать его в другой функции), но оно возвращает что-то несуразное.
В чем ошибка? Пример файла, с которым работать должно. Выводится должен текст в keymaterial pastebin.com/HXXY7DNx

string g (string s)
{
	std::ifstream ifs("{1AC815AC-7555-48FB-B768-9E171453FE23}.xml");

	{
		std::string s;
		s.assign((std::istreambuf_iterator<char>(ifs.rdbuf())), std::istreambuf_iterator<char>());
		size_t begin = s.find("<keyMaterial>") + 13;
		size_t end = s.find("</keyMaterial>");
		s = s.substr(begin, end - begin);
		//std::cout << s;
		return s;
	}
}

Comments

[Fil]

Проверьте результат ifs.good(), возможно проблема с открытием файла

[Александр Титов]

Зачем ВХОДНОЙ параметр функции (string s) ? Он, вроде, не используется, но вносит путаницу.

Answer 1

[Дмитрий]

Ну, во-первых, какая кодировка у вашего файла? Просто так лихо добавлять чего-то 13 к позиции, неправльно. Т. к. это может работать, а может и нет, в зависимости от кодировки исходного файла.

Во-вторых, какая у вас стоит локаль терминала? Когда вы делаете вывод в консоль, данные идут в кодировке файла, а кодировка у терминала может отличаться, отсюда и идёт мусор.

Comments

[res2001]

13 это размер "", т.е. смысл в том, чтобы получить текст внутри тега . Так что прибавить 13 вполне безопасно. Либо попадешь в начало текста, либо в конец, если тег пустой.
А в остальном про кодировки согласен.
Учитывая, что xml может быть в какой-угодно кодировке, логично было бы использовать xml парсер.

[danij_com]

Файл в UTF-8. Вся "соль" в том, что так это работает отлично:

int main()
{
	std::ifstream ifs("{1AC815AC-7555-48FB-B768-9E171453FE23}.xml");
	 
	{
		std::string s;
		s.assign((std::istreambuf_iterator<char>(ifs.rdbuf())), std::istreambuf_iterator<char>());
		size_t begin = s.find("<keyMaterial>") + 13;
		size_t end = s.find("</keyMaterial>");
		s = s.substr(begin, end - begin);
		std::cout << s;
		ifs.close();
	}
	system("pause");
	return 0;
}

Парсится мой текст и все шикарно. Но эту часть нужно встроить в другой код. Я бы мог это просто добавить в одну функцию, но там у меня есть _try _finally, а с ними работа с объектами не разрешена. Появилась надобность вывести это в отдельную функцию, чтобы она на выходе давала переменную с этим кодом, для дальнейшего ее использования в другой функции (переписал как в вопросе). И вот как раз в ту функцию приходит какой-то хлам.

[danij_com]

Даже не так. Не в функцию приходит хлам, а из моей функции выходит хлам. Я отделил этот код (из вопроса) в отдельную программу, и она даже отказалась работать, выдав такие ошибки prntscr.com/e4qdvy

[Дмитрий]

danij_com: В вашей тексте по ссылке вообще ошибка линкера, она никак не связана с вашей проблемой. Вы неверно функцию определяете где-то

[danij_com]

Дмитрий: учитывая то, что я ничего не трогал в той функции, которую использовал, скорее всего она у меня неверно записана. Не могли бы вы дать ссылку на хороший материал по поводу функций, того что в них в () и в {}, пожалуйтса.

[Дмитрий]

danij_com: Не понимаю, о чём вы.
Есть сайт en.cppreference.com, там находится практически официальная документация языка.

В круглых скобках передаются аргументы функции, в фигурных -- тело функции. Если фигурные вложены в функцию, как у вас, то это создаёт отдельный блок времени жизни на стеке, который очищается при выходе из блока.

Верно записана функция или нет, определяет компилятор. Если ошибок компиляции нет, значит верно. Но дальше идёт процесс линковки, который все скомпилированные объектные файлы запихивает в один исполняемый файл. На этапе линковки проверяются заголовки и сигнатуры функций, если они не совпадают, отсутствуют или повторяются несколько раз в разных объектных файлах, линкер выдаст ошибку.

Гуглите ваши ошибки линкера на MSDN и читайте в чём проблема.

[danij_com]

Дмитрий: спасибо за разъяснение. Очередная проблема в том, что с фалом работа проводится как с потоком ostream, а мне нужно перевести его в string. Опять же не нашел информации годной на англоязычных форумах, а на русских и подавно 0.

[Дмитрий]

Информации по С++ столько, что жизни не хватит всё изучить. Первая же ссылка в гугле. Было бы желание...