Question

The following C++ code include 3 files: Patient.h, Patient.cpp and Main.cpp. The program basically creates and stores patient records. The original code has everything in a single .cpp file. I tried to divide the code in 3 parts (Patient.h, Patient.cpp and Main.cpp), but it is giving me errors.

Patient.h

#ifndef PATIENT_H
#define PATIENT_H

#include <string>
#include "Patient.cpp"
using namespace std;

class Patient{
private :
string firstname;
string lastname;
string location;
static int cnt;
int id;

public :
Patient(string, string, string);
string toString();
void setFirstName(string);
void setLastName(string);
void setLocation(string);
string getFirstName();
string getLastName();
string getLocation();
int getId();
};

#endif // PATIENT_H

Patient.cpp

#include "Patient.h"
#include <iostream>
#include <iomanip>
#include <vector>
#include <sstream>
#include <fstream>
#include <string>
using namespace std;

Patient::Patient(string firstname,string lastname,string location){
cnt++;
this->firstname=firstname;
this->lastname=lastname;
this->location=location;
this->id=cnt;
}

string Patient::toString(){
stringstream ss;
if(id<=9){
ss << "Patient ID: " << "0" << id << endl;
}else{
ss<<"Patient ID:"<<id<<endl;
ss<<firstname<<endl;
ss<<lastname<<endl;
ss<<location<<endl;
}
return ss.str();
}

void Patient::setFirstName(string fname){
this->firstname=fname;
}

void Patient::setLastName(string lname){
this->lastname=lname;
}

void Patient::setLocation(string loc){
this->location=loc;
}

string Patient::getFirstName(){
return firstname;
}

string Patient::getLastName(){
return lastname;
}

string Patient::getLocation(){
return location;
}

int Patient::getId(){
return id;
}

int Patient::cnt=0;

Main.cpp

#include <iostream>
#include <iomanip>
#include <vector>
#include <sstream>
#include <fstream>
#include "Patient.h"
#include "Patient.cpp"
using namespace std;

int search(vector<Patient> patients,int id);

int main(){
string fname,lname,loc;
int id;
int choice ;
vector<Patient> patients;
while(true){
cout<<"\nMENU:"<<endl;
cout<<"\t1) Enter a patient record"<<endl;
cout<<"\t2) Print all patient records"<<endl;
cout<<"\t3) Export list to a file"<<endl;
cout<<"\t4) Edit patient information"<<endl;
cout<<"\t5) EXIT"<<endl;
cout<<"\n\tEnter Option:";
cin>>choice;

switch(choice){
case 1:{
cout<<"Enter patient's first name :";
cin>>fname;
cout<<"Enter patient's last name :";
cin>>lname;
cin.ignore();
cout<<"Enter patient's location :";
getline(cin,loc);
Patient p(fname,lname,loc);
patients.push_back(p);
continue;
}
case 2:{
cout<<"Displaying all records :"<<endl;
for(int i=0;i<patients.size();i++){
cout<<patients[i].toString();
}
continue;
}
case 3:{
ofstream dataOut;
dataOut.open("patientsRecords.txt");
for(int i=0;i<patients.size();i++){
dataOut<<patients[i].toString();
}
dataOut.close();
continue;
}
case 4:{
cout<<"Enter patient id :";
cin>>id;
int indx=search(patients,id);
if(indx==-1){
cout<<id<<" not found."<<endl;
}else{
cout<<"Enter new firstname:";
cin>>fname;
cout<<"Enter new lastname:";
cin>>lname;
cin.ignore();
cout<<"Enter new location:";
getline(cin,loc);
patients[indx].setFirstName(fname);
patients[indx].setLastName(lname);
patients[indx].setLocation(loc);
}
continue;
}
case 5:{
break;
}
default:{
cout<<"** Invalid Choice **"<<endl;
continue;
}
}
break;
}
return 0;
}

int search(vector<Patient> patients,int id){
for(int i=0;i<patients.size();i++){
if(patients[i].getId()==id){
return i;
}
}
return -1;
}

Answer 1

use this, I have shuffled few imports

Patient.h

#ifndef PATIENT_H

#define PATIENT_H

#include <string>

//#include "Patient.cpp"

using namespace std;

class Patient{

private :

string firstname;

string lastname;

string location;

static int cnt;

int id;

public :

Patient(string, string, string);

string toString();

void setFirstName(string);

void setLastName(string);

void setLocation(string);

string getFirstName();

string getLastName();

string getLocation();

int getId();

};

#endif // PATIENT_H

----------------------------------------------------------------------------------------------------------------------------------------

Patient.cpp

#include <iostream>

#include <iomanip>

#include <vector>

#include <sstream>

#include <fstream>

#include <string>

#include "Patient.h"

using namespace std;

Patient::Patient(string firstname,string lastname,string location){

cnt++;

this->firstname=firstname;

this->lastname=lastname;

this->location=location;

this->id=cnt;

}

string Patient::toString(){

stringstream ss;

if(id<=9){

ss << "Patient ID: " << "0" << id << endl;

}else{

ss<<"Patient ID:"<<id<<endl;

ss<<firstname<<endl;

ss<<lastname<<endl;

ss<<location<<endl;

}

return ss.str();

}

void Patient::setFirstName(string fname){

this->firstname=fname;

}

void Patient::setLastName(string lname){

this->lastname=lname;

}

void Patient::setLocation(string loc){

this->location=loc;

}

string Patient::getFirstName(){

return firstname;

}

string Patient::getLastName(){

return lastname;

}

string Patient::getLocation(){

return location;

}

int Patient::getId(){

return id;

}

int Patient::cnt=0;

-------------------------------------------------------------------------------

//Main.cpp

#include <iostream>

#include <iomanip>

#include <vector>

#include <sstream>

#include <fstream>

#include "Patient.h"

using namespace std;

int search(vector<Patient> patients,int id);

int main(){

string fname,lname,loc;

int id;

int choice ;

vector<Patient> patients;

while(true){

cout<<"\nMENU:"<<endl;

cout<<"\t1) Enter a patient record"<<endl;

cout<<"\t2) Print all patient records"<<endl;

cout<<"\t3) Export list to a file"<<endl;

cout<<"\t4) Edit patient information"<<endl;

cout<<"\t5) EXIT"<<endl;

cout<<"\n\tEnter Option:";

cin>>choice;

switch(choice){

case 1:{

cout<<"Enter patient's first name :";

cin>>fname;

cout<<"Enter patient's last name :";

cin>>lname;

cin.ignore();

cout<<"Enter patient's location :";

getline(cin,loc);

Patient p(fname,lname,loc);

patients.push_back(p);

continue;

}

case 2:{

cout<<"Displaying all records :"<<endl;

for(int i=0;i<patients.size();i++){

cout<<patients[i].toString();

}

continue;

}

case 3:{

ofstream dataOut;

dataOut.open("patientsRecords.txt");

for(int i=0;i<patients.size();i++){

dataOut<<patients[i].toString();

}

dataOut.close();

continue;

}

case 4:{

cout<<"Enter patient id :";

cin>>id;

int indx=search(patients,id);

if(indx==-1){

cout<<id<<" not found."<<endl;

}else{

cout<<"Enter new firstname:";

cin>>fname;

cout<<"Enter new lastname:";

cin>>lname;

cin.ignore();

cout<<"Enter new location:";

getline(cin,loc);

patients[indx].setFirstName(fname);

patients[indx].setLastName(lname);

patients[indx].setLocation(loc);

}

continue;

}

case 5:{

break;

}

default:{

cout<<"** Invalid Choice **"<<endl;

continue;

}

}

break;

}

return 0;

}

int search(vector<Patient> patients,int id){

for(int i=0;i<patients.size();i++){

if(patients[i].getId()==id){

return i;

}

}

return -1;

}

----------------------------------------------------------------------------------------------------------------
SEE OUTPUT

Thanks, PLEASE COMMENT if there is any concern. PLEASE UPVOTE