2

Não consigo obter o valor retornado da função, qual seria o motivo? Alguém poderia me ajudar?

Em minha página HTML tenho a seguinte função:

function montarCursos(){
            var cursos = controller.getCursos();

            if(cursos != null){
                for(var i = 0; i < cursos.length; i++){
                    $("#cursos").append('<button class="ui-btn ui-corner-all" data-transition ="slide" onclick="controller.setNomeCurso("' + cursos[i].Value +'")">' + cursos[i].Name + '</button>')
                }
            }   
        }

Controller.js

var controller = {
getCursos: function () {

        function carregarCursos() {
            var resultado = [];
            var cursos = new cursosDao();
            resultado = cursos.getCursos();
            if (resultado.length > 0) {
                return resultado;
            } else {
                return null;
            }
        }
        loadDependence("Dao/cursosDao.js", carregarCursos);
    }
}

loadDependence.js:

function loadDependence(url, callback) {

    var head = document.getElementsByTagName('head')[0];
    var script = document.createElement('script');
    script.type = 'text/javascript';
    script.src = url;
    script.onreadystatechange = callback;
    script.onload = callback;
    head.appendChild(script);
}

cursosDao.js

function cursosDao() {
    this.getCursos = function() {
        var cursos = [];
        cursos.push({
            "Value" : "Anáse de Sistemas",
            "Name" : "Anáse de Sistemas"
        });

        cursos.push({
            "Value" : "Ciência da Computação",
            "Name" : "Ciência da Computação"
        });

        return cursos;
    }
}
2
  • 2
    Seria melhor você colocar o código aqui no lugar de um link para um site externo.
    – Otávio
    4/06/2015 às 21:23
  • Pergunta editada, me desculpem... 4/06/2015 às 21:30

1 Resposta 1

4

O problema é que o return dentro da funcão carregarCursos só se aplica ao carregarCursos. Ele não vai "pular pra fora" e servir para retornar um valor pro getCursos. Na forma que seu código está escrito, getCursos vai sempre retornar undefined, sem esperar que os cursos sejam carregados.

Uma maneira de verificar isso é botar um console.log no final:

getCursos: function () {
    /*...*/
    loadDependence("Dao/cursosDao.js", carregarCursos);
    console.log("Nao está esperando pelo loadDependence...");
    return 17; // E está chegando no final da função 
}

Infelizmente, não tem uma meneira bonita de resolver esse problema em Javascript. Você vai ter que fazer o getCursos usar uma API com callbacks ao invés de retornar a lista de cursos.

getCursos: function (onDone) {
    loadDependence("Dao/cursosDao.js", function() {
        var resultado = [];
        var cursos = new cursosDao();
        cursos.getCursos(function(resultado){
            if (resultado.length > 0) {
                onDone(resultado);
            } else {
                onDone(null);
            }
        });
    });
}

controller.getCursos(function(cursos){
  if(cursos != null){
    for(var i = 0; i < cursos.length; i++){
      /*...*/
    }
  }
}

Nesse exemplo também assumi que o cursos.getCursos pode ter que fazer uma requisição assíncrona e portanto também precisaria retornar o valor via callback. Se não precisar fazer request pode continuar usando a interface síncrona que você já estava usando.

4
  • muito obrigado pela resposta, mas mesmo depois da alteração infelizmente ainda não consigo obter o valor retornado. Existe alguma outra alternativa para fazer tudo isso? Eu gostaria de deixar bem estruturado o projeto para eventuais manutenções. 4/06/2015 às 21:57
  • Converter o código pra usar continuações ao invés de retornos é algo que vai dar trabalho e é uma das maiores chatices de trabalhar com Javascript. E o resultado fica feio mesmo. (Dê uma pesquisada por promises e por generators se você quiser domar as callbacks)
    – hugomg
    4/06/2015 às 22:12
  • Obrigado mesmo assim, vou aceitar sua resposta. Eu estava lutando pra não colocar as tags script na página HTML manualmente, estava deixando dinâmico. Mas acho melhor eu simplificar meu trabalho. 4/06/2015 às 22:16
  • Se o problema são só os módulos carregados na página, tem umas ferramentas como o browserify que permitem que você requisite os módulos no seu Javascript sem ter que adicionar um monte de tags no html. Basicamente, ele pega todos os arquivos que precisar e concatena num "arquivão" só.
    – hugomg
    4/06/2015 às 23:01

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .