Author Topic: For each loop  (Read 3901 times)

Offline spartan

  • Single posting newcomer
  • *
  • Posts: 5
For each loop
« on: November 07, 2017, 09:35:26 am »
I'm revising C++ and I got error, but I'm pretty sure this passed in Microsoft Visual Studio, is my syntax really incorrect or is this just Code::Blocks specific issue? Because I wrote for loop just as is in tutorial(version that is commented out works).

int* returnArray(int size)
{
    return new int[size];
}


int main()
{
    int *myarray = returnArray(10);
    for(auto const &index : myarray)
        std::cout << index;
//    for(int i = 0; i < 10; i++)
//        std::cout << myarray << "\n";
    delete[] myarray;
    return 0;
}

Build log:

-------------- Build: Debug in exercise (compiler: GNU GCC Compiler)---------------

g++ -Wall -fexceptions -g -std=c++14  -c /home/uros/Namizje/CodeblocksProjects/exercise/main.cpp -o obj/Debug/main.o
/home/uros/Namizje/CodeblocksProjects/exercise/main.cpp: In function ‘int main()’:
/home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:63:29: error: ‘begin’ was not declared in this scope
     for(auto const &index : myarray)
                             ^
/home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:63:29: note: suggested alternative:
In file included from /usr/include/c++/5/string:51:0,
                 from /usr/include/c++/5/bits/locale_classes.h:40,
                 from /usr/include/c++/5/bits/ios_base.h:41,
                 from /usr/include/c++/5/ios:42,
                 from /usr/include/c++/5/ostream:38,
                 from /usr/include/c++/5/iostream:39,
                 from /home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:1:
/usr/include/c++/5/bits/range_access.h:105:37: note:   ‘std::begin’
   template<typename _Tp> const _Tp* begin(const valarray<_Tp>&);
                                     ^
/home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:63:29: error: ‘end’ was not declared in this scope
     for(auto const &index : myarray)
                             ^
/home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:63:29: note: suggested alternative:
In file included from /usr/include/c++/5/string:51:0,
                 from /usr/include/c++/5/bits/locale_classes.h:40,
                 from /usr/include/c++/5/bits/ios_base.h:41,
                 from /usr/include/c++/5/ios:42,
                 from /usr/include/c++/5/ostream:38,
                 from /usr/include/c++/5/iostream:39,
                 from /home/uros/Namizje/CodeblocksProjects/exercise/main.cpp:1:
/usr/include/c++/5/bits/range_access.h:107:37: note:   ‘std::end’
   template<typename _Tp> const _Tp* end(const valarray<_Tp>&);
                                     ^
Process terminated with status 1 (0 minute(s), 0 second(s))
2 error(s), 0 warning(s) (0 minute(s), 0 second(s))
 

« Last Edit: November 07, 2017, 09:47:38 am by spartan »

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13413
    • Travis build status
Re: For each loop
« Reply #1 on: November 07, 2017, 10:25:01 am »
This is programming question, so a forum rules violation.

The problem is in your code for T *a, the compiler doesn't know the size of the array, so there is no way to implement correct begin and end. In VC you've probably done T a[10]...

You can check your code against different compilers here: https://godbolt.org/
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline spartan

  • Single posting newcomer
  • *
  • Posts: 5
Re: For each loop
« Reply #2 on: November 07, 2017, 10:49:21 am »
Sorry, moderators can delete this thread if neccessary, I copied this code from tutorial and seemed correct to me that's why I posted it here.
« Last Edit: November 07, 2017, 10:51:47 am by spartan »