Проблема с REST API (Node, Express, Mongo)?

Question

ummahusla @Antonoff

Разработчик

Проблема с REST API (Node, Express, Mongo)?

У меня есть апи с locations, для каждого location - есть список reviews. Написать api ко всем locations получилось, выбрать из списка определенный review зная его id, не вышло. Всё ломается тут

review = location.reviews.id(req.params.reviewid);

, хотя location.reviews =

[ { author: 'Simon Holmes',
    id: 57acda33cec290c7a1466cf2,
    rating: 5,
    timestamp: Thu Jul 16 2015 00:00:00 GMT+0100 (GMT Summer Time),
    reviewText: 'What a great place. I can\'t say enough good things about it.',
    createdOn: Sun Aug 14 2016 19:21:34 GMT+0100 (GMT Summer Time) },
  { author: 'Simon Holmes',
    id: 57acdcb3cec290c7a1466cf7,
    rating: 5,
    timestamp: Wed Jun 10 2015 00:00:00 GMT+0100 (GMT Summer Time),
    reviewText: 'What a great place. I can\'t say enough good things about it.',
    createdOn: Sun Aug 14 2016 19:21:34 GMT+0100 (GMT Summer Time) },
  { author: 'Simon Holmes',
    id: 57acdcd1cec290c7a1466cf8,
    rating: 2,
    timestamp: Mon Jun 29 2015 00:00:00 GMT+0100 (GMT Summer Time),
    reviewText: 'What a great place. I can\'t say enough good things about it.',
    createdOn: Sun Aug 14 2016 19:21:34 GMT+0100 (GMT Summer Time) } ]

а

req.params.reviewid

равняется

57acda33cec290c7a1466cf2

Controller:

module.exports.reviewsReadOne = function(req, res) {
    console.log("Getting a single review");
    if (req.params && req.params.locationid && req.params.reviewid) {
        Loc
            .findById(req.params.locationid)
            .select('name reviews')
            .exec(
                function(err, location) {
                    // console.log(location);
                    var response, review;

                    if (!location) {
                        sendJSONresponse(res, 404, {
                            "message": "locationid not found"
                        });
                        return;
                    } else if (err) {
                        sendJSONresponse(res, 400, err);
                        return;
                    }

                    console.log(req.params);
                    console.log(location);

                    if (location.reviews) {
                        review = location.reviews.id(req.params.reviewid); // null
                        if (!review) {
                            sendJSONresponse(res, 404, {
                                "message": "reviewid not found"
                            });
                        } else {
                            response = {
                                location: {
                                    name: location.name,
                                    id: req.params.locationid
                                },
                                review: review
                            };
                            sendJSONresponse(res, 200, response);
                        }
                    } else {
                        sendJSONresponse(res, 404, {
                            "message": "No reviews found"
                        });
                    }
                }
            );
    } else {
        sendJSONresponse(res, 404, {
            "message": "Not found, locationid and reviewid are both required"
        });
    }
};

Когда делаю запрос через Postman

http://localhost:1337/api/locations/57acd67fcec290c7a1466cf0/reviews/57acda33cec290c7a1466cf2

Получаю в ответ:

{
  "message": "reviewid not found"
}

Ответ в Node консоле:

Mongoose connected to mongodb://127.0.0.1:27017/Loc8r
Getting a single review
{ locationid: '57acd67fcec290c7a1466cf0',
  reviewid: '57acda33cec290c7a1466cf2' }
{ _id: 57acd67fcec290c7a1466cf0,
  name: 'Starcups',
  reviews:
   [ { author: 'Simon Holmes',
       id: 57acda33cec290c7a1466cf2,
       rating: 5,
       timestamp: Thu Jul 16 2015 00:00:00 GMT+0100 (GMT Summer Time),
       reviewText: 'What a great place. I can\'t say enough good things about it.',
       createdOn: Sun Aug 14 2016 19:14:38 GMT+0100 (GMT Summer Time) },
     { author: 'Simon Holmes',
       id: 57acdcb3cec290c7a1466cf7,
       rating: 5,
       timestamp: Wed Jun 10 2015 00:00:00 GMT+0100 (GMT Summer Time),
       reviewText: 'What a great place. I can\'t say enough good things about it.',
       createdOn: Sun Aug 14 2016 19:14:38 GMT+0100 (GMT Summer Time) },
     { author: 'Simon Holmes',
       id: 57acdcd1cec290c7a1466cf8,
       rating: 2,
       timestamp: Mon Jun 29 2015 00:00:00 GMT+0100 (GMT Summer Time),
       reviewText: 'What a great place. I can\'t say enough good things about it.',
       createdOn: Sun Aug 14 2016 19:14:38 GMT+0100 (GMT Summer Time) } ] }
GET /api/locations/57acd67fcec290c7a1466cf0/reviews/57acda33cec290c7a1466cf2 404 27.935 ms - 32

Вопрос задан более трёх лет назад
396 просмотров

12 комментариев

Подписаться 2 Оценить 12 комментариев

OVK2015 @OVK2015

location.reviews.id
так в ответе reviews ж массив.

Написано более трёх лет назад
ummahusla @Antonoff Автор вопроса

OVK2015: Ну так мне из этого масива надо выбрать review с нужным id и всю информацию по нему

Написано более трёх лет назад
OVK2015 @OVK2015

ummahusla: Я к тому, что у location.reviews нет св-ва Id, кот. вы хотите получить. У эл-ов массива location.reviews это св-во есть. Фильтруйте массив,- получайте нужный эл-т

Написано более трёх лет назад
Виталий @vshvydky

Железый ответ... не поспоришь. Поиск в массиве вам в помощь. А вообще судя по тому, что вы шлете запрос на поиск одной записи , а получаете несколько, то проблема наверное сразу в запросе...

Написано более трёх лет назад
ummahusla @Antonoff Автор вопроса

Виталий: Я делаю console.log запроса на location по его ip, и уже из него пытаюсь выбрать всю нужную информацию

Написано более трёх лет назад
Виталий @vshvydky

Из массива объектов нельзя возвратить объект, передав в массив в виде параметра поле объекта, это какой-то треш

Написано более трёх лет назад
Виталий @vshvydky

ummahusla: я вижу это как-то так, пишу с планшета, могут быть ошибки, ловите идею.
Array.prototype.getObject = function(id) {
this.forEach((item, index, arr)=>{
if(item.id == id){
return arr[index];
}
});
return false;
};
Далее берем ваш массив объектов
И делаем чет типа let obj = arrObj.getObject(objectId);

Написано более трёх лет назад
Виталий @vshvydky

В foreach я использовал стрелочную функцию, если ваша нода ее еще не умеет перепишите на обычную анонимную функцию

Написано более трёх лет назад
Кирилл Халитов @Voronar
Читаю туже книгу и тоже не могу понять этот момент.
Автор говорит, что в мангусте есть вспомогательный метод для получения поддокумента по ID. Для полученного массива поддокументов у мангуста есть метод id, который принимает ID поддокумента, который нас интересует.
Тогда судя по тексту получается, мангуст возвращает не чистый JS Array, а свою расширенную версию с дополнительными методами.
Есть профи по мангусту?

Цитата из книги:
Mongoose also offers a helper method for finding a subdocument by ID. Given an
array of subdocuments Mongoose has an id method that accepts the ID you want to
find. The id method will return the single matching subdocument, and it can be used
as follows:
Loc .findById(req.params.locationid) .select('name reviews') .exec(function(err, location) { var review; review = location.reviews.id(req.params.reviewid); } );
Написано более трёх лет назад
ummahusla @Antonoff Автор вопроса

Кирилл Халитов: хех, ну не я один такой. Если найдёте ответ, дайте мне знать пожалуйста.

Написано более трёх лет назад
ummahusla @Antonoff Автор вопроса

Кирилл Халитов: Написал я автору книги в твиттере, посмотрим что ответит

Написано более трёх лет назад
ummahusla @Antonoff Автор вопроса

Кирилл Халитов: А я как раз с автором переписываюсь

Написано более трёх лет назад

location.reviews.id
так в ответе reviews ж массив.
OVK2015: Ну так мне из этого масива надо выбрать review с нужным id и всю информацию по нему
ummahusla: Я к тому, что у location.reviews нет св-ва Id, кот. вы хотите получить. У эл-ов массива location.reviews это св-во есть. Фильтруйте массив,- получайте нужный эл-т
Железый ответ... не поспоришь. Поиск в массиве вам в помощь. А вообще судя по тому, что вы шлете запрос на поиск одной записи , а получаете несколько, то проблема наверное сразу в запросе...
Виталий: Я делаю console.log запроса на location по его ip, и уже из него пытаюсь выбрать всю нужную информацию
Из массива объектов нельзя возвратить объект, передав в массив в виде параметра поле объекта, это какой-то треш
ummahusla: я вижу это как-то так, пишу с планшета, могут быть ошибки, ловите идею.
Array.prototype.getObject = function(id) {
this.forEach((item, index, arr)=>{
if(item.id == id){
return arr[index];
}
});
return false;
};
Далее берем ваш массив объектов
И делаем чет типа let obj = arrObj.getObject(objectId);
В foreach я использовал стрелочную функцию, если ваша нода ее еще не умеет перепишите на обычную анонимную функцию
Читаю туже книгу и тоже не могу понять этот момент.
Автор говорит, что в мангусте есть вспомогательный метод для получения поддокумента по ID. Для полученного массива поддокументов у мангуста есть метод id, который принимает ID поддокумента, который нас интересует.
Тогда судя по тексту получается, мангуст возвращает не чистый JS Array, а свою расширенную версию с дополнительными методами.
Есть профи по мангусту?

Цитата из книги:
Mongoose also offers a helper method for finding a subdocument by ID. Given an
array of subdocuments Mongoose has an id method that accepts the ID you want to
find. The id method will return the single matching subdocument, and it can be used
as follows:
Loc .findById(req.params.locationid) .select('name reviews') .exec(function(err, location) { var review; review = location.reviews.id(req.params.reviewid); } );
Кирилл Халитов: хех, ну не я один такой. Если найдёте ответ, дайте мне знать пожалуйста.
Кирилл Халитов: Написал я автору книги в твиттере, посмотрим что ответит
Кирилл Халитов: А я как раз с автором переписываюсь

Answer 1 · 2016-08-17 13:58:20

То что ответил автор книги:

In the Mongo shell try something like this to add an _id to a review

db.locations.update(
  { "name": "INSERT A LOCATION NAME HERE" },
  {
    $set : {
      "reviews.0._id" : ObjectId()
    }
  }
)

This will add an _id property with an objectId value to the first review subdocument in the location you specify in the command.

Проблема с REST API (Node, Express, Mongo)?

Войдите на сайт