Simple Inheritance

[Shervan360]

Hello,

I want to implement a simple OOP program.
Could you please check my program? I use a good solution?
I don't want to use a virtual function.
Is this a good solution I call to read and display method in canderDisplay and cancerRead?

Thank you

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

class Person
{
private:
	string name;
	string family;
	int age;
	string sex;
	int id_number;
public:
	void Read()
	{
		cout << "Name: ";
		getline(cin,name);
		cout << "Family: ";
		getline(cin, family);
		cout << "Sex: ";
		getline(cin, sex);
		cout << "Age: ";
		cin >> age;
		cout << "ID Number: ";
		cin >> id_number;
	}
	void Display()
	{
		cout << "Name: " << name << endl;
		cout << "Family: " <<family << endl;
		cout << "Sex: " << sex << endl;
		cout << "Age: " << age << endl;
		cout << "ID Number: " << id_number << endl;
	}
};
class Cancer : public Person
{
private:
	int r;
	string position;
public:
	void CancerRead()
	{
		Read();
		cout << "Position: ";
		cin.ignore(1000,'\n');
		getline(cin,position);
		cout << "Radius: ";
		cin >> r;

	}
	void CancerDisplay()
	{
		Display();
		cout << "Area is: " << Area()<<endl;
		cout << "Perimeter is: " << Perimeter() << endl;
		cout << "Position is: " << position << endl;

	}

	double Area()
	{
		return 3.14 * r * r;
	}
	double Perimeter()
	{
		return 2 * 3.14 * r;
	}
};
int main()
{
	Cancer c;
	c.CancerRead();
	c.CancerDisplay();


	return 0;
}

[Thomas1965]

Normally you use inheritance for a is-a relationship. Do you think a Cancer is a Person ?
https://medium.com/thipwriteblog/short-description-of-is-a-and-has-a-relationship-in-oop-b3448f91767f

[Shervan360]

Thank you for your reply,
It means someone who has cancer.
Is this a good solution I call read and display methods in canderDisplay and cancerRead?

[TheToaster]

Then you shouldn't structure the program in this way. Inheritance models an is-a relationship. Cancer is not a person. A person can HAVE cancer. You could have a Person class with a private cancer member

[Thomas1965]

Better to create a class Patient that inherits from Person and has a has_cancer attribute