重构Node.JS回调以获取DRY代码

Question

在ifs内部的两个find调用都具有以函数（例如，docs）开头的回调。 有什么干净的方法可以将其重构为DRYer？ 谢谢。

    if (connection_id == null) {
        id_connectionsCollection.find({}, {}, function (e, docs) {
            if (e) {
                return callback(e);
            }
            var connectionDetails = null;
            if (docs == null || docs.length == 0) {//if no connections found, use default from config
                connectionDetails = defaultConnectionDetails
            }
            else {

                connectionDetails = docs[0];//just get the first one
            }

            return callback(null, connectionDetails);
        });

    }
    else {
        id_connectionsCollection.find({name: connection_id}, {sort: {updated_at: -1}}, function (e, docs) {
            if (e) {
                return callback(e);
            }
            var connectionDetails = null;
            if (docs == null || docs.length == 0) {
                connectionDetails = defaultConnectionDetails;
            }
            else {

                connectionDetails = docs[0];//just get the first one
            }
            return callback(null, connectionDetails);
        });
    }

Answer 1

使代码干燥的最明显方法是将回调提取到一个命名函数，该函数可以在find方法的最后一个arg上作为回调传递：

 // Can probably think of a better name here...
 doCallback = function(e, docs) {
      if (e)
        return callback(e);

      var connectionDetails = null;

      if (docs == null || docs.length == 0)
        connectionDetails = defaultConnectionDetails;
      else 
        connectionDetails = docs[0];//just get the first one

      return callback(null, connectionDetails);
}

if (connection_id == null)
  id_connectionsCollection.find({}, {}, doCallback);
else 
  id_connectionsCollection.find({name: connection_id}, {sort: {updated_at: -1}}, doCallback);