Why is my insertion sort code not working for certain array sizes?

[waver.]

hi Guys
i wrote a insertion sort code which is

#include <iostream> /*Waver*/
using namespace std;
int main()

{
int n,i,m;
float a[100],k;

cout<<"enter the amount of number";
cin>>n;

for(i=0; i<n; i++)
cin>>a[i];

for(i=1; i<=n; i++)
{
    for(m=1 ; m<=i; m++)
    {
      if(a[m]<a[m-1])
      {
          k=a[m-1];
          a[m-1]=a[m];
          a[m]=k;
      }

    }
}
for(i=0; i<n; i++)
cout<<a[i] <<"  ";
return 0;
}

it work fine with some size used in array and won't work with other size
i mean when i enter n=5 it work fine when i enter n=4 it won't work and sort is wrong

when n=5

and when n=4
the sort is wrong and error appear with last number

i just want to know what i did wrong . thanks in advance

 

[Integrand]

Hint: You are mixing 0-based and 1-based indexing in your logic. This is not a good idea. Read about off-by-one errors, and go from there.

 

[jedishrfu]

I would use your debugger and step through the code making sure you agree with what values the computer has set. Doing this you should be able to locate your mistake.

If you think that's too difficult then sprinkle print statements after key lines to see values of indexes and other key variables.

 

[waver.]

I would use your debugger and step through the code making sure you agree with what values the computer has set. Doing this you should be able to locate your mistake.

If you think that's too difficult then sprinkle print statements after key lines to see values of indexes and other key variables.

the debugger didnt cought any thing

 

[rcgldr]

Change the second (or the third) for loop to use < instead of <=. You're indexing beyond the end of the array.

 

[jedishrfu]

When I said use the debugger to step through your code, I didn't mean run your code in the debugger.

The debugger can literally step one line at a time and as you do that you can inspect the values of any variable to see if they are what you expect. Many times you will see your error right away.

Or in the case of @rcgldr's hint you will discover that you looped one too many times.