函数返回undefined时使用

Question

我有一个连接到MsSQL数据库的函数来获取一些数据并构建一个文本语句，这里是：

function getActions(issue_id, callback){
    var action = '';
    var conn2 = new sql.Connection(config, function(err) {
        if(err){
            showNotification('error connecting for selecting actions for ALL issues: ' + err.message, 'danger', 'glyphicon glyphicon-tasks');
        } else {
            var request = new sql.Request(conn2);
            request
            .input('issue_id', sql.Int,issue_id)
            .query('SELECT [date], [description] FROM [actions] WHERE [issue_id] = @issue_id')
            .then(function(data2) {
                action ='<tr>'+
                                '<td style="vertical-align: top;" class="bold">action:</td>'+
                                '<td>'+
                                '<table>';
                data2.forEach(function(data21){
                        action +='<tr>'+
                                    '<td>'+data21.date+'</td>'+
                                    '<td>'+data21.description+'</td>'+
                                    '</tr>';

                });
                action += '</table>'+
                            '</td>'+
                            '</tr>'+
                            '</tbody>'+
                            '</table>';
            console.log(action);
            callback(null, action);
        }).catch(function(error) {
            showNotification('Error on selecting actions for ALL issues:' + error.message, 'danger', 'glyphicon glyphicon-tasks');
        });

    }

});    
}

但是当我在我的代码中的另一个地方使用该函数它返回undefined而不是text语句时，我在return语句之前创建一个console.log（action）它返回正确的文本.....我没有得到它为什么在使用函数时返回undefined

更新当我使用Aruna的功能我正在使用这样的功能：

for(let b=0;b<id.length;b++){
    getActions(id[b],function(err, action) {
        arrAction[b] = action;
    });
 }

Answer 1

当您异步连接和查询MySQL数据库时，您不能指望同步返回数据。

您应该使用'callback'来处理这个问题。

所以你的代码可以是这样的，

function getActions(issue_id, callback){
var action = '';
var conn2 = new sql.Connection(config, function(err) {
    if(err){
        showNotification('error connecting for selecting actions for ALL issues: ' + err.message, 'danger', 'glyphicon glyphicon-tasks');
    } else {
        var request = new sql.Request(conn2);
        request
        .input('issue_id', sql.Int,issue_id)
        .query('SELECT [date], [description] FROM [actions] WHERE [issue_id] = @issue_id')
        .then(function(data2) {
            action ='<tr>'+
                            '<td style="vertical-align: top;" class="bold">action:</td>'+
                            '<td>'+
                            '<table>';
            data2.forEach(function(data21){
                action +='<tr>'+
                                '<td>'+data21.date+'</td>'+
                                '<td>'+data21.description+'</td>'+
                                '</tr>';

            });
            action += '</table>'+
                            '</td>'+
                            '</tr>'+
                            '</tbody>'+
                            '</table>';
            console.log(action);
            callback(null, action);
        }).catch(function(error) {
            showNotification('Error on selecting actions for ALL issues:' + error.message, 'danger', 'glyphicon glyphicon-tasks');
        });

    }

});    
}

你应该像这样打电话，

var issue_id = 12;
getActions(issue_id, function(err, action) {
   // to do
});

另外，像这样改变你的for循环，就像在循环中调用异步函数一样，for循环变量“b”将在响应来自函数之前被更改

var arrAction = [];
for(let b=0;b<id.length;b++){
    getActions(id[b],function(err, action) {
        arrAction[arrAction.length] = action;
    });
 }

Answer 2

getActions不会返回任何内容。

在内部函数中调用return action ，与sql.Connection类有关。 也许你应该return conn2; 在getActions结束时？ 没有关于你在做什么的背景很难说。