Проблемы с динамическим массивом внутри класса

[Антон (LogRus)]

Finch, не забывай, что коллега не позаботился от инициализации массива Chromos пустыми значениями перед тем как заполнять его данными, так что если вызовется деструктор после неудачного конструирования, то есть большие шансы огрести по полной программе.

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

[romeo]

Еще кто-нибудь может подсказать почему у меня при вызове функции-члена FillRandChrom генерируется одна и та же случайная последовательность для разных экземпляров?  По идее при каждом вызове функции должна генерироваться своя последовательность, тем более, что srand проинициализирована функцией времени. ctime я подключил

Код:

void Chromosome::FillRandChrom()	//заполнение хромосомы случайными значениями генов
{
	srand(time(0));
	for (int i=0; i<GeneQty; i++)
		for (int j=0; j<GeneSize; j++) Chromos[i][j]=(bool)(rand()%2);
}

[Алексей++]

почему одинаковая - не знаю, но вот это

(bool)(rand()%2);

не гарантирует 50/50

если надо примерно 50/50 , то так сделай:

(bool) (RAND_MAX-rand()/2 > 0)

[romeo]

Не знай-не знай, так вообще все единицы генерируются

[Вад]

Может, ( (double)rand() ) / RAND_MAX > 0.5?

[Алексей++]

гы,сорри, наоборот написал, надо было так
(bool) (rand()-RAND_MAX/2 > 0)

или
(bool) (rand() > RAND_MAX/2)


(вот что значит - не тестировать )

[Вад]

Да, последний вариант пооптимальнее с точки зрения вычислений

[Антон (LogRus)]

оптимальней вот так
rand() > (RAND_MAX >> 1)

нафига приведение к bool?

[Алексей++]

деление константы или сдвиг - без разницы, будет всё константа )

а к bool приведено просто так, для наглядности (компилятор всё равно это сделает)