// CHANGE:// iostream is the correct header// conio.h doesn't exist in C+// cstdlib is the correct header, but it isn't used so we remove it.// string added so we can use strings#include <iostream>#include <string>class Blood_Pressure{private:float upper;float lower;public:Blood_Pressure(){// NOTE:// 0.0 is a double, replaced with 0.0F which is a float, not an error, but bad programming style// NOTE:// Initialization lists is better for initializing variablesupper=lower=0.0; }~Blood_Pressure(){}// NOTE: Some names starting with _ are reserved for implementation// NOTE: When passing by value you shouldn't make the value const, there is no reason why you would.void setupper(const float _upper){upper=_upper;}// NOTE: Some names starting with _ are reserved for implementation// NOTE: When passing by value you shouldn't make the value const, there is no reason why you would.void setlower(const float _lower){lower=_lower;}// CHANGED: void to indicate 0 paramters is a C habit, not C++// CHANGED: This function has been made const, because it doesn't change the state of the object.float getupper()const{return upper;}// CHANGED: void to indicate 0 paramters is a C habit, not C++// CHANGED: This function has been made const, because it doesn't change the state of the object.float getlower()const{return lower;}};class Patient{private:// CHANGED:// Using a real string instead of a finite sequence of bytesstd::string name;// NOTE: Inconsistent naming convention, before you used lowercase for member variables.Blood_Pressure BP;public:Patient(){// CHANGE:// Changed to a real string.std::string name;// Change it is NOT cout and cin, it's std::cout and std::cin.std::cout <<"enter your name..";std::cin>>name;// NOTE: Some names starting with _ are reserved for implementationfloat _upper,_lower;std::cout<<"enter upper limit..";std::cin>>_upper;BP.setupper(_upper);std::cout<<"enter lower limit..";std::cin>>_lower;BP.setlower(_lower);CalculateReport();}~Patient(){}// NOTE: Some names starting with _ are reserved for implementation// CHANGE: Changed to a real string. Passed by const-reference.void setname(const std::string& _name){// CHANGE: using different function to copy a real string.// NOTE: When using the C library you should always make sure to use the bounded versions of the functions. Then you could tell the compiler to not copy more than 20 bytes. One of the many reasons we use real strings.// NOTE: You should use std::strcpy instead of strcpyname=_name;}// CHANGED: void to indicate 0 parameters is a C habit, not C++// CHANGED: real string as return value.// CHANGED: This function has been made const, because it doesn't change the state of the object.// NOTE: That function was a potential bug anyway, you can't just return a pointe like that// you need to make a copy of that and return that and then make it the caller's responsibility// to delete it. So if someone had got your string deleted your patient, then the string they got// would be invalid because the patient destroyed it. // One of the many reasons we use real strings.std::string getname() const{return name;}// CHANGED: void to indicate 0 paramters is a C habit, not C++// CHANGED: This function has been made const, because it doesn't change the state of the object.void CalculateReport()const// CHANGED: { missing, added it.{// NOTE: Don't you consider 120.00000001 and 80.0000001 normal? Also you can't just compare floating point numbers like that, you need to accept all numbers within a certain range (because 120 and 80 might not be representable by your floating point format. For example if( getupper()-120 < 0.001 ) would accept all values which aren't farther than 0.001 away.if((getupper()==120) && (getlower()==80))cout<<"Blood pressure is normal...";elsecout<<"blood pressure is not normal..";// CHANGED: } missing, added it. Functions ALWAYS need brackets.}};// CHANGED: void to indicate 0 parameters is a C habit, not C++int main(){Patient P;// CHANGED: system("pause") aren't guaranteed to work, use while(std::cin.get()); instead.while(std::cin.get() );return 0;}