feat: Add null check and error handling in PostHero method for better…

… stability 🚀
0GiS0 · Dec 29, 2024 · 0b4edee · 0b4edee
1 parent a97d743
commit 0b4edee
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/src/Controllers/HeroController.cs b/src/Controllers/HeroController.cs
@@ -67,11 +67,21 @@ public ActionResult PutHero(int id, Hero hero)
         [HttpPost]
         public ActionResult<Hero> PostHero(Hero hero)
         {
+            if (hero == null)
+            {
+                return BadRequest("Hero cannot be null.");
+            }
 
-            _heroRepository.Add(hero);
-
-            return Ok(hero);
-
+            try
+            {
+                _heroRepository.Add(hero);
+                return CreatedAtAction(nameof(GetHero), new { id = hero.Id }, hero);
+            }
+            catch (Exception ex)
+            {
+                // Log the exception (ex) here if necessary
+                return StatusCode(500, "Internal server error");
+            }
         }
 
         // DELETE: api/Hero/5