Solved

simple problem

Posted on 1998-09-13
11
221 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
  • 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
Free Tool: SSL Checker

Scans your site and returns information about your SSL implementation and certificate. Helpful for debugging and validating your SSL configuration.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

 

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

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

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

  Included as part of the C++ Standard Template Library (STL) is a collection of generic containers. Each of these containers serves a different purpose and has different pros and cons. It is often difficult to decide which container to use and …
Go is an acronym of golang, is a programming language developed Google in 2007. Go is a new language that is mostly in the C family, with significant input from Pascal/Modula/Oberon family. Hence Go arisen as low-level language with fast compilation…
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++.

830 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