15,943,245 members
See more:

What I have tried:

for(i=0;i<n-1;i++)
{
{
if(a[i]<a[i+1])
swap(a[i],a[i+1])
}
if(a[i]>a[i+1])
swap(a[i],a[i+1])
}
Posted
Updated 20-Nov-22 10:38am
Rick York 20-Nov-22 14:42pm
We don't always get what we want.

## Solution 1

Quote:
Well that is a very rude way to ask for help. So I suggest you change that sentence to explain exactly what that code is supposed to do and what problem you are having with it.

## Solution 2

Despite your rudeness, I will remain polite.

While we are more than willing to help those that are stuck, that doesn't mean that we are here to do it all for you! We can't do all the work, you are either getting paid for this, or it's part of your grades and it wouldn't be at all fair for us to do it all for you.

So we need you to do the work, and we will help you when you get stuck. That doesn't mean we will give you a step by step solution you can hand in!
Start by explaining where you are at the moment, and what the next step in the process is. Then tell us what you have tried to get that next step working, and what happened when you did.

If you are having problems getting started at all, then this may help: How to Write Code to Solve a Problem, A Beginner's Guide[^]

## Solution 3

Quote:

C
```for(i=0;i<n-1;i++)
{
if(a[i]<a[i+1])
swap(a[i],a[i+1])

if(a[i]>a[i+1])
swap(a[i],a[i+1])
}```

The code is nonsensical for two reasons:
1. although the definition of a[] is missing, it can be assumed that the swap() function cannot work like this.
2. even if it would work, the second swap() function would undo the result of the first one.