Math.random, Javascript. Что-то пошло не так

Question

var GoodObj = {};

$(document).ready(function() {
    var a = Math.floor(Math.random() * 50);
    var b = Math.floor(Math.random() * 50);
    var trololo = parseInt(a + b);
    GoodObj.lala = trololo;
    document.getElementById('number').innerHTML += a;
    document.getElementById('number').innerHTML += '+';
    document.getElementById('number').innerHTML += b;
});

function myFunction (){
    var  myTrololo = GoodObj.lala;
    var stop = false;
    if (document.getElementById('timer').innerHTML == myTrololo) {
        alert('right');
        return true;
    }
    else {
        alert('false');
        return stop;
    }
}

function randomInt(min,max){
    return Math.floor(Math.random() * (max - min) + min);
}
var param = randomInt(14, GoodObj.lala);
alert(randomInt(14, GoodObj.lala));

Здесь GoodObj.lala - сума двух рандомных чисел. Не думаю, что тот кусок кода нужен. В общем, выводит Nan. В других случаях перемещения переменной или еще чего - что код слишком рекурсивен. Уже совсем запуталась

Тот кусок кода нужен, так как если бы он был правильный, то всё бы работало. — diralik, Sep 11 '17 at 16:36
выводит Nan - значит GoodObj.lala не число... — Rostyslav Kuzmovych, Sep 11 '17 at 16:37
https://pp.userapi.com/c638629/v638629157/5fe86/iI5FmYrALdo.jpg Число же — Bipa, Sep 11 '17 at 16:38
просто посмотрите что у вас в GoodObj.lala, не надо ссылок на картинки, которые у меня еще и не грузятся :) — Rostyslav Kuzmovych, Sep 11 '17 at 16:39
Скорее всего у вас не подключен jQuery, а вы пытаетесь делать $(document).ready() — MedvedevDev, Sep 11 '17 at 16:42
@MedvedevDev как ты это понял ?) на картинке что-то есть ?) — Rostyslav Kuzmovych, Sep 11 '17 at 16:43
уберите эту конструкцию, либо замените на window.onload = () => { /* code */ } — MedvedevDev, Sep 11 '17 at 16:43
@Rostyslav Kuzmovych, ссылка битая, лишние символы поудаляй просто. ВОТ — MedvedevDev, Sep 11 '17 at 16:44
@MedvedevDev у меня всеравно не грузит, видимо изза того что я с Украины :) — Rostyslav Kuzmovych, Sep 11 '17 at 16:45
@MedvedevDevНу, во-первых, jQuery подключен. Во-вторых, код вот там, выше. Можете посмотреть, пожалуйста? — Bipa, Sep 11 '17 at 16:48

score 4 · Accepted Answer · answered Sep 11 '17 at 16:51

4

var GoodObj = {};
$(document).ready(function() {
  var a = Math.floor(Math.random() * 50);
  var b = Math.floor(Math.random() * 50);
  var trololo = parseInt(a + b);
  GoodObj.lala = trololo;
  document.getElementById('number').innerHTML += a;
  document.getElementById('number').innerHTML += '+';
  document.getElementById('number').innerHTML += b;
var param = randomInt(14, GoodObj.lala);
  console.log(randomInt(14, GoodObj.lala));
});
function randomInt(min, max) {
  return Math.floor(Math.random() * (max - min) + min);
}
// этот кусок кода выполняется сразу, тогда, когда инициализация 
// var trololo = parseInt(a + b);
// GoodObj.lala = trololo;
// только после загрузки всего ДОМ дерева
// в итоге GoodObj.lala попросту не существует и ровняется undefined что и дает нам NaN
// просто перенесите в $(document).ready
//var param = randomInt(14, GoodObj.lala);
//console.log(randomInt(14, GoodObj.lala));

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<div id="number"></div>

answered Sep 11 '17 at 16:51

Rostyslav Kuzmovych

5,980

Спасибо огромное С: – Bipa Sep 11 '17 at 17:28
@Біпач незачто, в следующий раз сразу давайте весь код, быстрее будет :) – Rostyslav Kuzmovych Sep 11 '17 at 17:33
1

@RostyslavKuzmovych не учите плохому. Давать надо MCVE, а не "весь код". – Pavel Mayorov Sep 12 '17 at 08:25
@PavelMayorov вы правы :) я чтото на подобе этого и имел в виду. – Rostyslav Kuzmovych Sep 12 '17 at 08:28
@PavelMayorov а что такое MCVE? – Bipa Sep 12 '17 at 10:59
1

@Біпач Вот тут почитайте – Rostyslav Kuzmovych Sep 12 '17 at 11:03
@PavelMayorov, это иногда сложно сочетать – Bipa Sep 12 '17 at 11:05
1

@Біпач ничего сложного, надо просто сокращать код пока ошибка продолжает воспроизводиться. К примеру, в вашем коде можно было бы убрать переменные b и trololo, а также функции myFunction и randomInt. – Pavel Mayorov Sep 12 '17 at 11:28

Math.random, Javascript. Что-то пошло не так

1 Answers1