Чтение содержимого файла в массив. С++

[Warchief]

Сама задача следующая: посчитать кол-во строк в файле, передать их в дин. массив и отсортировать.
Нужно прочитать содержимое файла(содержит числа в строчку, одно число одна строка). Так я считаю кол-во строк.

#include <iostream>
#include <fstream>
using namespace std;
int main()
{
ifstream in("input.txt");
if(!in) {
cout << "Cannot open file.\n";
return 1;
}

char ch[255];
int lines=0;
while(in){
in.getline(ch,255);
lines++;
if(in)
cout<<ch<<endl;}

in.close(); //close stream



ofstream out("output.txt");
if(!out){
	cout <<"error";
    return 1;
}
out << lines;
out.close();
return 0;
}

Потом мне надо прочитать тот-же файл ещё раз и загрузить числа в массив(одна строка, одна ячейка в массиве.)Нужно для последующей сортировки(как сортировать я знаю.)Проблема в том, что я не знаю как записать тип int в массив.


Пробовал записать так:

#include<fstream>
#include<iostream>
using namespace std;

int main()
{
    int	input[1001];

	fstream infile;
	infile.open("input.txt",ios::in);

		for (int i=0;i<1001;i++)
		{
			if(infile.eof())
				return -1;
			infile >>input;
			cout<<input<<endl;
		}
		return 0;
}

Компайлер спотыкается на строке infile >>input;
Пробовал компилировать в VS8 и Dev++.

[Drakosha]

скорее всего ты забыл индекс:
infile >>input[i];

[Bishop]

Warchief,
infile >> input[i];

[upd] Долго писал Drakosha опередил.

[Warchief]

Drakosha,
Bishop,

Ага забыл...


А, что нужно изменить в первом куске кода, чтобы можно было записать в ch[255] сразу значение интегер, а не char? Если просто поменять char на int, то естественно код не компилируется.

[Drakosha]

по моему можно написать:

int xx;
in >> xx;

[PSyton]

Информация к размышлению (точнее вопросы):
А почему бы не делать все за раз?
Складывать в массив и попутно считать?
Складывать сразу в отсортированом виде?
А если в строке не число, пусть все умрет?
А если строк больше чем тысяча одна, на оставшиеся пофиг?
А почему бы не использовать std::vector?

Я так понял это учебная задача, так вот практика показывает что если учебные задачи решаются абы как, то потом и на практике все будет решаться также. Извините если кого обидел. Просто когда начинаешь работать с каким-то крупным проектом и видишь там куски отвратительного когда, становится как-то грустно...

P.S. а да еще - при чтении потока через >> совсем не очевидно что считается целая строка.

[Warchief]

PSyton,

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

Информация к размышлению (точнее вопросы):
А почему бы не делать все за раз?
Складывать в массив и попутно считать?
Складывать сразу в отсортированом виде?
А если в строке не число, пусть все умрет?
А если строк больше чем тысяча одна, на оставшиеся пофиг?
А почему бы не использовать std::vector?

Я так понял это учебная задача, так вот практика показывает что если учебные задачи решаются абы как, то потом и на практике все будет решаться также. Извините если кого обидел. Просто когда начинаешь работать с каким-то крупным проектом и видишь там куски отвратительного когда, становится как-то грустно...

Ты прав, задача учебная. Дело в том, что до этого момента я нокогда не сталкивался с с++, времени дают очень мало, поэтому просто не успеваю разбираться с заданием как следует, к тому-же задания у нас дают в рамках совсем другого предмета(програмирования у нас вообще нет ). Никаких определённых рекомендаций как решать эти задания тоже нет.

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

P.S. а да еще - при чтении потока через >> совсем не очевидно что считается целая строка.

Объясни пожалуйста почему, и если не трудно, объясни как сделать лучше?

[PSyton]

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

Объясни пожалуйста почему, и если не трудно, объясни как сделать лучше?

Оператор >> читает из потока до первого разделителя, коим может быть конец строки или пробел или табуляция, потому если в строке есть что-то типа "abc 2234", то << считает только "abc". Потому в подобных случаях целесообразнее испоользовать cin.getline(), если предполагать что размер строки заведомо меньше заданного в функции. В случае же когда непонятно какого размера строки могут встретиться из потока читается какое-то количество байт и потом там ищется символ конца строки. И этот процесс продолжается итеративно, с последующим добавлением следующих байтов из потока, пока он не закончится.
А как пример одной из множества реализаций можно написать так:

#include <iostream>
#include <set>
#include<fstream>
#include<iostream>
using namespace std;

int main()
{
	multiset<int> sorted_data; //Объявляем шаблонный класс упорядоченного множетсва.
	ifstream in("input.txt");
	if(!in)
	{
		cout << "Cannot open file." << endl;
		return -1;
	}
	else
		while (in) //Читаем файл
		{
			char line[255];
			in.getline(line, 255);
			sorted_data.insert(atoi(line));
		}
		in.close();
		cout << "Lines count: " << sorted_data.size() << endl; //Количество прочитанных строк.
		ofstream out("output.txt");
		if(!out) 
		{
			cout << "Can't open output file";
			return -1;
		}
		//Записываем в выходной файл числа в отсотртированном виде.
		multiset<int>::const_iterator i = sorted_data.begin();
		multiset<int>::const_iterator ei = sorted_data.end();
		for (; i != ei; ++i)
		{
			out << *i << endl;
		}
		out.close();
		return 0;
}

Судя по задаче доподлинно известно что во входящем файле только числа на каждой строке, потому

		char line[255];
		in.getline(line, 255);
		sorted_data.insert(atoi(line));
//можно, как заметил Drakosha, заменить на 
		int tmp;
		in >> tmp;
		sorted_data.insert(tmp);