Как получить значение из вызова API и сохранить его в переменной и обновить запись DynamodB - PullRequest
Новая
       6

Как получить значение из вызова API и сохранить его в переменной и обновить запись DynamodB

0 голосов
/ 22 января 2019

У меня есть веб-запрос, который получает некоторые данные и возвращает ответ. Я ищу, как сохранить ответ в переменной, чтобы его можно было использовать позже в коде.

Это для узла js, работающего в лямбда-функции

/ ** * Выполняет операции по управлению транспортными средствами, взаимодействуя в основном с * Amazon DynamoDB таблица. * * @class автомобиль * / 

/**
 * Registers a vehicle to and owner.
 * @param {JSON} ticket - authentication ticket
 * @param {JSON} vehicle - vehicle object
 * @param {createVehicle~callback} cb - The callback that handles the response.
 */
vehicle.prototype.createVehicle = function(ticket, vehicle, cb) {
    let vehicle_data = [];
    vehicle_data.push(vehicle);
    let vin_data = _.pluck(vehicle_data, 'vin');
    let vin_number = vin_data[0];
    console.log(vin_number);

    var options = {
        url: 'https://vindecoder.p.mashape.com/decode_vin?' + 'vin=' + vin_number,
        headers: {"X-Mashape-Key": "XXXXXXXXXXXXXXXXXXXXXXXXXX","Accept": "application/json"}
      };

      var data;

      function callback(error, response, body) {
        if (!error && response.statusCode == 200) {
          var result = JSON.parse(body);
          var data = result['specification'];
          //console.log(data);
        }
      }
    request(options, callback);

    var year = data['year'];
    var make = data['make'];
    var model = data['model'];
    var trim_level = data['trim_level'];
    var engine = data['engine'];
    var body_style = data['style'];
    var made_in = data['made_in'];
    var steering_type = data['steering_type'];
    var anti_brake_system = data['anti_brake_system'];
    var fuel_tank = data['tank_size'];
    var overall_height = data['overall_height'];
    var overall_length = data['overall_length'];
    var overall_width = data['overall_width'];
    var standard_seating = data['standard_seating'];
    var optional_seating = data['optional_seating'];
    var highway_mileage = data['highway_mileage'];
    var city_mileage = data['city_mileage'];

    vehicle.owner_id = ticket['cognito:username'];
    // vehicle.vehicle_year = year;
    // vehicle.make = make;
    // vehicle.model = model;
    // vehicle.trim_level = trim_level;
    // vehicle.engine = engine;
    // vehicle.body_style = style;
    // vehicle.made_in = made_in;
    // vehicle.steering_type = steering_type;
    // vehicle.anti_brake_system = anti_brake_system;
    // vehicle.fuel_tank = fuel_tank;
    // vehicle.overall_height = overall_height;
    // vehicle.overall_length = overall_length;
    // vehicle.overall_width = overall_width;
    // vehicle.standard_seating = standard_seating;
    // vehicle.optional_seating = optional_seating;
    // vehicle.highway_mileage = highway_mileage;
    // vehicle.city_mileage = city_mileage;

    let params = {
        TableName: ddbTable,
        Item: vehicle
    };

    let docClient = new AWS.DynamoDB.DocumentClient(dynamoConfig);
    docClient.put(params, function(err, data) {
        if (err) {
            console.log(err);
            return cb(err, null);
        }

        return cb(null, vehicle);
    });

};

Я ожидаю, что ответ от вызова API будет сохранен в объекте, чтобы его можно было использовать для обновления записи DynamodB

1 Ответ

0 голосов
/ 22 января 2019

В вашем коде есть различные проблемы. Смотрите комментарии, которые я оставил в коде. Ваша функция не работает из-за асинхронного характера JavaScript. По сути, у вас есть обратный вызов запроса с результатом, который никогда не будет виден остальной частью вашего кода. Использование обещаний и async / await - один из простых способов решения этой проблемы. Смотрите ниже: 

    // use request promise instead of request
    // promises will make your life much easier
    // https://hackernoon.com/javascript-promises-and-why-async-await-wins-the-battle-4fc9d15d509f
    // https://github.com/request/request-promise
    const request = require('request-promise')

    vehicle.prototype.createVehicle = function(ticket, vehicle, cb) {
        // No need to use var anymore with es6, just use let and const
        // https://www.sitepoint.com/es6-let-const/

        let vehicle_data = [];
        vehicle_data.push(vehicle);
        let vin_data = _.pluck(vehicle_data, "vin");
        let vin_number = vin_data[0];
        console.log(vin_number);

        const options = {
            uri: "https://vindecoder.p.mashape.com/decode_vin?" + "vin=" + vin_number,
            headers: {
                "X-Mashape-Key": "XXXXXXXXXXXXXXXXXXXXXXXXXX",
                Accept: "application/json"
            }
        };


      // Here's the main mistake
      // request has a callback function that is asynchronous
      // the rest of your code never sees the result because the result doesn't leave the callback
      // your code continues to execute without waiting for the result (this is the gist of asynchronity in js)

      // function callback(error, response, body) {
      //   if (!error && response.statusCode == 200) {
      //     const result = JSON.parse(body);
      //     const data = result["specification"]; // this variable doesn't leave the scope of this callback
      //     //console.log(data);
      //   }
      // }

      // therefore this is the failure point
      // request(options, callback);

      // Do this instead
      // here I utilize promises and async/ await
      // https://medium.com/codebuddies/getting-to-know-asynchronous-javascript-callbacks-promises-and-async-await-17e0673281ee
      try {
        const result = await request(options);
        const data = result["specification"];

      } catch (error) {
        console.log(error);
      }

      // Now data is available to be used below in dynamodb

      // also, utilize objects, its much cleaner
      const carVariables = {
          year: data["year"],
          make: data["make"],
          model: data["model"],
          trim_level: data["trim_level"],
          engine: data["engine"],
          body_style: data["style"],
          made_in: data["made_in"],
          steering_type: data["steering_type"],
          anti_brake_system: data["anti_brake_system"],
          fuel_tank: data["tank_size"],
          overall_height: data["overall_height"],
          overall_length: data["overall_length"],
          overall_width: data["overall_width"],
          standard_seating: data["standard_seating"],
          optional_seating: data["optional_seating"],
          highway_mileage: data["highway_mileage"],
          city_mileage: data["city_mileage"],
      };


        vehicle.owner_id = ticket["cognito:username"];
        vehicle = { ...vehicle, ...carVariables} // ES6 spread operator does the below code for you:
        // one line versus 20. win. // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Spread_syntax

        // vehicle.vehicle_year = year;
        // vehicle.make = make;
        // vehicle.model = model;
        // vehicle.trim_level = trim_level;
        // vehicle.engine = engine;
        // vehicle.body_style = style;
        // vehicle.made_in = made_in;
        // vehicle.steering_type = steering_type;
        // vehicle.anti_brake_system = anti_brake_system;
        // vehicle.fuel_tank = fuel_tank;
        // vehicle.overall_height = overall_height;
        // vehicle.overall_length = overall_length;
        // vehicle.overall_width = overall_width;
        // vehicle.standard_seating = standard_seating;
        // vehicle.optional_seating = optional_seating;
        // vehicle.highway_mileage = highway_mileage;
        // vehicle.city_mileage = city_mileage;

        let params = {
            TableName: ddbTable,
            Item: vehicle
        };

        // This will now probably work. yay!

        let docClient = new AWS.DynamoDB.DocumentClient(dynamoConfig);
        docClient.put(params, function(err, data) {
            if (err) {
                console.log(err);
                return cb(err, null);
            }

            return cb(null, vehicle);
        });
    };

Это не проверено, но я считаю, что это должно работать. И здесь могут быть некоторые опечатки или что-то еще, но главное здесь - использование обещаний и async / await для ожидания и предоставления результата из запроса. См. Ссылки и ссылки в комментариях для дальнейшего чтения.

...