Good way to call overloaded const vs non-const method in C++ TDD?

Question

All

wrote some fancy container with iterators. So I have

iterator begin() {
}

iterator end() {
}

const_iterator begin() const {
}

const_iterator end() const {
}

In test I instantiate container, fill it and testing it. What is an accepted/good way to call both const and non-const version? Say, simple test like

TEST( c.end() - c.begin() == c.size() );

should run for iterators as well as for const iterators. Const reference? Some ugly cast?

score 5 · Accepted Answer · answered Jul 29 '16 at 01:18

5

I would just make a const& to it:

const container& constC= c;
TEST(constC.end() - constC.begin() == constC.size());

answered Jul 29 '16 at 01:18

David

score 1 · Answer 2 · answered Jul 29 '16 at 01:15

1

I think not overloading them is a better solution as per std convention.

Declare cbegin() and cend() instead.

answered Jul 29 '16 at 01:15

Dafang Cao

3

the std convention is to do all of the above: `iterator begin();`, `const_iterator begin() const;`, and `const_iterator cbegin() const;` – David Jul 29 '16 at 01:16

2 Answers2