use primary constructors (#59712)

Co-authored-by: Safia Abdalla <[email protected]>

Author: Varorbc

src/ProjectTemplates/Web.ProjectTemplates/content/BlazorWeb-CSharp/BlazorWeb-CSharp/Program.cs

@@ -102,7 +102,6 @@
 app.UseHttpsRedirection();
 
 #endif
-
 app.UseAntiforgery();
 
 app.MapStaticAssets();

src/ProjectTemplates/Web.ProjectTemplates/content/GrpcService-CSharp/Program.Main.cs

@@ -19,4 +19,4 @@ public static void Main(string[] args)
 
         app.Run();
     }
-}
+}

src/ProjectTemplates/Web.ProjectTemplates/content/GrpcService-CSharp/Services/GreeterService.cs

@@ -1,18 +1,13 @@
 using Grpc.Core;
-using GrpcService_CSharp;
 
 namespace GrpcService_CSharp.Services;
 
-public class GreeterService : Greeter.GreeterBase
+public class GreeterService(ILogger<GreeterService> logger) : Greeter.GreeterBase
 {
-    private readonly ILogger<GreeterService> _logger;
-    public GreeterService(ILogger<GreeterService> logger)
-    {
-        _logger = logger;
-    }
-
     public override Task<HelloReply> SayHello(HelloRequest request, ServerCallContext context)
     {
+        logger.LogInformation("The message is received from {Name}", request.Name);
+
         return Task.FromResult(new HelloReply
         {
             Message = "Hello " + request.Name

src/ProjectTemplates/Web.ProjectTemplates/content/RazorClassLibrary-CSharp/ExampleJsInterop.cs

@@ -9,15 +9,10 @@ namespace Company.RazorClassLibrary1;
 // This class can be registered as scoped DI service and then injected into Blazor
 // components for use.
 
-public class ExampleJsInterop : IAsyncDisposable
+public class ExampleJsInterop(IJSRuntime jsRuntime) : IAsyncDisposable
 {
-    private readonly Lazy<Task<IJSObjectReference>> moduleTask;
-
-    public ExampleJsInterop(IJSRuntime jsRuntime)
-    {
-        moduleTask = new (() => jsRuntime.InvokeAsync<IJSObjectReference>(
+    private readonly Lazy<Task<IJSObjectReference>> moduleTask = new(() => jsRuntime.InvokeAsync<IJSObjectReference>(
             "import", "./_content/Company.RazorClassLibrary1/exampleJsInterop.js").AsTask());
-    }
 
     public async ValueTask<string> Prompt(string message)
     {

src/ProjectTemplates/Web.ProjectTemplates/content/RazorPagesWeb-CSharp/Data/ApplicationDbContext.cs

@@ -3,10 +3,6 @@
 
 namespace Company.WebApplication1.Data;
 
-public class ApplicationDbContext : IdentityDbContext
+public class ApplicationDbContext(DbContextOptions<ApplicationDbContext> options) : IdentityDbContext(options)
 {
-    public ApplicationDbContext(DbContextOptions<ApplicationDbContext> options)
-        : base(options)
-    {
-    }
 }

src/ProjectTemplates/Web.ProjectTemplates/content/RazorPagesWeb-CSharp/Pages/Error.cshtml.cs

@@ -12,13 +12,6 @@ public class ErrorModel : PageModel
 
     public bool ShowRequestId => !string.IsNullOrEmpty(RequestId);
 
-    private readonly ILogger<ErrorModel> _logger;
-
-    public ErrorModel(ILogger<ErrorModel> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnGet()
     {
         RequestId = Activity.Current?.Id ?? HttpContext.TraceIdentifier;

src/ProjectTemplates/Web.ProjectTemplates/content/RazorPagesWeb-CSharp/Pages/Index.cshtml.cs

@@ -16,23 +16,18 @@ namespace Company.WebApplication1.Pages;
 #if (GenerateApiOrGraph)
 [AuthorizeForScopes(ScopeKeySection = "DownstreamApi:Scopes")]
 #endif
+#if (GenerateApi)
+public class IndexModel(IDownstreamApi downstreamApi) : PageModel
+#elseif (GenerateGraph)
+public class IndexModel(GraphServiceClient graphServiceClient) : PageModel
+#else
 public class IndexModel : PageModel
+#endif
 {
-    private readonly ILogger<IndexModel> _logger;
-
 #if (GenerateApi)
-    private readonly IDownstreamApi _downstreamApi;
-
-    public IndexModel(ILogger<IndexModel> logger,
-                        IDownstreamApi downstreamApi)
-    {
-            _logger = logger;
-        _downstreamApi = downstreamApi;
-    }
-
     public async Task OnGet()
     {
-        using var response = await _downstreamApi.CallApiForUserAsync("DownstreamApi").ConfigureAwait(false);
+        using var response = await downstreamApi.CallApiForUserAsync("DownstreamApi").ConfigureAwait(false);
         if (response.StatusCode == System.Net.HttpStatusCode.OK)
         {
             var apiResult = await response.Content.ReadAsStringAsync().ConfigureAwait(false);
@@ -45,27 +40,13 @@ public async Task OnGet()
         }
     }
 #elseif (GenerateGraph)
-    private readonly GraphServiceClient _graphServiceClient;
-
-    public IndexModel(ILogger<IndexModel> logger,
-                        GraphServiceClient graphServiceClient)
-    {
-        _logger = logger;
-        _graphServiceClient = graphServiceClient;
-    }
-
     public async Task OnGet()
     {
-        var user = await _graphServiceClient.Me.GetAsync();
+        var user = await graphServiceClient.Me.GetAsync();
 
         ViewData["ApiResult"] = user?.DisplayName;
     }
 #else
-    public IndexModel(ILogger<IndexModel> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnGet()
     {
 

src/ProjectTemplates/Web.ProjectTemplates/content/RazorPagesWeb-CSharp/Pages/Privacy.cshtml.cs

@@ -5,13 +5,6 @@ namespace Company.WebApplication1.Pages;
 
 public class PrivacyModel : PageModel
 {
-    private readonly ILogger<PrivacyModel> _logger;
-
-    public PrivacyModel(ILogger<PrivacyModel> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnGet()
     {
     }

src/ProjectTemplates/Web.ProjectTemplates/content/StarterWeb-CSharp/Controllers/HomeController.cs

@@ -20,24 +20,19 @@ namespace Company.WebApplication1.Controllers;
 #if (OrganizationalAuth)
 [Authorize]
 #endif
+#if (GenerateApi)
+public class HomeController(IDownstreamApi downstreamApi) : Controller
+#elseif (GenerateGraph)
+public class HomeController(GraphServiceClient graphServiceClient) : Controller
+#else
 public class HomeController : Controller
+#endif
 {
-    private readonly ILogger<HomeController> _logger;
-
 #if (GenerateApi)
-    private readonly IDownstreamApi _downstreamApi;
-
-    public HomeController(ILogger<HomeController> logger,
-                            IDownstreamApi downstreamApi)
-    {
-            _logger = logger;
-        _downstreamApi = downstreamApi;
-    }
-
     [AuthorizeForScopes(ScopeKeySection = "DownstreamApi:Scopes")]
     public async Task<IActionResult> Index()
     {
-        using var response = await _downstreamApi.CallApiForUserAsync("DownstreamApi").ConfigureAwait(false);
+        using var response = await downstreamApi.CallApiForUserAsync("DownstreamApi").ConfigureAwait(false);
         if (response.StatusCode == System.Net.HttpStatusCode.OK)
         {
             var apiResult = await response.Content.ReadAsStringAsync().ConfigureAwait(false);
@@ -51,29 +46,15 @@ public async Task<IActionResult> Index()
         return View();
     }
 #elseif (GenerateGraph)
-    private readonly GraphServiceClient _graphServiceClient;
-
-    public HomeController(ILogger<HomeController> logger,
-                        GraphServiceClient graphServiceClient)
-    {
-            _logger = logger;
-        _graphServiceClient = graphServiceClient;
-    }
-
     [AuthorizeForScopes(ScopeKeySection = "DownstreamApi:Scopes")]
     public async Task<IActionResult> Index()
     {
-        var user = await _graphServiceClient.Me.GetAsync();
+        var user = await graphServiceClient.Me.GetAsync();
         ViewData["ApiResult"] = user?.DisplayName;
 
         return View();
     }
 #else
-    public HomeController(ILogger<HomeController> logger)
-    {
-        _logger = logger;
-    }
-
     public IActionResult Index()
     {
         return View();

src/ProjectTemplates/Web.ProjectTemplates/content/StarterWeb-CSharp/Data/ApplicationDbContext.cs

@@ -3,10 +3,6 @@
 
 namespace Company.WebApplication1.Data;
 
-public class ApplicationDbContext : IdentityDbContext
+public class ApplicationDbContext(DbContextOptions<ApplicationDbContext> options) : IdentityDbContext(options)
 {
-    public ApplicationDbContext(DbContextOptions<ApplicationDbContext> options)
-        : base(options)
-    {
-    }
 }