Help moving my bool function from my .h to .cpp file

[rex1200]

Ah I think I've got it. I want to return servers[I] because that is what we are decrementing previously correct?

How is this?

int ServerGroup::decServers()
{
  for(int I(0); I < size; ++I) 
  {
    if(servers[I] != 0)
      --servers[I];
}
  if(spServer!= 0)
  spServer -= 1;
  
  return servers[I];
}

[Daleth]

The variable I only exists within the for loop body. If you want to return the last item decremented, declare I outside of the for loop.

[rex1200]

Declare I outside of the for loop? Do you mean something like this?

int ServerGroup::decServers()
{
  for(int I(0); I < size; ++I)
  {
    if(servers[I] != 0)
      --servers[I];
  }
  if(spServer!= 0)
    spServer -= 1;
 
  int I(0);
  return servers[I];
}

[Daleth]

No... Because that is as good as return servers[0];. You stated the returned value will be the last decremented item, correct?

int ServerGroup::decServers()
{
  int I(0); //<--Here
  for(; I < size; ++I)
  {
    if(servers[I] != 0)
      --servers[I];
  }
//By this point, I will be the index of the last decremented element
  if(spServer!= 0)
    spServer -= 1;
 
  return servers[I];
}