Solved

simple problem

Posted on 1998-09-13
11
229 Views
Last Modified: 2010-04-02
this is a simple problem
I need to find the highest and lowest number of the numbers that have been entered.

#include <iostream.h>

int main(void)
{
       int n = 0;
       int high=0;
       int low =0;

       while(!cin.eof())
       {
              int x = 0;
              cin >> x;

              if(x>high)
                   high=n; //this works fine

              if(x<low)
             low=n;  // I always get a 0
              n += x;
       }
       cout << n << endl;
       cout << high<< endl;
       cout << low << endl;
       return 0;
any help would be great.
}
0
Comment
Question by:strmtrpr
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 6
  • 5
11 Comments
 
LVL 2

Accepted Solution

by:
prasanth earned 120 total points
ID: 1172608
try

#include <iostream.h>

int main(void)
{
int n = 0;
int high=0;
int low =0;
int highidx = 0;
int lowidx = 0;

while(!cin.eof())
{
  int x = 0;
  cin >> x;

  if(x>high)
  {
    high=x;
    highidx = n;
  }
  if(x<low)
  {
    low=x;
    lowidx = n;
  }
  n++;
}
cout << n << endl;
cout << highidx << ":" << high << endl;
cout << lowidx << ":" low << endl;
return 0;
}
0
 

Author Comment

by:strmtrpr
ID: 1172609
if(x<low)
      {
        low=x;
        lowidx = n;
      } // still  getting a zero on the output;
 
0
 
LVL 2

Expert Comment

by:prasanth
ID: 1172610
did you enter any negative numbers? If not, 0 (the initial value of low) will end up being the lowest number.
0
Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 

Author Comment

by:strmtrpr
ID: 1172611
the program is supposed to accept numbers until cin.eof() is encountered
then  display  the higest lowest and sum of the numbers entered
then only reason I am asking for help is because I am out of Ideas, I really think that I am
trying to make this to difficult.
thank you for your help.
how do I find the lowest number entered.
I would think that it would be the opposite of the high but that isn,t working.

thank you for your time

0
 
LVL 2

Expert Comment

by:prasanth
ID: 1172612
this should do it

#include <iostream.h>

int main(void)
{
int n = 0;
int high=0;
bool highn = false;
int low =0;
bool lown = false;

while(!cin.eof())
{
  int x = 0;
  cin >> x;

  if(x>high || !highn)
  {
     high=n;
     highn = true;
  }

  if(x<low || !lown)
  {
     low=n;
     lown = true;
  }
  n += x;
}
cout << n << endl;
cout << high<< endl;
cout << low << endl;
return 0;

0
 

Author Comment

by:strmtrpr
ID: 1172613
thanks for the help
the last code you sent wouldn't compile under borland c++ 4.0 or g++under linux.
thanks anyway. I am not sure why this is so hard for me to figure out.
thanks for your time anyway.

0
 
LVL 2

Expert Comment

by:prasanth
ID: 1172614
replace

bool with int
true with 1
false with 0

and it should compile.
0
 

Author Comment

by:strmtrpr
ID: 1172615
it compiles but stilling getting a zero for the output of low

0
 
LVL 2

Expert Comment

by:prasanth
ID: 1172616
I tried this and it works:

#include <iostream.h>

int main(void)
{
int n = 0;
int high=0;
bool highn = false;
int low =0;
bool lown = false;

while(!cin.eof())
{
  int x = 0;
  cin >> x;

  if (!cin.eof())
  {
  if(x>high || !highn)
  {
     high=x;
     highn = true;
  }

  if(x<low || !lown)
  {
     cout << "here" << x <<endl;
     low=x;
     lown = true;
  }
  n += x;
  }
}
cout << n << endl;
cout << high<< endl;
cout << low << endl;
return 0;
}


basically what was happening was that when you pressed CTRL+Z to do eof, the character read was EOF which isn't a number and was being translated to 0.
0
 

Author Comment

by:strmtrpr
ID: 1172617
I owe you  big time
thanks very much now I can get some sleep.

0
 

Author Comment

by:strmtrpr
ID: 1172618
if you ever run into this  again try this instead
while(!(cin>>x).eof)
this fixed the original code the funny thing I had the answer sitting in front of the whole time
ans over looked it
0

Featured Post

Industry Leaders: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
C++ question 3 81
Printing the elements of a set declared inside a map in C++. 3 67
Embarcadero WebBroker REST server endpoint parameters 2 115
COM server issue 2 23
Often, when implementing a feature, you won't know how certain events should be handled at the point where they occur and you'd rather defer to the user of your function or class. For example, a XML parser will extract a tag from the source code, wh…
IntroductionThis article is the second in a three part article series on the Visual Studio 2008 Debugger.  It provides tips in setting and using breakpoints. If not familiar with this debugger, you can find a basic introduction in the EE article loc…
The viewer will learn additional member functions of the vector class. Specifically, the capacity and swap member functions will be introduced.
The viewer will learn how to clear a vector as well as how to detect empty vectors in C++.

732 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question