comparison logic!!

[curiousEngine]

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.
//TO DO RIGOUROUS TESTING

#include<iostre am>
using namespace std;

void main(){
int A(0), B(0), C(0);
int max(0), min(0);

cout<<"Enter 3 integers:";
cin >A >B >C;

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}
if (A < B){
if (A < C)
min = A;
} else {
if (B < C)
min = C;
else if (C < A)
min = C;
}
cout <<"Max integer: "<<max<<end l;
cout <<"Min integer: "<<min<<end l;

}

[Markus Moll]

Hi

curiousEngine wrote:

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.

So what do you want us to do?

//TO DO RIGOUROUS TESTING

That's a joke, right?

#include<iostre am>
using namespace std;

void main(){
int A(0), B(0), C(0);
int max(0), min(0);

cout<<"Enter 3 integers:";
cin >A >B >C;

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}

You have a problem. One of your if's is missing an else-branch. So either
one of your conditions is always true or there is a path where no maximum
element is found. Try drawing three elements and your knowledge about them
in a diagram where when A < B then A is located lower than B on your paper
and there's a line connecting them. This helps you to find the necessary
comparisons. (And it has nothing to do with C++)

Markus

[Anony Mouse]

Markus Moll wrote:

Hi

curiousEngine wrote:

>Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.

So what do you want us to do?

>//TO DO RIGOUROUS TESTING

That's a joke, right?

>#include<iostr eam>
using namespace std;

void main(){
int A(0), B(0), C(0);
int max(0), min(0);

cout<<"Enter 3 integers:";
cin >A >B >C;

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}

You have a problem. One of your if's is missing an else-branch. So either
one of your conditions is always true or there is a path where no maximum
element is found. Try drawing three elements and your knowledge about them
in a diagram where when A < B then A is located lower than B on your paper
and there's a line connecting them. This helps you to find the necessary
comparisons. (And it has nothing to do with C++)

Markus

Actually, that doesn't matter as 'if (A C)' is only called when 'if (A

B) returns true, and if it doesn't, the line is skipped either way.

I'm not at my home comp right now, so I can't mess around with some code
myself, but I played all combinations I could think of through in my
head, and that piece of code always seems to set the right result.

What I don't get, though:
You could either make the code more readable by using two ifs per
result. Not really more efficient, but a lot easier to play through in
your head.

e.g.

if (A B) {
if (A C)
max = A; }
else if (B A) {
if (B C)
max = B; }
else if (C A) {
if (C B)
max = C; }

Not that much slower. True, yours is more efficient, but really gives
one a headache when one tries to follow it in ones head.

What I also noticed, your int min(0) is currently unused.

Thing with either of the solutions is, both handle only three integers.
Why not use an array and a sorting loop? I like flexibility~

[red floyd]

curiousEngine wrote:

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.
//TO DO RIGOUROUS TESTING

#include<iostre am>
using namespace std;

void main(){

At this point, you have an ill-formed program and anything can happen,
including your compiler not accepting it.

main() returns int. Period.

int A(0), B(0), C(0);
int max(0), min(0);

cout<<"Enter 3 integers:";
cin >A >B >C;

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}
if (A < B){
if (A < C)
min = A;
} else {
if (B < C)
min = C;
else if (C < A)
min = C;
}
cout <<"Max integer: "<<max<<end l;
cout <<"Min integer: "<<min<<end l;

}

[Anony Mouse]

Anony Mouse wrote:

What I also noticed, your int min(0) is currently unused.

Woops. That's what I get for only reading Markus' quote. Same thing for
that one, though.

[Puppet_Sock]

On Oct 12, 11:51 am, curiousEngine <curious.eng... @gmail.comwrote :

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.

//TO DO RIGOUROUS TESTING

#include<iostre am>
using namespace std;

Try to avoid this. It dumps everything in std into
your namespace, and basically defeats the purpose of
having the std namespace in the first place. Learn to
use the std:: decorator.

void main(){

Don't use void main. Using void main allows the Vorlons
to contact the Foul Marmidons. And nobody wants that.
Not even the Vorlons.

int A(0), B(0), C(0);
int max(0), min(0);

cout<<"Enter 3 integers:";
cin >A >B >C;

Learn to use white space to make stuff readable.

cout << "Enter 3 integers: ";

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}

This point is pretty personal, and lots of people
will disagree with me. But I like my { and } to line
up so as to make things readable. Plus, I usually put
in the braces for if statements, even when it's only
one statement and they are not needed.

if (A B)
{
if (A C)
{
max = A;
}
}
else
{
if (B C)
{
max = B;
}
else if(C A)
{
max = C;
}
}

Lots of people prefer how you wrote it to
how I wrote it. This is a "religious" issue
that many people get very upset over. Usually
it's fine either way as long as you stick to it.

It's a lot easier to understand this if you
do max as follows.

max = A;
if(B max)
{
max = B;
}
if(C max)
{
max = C;
}

This way it is more clear that you have covered
every possible case of relative sizes, and it
uses, at worst, the same number of tests. It is
also a lot easier to add more variables. It's
even very easy to do it for a collection.
Socks

[Pete Becker]

On 2007-10-12 05:51:42 -1000, curiousEngine <cu************ @gmail.comsaid:

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.

Start out by setting the maximum value to the first value. Then look at
the second value, and if its greater than the maximum so far, store its
value in the maximum. Then move on to the third. This has the advantage
of working for any number of values, not just three. And its much
simpler and clearer than all those messy if statements.

--
Pete
Roundhouse Consulting, Ltd. (www.versatilecoding.com) Author of "The
Standard C++ Library Extensions: a Tutorial and Reference
(www.petebecker.com/tr1book)

[James Kanze]

On Oct 12, 8:01 pm, Pete Becker <p...@versatile coding.comwrote :

On 2007-10-12 05:51:42 -1000, curiousEngine <curious.eng... @gmail.comsaid:

Write a program that allows you to input 3 integer values A, B and C
and output the largest of the 3 values.

Start out by setting the maximum value to the first value.
Then look at the second value, and if its greater than the
maximum so far, store its value in the maximum. Then move on
to the third. This has the advantage of working for any number
of values, not just three. And its much simpler and clearer
than all those messy if statements.

In fact, the best solution is to get rid of the if's entirely:

std::cout << "max is: " << std::max( std::max( A, B (, C ) ))
std::endl ;
std::cout << "min is: " << std::min( std::min( A, B (, C ) ))
std::endl ;

More generally, I'd put the values in an array (std::vector),
and use std::max_elemen t.

--
James Kanze (GABI Software) email:ja******* **@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientier ter Datenverarbeitu ng
9 place Sémard, 78210 St.-Cyr-l'École, France, [phone]

[Juha Nieminen]

curiousEngine wrote:

if (A B){
if (A C)
max = A;
} else {
if (B C)
max = B;
else if (C A)
max = C;
}

If you want to do your homework in a bit more obfuscated way, you can
do that with a one-liner:

max = A B ? A C ? A : C : B C ? B : C;

[Puppet_Sock]

On Oct 15, 1:01 pm, Juha Nieminen <nos...@thanks. invalidwrote:
[snip]

If you want to do your homework in a bit more obfuscated way, you can
do that with a one-liner:

max = A B ? A C ? A : C : B C ? B : C;

I compiled this, with appropriate int main etc.
added to make a complete program. It printed out
the words to Aida.
Socks