From d23fa693ea2ff01554ee83f2cc6a7cd2bb787808 Mon Sep 17 00:00:00 2001
From: jaanvi <jaanvi.jaanvi@digimantra.com>
Date: Tue, 3 Jun 2025 09:35:03 +0530
Subject: [PATCH] CRUD created

---
 app.js                        |  4 ++++
 controllers/userController.js | 20 +++++++++++---------
 package-lock.json             |  4 ----
 package.json                  |  2 +-
 routes/userRoutes.js          |  4 +++-
 5 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/app.js b/app.js
index 20a9d3f..540e671 100644
--- a/app.js
+++ b/app.js
@@ -7,8 +7,12 @@ const PORT = process.env.PORT || 5000;
 //DB connection:Done
 connectDB();
 
+//use in built-middleware to parse the data form body
+app.use(express.urlencoded({ extended: false }))
 //Routes
 app.use("/api/users",userRoutes);
+
+
 app.listen(PORT,()=>{
     console.log(`Server is running on port ${PORT}`);
 })
diff --git a/controllers/userController.js b/controllers/userController.js
index 424b685..94ff8da 100644
--- a/controllers/userController.js
+++ b/controllers/userController.js
@@ -6,17 +6,18 @@ const createUser = async (req, res) => {
     const user = await User.create(req.body);
     res.status(201).json(user);
   } catch (err) {
-    res.status(400).json({ error: err.message });
+    res.status(500).json({ error: err.message });
   }
 };
 
-//GET
+//GET  
+//if i use findOne then it fetching the first match data
 const getUsers = async (req, res) => {
   try {
     const users = await User.find();
     res.status(200).json(users);
   } catch (err) {
-    res.status(400).json({ error: err.message });
+    res.status(500).json({ error: err.message });
   }
 };
 
@@ -29,18 +30,19 @@ const getUserById = async (req,res)=>{
         }
         res.status(200).json(user);
     } catch (err) {
-       res.status(400).json({ error: err.message });
+       res.status(500).json({ error: err.message });
     }
 };
-//UPDATE
+//UPDATE 
 const updateUser = async(req,res)=>{
     try {
         const user = await User.findByIdAndUpdate(req.params.id,req.body,{
-            new:true,
+            new:true,//return updated document
         });
+        if(!user) return res.status(404).json({message: "User Not Found."})
         res.status(200).json(user);
     } catch (err) {
-        res.status(400).json({ error: err.message });
+        res.status(500).json({ error: err.message });
     }
 };
 //DELETE 
@@ -50,8 +52,8 @@ const deleteUser= async(req,res)=>{
          if(!user){
             return res.status(404).json({message: "User not found"});
          }
-    } catch (error) {
-         res.status(400).json({ error: err.message });
+    } catch (err) {
+         res.status(500).json({ error: err.message });
     }
    
 
diff --git a/package-lock.json b/package-lock.json
index 532bbbd..cb6f92b 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -272,7 +272,6 @@
       "version": "16.5.0",
       "resolved": "https://registry.npmjs.org/dotenv/-/dotenv-16.5.0.tgz",
       "integrity": "sha512-m/C+AwOAr9/W1UOIZUo232ejMNnJAJtYQjUbHoNTBNTJSvqzzDh7vnrei3o3r3m9blf6ZoDkvcw0VmozNRFJxg==",
-      "license": "BSD-2-Clause",
       "engines": {
         "node": ">=12"
       },
@@ -358,7 +357,6 @@
       "version": "5.1.0",
       "resolved": "https://registry.npmjs.org/express/-/express-5.1.0.tgz",
       "integrity": "sha512-DT9ck5YIRU+8GYzzU5kT3eHGA5iL+1Zd0EutOmTE9Dtk+Tvuzd23VBU+ec7HPNSTxXYO55gPV/hq4pSBJDjFpA==",
-      "license": "MIT",
       "dependencies": {
         "accepts": "^2.0.0",
         "body-parser": "^2.2.0",
@@ -739,7 +737,6 @@
       "version": "6.16.0",
       "resolved": "https://registry.npmjs.org/mongodb/-/mongodb-6.16.0.tgz",
       "integrity": "sha512-D1PNcdT0y4Grhou5Zi/qgipZOYeWrhLEpk33n3nm6LGtz61jvO88WlrWCK/bigMjpnOdAUKKQwsGIl0NtWMyYw==",
-      "license": "Apache-2.0",
       "dependencies": {
         "@mongodb-js/saslprep": "^1.1.9",
         "bson": "^6.10.3",
@@ -795,7 +792,6 @@
       "version": "8.15.1",
       "resolved": "https://registry.npmjs.org/mongoose/-/mongoose-8.15.1.tgz",
       "integrity": "sha512-RhQ4DzmBi5BNGcS0w4u1vdMRIKcteXTCNzDt1j7XRcdWYBz1MjMjulBhPaeC5jBCHOD1yinuOFTTSOWLLGexWw==",
-      "license": "MIT",
       "dependencies": {
         "bson": "^6.10.3",
         "kareem": "2.6.3",
diff --git a/package.json b/package.json
index 265257a..b2dec8b 100644
--- a/package.json
+++ b/package.json
@@ -3,7 +3,7 @@
   "version": "1.0.0",
   "main": "app.js",
   "scripts": {
-    "start":"nodemon app.js",
+    "start": "nodemon app.js",
     "test": "echo \"Error: no test specified\" && exit 1"
   },
   "keywords": [],
diff --git a/routes/userRoutes.js b/routes/userRoutes.js
index e6a5519..c0734a3 100644
--- a/routes/userRoutes.js
+++ b/routes/userRoutes.js
@@ -6,4 +6,6 @@ router.post("/", createUser);
 router.get('/',getUsers);
 router.get('/:id',getUserById);
 router.put('/:id',updateUser);
-router.delete(':/id',deleteUser)
+router.delete('/:id',deleteUser)
+
+module.exports = router;