error?? name lookup of 'i' changed for ISO 'for' scoping

Code:

#include <iostream>


using namespace std;


void swap (int array[], int first_index, int second_index);
int findSmallestRemainingElement(int array[], int size, int index);


int store (int array[], int size)
{
    std::cout << "Enter the number(s) you want to store: \n";
    int values, count = size;
    for (int i=0;i<size;i++)
    {
        std::cin >> values;
        array[i] = values;
        count--;
        std::cout << "-----" << count << "----- values left to be entered.\n";
    }


    return true;
}




void sort(int array[], int size)
{
    for ( int i=0; i<size ;i++)
    {
        int index = findSmallestRemainingElement(array,size,i);
        swap(array, i, index );
    }


}


int findSmallestRemainingElement(int array[], int size, int index)
{
    int index_of_smallest_value = index;
    for (int i = index+1 ; i<size ; i++)
    {
        if ( array[i] < array [index_of_smallest_value])
        {
            index_of_smallest_value = i;
        }
    }
    return index_of_smallest_value;
}


void swap (int array[], int first_index, int second_index )
{
    int temp = array[first_index];
    array[first_index] = array[second_index];
   array[second_index] = temp;
}


void displayarray (int array[], int size)
{
    std::cout << "{";


        for (int i = 0 ;i <size ; i++)
        {
            if (i !=0)
            {
                std::cout << ", ";
            }
        }   std::cout << array[i];
    std::cout << "}";
}


int main()
{   std::cout << "How many values would you like to store? \n";
    int size;
    std::cin >> size;


    int array[size];
    store(array, size);
    sort(array,size);
    displayarray(array,size);
    return 0;
}

Compiler error:

\main.cpp||In function 'void displayarray(int*, int)':|
\main.cpp|64|error: name lookup of 'i' changed for ISO 'for' scoping|
\main.cpp|64|note: (if you use '-fpermissive' G++ will accept your code)|
||=== Build finished: 1 errors, 0 warnings ===|

Code:

        for (int i = 0 ;i <size ; i++)
        {
            if (i !=0)
            {
                std::cout << ", ";
            }
        }   std::cout << array[i];

First of all, this demonstrates the importance of really good indentation.
Putting code AFTER a brace results in pretty unreadable code.
Look carefully, does the cout array[i] run for every loop iteration, or just once when the loop exits?

It would help if you indented your code better (and gave the line number):

Code:

void displayarray (int array[], int size)
{
    std::cout << "{";
    for (int i = 0 ;i <size ; i++)
    {
        if (i !=0)
        {
            std::cout << ", ";
        }
    }
    std::cout << array[i];
    std::cout << "}";
}

The problem is that i is declared in the for loop, but you use it after the for loop. Some old compilers may still accept this code as being roughly equivalent to:

Code:

void displayarray (int array[], int size)
{
    std::cout << "{";
    int i;
    for (i = 0 ;i <size ; i++)
    {
        if (i !=0)
        {
            std::cout << ", ";
        }
    }
    std::cout << array[i];
    std::cout << "}";
}

in which case there would be no error, hence the "name lookup of 'i' changed for ISO 'for' scoping" error message.

eek... I didn't intend to put the array out of the for loop. Thanks I'll remember this mistake. Putting code after braces is bad is noted.