mongoose findById не работает.

https://stackoverflow.com/a/41474798

Только вместо прямого вызова `new ObjectId()` лучше врапер с try/catch и проверкой результата. Иначе при не валидных данных будет бабахать эксепшен мимо колбеков.

Не могу разобраться.
Привел код к таком виду:

var mongoose = require('mongoose')
var ObjectId = mongoose.Schema.Types.ObjectId;

app.get('/user/:id', function(req, res, next){
    var param = req.param[this.name + 'Id'];
    var search = {'_id': new ObjectId(param)};
User.findById(search, function(err, user){
    if(err) return next(err);
    res.json(user);
  });
});

В User.findById нужно передавать само _id, а не объект с ключем '_id'.

То-есть вот так: User.findById(search._id, function(err, user)
?

Да. Либо даже просто не создавать search, а сразу конвертировать ваш параметр, как здесь:

var id = ObjectId(req.params.id)

Вы скопировали со stackoverflow строку

var param = req.param[this.name + 'Id'];

Не очень удобно. В Mongoid можно сделать

Mongoid.raise_not_found_error = false

И строки Mongoid тоже автоматически в ObjectId конвертирует.

app.get('/user/:id', function(req, res, next){
    var id = ObjectId(req.params.id);
    console.log(id);
    User.findById(id, function(err, user){
    if(err) return next(err);
    res.json(user);
  });
});

console.log(id) выдает:

undefined

И в браузере по адресу
http://127.0.0.1:3000/user/586f2862632df73c31f25df3
Выводится:
null

Сделал даже так:

app.get('/user/:id', function(req, res, next){
    var id = ObjectId('586f2862632df73c31f25df2');
    console.log(id);
    User.findById(id, function(err, user){
    if(err) return next(err);
    res.json(user);
  });
});

Попробуйте в каком-нибудь repl'e поэкспериментировать с mongoose без express'a.

лучше начать с голого драйвера mongo, освоить технологию, разобраться с апи. mongoose - тормозное поделие без профитов со своим api от балды.

в консоли mongo db.users.findOne({_id: ObjectId(" что вернет? вам бы сначала отладку освоить, а потом в асинхронные дебри нырять

у вас ObjectId это разве BSON'новский класс? Это дескриптор типа для описания схемы документа. Начните с изучения mongo без mongoose & express.

db.users.findOne()

{
	"_id" : ObjectId("586f2862632df73c31f25df2"),
	"username" : "Вася",
	"hashedPassword" : "cc4c5d195b9e1956f2f80eb34db23f85c3b3e02d",
	"salt" : "0.6767356628552079",
	"created" : ISODate("2017-01-06T05:17:22.458Z"),
	"__v" : 0
}

ObjectID = require('mongodb').ObjectID

запомните, если что-то не работает, значит ошибку допустили Вы. ошибки в технологии, в вашем случае, исключены. начните с простого, с примеров в консоли.

Я следовал примеру из урока, у автора все один в один написано, но у него работает.

Я следовал примеру из урока, у автора все один в один написано, но у него работает.

Тогда лучше используйте ту же версию mongoose, что и у автора. Похоже, в mongoose часто что-то меняется.

Меня уже тоже эта мысль посетила.

это вы ему через плечо заглядывали и видели что работает, или на слово поверили? очевидно что вы не понимаете что делаете, сравните код Objectid из этих пакетов и убедитесь.

https://youtu.be/YZwAVRsa1O4?list=PLDyvV36pndZFWfEQpNixIHVvp191Hb3Gg&t=116

Собственно помогло:
mongodb 3 версия.
mongoose 2 версия.
Всем спасибо за советы

Лучше вообще использовать async/await уже наконец, с нормальной обработкой ошибок вместо протаскивания руками через колбеки же. У меня на календаре 2017.

А, ещё (анрелейтед) стоит использовать const/let вместо var и подключить eslint, который это (и многое другое) проверяет.

Спасибо за советы, но я еще только учусь.Был бы рад конкретным примерам.

Это из ES7 драфт с бабелем? Или бубелем?

лет 7 назад использовал это в расширении iced-coffee-script, - работало

нет. в ес7 только две фичи: array.includes и x ** y. а это лишь один из вариантов замены коллбеков. очевидно, синтаксис async/await более привычен и близок к другим языкам

Если этой фичи нет в стандарте, тогда зачем о ней говорить?

http://mongoosejs.com/docs/api.html#model_Model.findById

Вообще вроде и строки должно хавать, судя по документации. В console.log() выведите реальное значение, может из экспресса мусор прет.

Примеры, честно говоря, старые. Я сильно сомневаюсь, что автор сейчас стал бы хреначить колбеками. Ну и про то что на кривом урле эксепшены в космос улетят в таком коде я уже объяснял.

bluebird Coroutine + yield наше всё. Пока async/await без ключиков хотя бы в 7 версии не заработет - в жопу такое счастье.

А чем тебе бабель не угодил?

На йелдах писать чтобы через несколько месяцев всё переписывать? Зачем?

По поводу async/await 7 — там это зависит от того, получится ли v8 5.5 в 7 ветку посадить, не сломав совместимость. Я бы сказал, что скорее нет, но я могу ошибаться — надо смотреть внимательнее, после того, как в мастер войдёт. В 8.0.0 в апреле точно будет.

Если придерживаться такой терминологии, то это ES8. Но вообще-то он ECMAScript 2017.

И нет, это не драфт, спецификация async/await полностью завершена, принята, и войдёт в стандарт в этом году.

Как-то не прет препроцессить код на сервере.

Восьмерка пойдет. А разве туда станут новые фичи втягивать? Она же LTS.

Как-то не прет препроцессить код на сервере.

Почему?

А разве туда станут новые фичи втягивать? Она же LTS.

Восьмёрка не LTS изначально, она станет LTS только через полгода после выхода, в октябре, одновременно с выходом 9.0.0. Ознакомься с https://github.com/nodejs/LTS.