Creating a new Home controller action Details(int id), updating servi…

…ces with a new Get(int id) method to return details for a particular restaurant, creating a simple Details view, updating the Index view to use tag helpers and creating a required _ViewImports.cshtml along the way.
gorohoroh · Jul 4, 2018 · ddc5a0e · ddc5a0e
1 parent fde46ac
commit ddc5a0e
Show file tree

Hide file tree

Showing 12 changed files with 77 additions and 0 deletions.
diff --git a/OdeToFoodRider/OdeToFoodRider/Controllers/HomeController.cs b/OdeToFoodRider/OdeToFoodRider/Controllers/HomeController.cs
@@ -1,3 +1,4 @@
+using System;
 using Microsoft.AspNetCore.Mvc;
 using OdeToFoodRider.Models;
 using OdeToFoodRider.Services;
@@ -27,5 +28,17 @@ public IActionResult Index()
 
             return View(model);
         }
+
+        public IActionResult Details(int id)
+        {
+            // VSRD: Rider's completion misfires here, too: the opening parentheses completes the existing GetAll() method, too
+            var model = _restaurantData.Get(id); // VSRD: In Rider, a "Check variable for null" CA is available, along with a quick path to null check pattern settings 
+            if (model == null)
+            {
+                return RedirectToAction(nameof(Index));
+            }
+
+            return View(model);
+        }
     }
 }
diff --git a/OdeToFoodRider/OdeToFoodRider/Services/IRestaurantData.cs b/OdeToFoodRider/OdeToFoodRider/Services/IRestaurantData.cs
@@ -7,5 +7,6 @@ public interface IRestaurantData
     {
         // VSRD: Complete Statement at GetAll{caret} generates both the parentheses and the semicolon
         IEnumerable<Restaurant> GetAll();
+        Restaurant Get(int id);
     }
 }
diff --git a/OdeToFoodRider/OdeToFoodRider/Services/InMemoryRestaurantData.cs b/OdeToFoodRider/OdeToFoodRider/Services/InMemoryRestaurantData.cs
@@ -28,5 +28,10 @@ public IEnumerable<Restaurant> GetAll()
         {
             return _restaurants.OrderBy(r => r.Name);
         }
+
+        public Restaurant Get(int id)
+        {
+            return _restaurants.FirstOrDefault(r => r.Id == id);
+        }
     }
 }
diff --git a/OdeToFoodRider/OdeToFoodRider/Views/Home/Details.cshtml b/OdeToFoodRider/OdeToFoodRider/Views/Home/Details.cshtml
@@ -0,0 +1,15 @@
+@using OdeToFoodRider.Models
+@model Restaurant
+
+<!DOCTYPE html>
+
+<html>
+<head>
+    <title>Restaurant details</title>
+</head>
+<body>
+    <h1>@Model.Name</h1>
+<div>... details ...</div>
+<a asp-action="Index" asp-controller="Home">Home</a>
+</body>
+</html>
diff --git a/OdeToFoodRider/OdeToFoodRider/Views/Home/Index.cshtml b/OdeToFoodRider/OdeToFoodRider/Views/Home/Index.cshtml
@@ -17,6 +17,12 @@
         <tr>
             <td>@restaurant.Id</td>
             <td>@restaurant.Name</td>
+            <td>
+                @* VSRD: Rider's path mapping functionality acts weird with the path below *@
+                <a href="/home/details/@restaurant.Id">Details</a>
+                @Html.ActionLink("Go", "Details", new {id = restaurant.Id})
+                <a asp-action="Details" asp-route-id="@restaurant.Id">More</a>
+            </td>
         </tr>
     }
 </table>

diff --git a/OdeToFoodRider/OdeToFoodRider/Views/_ViewImports.cshtml b/OdeToFoodRider/OdeToFoodRider/Views/_ViewImports.cshtml
@@ -0,0 +1,2 @@
+@addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers
+@* VSRD: No completion suggestions for assembly name in Rider, too :( - completion for @addTagHelper is available though *@
diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Controllers/HomeController.cs b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Controllers/HomeController.cs
@@ -30,5 +30,19 @@ public IActionResult Index()
             return View(model);
 
         }
+
+        public IActionResult Details(int id)
+        {
+            // VSRD: at _restaurantData.Get{caret}, I want to use a method called Get() before declaring it but entering an opening parentheses completes the existing GetAll() method. Not good.
+            // VSRD: However, creating the method from usage here does in fact create "object Get()" in the IRestaurantData interface
+            var model = _restaurantData.Get(id);
+            if (model == null)
+            {
+                return RedirectToAction(nameof(Index)); // VSRD: if we type RedirectToAction("Index"), Visual Studio doesn't provide view completion in the string literal, which is why Scott prefers the nameof syntax
+            }
+
+            // VSRD: Visual Studio doesn't see that the view isn't resolved and doesn't suggest to create one.
+            return View(model);
+        }
     }
 }
diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Services/IRestaurantData.cs b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Services/IRestaurantData.cs
@@ -6,8 +6,10 @@
 
 namespace OdeToFoodVisualStudio.Services
 {
+    // VSRD: "Go to Implementation" is available in Visual Studio
     public interface IRestaurantData
     {
         IEnumerable<Restaurant> GetAll();
+        Restaurant Get(int id);
     }
 }
diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Services/InMemoryRestaurantData.cs b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Services/InMemoryRestaurantData.cs
@@ -28,5 +28,10 @@ public IEnumerable<Restaurant> GetAll()
         {
             return _restaurants.OrderBy(r => r.Name);
         }
+
+        public Restaurant Get(int id)
+        {
+            return _restaurants.FirstOrDefault(r => r.Id == id);
+        }
     }
 }
diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/Home/Details.cshtml b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/Home/Details.cshtml
@@ -0,0 +1,7 @@
+@model OdeToFoodVisualStudio.Models.Restaurant
+
+<h1>@Model.Name</h1>
+
+<div>... details ...</div>
+@* VSRD: no completion or resolve for controller and action names in tag helpers in Visual Studio *@
+<a asp-action="Index" asp-controller="Home">Home</a>
diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/Home/Index.cshtml b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/Home/Index.cshtml
@@ -16,6 +16,11 @@
             <tr>
                 <td>@restaurant.Id</td>
                 <td>@restaurant.Name</td>
+                <td>
+                    <a href="/home/details/@restaurant.Id">Details</a>
+                    @Html.ActionLink("Go", "Details", new { id = restaurant.Id })
+                    <a asp-action="Details" asp-route-id="@restaurant.Id">More</a>
+                </td>
             </tr>
         }
     </table>

diff --git a/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/_ViewImports.cshtml b/OdeToFoodVisualStudio/OdeToFoodVisualStudio/Views/_ViewImports.cshtml
@@ -0,0 +1,2 @@
+@addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers
+@* VSRD: Visual Studio provides no completion for the assembly name above *@