solution

[00Mass00]

• DEMO LINK

[dffUqp]

Almost done:smiley:

[dffUqp]

Suggested change

	{age && <p className="Person__age">{`I am ${age}`}</p>}
	{age && (
	<p className="Person__age">{`I am ${age}`}</p>
	)}

[dffUqp]

I think will be better to refactor this solution and remove getPartner function.

Instead of calling getPartner the second time, we can use 'I am not married' string.

[tiserett]

'I'm not married' doesn't sound like partner, I'd prefer to create variable and use ternary

[roman-mirzoian]

Well done 👍

[roman-mirzoian]

Suggested change

	const NOT_MARRIED_PERSON = 'I am not married';
	const NOT_MARRIED_PERSON_MESSAGE = 'I am not married';

[roman-mirzoian]

Now everything is written correctly, but I offer a more optimized version:

Suggested change

	export const Person = ({ person }) => {
	const {
	name,
	age,
	sex,
	isMarried,
	partnerName,
	} = person;
	const getPartner = () => {
	if (isMarried) {
	return sex === MALE
	? WIFE
	: HUSBAND;
	}
	return NOT_MARRIED_PERSON;
	};
	return (
	<section className="Person">
	<h2 className="Person__name">
	{`My name is ${name}`}
	</h2>
	{age && (
	<p className="Person__age">{`I am ${age}`}</p>
	)}
	<p className="Person__partner">
	{partnerName
	? `${partnerName} is my ${getPartner()}`
	: NOT_MARRIED_PERSON
	}
	</p>
	</section>
	);
	};
	const getPartner = (isMarried) => {
	if (isMarried) {
	return sex === MALE
	? WIFE
	: HUSBAND;
	}
	return NOT_MARRIED_PERSON_MESSAGE;
	};
	export const Person = ({ person }) => {
	const {
	name,
	age,
	sex,
	isMarried,
	partnerName,
	} = person;
	const partnerInfo = partnerName
	? `${partnerName} is my ${getPartner(isMarried)}`
	: NOT_MARRIED_PERSON_MESSAGE;
	return (
	<section className="Person">
	<h2 className="Person__name">
	{`My name is ${name}`}
	</h2>
	{age && (
	<p className="Person__age">{`I am ${age}`}</p>
	)}
	<p className="Person__partner">
	{ partnerInfo }
	</p>
	</section>
	);
	};

[00Mass00]

done