我无法访问我创建的对象中的数据。我是JS和节点的新手,我认为我的问题是我如何初始化变量,但我不知道。
这是我的初始化:
var http = require('http');
var MongoClient = require('mongodb').MongoClient;
var async = require('async');
var currentBoatList = [];
var BoatObjectList = [];
我有一个班级来创建船的当前信息(取自数据库):
function CurrentBoatInfo(boatName) {
var name,MMSI,callSign,currentDate,positionJSON,status,speed,course;
database.collection('Vessels').find({"BoatName":boatName},{"sort":{DateTime:-1}}).toArray(function(error1,vessel) {
name = vessel[0].BoatName;
MMSI = vessel[0].MMSI;
callSign = vessel[0].VesselCallSign;
console.log(name); \\logs the boats name, so the variable is there
});
});
}
我的数据库功能可以提取最近的船只,将其名称放在一个列表中,然后在另一个列表中为列表中的每个船名创建对象:
编辑:我发现我多次不必要地连接到mongoDB,使用代码来解决这个问题,并清除'db'变量名。
var createBoats = function() {
MongoClient.connect('mongodb://localhost:27017/tracks', function(err,database){
if (err) {return console.dir(err); }
else {console.log("connect to db");}
database.collection('Vessels').find({"MostRecentContact": { "$gte": (new Date((new Date()).getTime() - (365*24*60*60*1000)))}}).toArray(function(error,docs) { //within a year
docs.forEach(function(entry, index, array) {
currentBoatList.push(entry.BoatName); //create list of boats
BoatObjectList.push(new CurrentBoatInfo(entry.BoatName,database));
});
server();
});
});
};
最后我的服务器代码只是创建一个服务器,并且应该记录上面创建的每个对象的一些信息,但由于某种原因没有(在下面输出):
var server = function() {
http.createServer(function handler(req, res) {
res.writeHead(200, {'Content-Type': 'text/plain'});
console.log(BoatObjectList); //array of CurrentBoatInfo objects, prints [CurrentBoatInfo {}, CurrentBoatInfo {}, CurrentBoatInfo {}]
console.log(BoatObjectList[0].name); //prints undefined
BoatObjectList.forEach(function(entry) {
var count = 0;
for(var propertyName in entry.CurrentBoatInfo) { //nothing from here prints
console.log(JSON.stringify(propertyName));
count++;
console.log(count);
}
});
res.end();
}).listen(1337, '127.0.0.1');
};
我看到的输出类似于:
connect to db
[ 'DOCK HOLIDAY', 'BOATY MCBOATFACE', 'PIER PRESSURE' ] //list of boats
DOCK HOLIDAY //boat names as they're being instantiated
BOATY MCBOATFACE
PIER PRESSURE
[ CurrentBoatInfo {}, CurrentBoatInfo {}, CurrentBoatInfo {} ] //list of boat objects
undefined //the name of the first boat in the object list
[ CurrentBoatInfo {}, CurrentBoatInfo {}, CurrentBoatInfo {} ]
undefined
考虑到这一点,我现在认为我的问题是createServer代码运行,但不记录,然后当我访问127.0.0.1:1337时,它记录名称(实例化时未定义)...但是如何让createServer等待实例化对象?
答案 0 :(得分:0)
我能发现的一个明显问题是这段代码:
docs.forEach(function(entry, index, array) {
currentBoatList.push(entry.BoatName); //create list of boats
if (currentBoatList.length === array.length) {
console.log(currentBoatList);
async.eachOf(currentBoatList, function(entry,index,array) {
BoatObjectList[index] = new CurrentBoatInfo(currentBoatList[index]); //create object of the boat's info
}, server()); //create the server after creating the objects
}
});
这里的问题是async.eachOf是在同步docs.forEach函数内运行的异步函数。此外,async.eachOf的第二个参数应该有一个必须为数组中的每个项调用的回调,如:
async.eachOf(array, function(item, index, callback) {
doSomethingAsync(function(err, result) {
if (err) return callback(err);
// do something with result
callback();
});
}, function(err) {
// this is called when all items of the array are processed
// or if any of them had error, err here will contain the error
if(err) {
console.log("something went wrong", err);
} else {
console.log("success");
}
});
正如您所看到的那样,运行server()作为回调看起来并不正确,因为您应该确保没有错误首先传递给最终回调然后继续。
对于你的情况,我不明白为什么你使用async.eachOf而不是简单 currentBoatList.forEach因为你没有在循环中进行任何异步操作,所以你只是填充了BoatObjectList。
UPD您的问题是您在假设变量准备好之前不等待异步操作完成。我希望以下内容能让您了解如何实现您的需求(请注意,我已经重命名了一些变量和函数,以使它们更清晰易懂):
database.collection('Vessels').find(query).toArray(function(err, vessels) {
if (err) {
// process error
return;
}
async.eachOf(vessels, function(vessel, index, next) {
currentBoatList.push(vessel.BoatName);
getVesselInfoByName(vessel.BoatName, function(err, info) {
if (err) {
return next(err);
}
vesselsInfoList.push(info);
next();
});
}, function(err) {
if (err) {
// process error
return;
}
// all vessels are processed without errors, vesselsInfoList is ready to be used
server();
});
});
function getVesselInfoByName(vesselName, callback) {
// do everything you need to get vessel info
// once you have received the vesselInfo call callback
// if any error happens return callback(err);
// otherwise return callback(null, vesselInfo);
}
一般来说,我建议您了解有关node.js asyncronius函数如何工作的更多信息,然后仔细查看异步库文档。