C++ operation overload, crash

Question

Why does this code crash the program when I run it

ostream& operator<<(ostream& cout, Array<int> a) {

    return cout;
}

and this doesn't

ostream& operator<<(ostream& cout, Array<int>& a) {

    return cout;
}

How do you run it? Could you post a little context? What is Array? — Juraj Blaho, Dec 12 '10 at 13:55
You have to show what is Array! Most probably you are allocating memory, and you have no copy constructor plus the destructor deallocates that memory. — Khaled Alshaya, Dec 12 '10 at 13:56

score 1 · Accepted Answer · edited May 23 '17 at 11:55

1

What does the copy constructor for Array<int> do ? See if reading the first answer to What is The Rule of Three? solves your problem (namely, that your class handles internally a pointer to a resource, but fails to perform a deep copy in its copy constructor, resulting in two instances deleting the same resource).

edited May 23 '17 at 11:55

Community

1
1

answered Dec 12 '10 at 13:56

Victor Nicollet

24,361
4
58
89

score 1 · Answer 2 · answered Dec 12 '10 at 13:59

1

The overwhelming probability is that your Array<int>'s copy constructor or destructor are screwed. In addition to that, you've got some serious namespacing problems- you've used namespace std for the ostream, but then called your argument cout, which is a conflict with std::cout. I'm amazed this code compiles- you should always use std:: for Standard names, because otherwise is just ambiguous.

answered Dec 12 '10 at 13:59

Puppy

144,682
38
256
465

1

There's no conflict: arguments shadow global and member variables with the same name silently. – Victor Nicollet Dec 12 '10 at 14:05

C++ operation overload, crash

2 Answers2