Errore in EF Core: una seconda operazione avviata in questo contesto prima che fosse completata un'operazione precedente
.net asp.net asp.net-core entity-framework-core
Domanda
Ho avuto uno strano errore nella mia app web API: una seconda operazione è iniziata in questo contesto prima che una precedente operazione fosse completata. Non è garantito che tutti i membri di istanza siano thread-safe.
Ho un processo per calcolare il giorno di paga dell'Utente e aggiornare i suoi saldi di Congedo degli utenti. quindi ho bisogno di iterare gli utenti e ottenere il suo saldo di conguaglio e quindi fare l'aggiornamento su ogni saldo. Non ho idea di come risolvere questo errore ancora. L'errore innescato quando chiamo this.SaveAll () che è containb
public async Task<bool> SaveAll()
{
return await _context.SaveChangesAsync() > 0;
}
anche a datacontext viene iniettato questo codice:
private readonly DataContext _context;
private readonly IAdminSettingsRepository _settingRepo;
private readonly IAppRepository _appRepository;
public PayrollRepository(DataContext context, IAdminSettingsRepository settingRepo, IAppRepository appRepository)
{
_context = context;
_settingRepo = settingRepo;
_appRepository = appRepository;
}
il processo come segue:
public async Task<bool> ProcessPayCalendar(PayCalendar payCalendar)
{
List<User> users = payCalendar.Users.ToList();
SickLeaveEntitlement sickLeaveEntitlement = await _settingRepo.GetSickLeaveEntitlement();
AnnualLeaveEntitlement annualLeaveEntitlement = await _settingRepo.GetAnnualLeaveEntitlement();
LongServiceLeaveEntitlement longServiceLeaveEntitlement = await _settingRepo.GetLongServiceLeaveEntitlement();
DateTime tenYearsAgo = DateTime.Today.AddYears(-10);
DateTime currentMonth = new DateTime(payCalendar.NextPaymentDate.Year, payCalendar.NextPaymentDate.Month, 1);
DateTime previousMonth = currentMonth.AddMonths(-1);
/// begin calculation for leave calendar
users.ForEach(async user =>
{
decimal hoursWorked = this.CalculateWorkingHours(user);
decimal hourlyRate = this.CalculateHourlyRate(user);
// create pay day for user
Payday payday = new Payday();
payday.UserId = user.Id;
payday.PayPeriodStart = payCalendar.PayPeriodStartDate;
payday.PayPeriodEnd = payCalendar.PayPeriodEndDate;
payday.PaymentDate = payCalendar.NextPaymentDate;
payday.HoursWorked = hoursWorked;
payday.SickLeaveAccrual = this.CalculateSickLeaveEntitlement(hoursWorked, sickLeaveEntitlement);
payday.AnnualLeaveAccrual = this.CalculateAnnualLeaveEntitlement(hoursWorked, annualLeaveEntitlement);
payday.LongServiceLeaveAccrual = (user.StartDateCurrentAnnualSalary > tenYearsAgo) ? 0m : this.CalculateLongServiceEntitlement(hoursWorked, longServiceLeaveEntitlement);
payday.SickLeaveAccrualValue = payday.SickLeaveAccrual * hourlyRate;
payday.AnnualLeaveAccrualValue = payday.AnnualLeaveAccrual * hourlyRate;
payday.LongServiceLeaveAccrualValue = payday.LongServiceLeaveAccrual * hourlyRate;
this.Add(payday);
// do calculation on leave balance
// jika leave balance di awal bulan maka lakukan replikasi dr bulan sebelumnya.
// closebalance menjadi opening balance bulan selanjutnya.
LeaveBalance sickLeaveBalance = await this.GetUserLeaveBalance(user.Id, sickLeaveEntitlement, "sickLeave");
if (sickLeaveBalance == null)
{
// sickLeaveBalance = await this.CreateLeaveBalance(user.Id,sickLeaveEntitlement,"sickLeave",payday.PaymentDate);
// sickLeaveBalance.CurrentBalance =+ payday.SickLeaveAccrual ;
// sickLeaveBalance.CurrentBalanceValue += payday.SickLeaveAccrualValue;
throw new Exception($"Sick Leave balance for user: {user.Username} is not found. Please report this as bug");
}
else
{
sickLeaveBalance.CurrentBalance = +payday.SickLeaveAccrual;
sickLeaveBalance.CurrentBalanceValue += payday.SickLeaveAccrualValue;
sickLeaveBalance.LastUpdate = payday.PaymentDate;
}
LeaveBalance annualLeaveBalance = await this.GetUserLeaveBalance(user.Id, annualLeaveEntitlement, "annualLeave");
if (annualLeaveBalance == null)
{
throw new Exception($"Annual Leave balance for user: {user.Username} is not found. Please report this as bug");
}
else
{
annualLeaveBalance.CurrentBalance = +payday.AnnualLeaveAccrual;
annualLeaveBalance.CurrentBalanceValue += payday.AnnualLeaveAccrualValue;
annualLeaveBalance.LastUpdate = payday.PaymentDate;
}
LeaveBalance longServiceLeaveBalance = await this.GetUserLeaveBalance(user.Id, longServiceLeaveEntitlement, "longServiceLeave");
if (longServiceLeaveBalance == null)
{
throw new Exception($"Long Service Leave balance for user: {user.Username} is not found. Please report this as bug");
}
else
{
longServiceLeaveBalance.CurrentBalance = +payday.LongServiceLeaveAccrual;
longServiceLeaveBalance.CurrentBalanceValue += payday.LongServiceLeaveAccrualValue;
longServiceLeaveBalance.LastUpdate = payday.PaymentDate;
}
});
return await this.SaveAll();
}
l'errore completo:
fail: Microsoft.EntityFrameworkCore.Update [10000] Si è verificata un'eccezione nel database durante il salvataggio delle modifiche per il tipo di contesto "CRSApp.API.Data.DataContext". System.InvalidOperationException: una seconda operazione avviata in questo contesto prima che fosse completata un'operazione precedente. Non è garantito che tutti i membri di istanza siano thread-safe. su Microsoft.EntityFrameworkCore.Internal.ConcurrencyDetector.EnterCriticalSection () su Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync (IReadOnlyList
1 entriesToSave, CancellationToken cancellationToken) at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken) at Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken) System.InvalidOperationException: A second operation started on this context before a previous operation completed. Any instance members are not guaranteed to be thread safe. at Microsoft.EntityFrameworkCore.Internal.ConcurrencyDetector.EnterCriticalSection() at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(IReadOnlyList1 entriesToSave, CancellationToken cancellationToken) su Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync (Booleano acceptAllChangesOnSuccess, CancellationToken cancellationToken ) in Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync (Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken) fail: Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware [1] Si è verificata un'eccezione non gestita durante l'esecuzione della richiesta. System.InvalidOperationException: una seconda operazione avviata in questo contesto prima che fosse completata un'operazione precedente. Non è garantito che tutti i membri di istanza siano thread-safe. su Microsoft.EntityFrameworkCore.Internal.ConcurrencyDetector.EnterCriticalSection () su Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync (IReadOnlyList1 entriesToSave, CancellationToken cancellationToken) at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken) at Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken) at CRSApp.API.Data.PayrollRepository.SaveAll() in E:\CRSApp\crsapp.api\Data\PayrollRepository.cs:line 35 at CRSApp.API.Data.PayrollRepository.ProcessPayCalendar(PayCalendar payCalendar) in E:\CRSApp\crsapp.api\Data\PayrollRepository.cs:line 229 at CRSApp.API.Controllers.Admin.PayrollController.ProcessPayCalendar(PayCalendarParam param) in E:\CRSApp\crsapp.api\Controllers\Admin\PayrollController.cs:line 68 at Microsoft.AspNetCore.Mvc.Internal.ActionMethodExecutor.TaskOfIActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments) at System.Threading.Tasks.ValueTask1.get_Result () in Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeActionMethodAsync () in Microsoft.AspNetCore. Mvc.Internal.ControllerActionInvoker.InvokeNextActionFilterAsync () in Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow (contesto ActionExecutedContext) in Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next (Stato e successivo, Scope & scope, Object & state, Boolean & isCompleted) in Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeInnerFilterAsync () in Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextResourceFilter () in Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow (contesto ResourceExecutedContext) in Microsoft.AspNetCore.Mvc .Internal.ResourceInvoker.Next (Stato e successivo, Scope & scope, Object & state, Boolean & isCompleted) su Microsoft.AspNetCore .Mvc.Internal.ResourceInvoker.InvokeFilterPipelineAsync () in Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeAsync () in Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke (HttpContext httpContext) in Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke (HttpContext context) in Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware.Invoke (contesto HttpContext) in Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke (contesto HttpContext) in Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.Invoke (contesto HttpContext)
Fyi: Io uso Dotnet Core 2.1.1 come indicato sotto.
<ItemGroup>
<PackageReference Include="Microsoft.AspNetCore.App" Version="2.1.1"/>
<PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="2.1.0" PrivateAssets="All"/>
<PackageReference Include="AutoMapper.Extensions.Microsoft.DependencyInjection" Version="4.0.1"/>
<PackageReference Include="Pomelo.EntityFrameworkCore.MySql" Version="2.1.1"/>
<PackageReference Include="MailKit" Version="2.0.6"/>
</ItemGroup>
Risposta popolare
DbContext non è thread-safe
Penso che questo ForEach è il vostro problema.
users.ForEach(async user =>
{
//....
LeaveBalance sickLeaveBalance = await this.GetUserLeaveBalance(user.Id,});
// .....
});
come hai usato la parola chiave async così per ogni utente un'azione verrà invocata in modo asincrono in modo che l'esecuzione del codice sarà simile all'esecuzione del codice seguente:
foreach(var user in users){
//without await
//it's an async method
DoSomeThingAsync(user); //you called GetUserLeaveBalance in DoSomeThingAsync
}
e suppongo che GetUserLeaveBalance stia usando DbContext quindi stai usando DbContext modo asincrono e ti troverai di fronte a un errore del genere
Una seconda operazione è iniziata in questo contesto prima che una precedente operazione fosse completata ...
Devi cambiare il tuo ForEach in qualcosa di simile
foreach(var user in users){
await DoSomeThingAsync(user); //you called GetUserLeaveBalance in DoSomeThingAsync
}
Autorizzato sotto: CC-BY-SA with attribution
Non affiliato con Stack Overflow