Где я ошибся в коде?

Question

[danij_com]

Код должен находить в файле текст, который находится между <keyMaterial></keyMaterial>и собственно выводить его в консоли.
Но где-то что-то не так, ибо в C++ я далеко не местный.
Укажите на ошибку.
Код:

// ConsoleApplication9.cpp: определяет точку входа для консольного приложения.
//

#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include <vector>
#include <fstream>
#include <string>


using namespace std;

int main() {
	std::vector< std::string > src;
	std::fstream file("{1AC815AC-7555-48FB-B768-9E171453FE23}.xml");

	std::string temp;
	while (!file.eof()) {
		std::getline(file, temp);
		src.push_back(temp);
		std::cout << temp << std::endl;

	}
	

	size_t begin = src.find("<keyMaterial>") + 13;
	size_t end = src.find("</keyMaterial>");
	src = src.substr(begin, end - begin);

	//Вывод
	for (int i = 0; i < 10; i++) {
		std::cout << src[i] << std::endl;
	}

	_getch();
	return 0;
}

А ошибки такие:

Comments

[Алексей С.]

кусок в 50-100 символов из файла приведите, ту часть где keymaterial начинается

[danij_com]

Алексей С.: пытаюсь прикрепить, но выходит криво. Вот весь файл pastebin.com/SW9tJdh6 .

[Максим]

danij_com: почему бы не использовать библиотеку для работы с XML?

[megorit]

Забудь про conio.h навсегда

[Алексей С.]

вот так работает:

#include <iostream>
#include <vector>
#include <fstream>
#include <string>

using namespace std;

int main() {
   
  std::string  src;
  std::string temp;
  src="";
  temp = "<keyMaterial>my text here!1" ;
  src += temp;
  temp = "dfvdfsdfdsfmy text here!2</keyMaterial>" ;
  src += temp;
  

    
    


  size_t begin = src.find("<keyMaterial>") + 13;
  size_t end = src.find("</keyMaterial>");
  src = src.substr(begin, end - begin);

  //Вывод
      std::cout << src;
  

  
  return 0;
}

[danij_com]

Егор Марчук: а как же _getch()?

[megorit]

danij_com: system("pause");

[danij_com]

Алексей С.: хм, но я не могу понять, что он делает. Точнее он выводит "my text here!1dfvdfsdfdsfmy text here!2".

[megorit]

danij_com: библиотека cstdlib

[Алексей С.]

Егор Марчук: посмотрите мой код и сравните со своим , начало вырезано и заменено на моё (где я ручками заполняю "файл" , что бы не возиться с файлом (вот он и выводит то что я поместил для теста между вашими тэгами) . Далее у вас в конце что за ерунда была в цикле , что вы там хотели делать ?
.. ну и главный вопрос , вы пишете не работает , а что именно то ? дайте то что у вас получилось

[megorit]

Алексей С.: адресатом ошиблись

[danij_com]

Алексей С.: вся проблема в том, что мне как раз и нужна эта морока с файлом, ибо их будет много разных, и из всех нужно это парсить. Вот и хочу делать все через массивы. У меня работал построчный парсинг файла в массив и вывод в консоль, а вот "забрать" из этого массива только нужный ключ у меня не выходило. Вообще оно должно было оставлять в массиве только лишь ключ и его выводить уже.
В цикле что именно? gtetch? Это я делал с целью того, чтобы окошко с выводом сразу не закрывалось. Выше мне указали, что можно заменить другим вариантом.

[Алексей С.]

danij_com:
вот этот кусок у вас ошибочный:

/Вывод
  for (int i = 0; i < 10; i++) {
    std::cout << src[i] << std::endl;
  }

. Далее , измените вопрос, добавьте конкретику, что у вас не работает , или результат выполнения программы или ошибки компиляции

[danij_com]

Алексей С.: этот кусок был взят с другого ресурса, но без него, когда я пытаюсь вывести массив (просто вывести, как он есть), получаю ошибку "бинарный "<<": не найден оператор, принимающий правый операнд типа "std::vector>" (или приемлемое преобразование отсутствует) ConsoleApplication9 c:\users\ddani\desktop\wfa\consoleapplication9\consoleapplication9\consoleapplication9.cpp 36
"

[danij_com]

И скриншот ошибок с моего первоначального кода приложил.

[Алексей С.]

danij_com: зачем вам вектор ? сделайте src строкой, во while-е вместо push просто дописывайте src, ну и вывод делайте как у меня просто одной строчкой без всяких for-ов

[danij_com]

Алексей С.: спасибо, проблему решил по Вашему совету.

[Adamos]

danij_com: вы не решили проблему, а обошли ее. Пока. Решение уже указано выше: использование библиотеки работы с XML, например pugi::xml.
А ваши велосипеды со строками - просто бомба замедленного действия. Получите файл чрезвычайно большого размера (или неверно сформированный, или в другой кодировке, или со значениями, которые нужно брать в зависимости от родителей... etc) - придется все просто выкинуть и начинать заново.

[danij_com]

Adamos: файлы всегда одинаковые, размер тоже варьируется порядка +-10 символов, что незначительно. Кодировка тоже одна и так же, так что все в порядке. С библиотеками я просто напросто не разобрался, так что решил сделать так.

Answer 1

[Denis Gershgoren]

Насчёт ошибки - std::vector<T> не имеет у себя функции поиска (find).
Так же ещё одна ошибка - невозможно преобразовать тип std::vector<std::string>> в std::string, как это и было сделано тут - src = src.substr(begin, end - begin);.

Предлагаю свой способ используя итераторы.
И на будущее - лучше не вставлять значения просто так в код, а и использовать #define / enum..

Чтобы напечатать саму строку, достаточно просто написать std::cout << str;.
Насчёт паузы перед выходом программы - в С++ лучше забыть про conio.h, всё уже есть в библиотеке iostream. Достаточно просто написать - std::getchar(); , а так же, если пишете программу для ОС Windows, то можно использовать std::system("pause");.

#define BEG_TAG_LEN 13
#define END_TAG_LEN 14

std::string str = "";
bool found = false;
size_t beg = 0;

for (std::vector<std::string>::iterator it = src.begin(); it != src.end() && !found; ++it)
{
	if ((beg = it->find("<keyMaterial>")) != std::string::npos)
	{
		found = !found;
		beg += BEG_TAG_LEN;
		str = it->substr(beg, (it->length() - beg - END_TAG_LEN));
	}
}

std::cout << str << std::endl << std::endl;