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;
});

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. В других случаях перемещения переменной или еще чего - что код слишком рекурсивен. Уже совсем запуталась

11
  • Тот кусок кода нужен, так как если бы он был правильный, то всё бы работало.
    – diralik
    Commented 11 сент. 2017 в 16:36
  • выводит Nan - значит GoodObj.lala не число... Commented 11 сент. 2017 в 16:37
  • 1
    просто посмотрите что у вас в GoodObj.lala, не надо ссылок на картинки, которые у меня еще и не грузятся :) Commented 11 сент. 2017 в 16:39
  • 1
    Скорее всего у вас не подключен jQuery, а вы пытаетесь делать $(document).ready() Commented 11 сент. 2017 в 16:42
  • 1
    @Rostyslav Kuzmovych, ссылка битая, лишние символы поудаляй просто. ВОТ Commented 11 сент. 2017 в 16:44

1 ответ 1

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>

8
  • Спасибо огромное С:
    – Bipa
    Commented 11 сент. 2017 в 17:28
  • @Біпач незачто, в следующий раз сразу давайте весь код, быстрее будет :) Commented 11 сент. 2017 в 17:33
  • 1
    @RostyslavKuzmovych не учите плохому. Давать надо MCVE, а не "весь код". Commented 12 сент. 2017 в 8:25
  • 1
    @Біпач Вот тут почитайте Commented 12 сент. 2017 в 11:03
  • 1
    @Біпач ничего сложного, надо просто сокращать код пока ошибка продолжает воспроизводиться. К примеру, в вашем коде можно было бы убрать переменные b и trololo, а также функции myFunction и randomInt. Commented 12 сент. 2017 в 11:28

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.