sorting program

Question

Srinivas0 0 Light Poster

12 Years Ago

write a program to accept the salaries of 10 employees from the user and display them in descending order for all the employees,if the user enters zero, the program should display the message"the amount should be greater than zero" and accept the value again.

#include <iostream>

using namespace std;

class Employee
{
    float salary[10];
    public:
  void sortdata()
  {
      int ctr;
      for(ctr=0;ctr<10;ctr++)
      {
          cout<<"enter the salary:";
          cin>>salary[ctr];
          if(salary[ctr] == 0)
          {
          cout<<"salary should be greater than zero"<<endl;
          continue;
          }
      }
  int sal=0;
  while (sal<9)
  {
      float temp;
      if(salary[sal]<salary[sal+1])
      {
          temp=salary[sal];
          salary[sal]=salary[sal+1];
          salary[sal+1]=temp;
          sal=0;
          continue;
      }
   sal++;
  }
  }
  void display()
  {
      for(int sal=0;sal<10;++sal)
      {
      cout<<"element"<<sal<<":"<<salary[sal]<<endl;
      }
  }
};
int main()
{
    Employee E;
    E.sortdata();
    E.display();
    return 0;
}

so i wrote the program but the last part of the question isn't working, if the user enters zero,the program should display the message"the amount should be greater than zero" and accept the values again. whats wrong with the code.:(

c++

3 Contributors
13 Replies
533 Views
1 Day Discussion Span
Latest Post 12 Years Ago Latest Post by Srinivas0

All 13 Replies

zeroliken 79 Nearly a Posting Virtuoso

12 Years Ago

continue; causes a while or for loop to begin again at the top of the loop.

Do you need to decrement the value of ctr to have another value?

Edited 12 Years Ago by zeroliken because: had a mistake

zeroliken 79 Nearly a Posting Virtuoso

12 Years Ago

So whats the problem?... If I understand what you just said that part of the code is doing what its suppose to do :?:

zeroliken 79 Nearly a Posting Virtuoso

12 Years Ago

Maybe instead of using continue why not change the value of ctr back to 0 whenever the user inputs 0?

zeroliken 79 Nearly a Posting Virtuoso

12 Years Ago

but if i do that wont the user have to input the whole values again ?

using continue does exactly the same thing

do you just want the salary to have a value not equal to 0?
then why not change the current value(0) to give way to a new value

for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
cin>>salary[ctr];
}

Edited 12 Years Ago by zeroliken because: n/a

Reply to this topic

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.

Srinivas0 0 Light Poster · Answer 1 · 2011-12-27T17:33:50+00:00

does it display "the amount should be greater than zero"?
Do you need to decrement the value of ctr to have another value?

yes it displays "the amount should be greater than zero" if i enter 0 and i dont think i need to decrement ctr value cause when 0 is entered the continue should work and loop should be returning to for

for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
continue;
}

Srinivas0 0 Light Poster · Answer 2 · 2011-12-27T17:48:07+00:00

http://i552.photobucket.com/albums/jj339/Kingofgames123/Untitled-2.png
so i entered the last one as 0, it showed "the amount should be greater than zero" but even 0 is getting sorted instead of this what i want if when i enter 0 it should exit the loop and ask for reentering the value and sort. hope you understood.

Srinivas0 0 Light Poster · Answer 3 · 2011-12-27T18:08:17+00:00

Maybe instead of using continue why not change the value of ctr back to 0 whenever the user inputs 0?

but if i do that wont the user have to input the whole values again ?

Srinivas0 0 Light Poster · Answer 4 · 2011-12-27T18:24:34+00:00

using continue does exactly the same thing
do you just want the salary to have a value not equal to 0?
then why not change the current value(0) to give way to a new value
for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
cin>>salary[ctr];
}

thx but it is still sorting the value 0 and it is not even showing "salary should be greater than zero" when 0 is entered . :confused:

zeroliken 79 Nearly a Posting Virtuoso · Answer 5 · 2011-12-27T19:51:42+00:00

try to add continue and see if it works you... though I think its suppose function normally without it

for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
cin>>salary[ctr];
continue;
}

Srinivas0 0 Light Poster · Answer 6 · 2011-12-27T20:27:42+00:00

try to add continue and see if it works you... though I think its suppose function normally without it
for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
cin>>salary[ctr];
continue;
}

not working can anyone else help me with this please.:sad:

Lerner 582 Nearly a Posting Maven · Answer 7 · 2011-12-27T21:27:43+00:00

Have you considered using a loop to request input into a given salary[ctr] as long as salary[ctr] equals zero?

Srinivas0 0 Light Poster · Answer 8 · 2011-12-28T20:18:06+00:00

Have you considered using a loop to request input into a given salary[ctr] as long as salary[ctr] equals zero?

hmm like this ? this is working when 0 is entered first time but if i enter zero again it is getting sorted.:-/

#include <iostream>

    using namespace std;

    class Employee
    {
    float salary[10];
    public:
    void sortdata()
    {
    int ctr;
    for(ctr=0;ctr<10;ctr++)
    {
    cout<<"enter the salary:";
    cin>>salary[ctr];
    if(salary[ctr] == 0)
    {
    cout<<"salary should be greater than zero"<<endl;
    for(ctr=0;ctr<10;ctr++)
    {
    cout<<"enter the salary:";
    cin>>salary[ctr];
    }
    }
    }
    int sal=0;
    while (sal<9)
    {
    float temp;
    if(salary[sal]<salary[sal+1])
    {
    temp=salary[sal];
    salary[sal]=salary[sal+1];
    salary[sal+1]=temp;
    sal=0;
    continue;
    }
    sal++;
    }
    }
    void display()
    {
    for(int sal=0;sal<10;++sal)
    {
    cout<<"element"<<sal<<":"<<salary[sal]<<endl;
    }
    }
    };
    int main()
    {
    Employee E;
    E.sortdata();
    E.display();
    return 0;
    }

Srinivas0 0 Light Poster · Answer 9 · 2011-12-28T23:50:36+00:00

finally i was able to figure out how to do it.

#include <iostream>

using namespace std;

class Employee
{
float salary[10];
public:
void sortdata()
{
int ctr;
for(ctr=0;ctr<10;ctr++)
{
cout<<"enter the salary:";
cin>>salary[ctr];
if(salary[ctr] == 0)
{
cout<<"salary should be greater than zero"<<endl;
cout<<"enter the salary:";
cin>>salary[ctr];
continue;
}
}
int sal=0;
while (sal<9)
{
float temp;
if(salary[sal]<salary[sal+1])
{
temp=salary[sal];
salary[sal]=salary[sal+1];
salary[sal+1]=temp;
sal=0;
continue;
}
sal++;
}
}
void display()
{
for(int sal=0;sal<10;++sal)
{
cout<<"element"<<sal<<":"<<salary[sal]<<endl;
}
}
};
int main()
{
Employee E;
E.sortdata();
E.display();
return 0;
}

sorting program

Recommended Answers Collapse Answers

All 13 Replies

Recommended Answers