запутался с индексом массивов

[rambler]

Доброго времени суток пишу метод класса, который должен возвращать случайный индекс. Точнее хочу написать только малость запутался буду рад любой помощи заранее спасибо

package alibrary{
 
	/**
	 * ...
	 * @author rambler
	 */
	public class ArrayAdvanced  {
 
		private static var endIndexArray:Array = [-1];
 
		public function ArrayAdvanced() {
 
 
		}
 
 
	internal function searchArray(theArray:Array, searchElement:Object):int 
	{
		// Проверять каждый элемент, чтобы определить, совпадает ли
		// его значение со значением параметра searchElement
		for (var i:int = 0; i < theArray.length; i++) {
 
			if (theArray[i] == searchElement) 
			{
				return i;
			}
		}
		return -1;
	}
	internal function randomIndexArray(theArray:Array):Array// Проблемный код
	{
		for (var i:int = 0; i < theArray.length; i++)// Обходим массив
		{
			if (theArray[i] == endIndexArray)// Если массив кончился, вернуть случайный индекс
			{
				var randomArray:Array = theArray[Math.floor(Math.random() * i)];
				return randomArray;
			}
 
		}
		return [i];
	}
 
	}
}

[TWETTI]

Зачем здесь цикл? Можно написать просто:

internal function randomIndexArray(theArray:Array):Array// Проблемный код
{
	return Math.round(Math.random() * (theArray.length - 1));
}

[rambler]

да цикл был лишним. немножко скорректировал. спасибо за подсказку совсем упустил .length

internal function randomIndexArray(theArray:Array):Number
	{
 
	return Math.round(Math.random() * (theArray.length - 1));
 
	}

[ChuwY]

round тут лишний.
Нужно просто:

internal function randomIndexArray(theArray:Array):int
{
	return Math.random() * theArray.length;
}

[neonoviiwolf]

ChuwY ага, ищем в массиве элемент с индексом 1,2456.

Добавлено через 2 минуты
TWETTI просто вопрос, а последний элемент когда нибудь сможет выпасть? может вместо "-1" написать "-0,99"?

[ObOs]

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

ChuwY ага, ищем в массиве элемент с индексом 1,2456.

Посмотрите внимательней на тип данных который возвращает метод.

[nubideus]

ага, ищем в массиве элемент с индексом 1,2456.

ага, у нас int с плавающей запятой?


round тут не просто лишний, он левый. тут нужен floor
a[Math.floor(Math.random() * a.length)]

зачем вообще писать целый метод для этого? индусостайл getArrayIndex1(theArray:Array):int{return 1};

a[Math.random() * a.length | 0]

> searchArray
ну есть же indexOf

[КорДум]

Конвертация в int и так сработает как Math.floor(), пусть даже она не явная.

[nubideus]

Конвертация в int и так сработает как Math.floor(), пусть даже она не явная.

это к чему?
надо же показать контекст, а то я могу написать ["asd"][0.5] // undefined
где конвертация?

[rambler]

спасибо за подсказку nubideus (indexOf)