Сallback request node js как получить данные этого метода в requeste?

Question

[lex bem]

Доброе время суток, господа и дамы)

недавно перешел на node js и программирую не так долго и что то из учебника явно не могу понять.

я использую node + ym и не могу понять как мне прокинуть данные после request внутри метода модуля

modules.define('clients', function(provide){

	provide({

		getClients: function(){

			/* Получаем список суб-клиентов агенства (или представителей) */

				var sendQ = {
				   "method": "GetClientsList",
				   "token": ""
				};

				var options = {
					url: "САЙТ",
					method: "POST",
					headers: {
					    'Content-Type': 'application/json; charset=utf-8',
					    'Authorization': 'Bearer'
					},
					json: true,
					body: sendQ
				};

				var clients = '';

				request(options, function (err, res, body) {
					var clients = body["data"];
					var client = clients.map(function(a){
						return a.Login;
					});
					// console.log(body);

					if(client[0] != false){
						console.log('Client list getty - OK - '+client[0]);
					}else {
						console.log('some problems');
					};
					return
				});

			return

			/* Список получен переходим к дальнейщим действиям */
		}

	});
});

Answer 1

[Дмитрий Беляев]

Во-первых, если request - это одноименный модуль из npm - то в параметр body он передает Buffer, если там json, то его надо спарситьvar clients = JSON.parse(body.toString()).data
Во-вторых, сам запрос асинхронный, проще всего обернуть его в промис, итого будет что-то вроде этого:

modules.define('clients', function(provide){

  provide({

    getClients() { //неименнованные функции - плохо для отладки, поэтому меняем на es2015 метод

      /* Получаем список суб-клиентов агенства (или представителей) */

        var sendQ = {
           "method": "GetClientsList",
           "token": ""
        };

        var options = {
          url: "САЙТ",
          method: "POST",
          headers: {
              'Content-Type': 'application/json; charset=utf-8',
              'Authorization': 'Bearer'
          },
          json: true,
          body: sendQ
        };

        //удалить, оно тут не надо
        //var clients = '';

        return new Promise((resolve, reject) => request(options, (err, res, body) => {
          if(err) {
            return reject(err); //прокидываем ошибку в промис
          }
          var clients = JSON.parse(body.toString()).data;
          var client = clients.map(a => a.Login);

          if(!client[0]) {
            console.log('Client list getty - OK - ' + client[0]);
            resolve(client[0]); //возвращаем client[0] из промиса
          } else {
            console.log('some problems');
            reject(new Error('some problems'));
          };
        })).then(client => {
      /* Список получен переходим к дальнейщим действиям */
        }).catch(err => {
          /* здесь обрабатываем ошибку */
        });
    }
  });
});

Comments

[Виталий]

Зря про кетч забыли, все ошибки утонут

[lex bem]

Виталий: Привет, утонут то есть не прокинуться дальше? если не выловить через catch ? наверно это не правильно но я пишу микротесты с консольлогом по всему приложение и ориентируюсь на него в плане ошибок глобальных, местные решаю в процессе написания.

[Виталий]

bemdev: все что попадет в реджект утонет

[lex bem]

Виталий: могли бы вы дополнить код или написать пример с кетчем?

[Дмитрий Беляев]

Виталий: bemdev: дополнил ответ

[lex bem]

Дмитрий Беляев: Promise pending когда использую модуль в другом модуле не пойму как результат всего кода грубо говоря прокинуть через метод в другой модуль

[lex bem]

Дмитрий Беляев: ну ладно я могу получить значение через .then в другом модуле. нельзя ли как то просто получить по методу конечное значение что бы не загромождать сл модуль. допустим я там пишу a.push(getClients()); и складываю результаты в новый объект и все. можно как то так сделать?

[Дмитрий Беляев]

bemdev: прочитайте про асинхронность и вопрос отпадет сам собой