intermediate/beginner question on classes

[pauldepstein]

I am coding for a large company and I have a block of code in front of
me that looks like the below, but with names changed for
confidentiality reasons.

SomeFunction(St udentClass Mary, StudentClass Jill, StudentClass Vijay)
// In reality there are about a dozen variable names

if (Mary.HasProble m() || Jill.HasProblem () || Vijay.HasProble m() )
// and on and on

{ return StudentClass("A student has a problem");}

So I'm changing it to

if(Mary.HasProb lem()) returnStudentCl ass("Mary has a problem.");
if(Jill.HasProb lem()) returnStudentCl ass("Jill has a problem.");
if(Vijay.HasPro blem()) returnStudentCl ass("Vijay has a problem.");

// many more similar statements.

I'm doing this to get more accurate diagnostic messages.

What is a more compact way of doing the above?

Thanks a lot for your help.

Paul Epstein

[=?ISO-8859-1?Q?Erik_Wikstr=F6m?=]

On 2007-07-24 10:36, pa**********@at t.net wrote:

I am coding for a large company and I have a block of code in front of
me that looks like the below, but with names changed for
confidentiality reasons.

SomeFunction(St udentClass Mary, StudentClass Jill, StudentClass Vijay)
// In reality there are about a dozen variable names

if (Mary.HasProble m() || Jill.HasProblem () || Vijay.HasProble m() )
// and on and on

{ return StudentClass("A student has a problem");}

So I'm changing it to

if(Mary.HasProb lem()) returnStudentCl ass("Mary has a problem.");
if(Jill.HasProb lem()) returnStudentCl ass("Jill has a problem.");
if(Vijay.HasPro blem()) returnStudentCl ass("Vijay has a problem.");

// many more similar statements.

I'm doing this to get more accurate diagnostic messages.

What is a more compact way of doing the above?

Probably not they way you are doing, what if more than one student have
a problem? Personally I find it quite dubious to return a StudentClass
object if a student have a problem, perhaps throw an exception instead?

The place where you should change the code is not in SomeFunction(), but
in the code that calls, it. If SomeFunction() returns with the status
that a student has problems you should then loop through the students
and check their status and print the names of those with problems.

By the way, it might be better to pass a collection of students (such as
vector or list) instead of passing each student.

--
Erik Wikström