Может ли кэшироваться данные в EF Core?

Question

[Александр Никульшин]

Подскажите, пожалуйста. Есть проект на winforms с DI контейнерами и EF Core база дынных sqllite . Каждая форма вызывается как scoped сервис. В констркуторе формы внедрен DBContextFactory. И возникает следующая проблема: открывается форма ShowDialog, меняется товар (его цена). После закрытия формы вызывается GoodList = await db.Goods.ToListAsync(). Но в GoodList отображается старая цена. Что не так?

ТОчнее, почему так работает. Я в вызываемой форме делаю сохранение. Такое ощущение, что EF Core кэширует данные

Comments

[Василий Банников]

Может быть даже не кэширование, а просто транзакция.
Например если на читающем контексте транзакция началась раньше, чем было сделано сохранение в пишущем - тогда сама база может отдать старые данные.

В самом EF, емнип, кэширования изначально нет.

Нужно смотреть код.

[Александр Никульшин]

Василий Банников, нет, транзакции нет. Раньше, когда в конструкторах формы было внедрение DBContext, то такой проблемы не возникало. Возникала проблема доступа из многих потоков, так как каждая форма отдельный сервис.
После того как я перевел на DBContextFactory, то стала возникать такая проблема

Код вызова второй формы из главной:

var fr = serviceProvider.GetRequiredService<FormMenu>();
fr.ShowDialog();
//После закрытие второй формы перечитываем список товаров, который поменялся
Task.Run(async () =>
            {
                GoodList = await db.Goods.Where(g=>g.IsDeleted==false).ToListAsync();
            });

Код второй формы, в нем происходит загрузка списка товаров с сервера

Task.Run(async () =>
            {
                bool statusSuccess = true;
                string messageRequest = "";
                try
                {
                    var str = await new HttpClient().GetAsync($"{serverName}/api/Goodssynchnew/{idShop}").Result.Content.ReadAsStringAsync();
                    List<GoodSynchDataModel> goods = JsonSerializer.Deserialize<List<GoodSynchDataModel>>(str);
                    foreach (var good in goods)
                    {
                        var goodDb = _db.Goods.Include(g=>g.BarCodes).Where(g => g.Uuid == good.Uuid).FirstOrDefault();
                        if (goodDb == null)
                        {
                            var newgood = new Good
                            {
                                Uuid = good.Uuid,
                                Name = good.Name,
                                Article = good.Name,
                                Unit = good.Unit,
                                Price = good.Price,
                                SpecialType=good.SpecialType,
                                VPackage=good.VPackage,
                                IsDeleted=good.IsDeleted
                            };
                            _db.Goods.Add(newgood);
                            //добавление штрих кодов
                            foreach (string barcode in good.Barcodes)
                                _db.BarCodes.Add(new BarCode
                                {
                                    Good = newgood,
                                    Code = barcode
                                });
                        }
                        else
                        {
                            goodDb.Name = good.Name;
                            goodDb.Unit = good.Unit;
                            goodDb.Price = good.Price;
                            goodDb.SpecialType = good.SpecialType;
                            goodDb.VPackage = good.VPackage;
                            goodDb.IsDeleted = good.IsDeleted;
                            //добавление новых или измененных штрих кодов
                            foreach (string barcode in good.Barcodes)
                                if (goodDb.BarCodes.Count(b => b.Code == barcode) == 0)
                                    _db.BarCodes.Add(new BarCode { Good = goodDb, Code = barcode });
                            //Удаление не зарегестрированных на сервере штрихкодов
                            foreach (var barcodeDb in goodDb.BarCodes)
                                if (good.Barcodes.Count(b => b == barcodeDb.Code) == 0)
                                    _db.BarCodes.Remove(barcodeDb);
                        }
                    };
                    await _db.SaveChangesAsync();
                    //System.Threading.Thread.Sleep(TimeSpan.FromSeconds(20));
                    messageRequest = "Перезыпаустите программу, для работы по новым ценам";
                }
                catch (SystemException ex)
                {
                    statusSuccess = false;
                    messageRequest = ex.Message;
                }
                var action = new Action(() =>
                  {
                      button1.BackColor = statusSuccess ? Color.LightGreen : Color.LightPink;
                      errorTextBox.Text = messageRequest;
                  });
                Invoke(action);
            });

Вот как у меня организован DI:

public static void ConfigureService(ServiceCollection services)
        {
            var builder = new ConfigurationBuilder()
            .SetBasePath(Path.Combine(AppContext.BaseDirectory))
            .AddJsonFile("appsettings.json", optional: true);
            services
                .AddDbContextFactory<DataContext>(opt=>opt.UseSqlite("Data Source=CustomerDB.db;"))
                .AddSingleton<BarCodeScanner>()
                .AddSingleton<ICashRegisterService, AtolService>()
                .AddLogging(configure => { configure.AddSerilog(); configure.SetMinimumLevel(LogLevel.Error | LogLevel.Warning); })
                .AddScoped<ISynchService, SynchService>()
                .AddSingleton<ISynchBackgroundService, SynchBackgroundService>()
                .AddTransient<IConfiguration>(_ => builder.Build())
                .AddScoped<Form1>()
                .AddScoped<FormMenu>()
                //.AddTransient<PayForm>()
                .AddTransient<FormPaymentCombine>()
                .AddTransient<FormWriteOf>()
                .AddTransient<FormArrival>()
                .AddTransient<FormStocktaking>()
                .AddTransient<FormCashMoney>()
                .AddTransient<FormNewGood>()
                .AddTransient<FormFindGood>()
                .AddTransient<FormHistory>()
                ;
        }

[Мирон]

Here is the answer
There is a reference to this blog post

Entity Framework Cache Busting
2016-02-19
The DbContext in Entity Framework 6 automatically caches data that it retrieves from your database. This is useful, but sometimes data changes outside your context (perhaps by another user) and you end up with stale data.

In my personal experience all of the above, plus whatever is in the blog, is a poor's man patch from водка и подорожник.
One should strive, as much as possible, to extend DBContext with methods and call in those stored procedures, where concurrent access to data is accounted for using
1 Query design
2 Transaction Isolation Level
3 Tuning options at all levels, including, query, database and server instance
Entities solve a lot of troubles, so, you don't have to code every POCO. The framework simplifies for developer data layer and magically extracts it from some foreign "database engine" into the application, as an opaque architecture, from which one can design entire application.
But beyond it, data access via some scaffolding based code, is utter nonsense.
I just can't believe, that someone with Russian Computer Science or Math University diploma would ask questions like this.
It's unbelievable!

[Василий Банников]

Александр Никульшин,
1. Нельзя к одному DbContext обращаться из нескольких потоков. Тем более, раз у тебя sqlite, то в принципе нельзя одновременно к одной базе из разных потоков обращаться.
2.

await new HttpClient()

Нельзя так писать. У тебя так просто сокеты все утекут и ты не сможешь больше запросы делать.
3.

.GetAsync($"{serverName}/api/Goodssynchnew/{idShop}").Result

await забыл. Нельзя .Result делать
4. Где создаётся экземпляр db и _db?

У меня больше всего подозрения на Task.Run, так как нельзя проверить, как в реальности у тебя код исполняется.
(Что и в каком порядке).
Из-за Task.Run - не известно, когда на самом деле оно всё завершается.

[Александр Никульшин]

2. А как тогда использовать вместо await new HttpClient() ?

4.
Я делаю через внедрение в конструктор:
Главная форма:

DataContext db;

public Form1(IServiceProvider serviceProvider, ILogger<Form1> logger, IDbContextFactory<DataContext> dbFactory, ISynchService synchService, BarCodeScanner barCodeScanner, ICashRegisterService cashService)
{
                db = dbFactory.CreateDbContext();
                _cashService = cashService;
                this.serviceProvider = serviceProvider;
                _logger = logger;

В форме загрузки товаров так же

DataContext _db;
        public FormMenu(IConfiguration configuration, IDbContextFactory<DataContext> dbFactory)
        {
            serverName = configuration.GetSection("serverName").Value;
            idShop = Convert.ToInt32(configuration.GetSection("idShop").Value);
            cashierName = configuration.GetSection("cashierName").Value;
            cashierInn = configuration.GetSection("cashierInn").Value;
            _configuration = configuration;
            _db = dbFactory.CreateDbContext();
            InitializeComponent();
        }

[Василий Банников]

Александр Никульшин,
2. Создавай HttpClient один раз в каком-нибудь одном месте и переиспользуй. Или используй IHttpClientFactory
4. А где dbContext диспозится?

И что там на счёт Task.Run?
Вполне возможно, что у тебя одновременно отрабатывает и сохранение и выгрузка.

[Василий Банников]

Мирон, а почему вы часть ответов пишите на русском, а часть на английском?

[Александр Никульшин]

Василий Банников, Решил проблему немного костыльно: после закрытия формы получаю список с использованием AsNoTracking. Мне список товаров нужен не для редактирования. Я так понял, это единственное решение если у меня dbcontextы разные?
Спасибо большое за помощь и за подсказки)))

Подскажите еще, di от Microsoft или Autofac какой лучше использовать для проектов winforms и wpf

[Мирон]

Василий Банников,
for security reasons I cannot install russian locale on this computer.
So, I have to enter russian text using Yandex keyboard, a mouse click per letter, which isn't really working for most of posts. But in some cases, depending on context, I feel obligated to answer in Russian, particularly when I feel vis-a-vis isn't really tolerant of english.
I try to be nice to people I speak to. But I have to think for myself, too.
In the end I am trying to help.
If you feel english isn't working, I will stop.
If english is acceptable ( I will learn from your future answer, just under supposition ) I try to write very concise english, very clear, as close to impeccable as I can, in the hope it would teach my vis-a-vis ways to read and express things in English.
In the end, pity it is, for the moment we don't have Russian speaking code. All of our code is English to some extent. So, I don't feel I am veering too far from the mark.
It would be very nice, speaking of the future, to have Russian environment, including cyrillic based vocabulary and semantical programming interface, end - to - end.
I'd work on it, if we start project like that.
If I may suggest, this post is grossly off top for the question asked. So, if moderators could move it to something like "off top answers" and provide link, for the future discussion, it would be very nice.

[Александр Никульшин]

Мирон, Спасибо тебе за наводку на AsNoTracking)

[Мирон]

Василий Банников,

И что там на счёт Task.Run?
Вполне возможно, что у тебя одновременно отрабатывает и сохранение и выгрузка.

One can easily test it by running same request 2nd time, after a bit of wait, when all database queries finished. In this second attempt he should see the change.
I was assuming the author of the question tried a few times, before coming to conclusion that the issue isn't concurrent access, but caching ( buffering ) of data on the front - end.
Besides, EF buffering ( if you dig into theoretical side of platform ) is an expression of IQueriable, which has caching defined as part of standard implementation, as opposed to IEnumerable, which requires hard reading of values from the media for each request.

[Василий Банников]

Мирон, понимаю.

[Василий Банников]

Александр Никульшин, разницы большой нет. autofac даёт чуть более простую регистрацию благодаря встроенному дискаверингу + из коробки есть autofac modules.
В то же время MS DI гораздо более легковесный и дискаверинг с модулями можно подключить отдельно.