obsolete binding at `a' error C++

[Nicanor]

Trying to compile a program, in my accounts.cpp I have 2 errors

accounts.cpp:240: error: name
   lookup of `a' changed for new ISO `for' scoping




accounts.cpp:234: error: 
    using obsolete binding at `a'

here's the code

bool cAccount::IsOnline( int acctnum )
{
    if (acctnum < 0) return false;

    map<int, acctman_st>::iterator iter_acctman;

    if ((iter_acctman = acctman.find(acctnum)) != acctman.end())
    {
        for (int a = 0; a < now; a++)
        {
            if (Client[a]->GetAccNum() == acctnum)
                break;
        }

        if (a == now)
        {
            SetOffline(acctnum);
            return false;
        }

any help would be awsome thanks :)

[Dave Sinkula]

Trying to compile a program, in my accounts.cpp I have 2 errorsp

accounts.cpp:240: error: name
lookup of `a' changed for new ISO `for' scoping
accounts.cpp:234: error:
using obsolete binding at `a'

here's the code

bool cAccount::IsOnline( int acctnum )
  {
  	if (acctnum < 0) return false;
  
  	map<int, acctman_st>::iterator iter_acctman;
  
  	if ((iter_acctman = acctman.find(acctnum)) != acctman.end())
  	{
  		for (int a = 0; a < now; a++)
  		{
  			if (Client[a]->GetAccNum() == acctnum)
  				break;
  		}
  
  		if (a == now)
  		{
  			SetOffline(acctnum);
  			return false;
  		}

any help would be awsome thanks :)

I believe the diagnostic messages are telling you that the scope of a should only be within the for loop -- that is, it is not available outside the for loop. So if you are using it outside the for loop, you should not declare it within the for loop.

bool cAccount::IsOnline( int acctnum )
   {
   	if (acctnum < 0) return false;
   
   	map<int, acctman_st>::iterator iter_acctman;
   
   	if ((iter_acctman = acctman.find(acctnum)) != acctman.end())
   	{
   		int a;
   		for (a = 0; a < now; a++)
   		{
   			if (Client[a]->GetAccNum() == acctnum)
   				break;
   		}
   
   		if (a == now)
   		{
   			SetOffline(acctnum);
   			return false;
   		}

[frogn8r]

I got this one :)
All you need is a final closing bracket at the bottom to finish your bool (function(?)