Help me with my C++ code please

[TheLoneWolf32]

void Compact(apvector<int> &List, int &N)
//Pre:List contains integers from 0...N-1
//PostCondition all 0's have been removed from the list; N is reduced as appropriate
{
for(int i = 0; N > i; i++)
{
while(List[i] == 0)
{
for(int c = 0; c < N; c++)
{
List[i] = List[i+1];
N--;
}
}
}

List.resize(N);
for(int b = 0; b != List.length(); b++)
{
cout << List[b] << " ";
}
}

and now I am in an endless loop...I am trying to make it so it reads the numbers off an array and removes all the zero and then put everything back into an array of vectors. and then resizes it to save memory...also a parameter is not to use a second array. I don't know whats wrong...this should work but it doesn't someone help me....also I am a high school student...also I am using very basic class's such as iostream, iomanip, apvector, math, fstream, limits and apstring. Also I am using MSVC 6(the latest one) if that helps

[TheLoneWolf32]

void Compact(apvector<int> &List, int &N)
//Pre:List contains integers from 0...N-1
//PostCondition all 0's have been removed from the list; N is reduced as appropriate
{
     for(int i = 0; N > i; i++)
     {
          while(List[i] == 0)
          {
               for(int c = 0; c < N; c++)
               {
                    List[i] = List[i+1];
                    N--;
               }
          }
     }

     List.resize(N);
     for(int b = 0; b != List.length(); b++)
     {
           cout << List[b] << " ";
     }
}

[PJYelton]

Errr.. your logic is a little messy. If you want to do it this way you need to change a few things. First off you have i swapping with its next partner N times over and over again instead of moving up the list. Also, you don't want to decrease N until you are done with all the swapping. You have it decreasing every swap. Also, you only want to swap the items from i to N, not from zero. When doing code like this it is very important at this stage in you programming to take a piece of paper and trace through your code step by step keeping track of what variable is equal to what. This will give you a better idea of what is going on and how to change it. Try this:

     for(int i = 0; N > i; i++)
     {

          while(List[i] == 0 && i<N)
          {
               for(int c = i; c < N-1; c++)
               {
                    List[c] = List[c+1];
               }
               N--;
          }
     }

Haven't tested this so I wouldn't be surprised if theres a bug...

[TheLoneWolf32]

thanks...I've been cracking at this program for some bit of time...I guess I'm not that logical...or I should write down my programs before hand instead of hacking away at code.