CascadeDelete SQLite unter .NetCore 3.1

MichaHo · 4. Februar 2020, 09:27

Hallo,
Folgende Umgebung:
WPF .NetCore 3.1 Projekt
Datenbank SQLite (local).
Tabellen: Person, GraduationEntry, PresentEntry
die Tabelle Person (representiert 1 Person):

Spoiler anzeigen

C#-Quellcode

public class Person : ModelBase
{
[Required(AllowEmptyStrings = false, ErrorMessage = "Bitte geben sie einen Vornamen an")]
[MinLength(3)]
[MaxLength(150)]
public virtual string FirstName { get; set; }
[Required(AllowEmptyStrings = false, ErrorMessage = "Bitte geben sie einen Nachnamen an")]
[MinLength(3)]
[MaxLength(150)]
public virtual string LastName { get; set; }
public virtual LoginData AcessData { get; set; }
public virtual string FullName => $"{LastName}, {FirstName}";
public virtual DateTime EntryDate { get; set; } = DateTime.Today;
public virtual GenderEnum Gender { get; set; } = GenderEnum.male;
public virtual PersonTypeEnum PersonType { get; set; } = PersonTypeEnum.Shinzan;
public virtual int MemberId { get; set; }
public virtual ICollection<GraduationEntry> Graduations { get; set; }
public virtual ICollection<PresentEntry> PresentEntries { get; set; }
public virtual bool IsAdmin { get; set; } = false;
public Person()
{
Graduations = new List<GraduationEntry>();
PresentEntries = new List<PresentEntry>();
}
}
public enum GenderEnum
{
male,
female
}
public enum PersonTypeEnum
{
Shinzan,
ShinzanKids
}

Tabelle GraduationEntry (representiert 1 Graduierung):

Spoiler anzeigen

Tabelle PresentEntry (representiert 1 Anwesend Eintrag):

Spoiler anzeigen

Der Context dazu:

Spoiler anzeigen

OK, soweit so gut. Die DB wird angelegt. Schaue ich nun in die DB Struktur, sehe ich auch, das EF Cascade Delete angelegt hat.
Hier am Beispiel GraduationEntry:

SQL-Abfrage

CREATE TABLE "GraduationEntry" (
"Id" INTEGER NOT NULL CONSTRAINT "PK_GraduationEntry" PRIMARY KEY AUTOINCREMENT,
"CreatedBy" INTEGER NOT NULL,
"CreationTimeStamp" TEXT NOT NULL,
"LastModifiedBy" INTEGER NOT NULL,
"LastModifiedTimeStamp" TEXT NOT NULL,
"DeletedFlag" INTEGER NOT NULL,
"GraduationId" INTEGER NOT NULL,
"GraduationReceived" TEXT NULL,
"GraduationHold" TEXT NULL,
"PersonId" INTEGER NOT NULL,
CONSTRAINT "FK_GraduationEntry_People_PersonId" FOREIGN KEY ("PersonId") REFERENCES "People" ("Id") ON DELETE CASCADE
)

Verstehe ich also so, das wenn ich bei der Person aus der Auflistung der GraduationEntry einen Eintrag gerausnehme (Auflistung.Remove), das dann automatisch in der GraduationEntry Tabelle der dazugehörige Eintrag auch gelsöcht wird.

Nun, wird es aber leider nicht.

Muss ich den Eintrag selbst löschen?

Ich hab auch versucht, im Context der Tabelle GraduationEntry explizit zu sagen, das er Cascade löschen soll, bringt aber auch nichts.
Fehlermeldung kommt keine, die Person wird richtig gespeichert (upgedatet) und beim Update ist der Eintrag auch wirklich weg, aber nach neuem Start der Anwendung ist der Eintrag wieder da.

Wie lösche ich denn den Eintrag in der Tabelle wenn ich einen Eintrag aus der Auflistung entferne?

Nofear23m · 4. Februar 2020, 09:40

MichaHo schrieb:

aber nach neuem Start der Anwendung ist der Eintrag wieder da.

Kann hier der Fehler liegen?
Wenn du einen Eintrag updatest was gibt SaveChanges() da zurück. Kommt hier die richtige Anzahl an Datensätzen zurück?

Hast du hier diesbezüglich noch etwas in OnModelCreating() konfiguriert??

Grüße
Sascha

MichaHo · 4. Februar 2020, 09:53

in OnModelCreating hab ich nichts configuriert.
SaveChanges hat auch tatsächlich nur noch 1 Graduierung drin, die, die ich gelöscht habe ist nicht mehr drin beim SaveChanges...
auch wenn ich durchsteppe und finde dann irgendwo die GraduationEntry, ist da nur noch 1 Eintrag drin.

EDIT:
auch wenn ich OnModelCreating nutze:

C#-Quellcode

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
modelBuilder.Entity<PresentEntry>().HasOne(p => p.Person).WithMany(e => e.PresentEntries).HasForeignKey(p => p.PersonId).OnDelete(DeleteBehavior.Cascade);
modelBuilder.Entity<GraduationEntry>().HasOne(p => p.Person).WithMany(g => g.Graduations).HasForeignKey(p => p.PersonId).OnDelete(DeleteBehavior.Cascade);
}

wird es zwar schön eingteragen im Create Statement, aber gelöscht wird da nix.

Muss ich die beiden Tabellen doch als DBSet anlegen und dann direkt in der Tabelle löschen?

Nofear23m · 4. Februar 2020, 10:20

MichaHo schrieb:

SaveChanges hat auch tatsächlich nur noch 1 Graduierung drin

Langsam. SaveChanges hat nix drin. Ist ne Methode mit einem Integer Rückgabewert welche zurückgibt wieviele Datensätze (Entitäten) vom Speichern betroffen waren. Du meinst den Context.

Gibt SaveChanges also 0 zurück ist was nicht in Ordnung. Stichwort Tracking.

Das du beim Debuggen im Context nur den soeben gelöschten Datensatz nicht mehr siehst bedeutet nicht das dieser in der DB gelöscht wurde. Er ist lediglich von Context entfernt. Hat der ChangeTracker dies aber nicht "mitbekommen" kannst du Savechanges() aufrufen sooft du willst, es wird kein Delete an die DB gesendet.

Meine Empfehlung: (Die gilt übrigens bei EF IMMER zur Entwicklungszeit)
EF Log einschalten und kontrollieren welche Querys an die DB gehen, erst dann kannst du sicher sein das alles passt.

Was gibt SaveChanges also zurück?

Grüße
Sascha

MichaHo · 4. Februar 2020, 10:28

Save Changes hat 3 zurück gegeben.
Ich schau mir das mit dem Log an

MichaHo · 4. Februar 2020, 11:17

OK, Logging habe ich eingeschaltet (hab dazu deine DbSimpleLogger usw. aus dem WPF Projekt genutzt).
Ich hab das Log direkt in ein File geschrieben.
Man sieht darin (ziemlich weit unten) das die Tanelle GraduationEntry nur upgedatet wird, ein Delete Statement passiert leider nicht....

Nofear23m · 4. Februar 2020, 12:04

Hallo

Moment mal....
Die von dir geposteten Klassen stimmen mal garnicht mit dem überein was hier im Log sichtbar wird.

SQL-Abfrage

UPDATE "GraduationEntry" SET "CreatedBy" = @p0, "CreationTimeStamp" = @p1, "DeletedFlag" = @p2, "GraduationHold" = @p3, "GraduationId" = @p4, "GraduationReceived" = @p5, "LastModifiedBy" = @p6, "LastModifiedTimeStamp" = @p7, "PersonId" = @p8
WHERE "Id" = @p9;

Hier hat GraduationEntry Spalten wie "DeletedFlag" und die TameStamp Spalten.

Verwendest du die Interfaces aus meinem WPFNotes Projekt?

Grüße
Sascha

MichaHo · 4. Februar 2020, 12:14

Ich verwende die ModelBase, die ist bei jeder Modelklasse hinterlegt:
einmal ein Interface:

C#-Quellcode

public interface IModel
{
bool IsValid { get; }
IList<ValidationResult> Validate();
}

und dann die ModelBase:

C#-Quellcode

public abstract class ModelBase : IModel
{
[Key]
public virtual int Id { get; set; }
public virtual CreationUserEnum CreatedBy { get; set; } = CreationUserEnum.System;
public virtual DateTime CreationTimeStamp { get; set; } = DateTime.Now;
public virtual CreationUserEnum LastModifiedBy { get; set; } = CreationUserEnum.System;
public virtual DateTime LastModifiedTimeStamp { get; set; } = DateTime.Now;
public virtual bool DeletedFlag { get; set; }
#region Validate Implementation
public bool IsValid => Validate() != null && Validate().Count == 0;
public IList<ValidationResult> Validate()
{
var validationResults = new List<ValidationResult>();
var validationContext = new ValidationContext(this);
Validator.TryValidateObject(this, validationContext, validationResults, true);
return validationResults;
}
#endregion
}
public enum CreationUserEnum
{
System = -1,
User = 1
}

Nofear23m · 4. Februar 2020, 12:17

OK, bitte Poste mal die Gesamte Context-Klasse.
Ich vermute fast du überschreibst auch die SaveChanges-Methode wie ich in meinen Projekten. Kann das sein?

MichaHo · 4. Februar 2020, 12:20

Nee, ich überschreib da nix:

Das ist die Gesamte Context Klasse:

C#-Quellcode

public class ShinzanDojoContext : DbContext
{
#region Private Member
private readonly string _dbFile = @"sdDb.db";
#endregion
public static readonly LoggerFactory MyLoggerFactory = new LoggerFactory(new[] { new DbLoggerProvider(s => File.AppendAllText("Log.txt",s))});
#region Constructor
public ShinzanDojoContext()
{
}
#endregion
#region DbSets
public virtual DbSet<Person> People { get; set; }
public virtual DbSet<Graduation> Graduations { get; set; }
#endregion
#region Overrides
protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
{
optionsBuilder.UseSqlite($"Data Source={_dbFile}");
optionsBuilder.UseLoggerFactory(MyLoggerFactory);
optionsBuilder.EnableSensitiveDataLogging();
//optionsBuilder.UseSqlite($"Data Source={_dbFile};foreign keys=true");
}
protected override void OnModelCreating(ModelBuilder modelBuilder)
{
modelBuilder.Entity<PresentEntry>().HasOne(p => p.Person).WithMany(e => e.PresentEntries).HasForeignKey(p => p.PersonId).OnDelete(DeleteBehavior.Cascade);
modelBuilder.Entity<GraduationEntry>().HasOne(p => p.Person).WithMany(g => g.Graduations).HasForeignKey(p => p.PersonId).OnDelete(DeleteBehavior.Cascade);
}
#endregion
}

ich nutze darin deine DbSimpleLogger, den DbLoggerProvider und die DbFacadeExtension

Nofear23m · 4. Februar 2020, 12:28

OK? Dann sehe ich keinen Grund warum EF den Datensatz "nur" aktualisiert anstatt ihn zu löschen.
Im gesamten Log sehe ich kein DELETE Statement.

Jetzt bleibt nur noch eines, der Code vom ViewModel bzw. von der BL und/oder des DataAccess Layer (Ich kenne deine genaue Struktur nicht) damit ich zum einen sehe wie du die Daten holst und zum anderen wie du sie löscht.
hier wäre eben auch wichtig wie du die Daten holst bezüglich des ChangeTrackers.
Sollte das alles in einem ViewModel passieren dann am besten das gesamte ViewModel bitte.

da kommen wir schon drauf was da nicht stimmt, ist hald ein wenig verzwickt.

Grüße
Sascha

MichaHo · 4. Februar 2020, 12:42

OK, dann schauen wir mal ob ich alles mitnehme....
Also, es geht hier erstmal nur darum, eine eingetragene Graduierung wieder entfernen zu können.
Ich hatte es zwischenzetlich mit dem setzen des DeletedFlag auf der selectierten Graduierung hinbekommen, das wäre PlanB.
So, das entsprechende ViewModel:

Spoiler anzeigen

C#-Quellcode

public class GraduationHistoryListViewModel : ViewModelBase
{
private readonly Person _personModel;
#region Public Properties
public string PersonName => _personModel.FullName;
public ObservableCollection<GraduationHistoryListItemViewModel> GraduationHistory { get; set; }
private GraduationHistoryListItemViewModel _selectedGraduation;
public GraduationHistoryListItemViewModel SelectedGraduation { get => _selectedGraduation; set => SetValue(ref _selectedGraduation, value); }
#endregion
#region Constructor
public GraduationHistoryListViewModel()
{
}
public GraduationHistoryListViewModel(Person person)
{
_personModel = person;
GetGraduationHistory();
BackCommand = new RelayCommand(b => IoC.Application.GoToPage(ApplicationPageEnum.AdminMain, new AdminMainViewModel(IoC.Application.CurrentPerson)));
DeleteGraduationCommand = new RelayCommand(d => Delete());
}
#endregion
#region Commands
public ICommand BackCommand { get; private set; }
public ICommand DeleteGraduationCommand { get; private set; }
#endregion
#region Command Methods
private void Delete()
{
if(GraduationHistory.Count > 1)
{
//Alternativ auf Deleted setzen
//_personModel.Graduations.FirstOrDefault(g => g.GraduationId == SelectedGraduation.GraduationEntry.Id).DeletedFlag = true;
_personModel.Graduations.Remove(SelectedGraduation.GraduationEntry);
if(PeopleLogic.UpdatePerson(_personModel))
GraduationHistory.Remove(SelectedGraduation);
}else
{
IoC.UI.ShowMessage(new MessageBoxDialogViewModel
{
ButtonText = "OK, Danke",
Message = String.Format("Sie können nicht alle Graduierungen löschen!{0}Im Zweifel, legen Sie bitte eine neue Graduierung{0}in den Teilnehmer Einstellungen an.{0}{0}", Environment.NewLine)
});
}
}
#endregion
#region Helper Methods
public ObservableCollection<GraduationHistoryListItemViewModel> GetGraduationHistory()
{
var graduationHistoryList = _personModel.Graduations.Where(g => !g.DeletedFlag).ToList();
GraduationHistory = new ObservableCollection<GraduationHistoryListItemViewModel>();
graduationHistoryList.ForEach(g => GraduationHistory.Add(new GraduationHistoryListItemViewModel(g)));
return GraduationHistory;
}
#endregion
}

Vielleicht kurz zur Erklärung noch... Ich arbeite hier mit Pages. Die Pages werden über das ApplicationViewModel gesteuert. Das hat eine Methode GoToPage und verlangt eine PageEnum und ein Viewmodel.
Navigiere ich nun zur Page, übergebe ich dem ViewModel eine Person, im Falle der Graduierung übergebe ich die Selected Person aus der vorherigen Page (das ist quasi eine Personenliste)

OK, weiter:
Zum obigen ViewModel gehört noch die ListItemViewModel:

Spoiler anzeigen

Das ApplicationViewModel hält eine Liste aller Graduierungen die es gibt in der Anwendung.

Hier die Komplette PeopleLogic Klasse:

Spoiler anzeigen

C#-Quellcode

public class PeopleLogic
{
#region Person Data Methods
/// <summary>
/// Legt eine übergebene Person in der Datenbank an
/// </summary>
/// <param name="person">die übergebene Person</param>
/// <returns></returns>
public static bool CreatePerson(Person person)
{
using (var prov = new PeopleProvider())
{
return prov.Insert(person);
}
}
public static bool CreateOrUpdatePerson(Person person)
{
if (PersonExists(person))
return UpdatePerson(person);
return CreatePerson(person);
}
/// <summary>
/// Aktualisiert die übergebene Person in der Datenbank
/// </summary>
/// <param name="person">die übergebene Person</param>
/// <returns></returns>
public static bool UpdatePerson(Person person)
{
using (var prov = new PeopleProvider())
{
return prov.Update(person);
}
}
public static bool DeletePerson(Person person)
{
using (var prov = new PeopleProvider())
{
return prov.Delete(person);
}
}
#endregion
#region Person Get Methods
/// <summary>
/// gibt die ID der ersten Person aus der Datenbank zurück
/// wenn keine Person in der Datenbank vorhanden ist
/// wird -1 zurück gegeben
/// </summary>
/// <returns></returns>
public static int GetLoggedInPersonId(string user, string pass)
{
using (var prov = new PeopleProvider())
{
var usr = prov.Get();
if (usr.AcessData.UserName == user && usr.AcessData.EncryptedPassword == pass.GetHash())
return usr.Id;
return -1;
}
}
public static int GetPresentDays(int personId)
{
using (var prov = new PeopleProvider())
{
return prov.Get(personId).PresentEntries.Count(e => !e.DeletedFlag);
}
}
public static List<Person> GetPresentPersons()
{
using (var prov = new PeopleProvider())
{
var entryList = new List<PresentEntry>();
var personList = new List<Person>();
foreach (var person in prov.GetList())
{
foreach (var item in person.PresentEntries)
{
if (item.Date != DateTime.Today && item.PresentState == PresentStateEnum.Present)
{
person.PresentEntries.FirstOrDefault(p => p.Id == item.Id).PresentState = PresentStateEnum.Absent;
UpdatePerson(person);
}
if (item.PresentState == PresentStateEnum.Present)
personList.Add(person);
}
}
return personList;
}
}
/// <summary>
/// gibt die erste Person aus der Datenbank zurück
/// </summary>
/// <returns></returns>
public static Person GetPerson(int id)
{
using (var prov = new PeopleProvider())
{
return prov.Get(id);
}
}
public static Person GetPersonByMemberId(int memberId)
{
using (var prov = new PeopleProvider())
{
return prov.GetByMemberId(memberId);
}
}
public static int GetLastMemberId()
{
using (var prov = new PeopleProvider())
{
return prov.GetLast().MemberId;
}
}
public static List<Person> GetPersons(PersonTypeEnum type, bool isDeactivated)
{
using (var prov = new PeopleProvider())
{
return prov.GetList().Where(p => !p.IsAdmin && p.PersonType == type && p.DeletedFlag == isDeactivated).ToList();
}
}
#endregion
#region Helper Methods
public static IEnumerable<PersonTypeEnum> GetPersonTypes()
{
return Enum.GetValues(typeof(PersonTypeEnum)).OfType<PersonTypeEnum>();
}
/// <summary>
/// gibt zurück ob die Tabelle People Einträge hat
/// </summary>
/// <returns></returns>
public static bool HasEntries()
{
using (var prov = new PeopleProvider())
{
return prov.HasEntries();
}
}
public static bool PersonExists(Person person)
{
return GetPerson(person.Id) != null;
}
#endregion
}

und der Provider dazu:

Spoiler anzeigen

C#-Quellcode

public class PeopleProvider : ShinzanDojoProvider, IGenericProvider<Person>
{
public bool Insert(Person item)
{
Context.People.Add(item);
return Context.SaveChanges() > 0;
}
public bool Update(Person item)
{
Context.People.Update(item);
return Context.SaveChanges() > 0;
}
public bool Delete(Person item)
{
item.DeletedFlag = true;
Context.People.Update(item);
return Context.SaveChanges() > 0;
}
public Person Get(int id)
{
return Context.People.Where(p => p.Id == id).Include(l => l.AcessData).Include(g => g.Graduations).Include(pe => pe.PresentEntries).FirstOrDefault();
}
public Person GetByMemberId(int id)
{
return Context.People.Where(p => p.MemberId == id && !p.DeletedFlag).Include(g => g.Graduations).Include(pe => pe.PresentEntries).FirstOrDefault();
}
public Person Get()
{
return Context.People.Include(l => l.AcessData).Include(g => g.Graduations).Include(pe => pe.PresentEntries).FirstOrDefault();
}
public Person GetLast()
{
return HasEntries() ? Context.People.Where(p => !p.DeletedFlag).Include(g => g.Graduations).Include(pe => pe.PresentEntries).ToList().LastOrDefault() : new Person();
}
public List<Person> GetList(int id)
{
return HasEntries() ? Context.People.Include(g => g.Graduations).Include(pe => pe.PresentEntries).ToList() : new List<Person>();
}
public List<Person> GetList()
{
return HasEntries() ? Context.People.Include(g => g.Graduations).Include(pe => pe.PresentEntries).ToList() : new List<Person>();
}
public bool HasEntries()
{
return Context.People.Any();
}
}

Dann noch der ProviderBase:

Spoiler anzeigen

und der IGenericProvider:

Spoiler anzeigen

ich geb noch die Provider Basisklasse mit:

Spoiler anzeigen

In der ViewModelklasse ist in Zeile 45 der entsprechende Remove Eintrag

Nofear23m · 4. Februar 2020, 13:19

Hallo

Irgendwie vermisse ich die Methode "GetGradiationhistory".

MichaHo · 4. Februar 2020, 13:47

MichaHo schrieb:

#region Helper Methods
public ObservableCollection<GraduationHistoryListItemViewModel> GetGraduationHistory()
{
var graduationHistoryList = _personModel.Graduations.Where(g => !g.DeletedFlag).ToList();
GraduationHistory = new ObservableCollection<GraduationHistoryListItemViewModel>();
graduationHistoryList.ForEach(g => GraduationHistory.Add(new GraduationHistoryListItemViewModel(g)));
return GraduationHistory;
}
#endregion

ist im ViewModel ganz unten.
Aufgerufen wird die Methode im Konstructor des ViewModel

Nofear23m · 4. Februar 2020, 14:20

Oh, sorry. Übersehen. Ohne F12 Funktion etwas mühsam, und die C# Syntax macht mir Kopfweh

OK, also so wie ich das sehe rufst du per Default ja ohne Tracking ab, was ja auch gut ist.

VB.NET-Quellcode

Context.ChangeTracker.AutoDetectChangesEnabled = false;

Aber dann bekommt der Context keine Änderungen mit. Du kannst in solch einem Fall mit .AsTracking() im Query dafür sorgen das die Entitäten getrackt werden.
Das könnte ein Grund sein.

Schau dir bitte beim ChangeTracker auch an welche Änderungen er mitbekommen hat. In meinem WPFNotes Projekt ist eine Methode im GenericRepository die dir das ausgibt.
Ansonsten melde dich bitte nochmals. Bei sowas komplexen wir einem Repository mit EF Core ist sowas schwer nachzuvollziehen.

Wenn ich am Abend daheim bin habe ich etwas mehr Zeit mir das genauer anzusehen, aber so finde ich jetzt ansonsten keinen Fehler auf die schnelle.

Grüße
Sascha

MichaHo · 4. Februar 2020, 15:23

Hi Sascha, Danke Dir für Deine Mühe...

Ich hab nun deine SetTracking eingebaut im PeopleProvider:

C#-Quellcode

public virtual IQueryable<Person> SetTracking(bool tracking, IQueryable<Person> query)
{
Debug.WriteLine($"SetTracking for Query of Type '{typeof(Person).ToString()}' to '{tracking.ToString()}' and Return modified Query");
if (tracking)
return query.AsTracking();
else
return query.AsNoTracking();
}

und dann in der Update Mezhode des Provider:

C#-Quellcode

public bool Update(Person item)
{
SetTracking(true, Context.People.Include(g => g.Graduations).Include(p => p.PresentEntries));
Context.People.Update(item);
return Context.SaveChanges() > 0;
}

Ich nutze es vermutlich falsch

SetTracking gibt mir ja ein IQueryable zurück, aber wie Update ich denn nu?

CascadeDelete SQLite unter .NetCore 3.1

CascadeDelete SQLite unter .NetCore 3.1

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

SQL-Abfrage

MichaHo schrieb:

C#-Quellcode

MichaHo schrieb:

SQL-Abfrage

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

C#-Quellcode

MichaHo schrieb:

VB.NET-Quellcode

C#-Quellcode

C#-Quellcode

Ähnliche Themen

2 Benutzer haben hier geschrieben