Je pense que vous devez reconcevoir vos schémas de manière plus simple, il y a trop de références entre les modèles, et cela pose des problèmes, par exemple vous avez un accès 5 db lorsque vous souhaitez créer un commentaire et un accès 6 db lorsque vous souhaitez supprimer un commentaire.
Je créerais le schéma utilisateur comme celui-ci en supprimant les publications et les références de commentaires, mais plus tard, lorsque nous voulons accéder aux publications des utilisateurs, je configure remplissage virtuel.
const UserSchema = new Schema(
{
name: {
type: String,
required: true
},
email: {
type: String,
required: true,
unique: true
},
password: {
type: String,
required: true
},
avatar: {
type: String
},
date: {
type: Date,
default: Date.now
}
},
{
toJSON: { virtuals: true }
}
);
UserSchema.virtual("posts", {
ref: "Post",
localField: "_id",
foreignField: "user"
});
Et dans le schéma des messages, j'ai supprimé les références des commentaires. (Pour plus de simplicité, j'ai supprimé les champs "J'aime" et "Je n'aime pas".)
const PostSchema = new Schema(
{
user: {
type: Schema.Types.ObjectId,
ref: "User"
},
text: {
type: String,
required: true
},
date: {
type: Date,
default: Date.now
}
},
{
toJSON: { virtuals: true }
}
);
PostSchema.virtual("comments", {
ref: "Comment",
localField: "_id",
foreignField: "post"
});
Le schéma de commentaire peut rester tel quel.
Maintenant, pour ajouter un commentaire à un article, nous n'avons besoin que de 2 accès db, un pour vérifier si le message existe et un pour créer le message.
router.post(
"/comment/:id",
[
auth,
[
check("text", "Text is required")
.not()
.isEmpty()
]
],
async (req, res) => {
const errors = validationResult(req);
if (!errors.isEmpty()) {
return res.status(400).json({ errors: errors.array() });
}
try {
const post = await Post.findById(req.params.id);
if (!post) {
return res.status(404).json({ msg: "Post not found" });
}
let comment = new Comment({
text: req.body.text,
post: req.params.id,
user: req.user.id
});
comment = await comment.save();
res.json(comment);
} catch (err) {
console.error(err.message);
res.status(500).send("Server Error");
}
}
);
Disons que nous avons ces 2 utilisateurs :
{
"_id" : ObjectId("5e216d74e7138b638cac040d"),
"name" : "user1"
}
{
"_id" : ObjectId("5e217192d204a26834d013e8"),
"name" : "user2"
}
Utilisateur1 avec _id:"5e216d74e7138b638cac040d"
a ce message.
{
"_id": "5e2170e7d204a26834d013e6",
"user": "5e216d74e7138b638cac040d",
"text": "Post 1",
"date": "2020-01-17T08:31:35.699Z",
"__v": 0,
"id": "5e2170e7d204a26834d013e6"
}
Disons user2 avec _id:"5e217192d204a26834d013e8"
a commenté ce post deux fois comme ceci :
{
"_id" : ObjectId("5e2172a4957c02689c9840d6"),
"text" : "User2 commented on user1 post1",
"post" : ObjectId("5e2170e7d204a26834d013e6"),
"user" : ObjectId("5e217192d204a26834d013e8"),
"date" : ISODate("2020-01-17T11:39:00.396+03:00"),
"__v" : 0
},
{
"_id": "5e21730d468bbb7ce8060ace",
"text": "User2 commented again on user1 post1",
"post": "5e2170e7d204a26834d013e6",
"user": "5e217192d204a26834d013e8",
"date": "2020-01-17T08:40:45.997Z",
"__v": 0
}
Pour supprimer un commentaire, nous pouvons utiliser la route suivante, comme vous le voyez, nous avons réduit l'accès à la base de données de 6 à 3, et le code est plus court et plus propre.
router.delete("/comment/:id/:comment_id", auth, async (req, res) => {
try {
const comment = await Comment.findById(req.params.comment_id);
if (!comment) {
return res.status(404).json({ msg: "Post do not have this comment" });
}
if (comment.user.toString() !== req.user.id) {
return res.status(401).json({ msg: "User not authorized" });
}
await comment.remove();
// resend the comments that belongs to that post
const postComments = await Comment.find({ post: req.params.id });
res.json(postComments);
} catch (err) {
console.error(err.message);
res.status(500).send("Server Error");
}
});
Maintenant, vous pouvez vous demander comment accéder aux publications d'un utilisateur ? Puisque nous avons configuré le peuplement virtuel dans notre schéma utilisateur, nous pouvons remplir les publications comme ceci :
router.get("/users/:id/posts", async (req, res) => {
const result = await User.findById(req.params.id).populate("posts");
res.send(result);
});