Javascript-форум (https://javascript.ru/forum/)
-   Events/DOM/Window (https://javascript.ru/forum/events/)
-   -   Вывести случайное число от 1 до 20 (https://javascript.ru/forum/events/20963-vyvesti-sluchajjnoe-chislo-ot-1-do-20-a.html)

devote 27.08.2011 01:47

Цитата:

Сообщение от nikita.mmf (Сообщение 122632)
Можно использовать еще вот такой способ, в нем не надо думать об индексах
for ( var list = [], i = 0; i < 20; ) {
	list[ i ] = ++i;
}
list.sort(function(){ return Math.random() - .5; });
  
function() { 
    var element = new Element('img', {src: 'kart/' + list.shift() + '.png'});
    $(myDiv).adopt(element); 
}

Во первых ему нужно от 1-20, во вторых ему нужно при клике что бы выводилась случайная картинка, и не повторялись индексы, а когда они закончатся все шло заново... в твоем варианте, заново ничего не будет идти, все индексы закончатся и будет пустой массив, тоесть он не обновиться... Читай вопрос внимательнее перед тем как приводить свой пример.

nikita.mmf 29.08.2011 13:07

devote,
специально прочитал вопрос и не нашел там место где требуется массив заполнять занова, и внимательно посмотрите код прежде чем критиковать, в list хранятся значения от 1 до 20 и каждый раз при вызове функции, будет браться число случайное число от 1 до 20, где не соответствие условию?

devote 29.08.2011 13:22

Цитата:

Сообщение от nikita.mmf
специально прочитал вопрос и не нашел там место где требуется массив заполнять занова

Верно говоришь, видимо я автоматом додумал сам... Ибо тупо делать кнопку на которую 20 раз жмакнул и потом ошибка.
Цитата:

Сообщение от nikita.mmf
и внимательно посмотрите код прежде чем критиковать

Да не заметил сложение перед назначением.


Часовой пояс GMT +3, время: 11:48.