Почему не выходит из цикла?

Question

[Денис Кузнецов]

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

using namespace std;

int main()
{
	ifstream in("E:\\f.txt");
	ofstream out1("E:\\h.txt"), out2("E:\\g.txt");
	if (!in)
	{
		cout << "Error" << endl;
		system("pause");
		return 0;
	}
	string s;
	s += ' ';
	while (in.peek() != EOF)
	{
		getline(in, s, '#');
	}
	string sub;
	int space = 0; 
	string::size_type len = s.size();
	for (unsigned int i = 0; i < len; i = space + 1)
	{
		space = s.find(' ',  i); 
		sub = s.substr(i, space - i);
		if (sub[0] == '-')
		{
			out1 << sub << ' ';
		}
		else
		{
			out2 << sub << ' ';
		} 
	}
	in.close();
	out1.close();
	out2.close();
	system("pause");
	return 0;
}

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

Comments

[Stalker_RED]

Используйте тег code (последняя кнопка в редакторе). Невозможно же читать.

[Adamos]

std::string::find возвращает -1, когда строка не найдена.
For, естественно, не закончится никогда.

[Roman]

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

вот например, и читать приятнее и вероятность ошибки снижается.

#include <iostream>
#include <fstream>
#include <vector>
#include <string>
using namespace std;
const string inputFile   = "C:\\f.txt";
const string outputFile  = "D:\\h.txt";//Для записи положительных чисел
const string outputFileN = "D:\\g.txt";//Для записи отрицательных чискл
ifstream& operator>>(ifstream& ifs, vector<int>& values)
{
    while(ifs)
    {
        int i=0;
        ifs>>i;
        values.push_back(i);
    }
    return ifs;
}


void writeIntegerToFile(ofstream& f,int& i)
{
    f<<i<<' ';
}

void error(string e)
{
    cout<<e<<endl;
}

int main()
{

    vector<int> inputDigits;
    ifstream in(inputFile);
    if(!in)
    {
        error("Can`t open file: " + inputFile);
        return -1;
    }
    in>>inputDigits;
    in.close();

    ofstream out1(outputFile);
    ofstream out2(outputFileN);

    if(!out1 || !out2)
    {
        string s = (!out1)?(outputFile):(outputFileN);
        error("Can`t open file: " + s);
        return -1;
    }

    for(int n:inputDigits)
    {
        if(n>=0)
        {
            writeIntegerToFile(out1,n);
        }
        else
        {
            writeIntegerToFile(out2,n);
        }

    }
    out1.close();
    out2.close();
return 0;
}

input
1 22 33 44 55 66 77 88 -99
-88 -987 77 66 0 101 -3000 55 0-

out1
1 22 33 44 55 66 77 88 77 66 0 101 55 0 0

out2
-99 -88 -987 -3000

[Денис Кузнецов]

Stalker_RED, спасибо , учел

[Денис Кузнецов]

Adamos, но там же в конец строки спецом добавляется пробел для считывания последнего числа , и индекс последнего пробела меньше чем len , а вот i = space + 1 будет уже больше и for завершится, расчет был на это

[Adamos]

Денис Кузнецов, надо делать не "расчет", а проверки. Сразу после получения результатов функции, в которой может быть не то, что ожидается.
Попробуйте представить, например, что у вас происходит в двух следующих строках при space = -1

Answer 1

[Codebaker]

Вы не проверяете на std::npos результат в space:

space = s.find(' ', i);