random_shuffle compilation problem

[Ganado]

When I first started C++, I used Dev-C++ 5.4.2. Recently I have tried out Code::Blocks 12.11 using the "GNU CCC Compiler", and I like it more, except for the following issue:

The following code compiles fine, and with no run-time errors, in Dev-C++:

#include <iostream>
#include <cstdlib>
#include <string>
#include <ctime>
#include <algorithm>
#include <iterator>

int main()
{
    std::srand(time(0));
    std::string colors[] = {"Red", "Orange", "Yellow", "Green", "Blue", "Violet", "White", "Gray", "Black", "Magenta", "Cyan", "Brown"};

    std::random_shuffle( begin(colors), end(colors) ) ;

    const int NUMBER_OF_COLORS = 6 ;
    for ( int i = 0; i < NUMBER_OF_COLORS; ++i )
    {
        std::cout << "  " << colors[i] << " Worm" << '\n' ;
    }
}

However, the same code will not compile in Code::Blocks.
The error output message is:

Line 13: error: 'begin' was not declared in this scope
Line 13: error: 'end' was not declared in this scope

Why won't it compile on Code::Blocks? Edit: Sorry if I should've just posted this in CodeBlocks forum, I just thought about that.

Edit 2: I went to Build Options... -> Have g++ follow 1998 ISO C++ language standard, thinking that would fix it, but same compiler issue.

[Hippogriff]

begin/end on arrays are a C++11 only feature. Following the 1998 iso standard will not work.

I have never used codeblocks so I can't be too much more help, but see if there is an option for C++11 in the build options. Make sure you have the latest version of the compiler as well.

[Ganado]

Oh thanks, I thought just having 1998 would be fine due to the C++98 tab on http://www.cplusplus.com/reference/algorithm/random_shuffle/

Yep, it has 2011 and I am pretty sure the compiler is up-to-date, I downloaded it via this link if I remember correctly: http://sourceforge.net/projects/mingw/files/latest/download?source=files

I did this in the settings:
http://i1093.photobucket.com/albums/i434/GanadoFO/CodeBlocks_ISO2011.png
And it compiled correctly.

Thanks a lot!

[cire]

std::begin and std::end are in the std namespace.

[Peter87]

cire wrote:

std::begin and std::end are in the std namespace.

I think you don't need std:: in front of begin and end because of argument-dependent lookup. At least the code compiles for me with GCC 4.7.1.

[cire]

I think you don't need std:: in front of begin and end because of argument-dependent lookup. At least the code compiles for me with GCC 4.7.1.

You are correct. It caught me off guard because colors is an array.

If the element type of colors is changed to int, it won't work -- might be that a using directive would be in order here in case the type is changed.