Solved

Sequence of input numbers

Posted on 2011-03-05
13
745 Views
Last Modified: 2012-05-11
Hello you more experienced programmers,
I am learning c++ programming on my free time, and now I have the following problem:
User is prompted to enter five numbers. The program should determine the highest value and lowest value of user’s input. Additionally, the program should determine the sequence of input numbers (i.e. ascending order, descending order, or no order).  Everything else is working fine but no matter in which order I input the numbers the message displays, “ Numbers are in ascending order”.  Any advice?

#include <iostream>

using namespace std;

int main()
{

int userInput; //number from user
int lowest_value = 9999999; 
int highest_value = 0;

bool ascending = true;
bool descending = true;

for(int i=1;i<=5; i++) //loop 5 times
{
cout<<"Enter your number:  ";
cin>>userInput;
//determine lowest and highest values
 if ( userInput < lowest_value )
 {
  lowest_value = userInput;
 }
 if ( userInput > highest_value )
 {
  highest_value = userInput;
 }
//determine if numbers are in ascending order,in descending order or in no order
    if(i <= i-1 && ascending == 1) {
            ascending = 0;
        }
    if (i >= i-1 && descending == 1){
        descending = 0;
    }
} // end loop
 cout <<"Lowest number is  " <<lowest_value <<endl;
 cout <<"Highest number is  " <<highest_value <<endl;

if (ascending == 1){
  cout <<"Numbers are in ascending order" <<endl;
}
else if (descending == 1){
  cout <<"Numbers are in descending order" <<endl;
}
else{
    cout <<"Numbers are in no order" <<endl;
}
return 0;
}

Open in new window

0
Comment
Question by:Mathilda10
[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
  • 5
  • 4
  • 2
  • +1
13 Comments
 
LVL 53

Expert Comment

by:Infinity08
ID: 35042778
Isn't this the same question as here :

        http://www.experts-exchange.com/Programming/Languages/CPP/Q_26860961.html

If not, what's different ? What is it that you need advice about ?
0
 
LVL 84

Expert Comment

by:ozo
ID: 35042918
i <= i-1 is always false
and i >= i-1 is always true.
neither are affected at all by userInput
0
 

Author Comment

by:Mathilda10
ID: 35042951
Infinity08,
this is not the same exercise. I need advice on how to determine the sequence of user's input numbers (i.e. if the numbers are in ascending, descending, or in no order).
0
What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

 
LVL 53

Expert Comment

by:Infinity08
ID: 35042957
>> I need advice on how to determine the sequence of user's input numbers (i.e. if the numbers are in ascending, descending, or in no order).

If you compare each entered value with the previous entered value, you can see if it's smaller, bigger or the same. If for each entered value, it is smaller than the previous one, then the numbers are descending. If for each entered value, it is bigger than the previous one, then the numbers are ascending. Otherwise the numbers have an indetermined ordering.
0
 
LVL 2

Expert Comment

by:preraksheth
ID: 35043006
You are probably using wrong variable in deciding whether ascending or descending. Use userInput instead of i to decide the order. It should work
0
 

Author Comment

by:Mathilda10
ID: 35043079
Thank you preraksheth!
I changed part of the code as:
//determine if numbers are in ascending order,in descending order or in no order
if (i>1);
    if(userInput <= i-1 && ascending == 1) {
            ascending = 0;
        }
    if (userInput >= i-1 && descending == 1){
        descending = 0;
    }
    i+1;

and now the program displays the correct message for numbers that are in ascending order and for numbers that are in no order. However, when I enter the numbers is descending order, the message displays "Numbers are in no order." Any advice?
0
 
LVL 84

Expert Comment

by:ozo
ID: 35043091
Try numbers greater than 5 in descending order
0
 
LVL 84

Expert Comment

by:ozo
ID: 35043102
or try numbers greater than 5 in ascending order, and numbers less than 0 in descending order
0
 
LVL 2

Accepted Solution

by:
preraksheth earned 500 total points
ID: 35043142
Mathilda10,

You still have a problem with the code.

Try the following
- Remove your current checking for ascending and descending checks
- use the following logic instead
- add a new variable noOrder, initialize to 0

 if ( userInput < lowest_value )
 {
  lowest_value = userInput;
  // added lines
  if(ascending == 0 && noOrder == 0) {descending = 1;}
  else {noOrder = 1;}
 }
 if ( userInput > highest_value )
 {
  highest_value = userInput;
  // added lines
  if(descending == 0 && noOrder == 0) {ascending = 1;}
  else {noOrder = 1;}

 }
0
 

Author Comment

by:Mathilda10
ID: 35043472
Thank you, I will change my code.
0
 

Author Comment

by:Mathilda10
ID: 35045754
I did change the code as you suggested but am still gettig the wrong displays. Maybe the reason for it is the last part of my code:
if (ascending == 1){
  cout <<"Numbers are in ascending order" <<endl;
}
else if (descending == 1){
  cout <<"Numbers are in descending order" <<endl;
}
else{
    cout <<"Numbers are in no order" <<endl;
}

I  guess I need to modify  the above code and ,hopefully, will get the right results.

0
 
LVL 84

Expert Comment

by:ozo
ID: 35045804
You are making it unnecessarily  complicated.
A descending sequence will change lowest_value each time.
So a non descending sequence will leave lowest_value unchanged at least once.
0
 

Author Comment

by:Mathilda10
ID: 35046187
Yes, you are right ozo: I think I make this “unnecessarily complicated”.  And I think that I should not try to learn C++ on my own; I should sign up for a C++ class. I understand  the rules for the expert's postings: you guys are not allowed to help people with their school homework/projects. Since I am not asking any advice with my school assignments,  I think I am in a wrong forum.  I talked with my friend who is experienced with C++ and he pointed me to the right direction (free of charge).
Thank you all for your comments and advice, I really appreciate your time and effort.

0

Featured Post

[Live Webinar] The Cloud Skills Gap

As Cloud technologies come of age, business leaders grapple with the impact it has on their team's skills and the gap associated with the use of a cloud platform.

Join experts from 451 Research and Concerto Cloud Services on July 27th where we will examine fact and fiction.

Question has a verified solution.

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

When writing generic code, using template meta-programming techniques, it is sometimes useful to know if a type is convertible to another type. A good example of when this might be is if you are writing diagnostic instrumentation for code to generat…
Basic understanding on "OO- Object Orientation" is needed for designing a logical solution to solve a problem. Basic OOAD is a prerequisite for a coder to ensure that they follow the basic design of OO. This would help developers to understand the b…
The goal of the video will be to teach the user the difference and consequence of passing data by value vs passing data by reference in C++. An example of passing data by value as well as an example of passing data by reference will be be given. Bot…
The viewer will learn how to user default arguments when defining functions. This method of defining functions will be contrasted with the non-default-argument of defining functions.

635 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