Sorting? Or Searching

[faieq92]

Urm basically its only letting me input for lastname and not first and same again with postcode . I know it doesn't make sense but its skipping some details

[faieq92]

so it skips first name and lets me input in 2nd name

[fun2code]

We're working with this code, right?

void SaveData(Details Data[],int size){
ofstream DataSave("test.txt");
	for(int i=0;i<size;i++){
	DataSave <<"\n";
	cout << "First Name: ";
    getline(cin, Data[i].FirstName);
	DataSave <<  Data[i].FirstName<<"\t";

    cout << "Last Name: ";
    getline(cin,  Data[i].LastName);
	DataSave << Data[i].LastName<<"\t";

	cout << "Age(Please Enter an Integer(Number)): ";
	DataSave <<  Data[i].Age<<"\t";

	cout << "Email: ";
	getline(cin,  Data[i].Email);
	DataSave <<  Data[i].Email<<"\t";

    cout << "Door Number(Please Enter an Integer(Number)): ";
    cin >>  Data[i].DoorNumber;
	DataSave <<  Data[i].DoorNumber<<"\t";

    cout << "Road Name: ";
    getline(cin,  Data[i].RoadName);
	DataSave <<  Data[i].RoadName<<"\t";

    cout << "Post Code(NOTE:Please use Lower Case): ";
	getline(cin,  Data[i].PostCode);
	DataSave <<  Data[i].PostCode<<"\t";
	DataSave <<"\n";
	}
	DataSave.close();

}

I see an error. You forgot to read in the age from the user. This would throw off whats being entered for what, which could easily cause cin to fail.
Data entry must be correct.

[faieq92]

fixed it with this
out << "Age(Please Enter an Integer(Number)): ";
cin >> Data[i].Age;
DataSave << Data[i].Age<<"\t";

still not working its skipping first name

[faieq92]

and its definetly a get line problem


it skips email and goes to door number

then skips road name and does the postcode

[fun2code]

Yeah, I think you are encountering the nasty problems that occur when you mix cin >> and getline(cin) calls.
It has something to do with a newline getting stuck in the stream.

My file I/O skills are limited because I don't do it much.

You can try stuff like adding this after any line where you use cin >> ( your newly added line and line 21 above). cin.ignore(256,'\n'); which may remove the stray newline (if that's what's causing the problem).

Or, try using getline for everything.

Someone else may have to help you with this part.

[faieq92]

Thanks for all your help. Also do you have any idea why Data[i]!=NULL doesn't work?

[fun2code]

You're welcome for the help. Did that cin.ignore(256,'\n'); fix anything?

Also do you have any idea why Data[i]!=NULL doesn't work?

Data[i] is not a pointer, so that comparison is illegal.

EDIT: On second thought, Data[i] is a Details object, so Data[i] != not-a-Details object will always be illegal.

[faieq92]

and no it didn't I might go back to just using cin without get line

so if that comparison to null doesn't work how do I check if array is full if so to skip :S

[fun2code]

and no it didn't I might go back to just using cin without get line

OK, but that will land you back onto problems with blank spaces in strings.

so if that comparison to null doesn't work how do I check if array is full if so to skip :S

Not sure what you mean by "array is full". The code above is done when i==size in the for loop.

[faieq92]

I mean to check if details have already been entered into data[I]

[fun2code]

Just check Data[i] for valid data. Your default constructor for a Details object assigns age = 0, if I recall.
I was going to suggest you make that -1 instead. If Data[i].age == -1 then details have not been entered.

[faieq92]

yes it is age=0; and I only need to check age right not the others such as name strings?

[faieq92]

im about to go crazy now its started to wipe the previous data in the file again...

[faieq92]

for(int j=0;j<x;j++){
				if ( Data[i].Age == 0){
				cout << "First Name: ";
				getline(cin, Data[i].FirstName);
				DataSave <<  Data[i].FirstName<<"\t";	
				cin.ignore(256,'\n'); 
				cout << "Last Name: ";
				getline(cin,Data[i].LastName);
				DataSave << Data[i].LastName<<"\t";
		
				cout << "Age(Please Enter an Integer(Number)): ";
				cin >>  Data[i].Age;
				DataSave <<  Data[i].Age<<"\t";
				cin.ignore(256,'\n'); 
				cout << "Email: ";
				getline(cin,  Data[i].Email);
				DataSave <<  Data[i].Email<<"\t";
			
				cout << "Door Number(Please Enter an Integer(Number)): ";
				cin >>  Data[i].DoorNumber;
				DataSave <<  Data[i].DoorNumber<<"\t";
				cin.ignore(256,'\n'); 
				cout << "Road Name: ";
				getline(cin,Data[i].RoadName);
				DataSave <<  Data[i].RoadName<<"\t";
			
				cout << "Post Code(NOTE:Please use Lower Case): ";
				getline(cin,Data[i].PostCode);
				DataSave <<  Data[i].PostCode<<"\t";
				DataSave <<"\n";
				
			}
			else{
				i++;
				x--;
				cout << "FULL";
			}
		}
	
	DataSave.close();

}

every time I re run the program it deletes old and puts in new

[Chervil]

every time I re run the program it deletes old and puts in new

Open the file with a mode of ios::app, in order to append the new data at the end of the existing file.
ofstream DataSave("test.txt", ios::out | ios::app);

That means the file will grow larger each time. I've lost track of the other parts of the code, if you are reading from the file into your array, then the size of the array may need to be increased. I did mention a vector at one point, and that may be useful, depending on the overall design and what it is you are trying to achieve.

I don't know whether this is a student project, where you must adhere to a strict specification, or your own where you can choose the requirements yourself.

[faieq92]

its a student project we have to use arrays

also I don't understand this bit of code

ofstream DataSave("test.txt", ios::out | ios::app);

[faieq92]

	#include <iostream> 
	#include <string>
	#include <math.h> 
	#include <fstream>
	#include <sstream>
	using namespace std;

class Details
{
public:
    Details();
	string FirstName;
    string LastName;
    int Age ;
    string Email;
    int DoorNumber;
    string RoadName;
    string PostCode;
	void SearchByAge(int);
	
};

	 Details::Details() : Age(0), DoorNumber(0) {}
//void DataEntry(Details &member){
//	ofstream DataSave("test.txt");
//	
//	cout << "First Name: ";
//    cin >> member.FirstName;
//	DataSave << member.FirstName<<"\t";
//
//    cout << "Last Name: ";
//    cin >> member.LastName;
//	DataSave << member.LastName<<"\t";
//
//	cout << "Age(Please Enter an Integer(Number)): ";
//	cin >> member.Age;
//	DataSave << member.Age<<"\t";
//
//	cout << "Email: ";
//	cin >> member.Email;
//	DataSave << member.Email<<"\t";
//
//    cout << "Door Number(Please Enter an Integer(Number)): ";
//    cin >> member.DoorNumber;
//	DataSave << member.DoorNumber<<"\t";
//
//    cout << "Road Name: ";
//    cin >> member.RoadName;
//	DataSave << member.RoadName<<"\t";
//
//    cout << "Post Code(NOTE:Please use Lower Case): ";
//	cin >> member.PostCode;
//	DataSave << member.PostCode<<"\t";
//	DataSave <<"\n";
//	
//	
//
//}
void GetData(Details &member){
	cout << "\nFirst Name: "<< member.FirstName;
	cout << "\nLast Name: " << member.LastName;
	cout << "\nAge: " << member.Age;
	cout << "\nEmail: " << member.Email;
	cout << "\nDoor Number: " <<  member.DoorNumber;
	cout << "\nRoad Name: " << member.RoadName;
	cout << "\nPost Code: " <<  member.PostCode<<"\n";
}
void Age_Search(Details info[], int size){
    int AgeSearch = 0;
	cin >> AgeSearch;
	for(int i=0; i < size; i++){
		if( info[i].Age == AgeSearch ){
		cout << info[i].FirstName << " "  << info[i].LastName << " is" << info[i].Age << " years old.\nHere are his/her Details:";
		GetData(info[i]);
		}
	}
}
void LastName_Search(Details info[],int size){
	string LastNameSearch;
	cin >> LastNameSearch;
	for(int i=0; i < size; i++){
		if( info[i].LastName == LastNameSearch ){
		cout << "Here are Details of People who have the Surname "<< LastNameSearch<<":";
		GetData(info[i]);
		}
	}

}
void PostCode_Search(Details info[],int size){
	string PostCodeSearch;
	cin >> PostCodeSearch;
	for(int i=0; i < size; ++i){
		if( info[i].PostCode == PostCodeSearch ){
		cout << "Here are Details of People Who Live in the '"<< PostCodeSearch<<"' Postal Area:";
		GetData(info[i]);
		}
	}
}
void Email_Search(Details info[],int size){
	string EmailSearch;
	cin >> EmailSearch;
	for(int i=0; i < 10; ++i){
		if( info[i].Email == EmailSearch ){
		cout << "This Person Has the email address '"<< EmailSearch<<"':";
		GetData(info[i]);
		}
	}
}
// define operator >> for a Details object

   istream& operator>>( istream& is, Details& detail ){
    string line;
    string word;
    char delim = '\t';
    getline(is, line);

    istringstream iss(line);

    getline(iss, word, delim);
    detail.FirstName = word;

    getline(iss, word, delim);
    detail.LastName = word;

    getline(iss, word, delim);
    detail.Age = atoi(word.c_str());          // convert string to integer

    getline(iss, word, delim);
    detail.Email = word;

    getline(iss, word, delim);
    detail.DoorNumber = atoi(word.c_str());   // convert string to integer

    getline(iss, word, delim);
    detail.RoadName = word;

    getline(iss, word, delim);
    detail.PostCode = word;

    return is;
}
void LoadData(Details Data[], int size){
    ifstream File_in("test.txt");// attempt to open file for reading
    if( File_in.is_open() )
    {
        for(int i=0; i<size;i++)
            File_in >> Data[i];
	        File_in.close();// details!
    }
    return;
}
	//void SaveData(Details Data[], int size){
//	ofstream DataSave("test.txt");
 //   if( Datasave.is_open() )
  //  {
   //     for(int i=0; i<size;i++){
    //       DataSave << Data[i];
	//	}
	 // DataSave.close();      // details!
  //  }

  //  return;
//}






    




void SaveData(Details Data[],int size){
	ofstream DataSave("test.txt");
	int x;
	int i;
	i=0;
		
	cout << "How Many Entries Would You Like to add";
	cin >> x;

	
		
	for(int j=0;j<x;j++){
				if ( Data[i].Age == 0){
				cout << "First Name: ";
				getline(cin, Data[i].FirstName);
				DataSave <<  Data[i].FirstName<<"\t";	
				cin.ignore(256,'\n'); 
				cout << "Last Name: ";
				getline(cin,Data[i].LastName);
				DataSave << Data[i].LastName<<"\t";
		
				cout << "Age(Please Enter an Integer(Number)): ";
				cin >>  Data[i].Age;
				DataSave <<  Data[i].Age<<"\t";
				cin.ignore(256,'\n'); 
				cout << "Email: ";
				getline(cin,  Data[i].Email);
				DataSave <<  Data[i].Email<<"\t";
			
				cout << "Door Number(Please Enter an Integer(Number)): ";
				cin >>  Data[i].DoorNumber;
				DataSave <<  Data[i].DoorNumber<<"\t";
				cin.ignore(256,'\n'); 
				cout << "Road Name: ";
				getline(cin,Data[i].RoadName);
				DataSave <<  Data[i].RoadName<<"\t";
			
				cout << "Post Code(NOTE:Please use Lower Case): ";
				getline(cin,Data[i].PostCode);
				DataSave <<  Data[i].PostCode<<"\t";
				DataSave <<"\n";
				
			}
			else{
				i++;
				j--;
				cout << "FULL";
			}
		}
	
	DataSave.close();

}
int main(){	
	
	int a=0;
	int b=0;
	int i = 0;
	string temp;
	Details	Data[10];
	LoadData(Data,10);
	
	
	do{	
		cout << "\n1) Input \n2) View \n3) Search\n4) Delete\nExit \n" ;
		cin >> a;
		switch(a){
			case 1:	
				//Input;
			SaveData(Data,10);
			break;
			case 2: 
				//View;
				cout << "Would You like to view the data?";
				cin >> i;
				GetData(Data[i]);	
			break;
			case 3: 
				//Search
				system("CLS");
				cout << "\n1) Input \n2) View";
				cout << "\n3Search\n   3.1)Search Using Age\n   3.2)Search Using Last Name\n   3.3)Search Using Post Code\n   3.4)Search Using Email";
				cout <<"\n4) Delete\nExit \n" ;
				cin >> b;
				switch(b){
				case 1:
					//Search With Age
					cout << "Enter the age of the person you are looking for: ";Age_Search(Data,10);break;
				case 2:
					//Search Using Last Name
					cout << "Enter the Last Name or Surname of the person you are looking for: ";LastName_Search(Data,10);break;
				case 3:
					//Search Using Post Code
					cout << "Enter the Post Code of the person/people you are searching for: ";PostCode_Search(Data,10);break;
				case 4:
					//Search Using Email Adress
					cout << "Enter the Email Address of the person you are searching for: ";Email_Search(Data,10);break;default:break;
				system("pause");					
				}//end of switch b(search)
			break;
			case 4: 
				//Delete
			break;
			case 5:
				//Exit
			break;default: break;       
		   }//end of switch a(main menu)
	}//end of do while loop if exit is chosen.
	while(a!=5);


return 0;}

and this is what I have atm

[Chervil]

Thanks for sharing your code. I haven't had time to look in detail, but I think it will be useful to all of us who try to help.

Opening a file using different mode flags such as append is described in the tutorial here: http://www.cplusplus.com/doc/tutorial/files/

[faieq92]

Also thanks for all the help so far from everyone in this topic. Don't know where id be without the help. I suppose it would be much more basic than it is now and I would've given up trying to add some features. Still have a long way to go but thanks. :D Really Appreciated.