Question

背景

我正在开发一个 .Net Core API 来驱动一个歌词应用程序。用户可以注册，提交艺术家和歌词，并在此过程中获得荣誉/ XP积分。基本上是一个社区驱动的歌词网站。

CODE

这是我的ArtistController类：

[Route("api/artists")]
public class ArtistsController : Controller
{
  private readonly IPermissionsService _permissionsService;
  private readonly IArtistsService _artistsService;

  public ArtistsController(IArtistsService artistsService, IPermissionsService permissionsService)
  {
    _permissionsService = permissionsService ?? throw new ArgumentNullException(nameof(permissionsService));
    _artistsService = artistsService ?? throw new ArgumentNullException(nameof(artistsService));
  }

  [HttpGet("{slug}")]
  [HttpGet("{slug}/lyrics", Name = "GetArtist")]
  public async Task<IActionResult> GetArtist(string slug)
  {
    if (!_artistsService.ArtistExists(slug)) return NotFound();
    var permissions = await _permissionsService.GetPermissions(HttpContext);
    var artist = _artistsService.GetArtistBySlug(slug, permissions.UserId, permissions.IsAdministrator);
    if (artist == null) return NotFound();
    return Ok(artist);
  }

  // other methods omitted
}

本着可测试性的精神，我创建了IPermissionsService，这样当我来单元测试控制器时，我可以轻松地这样做，而不必担心HttpContext和User。< / p>

以下是PermissionsService类的代码：

public class PermissionsService : IPermissionsService
{
  private string _userId;
  private bool _isAdministrator;
  private HttpContext _httpContext;
  private readonly UserManager<BbUser> _userManager;

  public PermissionsService(UserManager<BbUser> userManager)
  {
    _userManager = userManager;
  }

  public async Task<Permissions> GetPermissions(HttpContext httpContext)
  {
    _httpContext = httpContext;
    PopulateUserIdAndIsAdminFlag();
    var permissions = new Permissions
    {
      UserId = _userId,
      IsAdministrator = _isAdministrator
    };

    return await Task.Run(() => permissions);
  }

  private async void PopulateUserIdAndIsAdminFlag()
  {
    if (!IsAuthenticated()) return;
    var username = _httpContext.User.FindFirstValue(ClaimTypes.NameIdentifier);
    var user = await _userManager.FindByNameAsync(username);
    var roles = await _userManager.GetRolesAsync(user);
    _userId = user.Id;
    _isAdministrator = roles.Contains("Admin");
  }

  private bool IsAuthenticated()
  {
    return _httpContext.User.Identity.IsAuthenticated;
  }
}

问题

当我运行API并尝试调用该终点时。我收到以下错误：

在上一个操作完成之前，在此上下文中启动了第二个操作。不保证任何实例成员都是线程安全的。

消息很清楚，但我不知道如何克服这个错误。在将该逻辑移到PermissionsService之前，我没有收到错误，一切正常！

Answer 1

执行async工作的方法应该返回Task而不是void，除非它们是事件处理程序。这将允许等待结果Task。由于不等待PopulateUserIdAndIsAdminFlag，因此您跨线程同时调用同一个DbContext实例。如果你按照调用堆栈：

代码输入GetPermissions
您开始在PopulateUserIdAndIsAdminFlag 工作，但不要等待它完成
代码立即从GetPermissions返回（{{1>}中的代码仍在执行）
代码继续并在PopulateUserIdAndIsAdminFlag

这可能会导致多个线程同时调用DbContext，从而导致异常。

修复代码，以便等待_artistsService的结果。

将代码更改为等待PopulateUserIdAndIsAdminFlag方法，以便返回类型PopulateUserIdAndIsAdminFlag
等待Task
PopulateUserIdAndIsAdminFlag

Task

我还建议重命名并添加后缀GetPermissions，因为这被认为是返回类型Async的方法的正确命名约定。这将导致名为Task和GetPermissionsAsync

更改了代码：

PopulateUserIdAndIsAdminFlagAsync

在UserManager上等待导致错误在上一个操作完成之前，在此上下文中启动了第二个操作

1 个答案: