Передача ссылки в функцию класса на данный класс

[Impuls1989]

Доброго времени суток уважаемые эксперты. Для начала поздравляю всех с праздниками.
Ну а теперь к делу. Необходимо передать в функцию класса ссылку на другую переменную такого же класса (сам бы никогда не понял что тут написал). Собственно вот так:

class CNeuronLayer									
{
public:
	CNeuronLayer();
	void Create(int,const CNeuronLayer &);
	int GetCount();							
	~CNeuronLayer();
private:
	int kolNeuron;				
	CNeuron *neuron;
};

А вот как я описываю данную функцию:

void CNeuronLayer::Create(int KolNeuron,const CNeuronLayer &PredLayer)
{
	this->kolNeuron = KolNeuron;
	if (PredLayer.GetCount() != 0)
	{
		this->neuron = new CNeuron[KolNeuron];
		for(int i=0; i<KolNeuron; i++)
		{
			this->neuron[i].Create(PredLayer.GetCount());
		}
	}
}

Все бы ничего, но в итоге я получаю данную ошибку в двух местах:

error C2662: 'GetCount' : cannot convert 'this' pointer from 'const class CNeuronLayer' to 'class CNeuronLayer &'
        Conversion loses qualifiers

Без константы я получаю ошибку линковщика:

error LNK2001: unresolved external symbol "public: int __thiscall CNeuronLayer::GetCount(void)" (?GetCount@CNeuronLayer@@QAEHXZ)

Собственно весь вопрос состоит в том, чтобы узнать как решить мою проблему. Заранее спасибо.

[_Ч_]

в функцию Create передается константная ссылка на обьект, а внутри этой ф-ии у этого обьекта вызывается неконстантная функция:
if (PredLayer.GetCount() != 0)

Чтобы это вылечить, надо ф-ию GetCount обьявить константной:
void Create(int,const CNeuronLayer &) const;

Судя по второй ошибке (ошибка линковки) нигде не определена сама ф-ия GetCount. Ее надо определить.

[_Ч_]

this->kolNeuron и this->neuron[i] - вот тут this-> можно не писать.

Плюс ко всему вероятны еще утечки памяти. Сам класс как-то криво сделан. Но это уже другая история

[Impuls1989]

Сообщение от _Ч_

Чтобы это вылечить, надо ф-ию GetCount обьявить константной:
void Create(int,const CNeuronLayer &) const;

Я так понимаю - это просто опечатка
Все с этим разобрался - все замечательно заработало.
А можно уточнить почему тут будет утечка памяти и почему класс криво написан (хочется все же правильно написать)

[_Ч_]

нет. это не опечатка. ф-ия Create изменяет состояние обьекта (this->kolNeuron = KolNeuron) поэтому она не может быть константной. константкая именно GetCount

[Impuls1989]

Сообщение от _Ч_

нет. это не опечатка. ф-ия Create изменяет состояние обьекта (this->kolNeuron = KolNeuron) поэтому она не может быть константной. константкая именно GetCount

Я это знаю, просто вы написали Create(int,const CNeuronLayer &) const; а не GetCount() const;. Но не в этом дело. С этой своей ошибкой я уже справился. Хотелось бы все таки узнать: в каком месте произойдет утечка памяти?

[_Ч_]

зачем ф-ии void CNeuronLayer::Create(int KolNeuron,const CNeuronLayer &PredLayer) нужен const CNeuronLayer &PredLayer? чтобы позвать у него GetCount()? почему просто нельзя передвать это количество. (тут была глупость. удалил :-) )
Счетчики обьектов должны имень беззнаковый тип (unsigned например). Ведь количество обьектов не может быть отрицательным.
Это лишь вершина айсберга.
Чтобы написать правильно класс надо сперва описать для чего он нужен, что он делает. А так я вижу что он кривой, но как его сделать правильным я не знаю потому что не знаю для чего он.

Про утечку:
Что будет если два раза позвать ф-ию Create? правильно, будет утечка.

А что будет если я скопирую один обьект CNeuronLayer в другой? Если деструктор написан правильно, то потом может случится трындец.

[_Ч_]

Упс. void Create(int,const CNeuronLayer &) const; - да, это опечатка

[Impuls1989]

Чтобы написать правильно класс надо сперва описать для чего он нужен, что он делает. А так я вижу что он кривой, но как его сделать правильным я не знаю потому что не знаю для чего он.

Начну с этого. Класс является промежуточным между классом нейрон и классом нейронная сеть. Собственно он просто должен обеспечить связь нейронов в один слой, и обеспечить удобный доступ к ним.

зачем ф-ии void CNeuronLayer::Create(int KolNeuron,const CNeuronLayer &PredLayer) нужен const CNeuronLayer &PredLayer? чтобы позвать у него GetCount()? почему просто нельзя передвать это количество.

Признаюсь, сперва я так и планировал, но: тут довольно интересный момент - в процессе работы нейронной сети в слои могут добавляться (либо, что маловероятно, удаляться) нейроны, что, в свою очередь, приведет к автоматической переконфигурации всей сети. В любом случае понадобится дописывать Create (либо писать дополнительную функцию), но так мне будет удобнее брать параметры из предыдущего слоя.

А что будет если KolNeuron > PredLayer.GetCount()?

Так скорее всего и будет. Но, как вы должно быть заметили, KolNeuron - передается в функцию Create класс CNeuron, где он спокойненько себе обрабатывается.
Простите если ввел в заблуждения не приведя реализацию CNeuron

Счетчики объектов должны иметь беззнаковый тип (unsigned например). Ведь количество объектов не может быть отрицательным.

Вот с этим согласен на все 100%. Уже исправил.

Может есть еще какие-нибудь замечания?

[_Ч_]

ок. подумайте пока над этим:
Про утечку:
Что будет если два раза позвать ф-ию Create? правильно, будет утечка.

А что будет если я скопирую один обьект CNeuronLayer в другой? Если деструктор написан правильно, то потом может случится трындец.

Я пока остальное не осилил.