Iterator subclass separate inclusion











up vote
2
down vote

favorite












I'm new to the iterator class; not in how to use iterators for containers but how to implement an iterator class. I have two files, a .h and .cpp. Right now, I'm getting a compiler error in my .cpp file that displays "fatal error: 'iterator' is not a class, namespace, or enumeration". I don't understand why its telling me this.



Note: I'm sure there are other errors in my code, but right now I'm just trying to get the existing compiler error resolved.



Base.h



#ifndef Base_h
#define Base_h

class Base {
protected:
std::vector<std::vector<std::string> > vec;

public:

class iterator {

private:
Base* p = nullptr;
size_t x, y;

public:
iterator() = default;
iterator(Base *, size_t, size_t);
~iterator();
iterator &operator++();
Base operator*() const;
}

iterator begin() const;
iterator end() const;
// bunch of other iterator::functions and Base::functions
};

#endif


Base.cpp



iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
iterator::~iterator();
// Bunch of other iterator::functions including Base::functions


Compiler Error



Base.cc:254:13: fatal error: 'iterator is not a class, namespace, or enumeration
iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
^

/Library/Developer/CommandLineTools/usr/include/c++/v1/iterator:522:29: note: 'iterator' declared here
struct _LIBCPP_TEMPLATE_VIS iterator









share|improve this question


















  • 2




    iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
    – Gian Paolo
    Nov 19 at 23:31












  • I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
    – N. Colostate
    Nov 19 at 23:36






  • 1




    well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
    – Gian Paolo
    Nov 19 at 23:40










  • I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
    – N. Colostate
    Nov 19 at 23:44










  • the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
    – Phil1970
    Nov 20 at 2:01















up vote
2
down vote

favorite












I'm new to the iterator class; not in how to use iterators for containers but how to implement an iterator class. I have two files, a .h and .cpp. Right now, I'm getting a compiler error in my .cpp file that displays "fatal error: 'iterator' is not a class, namespace, or enumeration". I don't understand why its telling me this.



Note: I'm sure there are other errors in my code, but right now I'm just trying to get the existing compiler error resolved.



Base.h



#ifndef Base_h
#define Base_h

class Base {
protected:
std::vector<std::vector<std::string> > vec;

public:

class iterator {

private:
Base* p = nullptr;
size_t x, y;

public:
iterator() = default;
iterator(Base *, size_t, size_t);
~iterator();
iterator &operator++();
Base operator*() const;
}

iterator begin() const;
iterator end() const;
// bunch of other iterator::functions and Base::functions
};

#endif


Base.cpp



iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
iterator::~iterator();
// Bunch of other iterator::functions including Base::functions


Compiler Error



Base.cc:254:13: fatal error: 'iterator is not a class, namespace, or enumeration
iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
^

/Library/Developer/CommandLineTools/usr/include/c++/v1/iterator:522:29: note: 'iterator' declared here
struct _LIBCPP_TEMPLATE_VIS iterator









share|improve this question


















  • 2




    iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
    – Gian Paolo
    Nov 19 at 23:31












  • I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
    – N. Colostate
    Nov 19 at 23:36






  • 1




    well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
    – Gian Paolo
    Nov 19 at 23:40










  • I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
    – N. Colostate
    Nov 19 at 23:44










  • the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
    – Phil1970
    Nov 20 at 2:01













up vote
2
down vote

favorite









up vote
2
down vote

favorite











I'm new to the iterator class; not in how to use iterators for containers but how to implement an iterator class. I have two files, a .h and .cpp. Right now, I'm getting a compiler error in my .cpp file that displays "fatal error: 'iterator' is not a class, namespace, or enumeration". I don't understand why its telling me this.



Note: I'm sure there are other errors in my code, but right now I'm just trying to get the existing compiler error resolved.



Base.h



#ifndef Base_h
#define Base_h

class Base {
protected:
std::vector<std::vector<std::string> > vec;

public:

class iterator {

private:
Base* p = nullptr;
size_t x, y;

public:
iterator() = default;
iterator(Base *, size_t, size_t);
~iterator();
iterator &operator++();
Base operator*() const;
}

iterator begin() const;
iterator end() const;
// bunch of other iterator::functions and Base::functions
};

#endif


Base.cpp



iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
iterator::~iterator();
// Bunch of other iterator::functions including Base::functions


Compiler Error



Base.cc:254:13: fatal error: 'iterator is not a class, namespace, or enumeration
iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
^

/Library/Developer/CommandLineTools/usr/include/c++/v1/iterator:522:29: note: 'iterator' declared here
struct _LIBCPP_TEMPLATE_VIS iterator









share|improve this question













I'm new to the iterator class; not in how to use iterators for containers but how to implement an iterator class. I have two files, a .h and .cpp. Right now, I'm getting a compiler error in my .cpp file that displays "fatal error: 'iterator' is not a class, namespace, or enumeration". I don't understand why its telling me this.



Note: I'm sure there are other errors in my code, but right now I'm just trying to get the existing compiler error resolved.



Base.h



#ifndef Base_h
#define Base_h

class Base {
protected:
std::vector<std::vector<std::string> > vec;

public:

class iterator {

private:
Base* p = nullptr;
size_t x, y;

public:
iterator() = default;
iterator(Base *, size_t, size_t);
~iterator();
iterator &operator++();
Base operator*() const;
}

iterator begin() const;
iterator end() const;
// bunch of other iterator::functions and Base::functions
};

#endif


Base.cpp



iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
iterator::~iterator();
// Bunch of other iterator::functions including Base::functions


Compiler Error



Base.cc:254:13: fatal error: 'iterator is not a class, namespace, or enumeration
iterator::iterator(Base *b, size_t i, size_t j): p(b), x(i), y(j) {}
^

/Library/Developer/CommandLineTools/usr/include/c++/v1/iterator:522:29: note: 'iterator' declared here
struct _LIBCPP_TEMPLATE_VIS iterator






c++ iterator






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 19 at 23:27









N. Colostate

356




356








  • 2




    iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
    – Gian Paolo
    Nov 19 at 23:31












  • I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
    – N. Colostate
    Nov 19 at 23:36






  • 1




    well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
    – Gian Paolo
    Nov 19 at 23:40










  • I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
    – N. Colostate
    Nov 19 at 23:44










  • the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
    – Phil1970
    Nov 20 at 2:01














  • 2




    iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
    – Gian Paolo
    Nov 19 at 23:31












  • I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
    – N. Colostate
    Nov 19 at 23:36






  • 1




    well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
    – Gian Paolo
    Nov 19 at 23:40










  • I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
    – N. Colostate
    Nov 19 at 23:44










  • the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
    – Phil1970
    Nov 20 at 2:01








2




2




iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
– Gian Paolo
Nov 19 at 23:31






iterator is a nested class: try qualifiying the implementation in cpp file with Base::iterator::iterator Or try avoid to declare iterator as nested class
– Gian Paolo
Nov 19 at 23:31














I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
– N. Colostate
Nov 19 at 23:36




I believe that worked. Although, it came up with another compiler error but one I can deal with. Thanks. If all else fails, I may avoid a nested class. Thanks.
– N. Colostate
Nov 19 at 23:36




1




1




well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
– Gian Paolo
Nov 19 at 23:40




well, as a start, why do you need a nested class? (actually, I just asked this myself) see also stackoverflow.com/questions/4571355/…
– Gian Paolo
Nov 19 at 23:40












I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
– N. Colostate
Nov 19 at 23:44




I decided on using a nested class through examples provided from various .edu addresses. So, I assume it was standard practice. From the comments so far, it doesn't seem that way.
– N. Colostate
Nov 19 at 23:44












the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
– Phil1970
Nov 20 at 2:01




the following declaration Base operator*() const; seems suspicious. You really want to return a copy of Base?
– Phil1970
Nov 20 at 2:01

















active

oldest

votes











Your Answer






StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});














draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53384120%2fiterator-subclass-separate-inclusion%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown






























active

oldest

votes













active

oldest

votes









active

oldest

votes






active

oldest

votes
















draft saved

draft discarded




















































Thanks for contributing an answer to Stack Overflow!


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.





Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


Please pay close attention to the following guidance:


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53384120%2fiterator-subclass-separate-inclusion%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

Wiesbaden

Marschland

Dieringhausen