1

Имеется такой код:

    var RawRandom = Math.random();

function GetRoundRandom() {
    var MaxValue = document.getElementById('max').value;
    var MaxValueLength = MaxValue.length;
    console.log("MaxValueLength: ", MaxValueLength)
    var MaxValueLengthPow = Math.pow(10, MaxValueLength);
    console.log("MaxValueLengthPow: ", MaxValueLengthPow);
    var HalfRound = RawRandom * MaxValueLengthPow;
    console.log("HalfRound: ", HalfRound);
    var StringHalfRound = String(HalfRound);
    console.log("StringHalfRound: ", StringHalfRound);
    var i = 0;
    for (i = 0; i < StringHalfRound.length; i++) {
        if (StringHalfRound[i] == '.') {
            var StringRound = StringHalfRound.substring(0, i);
            console.log("StringRound: ", StringRound);
        }
    }
    var RoundRandom = Number(StringRound);
    console.log("RoundRandom: ", RoundRandom);
    return (RoundRandom);
}

Нужно сделать так чтобы RoundRandom можно было использовать вне функции.

Думал что для этого нужен return, но он не работает (как со скобками так и без них).

16
  • ну так убери var в строке var RoundRandom = Number(StringRound); ..... или пиши var test = GetRoundRandom() 10 дек 2017 в 10:59
  • @АлексейШиманский, и чем это должно помочь? 10 дек 2017 в 11:01
  • @VasiliyRusin что именно? а почему тогда кое-кто этот вариант описал в ответе? 10 дек 2017 в 11:05
  • @АлексейШиманский, речь шла о var RoundRandom = Number(StringRound);, до правки. Смысла данное изменение не имеет. 10 дек 2017 в 11:08
  • Товарищ, топикстартер, Вы бы уже попробовали внести изменения - да посмотрели результат
    – Dmytro
    10 дек 2017 в 11:09

2 ответа 2

1

Ваша функция возвращает RoundRandom cоответственно что бы использовать значение RoundRandom вне функции достаточно задать ее глобально.

var RoundRandom = GetRoundRandom();

UPD. Я бы не рекомендовал использование String() и Number() по идеологическим причинам. Гораздо чаще используются метод .toString() и функции parseInt() для целых, и parseFloat() для вещественных


Весь код

function GetRoundRandom() {
    var MaxValue = document.getElementById('max').value;
    var MaxValueLength = MaxValue.length;
    console.log("MaxValueLength: ", MaxValueLength)
    var MaxValueLengthPow = Math.pow(10, MaxValueLength);
    console.log("MaxValueLengthPow: ", MaxValueLengthPow);
    var HalfRound = RawRandom * MaxValueLengthPow;
    console.log("HalfRound: ", HalfRound);
    var StringHalfRound = String(HalfRound);
    console.log("StringHalfRound: ", StringHalfRound);
    var i = 0;
    for (i = 0; i < StringHalfRound.length; i++) {
        if (StringHalfRound[i] == '.') {
            var StringRound = StringHalfRound.substring(0, i);
            console.log("StringRound: ", StringRound);
        }
    }
    var RoundRandom = Number(StringRound);
    console.log("RoundRandom: ", RoundRandom);
    return RoundRandom;
}

var RoundRandom = GetRoundRandom();
11
  • А вернуть именно через return можно?
    – user272575
    10 дек 2017 в 11:04
  • Мне кажется, такой ответ запутывает. Особенно, если человек не знает про области видимости переменных. Для ответа лучше использовать промежуточную переменную
    – Dmytro
    10 дек 2017 в 11:05
  • Ну так в коде вы и возвращаете return RoundRandom; 10 дек 2017 в 11:06
  • Консоль выдает ошибку: RoundRandom is not defined
    – user272575
    10 дек 2017 в 11:08
  • @brachkoff, сверьте ваш код с тем что привел я. 10 дек 2017 в 11:10
1

Код получился такой:

function GetRandom() {
    var RawRandom = Math.random();

    function GetRoundRandom() {
        var MaxValue = document.getElementById('max').value;
        var MaxValueLength = MaxValue.length;
        console.log("MaxValueLength: ", MaxValueLength)
        var MaxValueLengthPow = Math.pow(10, MaxValueLength);
        console.log("MaxValueLengthPow: ", MaxValueLengthPow);
        var HalfRound = RawRandom * MaxValueLengthPow;
        console.log("HalfRound: ", HalfRound);
        var StringHalfRound = String(HalfRound);
        console.log("StringHalfRound: ", StringHalfRound);
        var i = 0;
        for (i = 0; i < StringHalfRound.length; i++) {
            if (StringHalfRound[i] == '.') {
                var StringRound = StringHalfRound.substring(0, i);
                console.log("StringRound: ", StringRound);
            }
        }
        var RoundRandom = Number(StringRound);
        console.log("RoundRandom: ", RoundRandom);
        return RoundRandom;
    }
    GetRoundRandom();
    var RoundRandom = GetRoundRandom();
    alert(RoundRandom);
}
1
  • 1
    Вам не нужно дважды вызвать GetRoundRandom(); 10 дек 2017 в 11:51

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.