Node.js - Javascript - Возвращение Asyn c Переменные - PullRequest
0 голосов
/ 07 января 2020

Я работаю над диссон-ботом, который будет принимать случайные значения из созданной мной таблицы Google. Я написал свой файл accessSpreadsheet. js, чтобы он получал данные из Google Sheets, это делается асинхронно. Я могу console.log получить данные, которые мне нужны, из функций asyn c, но я не могу установить другие переменные, возвращая асинхронные данные, вместо этого он возвращает неопределенное значение.

Для начала у меня разногласия. js который принимает входные данные от Discord и вызывает мою функцию giRandom и отвечает пользователю с выводом.

if(input === "!0"){
    var output = giRandom("1"); //this seed just guarantees a "true" outcome for  testing the boolSet function (located below)
    msg.reply(output); //Output's the returned giRandom data (currently replies but output is null)
}

Discord вызывает следующую функцию из giRandom. js Кажется, что все работает здесь, пока мы не достигнем var output = builder(seed_data);

//Returns a random number when fed a seed
function giRandom(seed) {

    //Seperate random seed js file. Working properly.
    var seedrandom = require('seedrandom');
    var rng = seedrandom(seed);

    var seed_Data = seedData(rng()); //Function below
    var output = builder(seed_Data); //Function below //Returning Undefined

    return output; //Returns output to discord.js (msg.reply(output))
}

exports.giRandom = giRandom; //located at end of file, not in a function

Моя функция giRandom вызывает функции seedData и построителя (обе расположены в giRandom. js)

функция seedData

//Takes the seed generated by seedrandom (something like 0.1189181568115987) and seperates each digit into an array, ignoring NaN digits (in this case the decimal point)
function seedData(data){
    var counter = 0; //used to determine amount of NaN digits

    //Counts each NaN in inputArray
    var inputArray = Array.from(data.toString()).map(Number);
    for (var i = 0; i < inputArray.length; i++) {
        if (inputArray[i] >= 0) { //Checks for NaN
            if (debug) {
                console.log("inputArray[" + i + "] | " + inputArray[i]);
            }
        } else { //If NaN counter++
            counter++;
        }
    }

    var outputArray = new Array(counter); //Creates output array with correct size for all NaN values
    counter = 0; //Reset counter to 0

    //Creates outputArray containing all seed digits
    for (var i = 0; i < inputArray.length; i++) {
        if (inputArray[i] >= 0) {
            outputArray[counter] = inputArray[i];
            counter++;
        }
    }

    //Debug function to view output values
    for (var i = 0; i < outputArray.length; i++) {
        if (debug) {
            console.log("outputArray[" + i + "] | " + outputArray[i]);
        }
    }

    return outputArray; //returns outputArray to seed_Data in giRandom function
}

функция компоновщика

//Takes seed_Data from giRandom function and determines the value's output
function builder(data) {

    //Booleans, determine if each category returns a value based on boolSet() (function below)
    var raceBool;

    //data[0] is always a value of 0. 16 Total Usable values 1-17
    raceBool = boolSet(data[1]);


    if (raceBool == true) {
        var raceData = sheetToArray(1);
        console.log("raceData | " + raceData);
        return raceData; //Returning undefined
    }
}

Моя функция компоновщика вызывает функции boolSet и sheetToArray (обе находятся в giRandom. js)

функция boolSet

//Each seed digit can be 0-9. If the digit passed (boolData) matches the value for valueTrue, valueFalse, or valuePick the output will be set accordingly. (Currently will always output True)
function boolSet(boolData) {
    var valueTrue = [0, 1, 2, 3];
    var valueFalse = [4, 5, 6, 7];
    var valuePick = [8, 9];

    //Loop through valueTrue and compare with boolData
    for (var i = 0; i <= valueTrue.length; i++) {
        if (boolData == valueTrue[i]) {
            if (debug) {
                console.log("boolData | " + boolData + " | true");
            }
            return true;
        }
    }

    //Loop through valueFalse and compare with boolData
    for (var i = 0; i <= valueFalse.length; i++) {
        if (boolData == valueFalse[i]) {
            if (debug) {
                console.log("boolData | " + boolData + " | false");
            }
            return false;
        }
    }
    //If !true && !false, must be "pick". This value will allow the end user to choose their data (in this case race)
    return "pick";
}

функция sheetToArray

//takes an int for sheetNum that determines which sheet in the project to load 
function sheetToArray(sheetNum){
    var sheetArray = []; //Array that holds all rows of a particular worksheet, used to build spreadsheetArray
    var cellArray = []; //Array that holds all cells in a particular row of the worksheet used to build sheetArray
    var rowArrayCount = 1; //Counter to determine rows in cells forEach loop

    /*I believe this is where my problem probably lies*/
    //Calling async function accessSpreadsheet from spreadsheet.js
    accessSpreadsheet(sheetNum).then(function (arr) {
        var cells = arr;
        //loop through all cells in current sheet
        cells.forEach(cell => {
            if (cell.row > rowArrayCount) { //True determines that data is now on the next row of current sheet
                rowArrayCount++;
                sheetArray.push(cellArray); //push last row to sheetArray
                cellArray = []; //reset cellArray for new row
            }
            cellArray.push(cell._value); //push final row to cellArray
        })//Exit forEach
        sheetArray.push(cellArray); //push all rows to sheetArray
        var raceLength = sheetArray.length; //Returns value correctly
        console.log("raceLength | " + raceLength); //Returns value correctly
        console.log("sheetArray | " + sheetArray); //Returns value correctly
        return sheetArray; //Returns undefined
    })
}

Для доступа к Google Sheets у меня есть отдельный файл js. Именно здесь асинхронная функция accessSpreadsheet () имеет вид

accessSpreadsheet. js

//Google Sheet Access Const
const GoogleSpreadsheet = require('google-spreadsheet');
const { promisify } = require('util');
const creds = require('./client_secret.json');
const doc = new GoogleSpreadsheet('**************'); //Google sheet we have access to (removed for security)

module.exports = accessSpreadsheet = async (sheetNum) => {
    await promisify(doc.useServiceAccountAuth)(creds);
    const info = await promisify(doc.getInfo)();
    const sheet = info.worksheets[sheetNum];

    //Async get sheet's cells without an offset
    const cells = await promisify(sheet.getCells)({
        offset: 0
    })

    return cells;
}

1 Ответ

0 голосов
/ 07 января 2020

Основы javascript обещаний:

Определение accessSpreadsheet - это что-то вроде

async accessSpreadsheet(args...){
   //Do stuff
   return something;
}

или как:

accessSpreadsheet(args...){
   return Promise(function (resolve,reject){
       // Do stuff
       resolve(return_value)
   })
}

Поскольку javascript является событием- управляемый, вы не можете заставить его ждать ввода / вывода, поскольку это наносит ущерб пользовательскому опыту.

Таким образом, вы можете решить вашу проблему, либо сделав весь код асинхронным c и используя ключевое слово await для ожидания accessSpreadsheet.

Пример:

await accessSpreadsheet()

, который будет ожидать возврата accessSpreadsheet перед переходом к следующей строке.

Это базовый c сценарий состояния гонки с обещаниями.

Вы должны удалить пустое обещание:

const info = await promisify(doc.getInfo)();

можно записать:

const info = doc.getInfo();

и зарегистрировать ошибки:

accessSpreadsheet(sheetNum).then(function (arr) {
   // somecode 
}).catch(console.error)

Это, вероятно, сделает вашу жизнь проще

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...