Почему не работат ObgectID?

Question

[lil_koi]

У меня есть код, который ищет по _id. Если делать напрямую по id, как из базы 5dd44dafc6f4762b80b8e5ff, то всё норм. Я посмотрел на stackoverflow.com как сделать поиск по id числу, то есть по "1", а не "5dd44dafc6f4762b80b8e5ff".

получилось это

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

exports.getById = async (req, res) => {
try {
const user = find({_id: ObjectId(req.params.id)});
res.send({ user });
} catch (err) {
res.send({status: 404, message: 'не найдено'})
}
}
но он не работает. Почему?

До этого было

exports.getById = async (req, res) => {
const id = req.params.id;
const user = await User.findById(id).exec(); // Венёт промис, поэтому необходим await. {} - найти все документы
res.send({ user });
}

Answer 1

[hzzzzl]

const user = await User.find({_id: ObjectId(req.params.id)});  // вот тут подождать надо
res.send({ user });

я вообще подозреваю, что mongoose сам приведет к нужному типу, и можно просто
await User.findOne({_id: req.params.id})

Comments

[lil_koi]

нет, это так не работает https://stackoverflow.com/questions/7878557/cant-f... Я сделал так же ,но у меня где-то ошибка, а мои -7 не дают её увидеть

[hzzzzl]

попробуй id вместо _id ?
https://stackoverflow.com/questions/15724272/what-...

const user = await User.find({ id: req.params.id })

Я сделал так же

там в ответе new ObjectId()

const user = find({_id: new ObjectId(req.params.id)});

[lil_koi]

так, теперь приходит не ошибка. Всё верно, но он из бд мне имя не приносит

const user = await User.find({ id: req.params.id }, name)
    res.send(user.name);

возвращает ошибку

[IDONTSUDO]

lil_koi,

const user = async  (req,res) =>{

User.find({ id: req.params.id })
.select("name")
.exec((err, result) => {
            if (err || !result) {
                return res.status(400).json({
                    error: err
                })
            }else{
               res.status(200).json({ result })
             }
}})

[lil_koi]

IDONTSUDO,

[IDONTSUDO]

lil_koi, внимательно посмотри на код

[IDONTSUDO]

lil_koi, 21 строка

[lil_koi]

IDONTSUDO, ? не вижу, что не так, но он возвращает имя из бд

[IDONTSUDO]

lil_koi, stastus

[lil_koi]

IDONTSUDO,

[IDONTSUDO]

lil_koi, и? зачем мне твои логи.

[IDONTSUDO]

lil_koi,

res.status(200).json({result})

[IDONTSUDO]

lil_koi, nodemoon установи пжл

[lil_koi]

IDONTSUDO, то что имя приходит, но он в 404 уходит.

Вот весь код export. Так будет проще

exports.getById = async (req, res) => {
try {
const user = await User.find({ id: req.params.id }).select("name").exec((err, result) => {
if (err || !result) {
return res.status(400).json({
error: err
})
}
})
res.status(200)(user.name);
res.send(JSON.parse({name:user.name}));
} catch (err) {
res.send({status: 404, message: 'не найдено'})
}
}

Так будет проще

[IDONTSUDO]

lil_koi, залогируй ошибку

[IDONTSUDO]

lil_koi, у тебя обьекта юзер нету

res.status(200)(user.name); убери
res.status(200).json({result}) вставь

[lil_koi]

IDONTSUDO,

[hzzzzl]

lil_koi, законсольлогь ошибку

catch (err) {
  console.log(err)   // ?
  res.send({status: 404, message: 'не найдено'})
}

await .. exec() фигня какая-то, выбирать проекцию так

const user = await User.findOne({ id: req.params.id }, 'name')
res.json({ user: user })

.find() вернет массив, даже если там один результат или пустой массив если нет результатов, выбирай через .findOne(), все равно юзер с айдишником же будет только один

[IDONTSUDO]

lil_koi,

exports.getById = async (req, res) => {
    const user = User.find({ id: req.params.id }).select("name").exec((err, result) => {
      if (err || !result) {
        console.log(err)
        console.log(result)
          return res.status(400).json({
              error: err
          })
      }
      res.status(200).json({result})
  })
}

[lil_koi]

hzzzzl, вернул null

[lil_koi]

IDONTSUDO, вернул {"result":[]}

[lil_koi]

{"_id":"5dd44db9c6f4762b80b8e600","
name":"Rusa"} в бд это вот так выглядит

[IDONTSUDO]

lil_koi, console.log(req.params.id) попробуй может не так обращаешься. У тебя точно в бд есть пользователь с id равным 1?

[IDONTSUDO]

lil_koi, дак с этим айди и обратись 5dd44db9c6f4762b80b8e600

[lil_koi]

IDONTSUDO, я делаю и на 2 проверку. У меня в носоли точно есть пользователь с id 5dd44db9c6f4762b80b8e600

[hzzzzl]

lil_koi, эээммм ты представляешь типа как в экселе ряды БД, и хочешь вызвать "второй ряд в базе данных"? не выйдет так

[lil_koi]

IDONTSUDO,

[lil_koi]

hzzzzl, у меня в коллекции два объекта, я хочу их получать по id 1 а не5dd44dafc6f4762b80b8e5ff В курсе на ютубе человек получал, я решил сам написать и у меня не выходит

[IDONTSUDO]

lil_koi,

exports.getById = async (req, res) => {
  let reqParams = req.params.id || "5dd44db9c6f4762b80b8e600"
    const user = User.find({ _id: reqParams }).select("name").exec((err, result) => {
      if (err || !result) {
        console.log(err)
          return res.status(400).json({
              error: err
          })
      }
      res.status(200).json({result})
  })
}

[lil_koi]

IDONTSUDO, в самом начале я рассписал, что я хочу получить

[lil_koi]

IDONTSUDO, https://www.youtube.com/watch?v=BmPUqg8A4PM&t=565s

[IDONTSUDO]

lil_koi,

[lil_koi]

IDONTSUDO, ну почему у него получилось, а у меня нет?

[IDONTSUDO]

lil_koi, потому что потому

Answer 2

[IDONTSUDO]

у твоего ID тип данных String, а монго ожидает Object

Comments

[lil_koi]

а как мне из него Object сделать?