Использую метод SaveChanges
чтобы сохранить изменения, внесенные пользователем на сайте. На локальном сервере все работает хорошо, как только загружаю на реальный сервер - метод вылетает на строке с db.SaveChangesAsync();
Вот код самого метода:
[HttpPost("[action]")]
public async Task<string> SaveChanges(string email, string name, string avatar, string password)
{
UserModel user = await db.Users.FirstOrDefaultAsync(u => u.EmailUser == User.Identity.Name);
List<UserModel> users = db.Users.ToList().FindAll(u => u.EmailUser == email);
user.EmailUser = email;
user.NameUser = name;
user.AvatarUser = avatar;
user.Password = password;
db.Users.Update(user);
await db.SaveChangesAsync(true);
return JsonConvert.SerializeObject("saved");
}
В чем может быть проблема?
метод вылетает на строке с db.SaveChanges();
у вас в коде нет методаSaveChanges
, ну и вы не пишете текст ошибки, с которой вылетаетUserModel user = await db.Users.FirstOrDefaultAsync(...)
может вернутьuser == null
, но вы не проверяете этот случай, а продолжаете работать с ним как не в чем ни бывало. Вторая строка... users = db.Users.ToList().FindAll(...)
бессмысленна, т.к. вы результат все равно нигде не используете, да еще сама по себе плохая, т.к. всех юзеров сначала выдергиваете в List, а потом что-то начинаете в них искать. А представьте на минуту, если пользователей миллион, надо ли нагружать так сервер СУБД и сеть, чтоб выдергивать такую коллекцию, из которой потом выбрать одного или двух?