Хочу получить рандомные числа. Что тут не так?

Question

[Айдар]

Есть массив. И в консоли я хочу через каждую секунду получать рандомное число. А у меня либо счетчик застывает после двух итераций или пишет NaN.

var array = ['1', '2', '3', '4', '5', '6', '7'];
var item = array[Math.floor(Math.random()*array.length)];
var i = 0;
  setInterval(function () {
        console.log(item);
  }, 1000);

Answer 1

[imhuman]

Не усложняйте вариант задающего вопрос, все у него правильно было, только item стоило сделать функцией, иначе всегда одинаковый результат был бы. Добавлять -1 к array.length не надо, иначе в сочетании с floor никогда не получим последний элемент массива.

var array = ['1', '2', '3', '4', '5', '6', '7'];
var item = function(){return array[Math.floor(Math.random()*array.length)]};
setInterval(function () {
        console.log(item());
}, 1000);

Comments

[Айдар]

imhuman, спасибо за пример. В консоли все перебирается, но вот на деле проблемы. Есть счетчик и мне нужно рандомно задавать скорость прокрутки (Затем и нужен был код выше).
Нужное место для рандома выделил жирным. Там должно меняться числа заданные в массиве. Но у меня почему-то не перебирает каждый раз новое число, а будто остается какое-то одно.

(function(){
        $('.odometer').each(function(){
            function time(){
                return parseInt(new Date().getTime()/1000);
            }
            
            var array = ['1000', '200', '3000'];
            
            function func() {
               return array[Math.floor(Math.random()*array.length)];
            }
            
            setInterval(func, 100);
            
            var v = time();
            //var v = date.getHours();
            var o = new Odometer({
                el: this,
                value: v,
                format:'',
                theme: $(this).data('theme')
            });
            o.render();
            setInterval(function(){
                //o.update(Math.floor(Math.random()));
                o.update(v++);
            }, func()); // ВОТ ТУТ ДОЛЖНО БЫТЬ РАНДОМНОЕ ЧИСЛО ИЗ МАССИВА
        });
    })();

Answer 2

[xmoonlight]

var item = array[Math.round(Math.random()*(array.length-1))];

Comments

[xmoonlight]

MaxKorz: заменил floor на round, чтобы работало как положено...

[napa3um]

xmoonlight: никогда там не будет круглой единицы.

[xmoonlight]

napa3um: точно, выколота... [0;1) пардон!