[citus]

in_stream >> Students_Number >> Quiz_1 >> Quiz_2 >> Test_1 >> Test_2;

Students_Number[i] = Student_Numbers;

Did you mean to put "Student_Numbers" after your first ">>" operand instead of what you currently have, which is "Students_Number" which happens to be an array of type int?

[citus]

No problem, glad I could help. I offer this advice to you as well, you can heed or disregard it. Creating variables with extremely similar names almost never turns out well, especially as your program grows. You might want to think about renaming a few of those variables if this program is going to get much bigger. Good luck to you.

[baavgai]

Also, use functions and methods. And, for the love of god, parallel arrays are evil; use a struct or a class.

e.g.

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

using namespace std;

struct Student {
	int number;
	int quiz_1, quiz_2, test_1, test_2;
};

void loadStudent(istream &in_stream, Student &student) {
	in_stream >> student.number >> student.quiz_1 >> student.quiz_2 >> student.test_1 >> student.test_2;
}

void loadStudents(istream &in_stream, Student list[], int size) {
	for(int i=0; i<size; i++) {
		loadStudent(in_stream, list[i]);
	}
}

bool loadStudentsFromFile(const char *filename, Student list[], int size) {
	ifstream in_stream;

	in_stream.open(filename);
	if (!in_stream.fail()) {
		loadStudents(in_stream, list, size);
		in_stream.close();
		return true;
	}
	return false;
}




int main() {
	const int StudentsSize = 5; 
	Student students[StudentsSize];

	if (!loadStudentsFromFile("HW_CH10_P1_Data.txt", students, StudentsSize)) {
		cout << "There was an error in reading the necessary file.\n";
		return 1;
	}
	
	cout << "Data Loaded" << endl;
	
		
	return 0
}

Note, the naming convention of lower case for variables. If you can distinguish between a variable and some other object, then your code will be easier to debug later.

Hope this helps.

This post has been edited by baavgai: 05 May 2010 - 04:05 AM
Reason for edit:: me also fail code tags