Finding the lowest score

[namcintosh]

First of all, here is my program:

#include <iostream>
#include <conio>
using namespace std;
 
//Function prototype
void getscore(int&, int&, int&, int&, int&);
void findLowest (int, int, int, int, int, int);
int main()
{
        int LO, score1, score2, score3, score4, score5;
 
        getscore (score1, score2, score3, score4, score5);
        findLowest(LO, score1, score2, score3, score4, score5);
 
        getch();
        return 0;
}
 
 
 
void getscore (int &score1, int &score2, int &score3, int &score4, int &score5)
{
        cout << "Enter first score: ";
        cin  >> score1;
        cout << "Enter second score: ";
        cin  >> score2;
        cout << "Enter third score: ";
        cin  >> score3;
        cout << "Enter fourth score: ";
        cin  >> score4;
        cout << "Enter fifth score: ";
        cin  >> score5;
}
 
void findLowest(int LO, int score1, int score2, int score3, int score4, int score5)
{
 
        LO = 1000;
 
        if (score1 < LO)
        {
            LO = score1;
            cout << "The lowest score is" <<LO <<endl;
        }
 
        else if (score2 < LO)
        {
            LO = score2;
            cout << "The lowest score is" <<LO <<endl;
        }
        else if (score3 < LO)
        {
            LO = score3;
            cout << "The lowest score is" <<LO <<endl;
        }
        else if (score4 < LO)
        {
            LO = score4;
            cout << "The lowest score is" <<LO <<endl;
        }
        else if (score5 < LO)
        {
            LO = score5;
            cout << "The lowest score is" <<LO <<endl;
        }
}

Okay, I got one part of the program to work right. The only problem is finding the lowest score.

I declared some if/else if statements and also assigned LO to a large number, such as 1000. Here's my output:

Enter first score: 100
Enter second score: 75
Enter third score: 55
Enter fourth score: 100
Enter fifth score: 85
 The lowest score is100

Do you see what I mean? It declares 100 as the lowest number. Why is it doing that?

[Savage]

Yes,and I know it.I wish that u learn so I have left one for u!! :D

Savage

[Savage]

Another thing:

Functions like these are not modular so I would rather use arrays for such function just in case that I feel need for it some time in future.


Savage

[namcintosh]

Yes,and I know it.I wish that u learn so I have left one for u!! :D

Savage

You have one more what

Is there any way that I can use it doing the if/else if statements

[Savage]

You have one more what

Is there any way that I can use it doing the if/else if statements

Well,in that case do it like:

S=score1;

if(s>score2) s=score2;
else if(s>score3) s=score3;
       else if(s>score4) s= score4;
              else s=score5

this approach to reguires a same loop

[namcintosh]

Well,in that case do it like:

S=score1;

if(s>score2) s=score2;
else if(s>score3) s=score3;
       else if(s>score4) s= score4;
              else s=score5

this approach to reguires a same loop

Oh. I though that at first all I could do was just declare LOW to be a large number and then do the if/else if statements to test it out. But I guess it didn't work because all of the numbers are smaller than 1000.

[Savage]

Oh. I though that at first all I could do was just declare LOW to be a large number and then do the if/else if statements to test it out. But I guess it didn't work because all of the numbers are smaller than 1000.

Even if there were number larger than 1000 it would not work as u planed.One more thing:

It can be done without using loop but then it would be done only and only by
using if,not if-else.

If u are intrested/intrigued by this let me know!!!

:D

Savage

[namcintosh]

Even if there were number larger than 1000 it would not work as u planed.One more thing:

It can be done without using loop but then it would be done only and only by
using if,not if-else.

If u are intrested/intrigued by this let me know!!!

:D

Savage

Oh, why Savage, please let me know. I am very interested. :-)

[Savage]

Glad to hear that!!

So here we go:

Start is the same:

int S=score1;

now to the important part:

if(S>score2) s=score2;
if(S>score3) s=score3;
if(s>score4) s=score4;
if(s>score5) s=score5;

this is it.

Now to test it:

score1=10;
score2=15;
score3=8;
score4=12;
score5=9;

1) s=10;
2) s>15 no;
3) s>8 yes-> s=8;
4) s>12 no;
5) s>9 no;

6) s=8

As u can see it's 'alive' and working properly.

And why it would not work with LO and if-else?

It would not work becasue it's wrong logic to set number to random value,always instead set it to one of the scores and then search if there is a smaller number and secound that if-else chain would work only in loop.


Savage

[fantasticamir]

remove all of "else"s and it is gonna work!

[namcintosh]

Glad to hear that!!

So here we go:

Start is the same:

int S=score1;

now to the important part:

if(S>score2) s=score2;
if(S>score3) s=score3;
if(s>score4) s=score4;
if(s>score5) s=score5;

this is it.

Now to test it:

score1=10;
score2=15;
score3=8;
score4=12;
score5=9;

1) s=10;
2) s>15 no;
3) s>8 yes-> s=8;
4) s>12 no;
5) s>9 no;

6) s=8

As u can see it's 'alive' and working properly.

And why it would not work with LO and if-else?

It would not work becasue it's wrong logic to set number to random value,always instead set it to one of the scores and then search if there is a smaller number and secound that if-else chain would work only in loop.


Savage

OK, so how can I get it to return a value