помогите разобраться с substr

[swswsss]

написал вот такую программу.по моему замыслу она должна записывать в вектор емейлы без повторов
вот код

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

#include "stdafx.h"
#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
#include <iterator>
using namespace std;

int main()
{
	string str = "a1@ya.ru,a1@ya.ru";
	string str1 = "";

	vector<string> s = {};
	int q = 0;

	for (int i = 0; i<(int)str.size(); i++)
	{
		if (str[i] == ',')
		{

			string str2 = str.substr(q, i);

			cout << str2 << endl;
			//auto it = std::find(s.begin(), s.end(), str2);
			auto it = find(s.begin(), s.end(), str2);
			if (it != s.end())
			{
				cout << "POVTOR" << endl;
			}
			else
			{
				cout << "DOBAVLEN" << endl;
				s.push_back(str2);
				q = i;
			}
			

		}

	}
	for (int i = 0; i<(int)s.size(); i++)
	{
		cout << s[i];
	}
}

если строка вот такая string str = "a1@ya.ru,a1@ya.ru"; то все ок в вектор записывается только 1 емейл. но если допустим string str = "a1@ya.ru,a1@ya.ru,a1@ya.ru"; то substr работает криво, и соответственно find не работает как нужно,подскажите как поправить

[p51x]

1. Вы неправильно вызываете str.substr, надо str.substr(q, i-q-1)
2. Вы неверный индекс сохраняете не q = i; а q = i+1;
3. Не надо приводит сайз к инту, используете правильные типы
4. Вместо

	for (int i = 0; i<(int)s.size(); i++)
	{
		cout << s[i];
	}

используйте for range

[swswsss]

Сообщение от p51x

1. Вы неправильно вызываете str.substr, надо str.substr(q, i-q-1)
2. Вы неверный индекс сохраняете не q = i; а q = i+1;
3. Не надо приводит сайз к инту, используете правильные типы
4. Вместо

	for (int i = 0; i<(int)s.size(); i++)
	{
		cout << s[i];
	}

используйте for range

спасибо если "Не надо приводит сайз к инту, используете правильные типы" то компилятор выдает ворнинг,я понимаю что это не критично,но все же

[p51x]

Вы вместо int i в объявлении цикла напишите правильный тип и варнингов не будет, а так варнинги дело говорят (о граблях).