Retorno assincrono encadeando promessas

[Luciano Silveira Marco 0]

Pessoal, boa noite!

 

Gostaria de ver a possibilidade de implementar o "Encadeando promessas" neste código, de modo em que eu possa atribuir o retorno da funcão CalculaDistancia para alguma variável, o "xtalk" é o valor que eu preciso, mas só consigo utilizar dentro da função, preciso deste valor atribuído:

var xvar = CalculaDistancia(xxxxxx,xxxxxx);

O resultado desta função já vem de um encadeamento que consegui fazer rodar olhando exemplos, só que agora preciso subir mais um nível conforme descrevi.

Agradeço quem puder ajudar ou tiver alguma idéia.

function testadistancia() {          
          CalculaDistancia(mapcord[0].varlat+','+mapcord[0].varlon,mapcord[1].varlat+','+mapcord[1].varlon)
          .done(function(response){
                var origins = response.originAddresses;
                for (var i = 0; i < origins.length; i++) {
                     var results = response.rows[i].elements;
                     for (var j = 0; j < results.length; j++) { 
                          var xtroca = substituiVirgula(results[j].distance.text);
                          xtalk=parseFloat(xtroca);
                          alert(xtalk);
                      }
                }
          })         
        }

[William Bruno 1501]

você nunca vai ser capaz de fazer isso:

var xvar = CalculaDistancia(xxxxxx,xxxxxx);

porque é algo assincrono.

Encadear promessas vai melhorar a legibilidade do código, mas não vai te dar a possibilidade de atribuir, porque é assíncrono.

[Luciano Silveira Marco 0]

 

Hum, bom ok, achei que encadear promessas pudesse aproximar pelo menos a solução, teria alguma outra idéia. Obrigado!

[William Bruno 1501]

Você não precisa realmente atribuir. Você pode sim trabalhar com um sistema assíncrono sem isso.

 

Faça um callback:

 

function testadistancia(callback) {
  CalculaDistancia(mapcord[0].varlat+','+mapcord[0].varlon,mapcord[1].varlat+','+mapcord[1].varlon)
  .done(function(response){
        var origins = response.originAddresses;
        for (var i = 0; i < origins.length; i++) {
             var results = response.rows[i].elements;
             for (var j = 0; j < results.length; j++) {
                  var xtroca = substituiVirgula(results[j].distance.text);
                  xtalk=parseFloat(xtroca);
                  callback(null, xtalk);
              }
        }
  })
}

ai você usa:

 testadistancia(function(err, xtalk){
   alert(xtalk);
});

Note que deixei o primeiro argumento como err, porque esse é um padrão bem aceito na comunidade.

 

Agora você só precisa ver se é realmente isso, porque está dentro de 2 loops ali.

O resultado esperado deveria ser o que? um array?

[Luciano Silveira Marco 0]

 

Sim, um array, vou estudar sua implementação e ver se consigo encaixar no problema e dou retorno se funcionou. Agradeço muito pela ajuda e pelas informações, muito obrigado mesmo!

[Luciano Silveira Marco 0]

 

 

Bruno, estou tentando fazer a implementação conforme seu código, e como eu não expliquei o formato em que está o array que preciso organizar, no final do exemplo do seu código eu acredito que está modificando os valores de origem da array e não a ordem:

o array de coordenada que tenho é mapcord{('varlat': xxxxxxx, 'varlon':xxxxx)}

esse array é preenchido aleatoriamente por coordenadas de endereços, logo preciso otimizar esta rota, então utilizo uma função que que retorna a distancia entre duas coordenadas partindo sempre da posição 0, então a idéia é manter o modelo do array mas trocar as posições conforme proximidade preservando somente a posição 0 que é base.

 

Aqui está o modelo que você sugeriu e tentei implementar, mas aparece um erro informando que a propriedade 'varlat' não existe, porque acredito que ela está sendo mudada no processo, confere?

function reoganizaproximos() {
      for(var i = 0; i < mapcord.length - 1; i++){
      var disMinima=[];
      disMinima.push({'index':0, 'distancia':0});
      for(var f = i + 1; f < mapcord.length; f++){
      testadistancia(function(err, xtalk, i, f){
		var distancia = xtalk;

		if(distancia < disMinima.distancia){
			disMinima.index = f;
			disMinima.distancia = distancia;
		}
	   });	
	 }

	 if(disMinima.index !== i + 1){

		var mapcordTemp = mapcord[i + 1];

		mapcord[i + 1] = mapcord[disMinima.index];
		mapcord[disMinima.index] = mapcordTemp;
	 }
      }
      
     }

[William Bruno 1501]

Putz... não cara.. isso tá muito complicado.

Veja, como é um array, deveríamos chamar o callback uma única vez, fora do loop:

 

function testadistancia(callback) {
  CalculaDistancia(mapcord[0].varlat+','+mapcord[0].varlon,mapcord[1].varlat+','+mapcord[1].varlon)
  .done(function(response){
        var origins = response.originAddresses;
        var talks = [];
        for (var i = 0; i < origins.length; i++) {
             var results = response.rows[i].elements;
             for (var j = 0; j < results.length; j++) {
                  var xtroca = substituiVirgula(results[j].distance.text);
                  xtalks.push(parseFloat(xtroca));

              }
        }
        callback(null, xtalks);//chamar aqui
  })
}

entende?

 

Assim no retorno do callback você tem um array, e ele será chamado uma única vez, quando terminar todo o loop.

 

Eu ainda estou um pouco perdido com o que isso faz e como faz, mas analisando só o fluxo, isso aqui:

 

 if(disMinima.index !== i + 1){

está fora de onde deveria, porque essa variável só existe dentro do callback lá em cima.

 

 

Além disso, outro loop? putz.. acho que você não precisa desse outro.

 

Tem alguma chance de escrevermos cenários de testes para o que você quer fazer?

E dai sim, implementarmos ?

[Luciano Silveira Marco 0]

Putz... não cara.. isso tá muito complicado.

Veja, como é um array, deveríamos chamar o callback uma única vez, fora do loop:

 

function testadistancia(callback) {
  CalculaDistancia(mapcord[0].varlat+','+mapcord[0].varlon,mapcord[1].varlat+','+mapcord[1].varlon)
  .done(function(response){
        var origins = response.originAddresses;
        var talks = [];
        for (var i = 0; i < origins.length; i++) {
             var results = response.rows[i].elements;
             for (var j = 0; j < results.length; j++) {
                  var xtroca = substituiVirgula(results[j].distance.text);
                  xtalks.push(parseFloat(xtroca));

              }
        }
        callback(null, xtalks);//chamar aqui
  })
}

entende?

 

Assim no retorno do callback você tem um array, e ele será chamado uma única vez, quando terminar todo o loop.

 

Eu ainda estou um pouco perdido com o que isso faz e como faz, mas analisando só o fluxo, isso aqui:

 

 if(disMinima.index !== i + 1){

está fora de onde deveria, porque essa variável só existe dentro do callback lá em cima.

 

 

Além disso, outro loop? putz.. acho que você não precisa desse outro.

 

Tem alguma chance de escrevermos cenários de testes para o que você quer fazer?

E dai sim, implementarmos ?

 

Claro, deixa eu tentar explicar ver se consigo sintetizar:

 

Conforme mencionei tenho um entrada de endereços que são transformados em coordenadas e inseridas num array chamado "mapcord{(varlat:xxxxx, varlon:xxxx)}" e tenho uma função chamada "testadistancia" que me devolve um valor float de distancia quando informo duas coordenadas.

Então o que eu preciso é uma função que organize as coordenadas (endereços) em sequência de menor distância partindo de A quem é mais perto, digamos que D, e quem é mais perto de D, digamos que B e quem é mais perto de B..... ou seja o array só preserva a posição 0 que é a base e as outras se organizam conforme distância, vou postar o código a partir do array já formado e quem sabe se conseguir reformular, porque não consegui achar uma forma ainda pela minha pouca experiência em javascript.

function reoganizaproximos() {
      for(var i = 0; i < mapcord.length - 1; i++){
      var disMinima=[];
      disMinima.push({'index':0, 'distancia':0});
      for(var f = i + 1; f < mapcord.length; f++){
      testadistancia(function(err, xtalk, i, f){
		var distancia = xtalk;

		if(distancia < disMinima.distancia){
			disMinima.index = f;
			disMinima.distancia = distancia;
		}
	   });	
	 }

	 if(disMinima.index !== i + 1){

		var mapcordTemp = mapcord[i + 1];

		mapcord[i + 1] = mapcord[disMinima.index];
		mapcord[disMinima.index] = mapcordTemp;
	 }
      }
      
 }



function testadistancia(callback) {
        CalculaDistancia(mapcord[i].varlat+','+mapcord[i].varlon,mapcord[f].varlat+','+mapcord[f].varlon)
        .done(function(response){
        var origins = response.originAddresses;
        for (var i = 0; i < origins.length; i++) {
             var results = response.rows[i].elements;
             for (var j = 0; j < results.length; j++) {
                  var xtroca = substituiVirgula(results[j].distance.text);
                  xtalk=parseFloat(xtroca);
                  callback(null, xtalk);
              }
         }
         })
 }

         function substituiVirgula(campo) {
         campo = campo.replace(/,/gi, ".");
         return campo;
         }
     
           function CalculaDistancia(pt1, pt2) { 
                var service = new google.maps.DistanceMatrixService(); 
                var d = $.Deferred();
                service.getDistanceMatrix(
                  { 
                      origins: [pt1], 
                      destinations: [pt2], 
                      travelMode: google.maps.TravelMode.DRIVING, 
                      unitSystem: google.maps.UnitSystem.METRIC,
                      avoidHighways: false,
                      avoidTolls: false
					},   
					 function(response, status){
                      if (status != google.maps.DistanceMatrixStatus.OK) {
                         d.reject(status);
                      } else {
                         d.resolve(response);
                      }
                   });
                   return d.promise(); 
            }

[Luciano Silveira Marco 0]

 

Bom dia William, postei ontem a noite já bem tarde, caso julgue necessárias mais informações fico aguardando e estou tentando achar um caminho aqui também enquanto você analisa o código. Obrigado!