chore: 优化代码注释

src/Infrastructure/TakeoutSaaS.Infrastructure/Identity/Persistence/IdentityDataSeeder.cs

@@ -1,14 +1,9 @@
-using System;
-using System.Linq;
-using System.Threading;
-using System.Threading.Tasks;
 using Microsoft.AspNetCore.Identity;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
-using TakeoutSaaS.Domain.Identity.Entities;
 using TakeoutSaaS.Infrastructure.Identity.Options;
 using DomainIdentityUser = TakeoutSaaS.Domain.Identity.Entities.IdentityUser;
 
@@ -17,29 +12,20 @@ namespace TakeoutSaaS.Infrastructure.Identity.Persistence;
 /// <summary>
 /// 后台账号初始化种子任务
 /// </summary>
-public sealed class IdentityDataSeeder : IHostedService
+public sealed class IdentityDataSeeder(IServiceProvider serviceProvider, ILogger<IdentityDataSeeder> logger) : IHostedService
 {
-    private readonly IServiceProvider _serviceProvider;
-    private readonly ILogger<IdentityDataSeeder> _logger;
-
-    public IdentityDataSeeder(IServiceProvider serviceProvider, ILogger<IdentityDataSeeder> logger)
-    {
-        _serviceProvider = serviceProvider;
-        _logger = logger;
-    }
-
     public async Task StartAsync(CancellationToken cancellationToken)
     {
-        using var scope = _serviceProvider.CreateScope();
+        using var scope = serviceProvider.CreateScope();
         var context = scope.ServiceProvider.GetRequiredService<IdentityDbContext>();
         var options = scope.ServiceProvider.GetRequiredService<IOptions<AdminSeedOptions>>().Value;
         var passwordHasher = scope.ServiceProvider.GetRequiredService<IPasswordHasher<DomainIdentityUser>>();
 
         await context.Database.MigrateAsync(cancellationToken);
 
-        if (options.Users == null || options.Users.Count == 0)
+        if (options.Users is null or { Count: 0 })
         {
-            _logger.LogInformation("AdminSeed 未配置账号，跳过后台账号初始化");
+            logger.LogInformation("AdminSeed 未配置账号，跳过后台账号初始化");
             return;
         }
 
@@ -64,7 +50,7 @@ public sealed class IdentityDataSeeder : IHostedService
                 };
                 user.PasswordHash = passwordHasher.HashPassword(user, userOptions.Password);
                 context.IdentityUsers.Add(user);
-                _logger.LogInformation("已创建后台账号 {Account}", user.Account);
+                logger.LogInformation("已创建后台账号 {Account}", user.Account);
             }
             else
             {
@@ -74,7 +60,7 @@ public sealed class IdentityDataSeeder : IHostedService
                 user.Roles = roles;
                 user.Permissions = permissions;
                 user.PasswordHash = passwordHasher.HashPassword(user, userOptions.Password);
-                _logger.LogInformation("已更新后台账号 {Account}", user.Account);
+                logger.LogInformation("已更新后台账号 {Account}", user.Account);
             }
         }
 
@@ -85,10 +71,9 @@ public sealed class IdentityDataSeeder : IHostedService
 
     private static string[] NormalizeValues(string[]? values)
         => values == null
-            ? Array.Empty<string>()
-            : values
+            ? []
+            : [.. values
                 .Where(v => !string.IsNullOrWhiteSpace(v))
                 .Select(v => v.Trim())
-                .Distinct(StringComparer.OrdinalIgnoreCase)
-                .ToArray();
+                .Distinct(StringComparer.OrdinalIgnoreCase)];
 }

src/Infrastructure/TakeoutSaaS.Infrastructure/Identity/Persistence/IdentityDbContext.cs

@@ -1,4 +1,3 @@
-using System;
 using System.Linq;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.ChangeTracking;
@@ -11,12 +10,8 @@ namespace TakeoutSaaS.Infrastructure.Identity.Persistence;
 /// <summary>
 /// 身份认证 DbContext。
 /// </summary>
-public sealed class IdentityDbContext : DbContext
+public sealed class IdentityDbContext(DbContextOptions<IdentityDbContext> options) : DbContext(options)
 {
-    public IdentityDbContext(DbContextOptions<IdentityDbContext> options)
-        : base(options)
-    {
-    }
 
     public DbSet<IdentityUser> IdentityUsers => Set<IdentityUser>();
     public DbSet<MiniUser> MiniUsers => Set<MiniUser>();
@@ -37,11 +32,11 @@ public sealed class IdentityDbContext : DbContext
         builder.Property(x => x.Avatar).HasMaxLength(256);
 
         var converter = new ValueConverter<string[], string>(
-            v => string.Join(',', v ?? Array.Empty<string>()),
+            v => string.Join(',', v),
             v => string.IsNullOrWhiteSpace(v) ? Array.Empty<string>() : v.Split(',', StringSplitOptions.RemoveEmptyEntries));
 
         var comparer = new ValueComparer<string[]>(
-            (l, r) => l!.SequenceEqual(r!),
+            (l, r) => (l == null && r == null) || (l != null && r != null && Enumerable.SequenceEqual(l, r)),
             v => v.Aggregate(0, (current, item) => HashCode.Combine(current, item.GetHashCode())),
             v => v.ToArray());