issues with objects storing lists

[badass]

so basically i have a row class which stores a list of characters as a doubly linked list
and i have a maze class which i want to store row objects as a doubly linked list

below is my code for the 2 classes

 
class row
{
private:
	list<char> chars;

public:
	row(){}

	row(row &other) {
		chars.clear();
		for(list<char>::iterator itr=other.getFirst(); itr != other.getLast(); itr++)
			chars.push_back(*itr);
		}

	void addChar(char temp){
		chars.push_back(temp);
	}

	list<char>::iterator getFirst() {
		return chars.begin();
	}

	list<char>::iterator getLast() {
		return chars.end();
	}

	int getSize()  {
		return chars.size();
	}

	void printRow() {
		for(list<char>::iterator itr= chars.begin();itr != chars.end();itr++)
			cout << *itr;
	}

};

		
class maze
{
private:
	list<row> rows;

public:
	maze() {}

	maze(maze &other) {
	   rows.clear();
	//   for(list<row>::iterator itr= other.getFirst(); itr != other.getLast(); itr++)
	//	   row.push_back(*itr);
	}

	void addRow(row rows1) {
        rows.push_back(rows1);
	}

	int getSize() {
		return rows.getSize();
	}

	

};

below is the error i get which i have no clue about

1>c:\program files\microsoft visual studio 9.0\vc\include\xmemory(52) : error C2558: class 'row' : no copy constructor available or copy constructor is declared 'explicit'

which takes me to somewhere in a STL template and thats where i lose it :(

any help would be greatly helpful

thanks

[firedraco]

Couldn't you just do:

this->rows = other.rows;

[Albatross]

You should probably use vectors or deques instead... you don't have at with lists, and trust me, you'll want it.

-Albatross

[moorecm]

How about:

typedef list<char> Row;
typedef list<Row> Mase;

[moorecm]

Anyway, on line 10, try:

row( const row & other ) {

EDIT: Also, you'll probably need to use const_iterator--that is, if you do not just use the assignment operator mentioned above.

[badass]

i figured out what the problem is but dont know how to solve...

its when i declare a row object in the main...it doesnt know which constructor to call..thnx to g++ compiler it said candidates:row and row(&row)...the default or the copy....it should be clear that im calling the default constructor since im doing

row rows;

my program works fine if i kill the copy constructor but still im curious

[Albatross]

Well, try explicitly stating the default constructor.

Also, you will need to do what morrecm suggested.

And also, you will need to do what I suggested.

-Albatross

[badass]

and how do u explicitly call default constructor?

i tried

row rows=new rows(); 

but got an error...

also i HAVE to use linked lists since its in the specs

thanks everyone for the great help!
c++ is a monster and we need to join forces to bring it down :)

[jsmith]

+1 moorecm

What OP is calling his copy constructor is not a copy constructor. Copy constructors
must take their argument by const reference.