Оптимизация рандомного выбора

[scorp2007]

Здраствуйте, имеется вопрос, вот допустим у меня в игре есть враги, которых можно убивать, я хочу чтобы при смерти они издавали разные вопли (разные звуки), так вот реализовываю так:

int_zvmol = random(1, 5);
switch (int_zvmol)
{
	case 1: 
		zv_mol1s.play();
		break;
	case 2: 
		zv_mol2s.play();
		break;
	case 3: 
		zv_mol3s.play();
		break;
	case 4: 
		zv_mol4s.play();
		break;
}

также делал вот так:

int_zvmol = random(1, 5);
if (int_zvmol == 1)
{
	zv_mol1s.play();
}
if (int_zvmol == 2)
{
	zv_mol2s.play();
}
if (int_zvmol == 3)
{
	zv_mol3s.play();
}
if (int_zvmol == 4)
{
	zv_mol4s.play();
}

Но так и знаю, с точки зрения быстродействия и правильности - какой код лучше? Не будет ли сильно нагружать проверкой такой если в секунду могут умирать до 5-10 врагов (бомбу например скинуть или еще что-то подобное) ? В общем подскажите пожалуйста как правильно такое организовать, или может функцию рандома изменить, сейчас использую такую, нашел где-то, может как-то сократить можно именно под мою нужду.

public function random(min:Number, max:Number):Number
{
	return Math.round(Math.random() * (max - min)) + min;
}

Спасибо.

[Hauts]

Как вариант, можно все звуки смертей врагов записать в массив:

var deathSounds:Array = [zv_mol1s, zv_mol2s, zv_mol3s, zv_mol4s];

А включать произвольный звук так:

deathSounds[Math.floor(deathSounds.length * Math.random())].play();

Разница в том, что вы не получаете некоторый индекс и потом сравниваете его, чтобы найти нужный звук, а сразу же выбираете один из доступных и включаете его. При этом вам не нужно заботиться о том, сколько звуков вообще есть (хоть 5, хоть 105), соответственно, не нужно писать кучу if`ов.

[iNils]

scorp2007, в оформлении кода должна присутствовать табуляция.

[scorp2007]

Сообщение от Hauts

Как вариант, можно все звуки смертей врагов записать в массив:

var deathSounds:Array = [zv_mol1s, zv_mol2s, zv_mol3s, zv_mol4s];

А включать произвольный звук так:

deathSounds[Math.floor(deathSounds.length * Math.random())].play();

Разница в том, что вы не получаете некоторый индекс и потом сравниваете его, чтобы найти нужный звук, а сразу же выбираете один из доступных и включаете его. При этом вам не нужно заботиться о том, сколько звуков вообще есть (хоть 5, хоть 105), соответственно, не нужно писать кучу if`ов.

Спасибо, этим кодом и воспользуюсь, компактнее намного

[expl]

Для 4-х вариантов switch будет работать не медленнее, чем получение одного значения массива через [].
Да и вообще - этот свитч вызывается только один раз в момент смерти, а сколько другого всякого разного при этом происходит, хорошо если switch занимает от этого всего 0.1%.
И надо оно, здесь оптимизировать?

Если бы свитч выдавал значения для какого-нибудь алгоритма, и вызывался раз 1000 за кадр - можно было бы подумать, посмотреть что быстрее работает. А так - руководствуйтесь соображениями удобства/расширяемости.

deathSounds[Math.floor(deathSounds.length * Math.random())].play();

С точки зрения типизации - не очень.
Лучше так:

Sound(deathSounds[Math.floor(deathSounds.length * Math.random())]).play();

Массив по природе - не типизированный, но хоть с написанием play не ошибётесь.
Или так:

var deathSounds:Vector.<Sound> = new <Sound>[zv_mol1s, zv_mol2s, zv_mol3s, zv_mol4s];
...
deathSounds[Math.floor(deathSounds.length * Math.random())].play();

[GBee]

можно и без массива, если все звуки будут называться подобно

int_zvmol = random(1, 5);
this['zv_mol'+int_zvmol+'s'].play()

[Hauts]

expl, вы правы. Просто исходя из сложности вопроса решил не углубляться в типизацию, а ограничиться простой реализацией

[scorp2007]

Всем спасибо, не знал что будет столько вариантов, последний вообще супер