EF Core BusinessLogic Ohne Repository

  • VB.NET

Es gibt 7 Antworten in diesem Thema. Der letzte Beitrag () ist von Nofear23m.

    EF Core BusinessLogic Ohne Repository

    Hallo zusammen,

    ich versuche mich noch immer an EF Core in meiner (MVVM) Anwendung. Da ein GernericRepository noch mein Verständnis übersteigt dachte ich mir ich kann den Layer erstmal weglassen.
    Also erstmal einen BusinessLogic Layer hinzugefügt. Das sollte doch auch gehen. Allerdings versteh ich das ganze zusammenspiel irgendwie nicht.

    Wäre jemand so nett und würde mir erklären wie genau das zusammenspiel funktioniert? Also was alles würde in meinem Fall in die BL gehören und wo wird sie dann aufgerufen oder wo wird sie hinein gereicht? Im/Ins Viewmodel?
    :/
    Rechtschreibfehler betonen den künstlerischen Charakter des Autors.

    Hallo

    Ich dachte das haben wir bereits besprochen?

    Im Grunde ist ja die BL nur eine "Hilfe" um den Code vom ViewModel sauber zu halten bzw. um die dll von einer anderen App auch dann auch nutzen zu können. Also wenn du eine zweite App machst. Handy, Website oder einfach ein Admintool peralell zu deiner "normalen" App dann kannst du die dll einbinden und hast Zugriff auf die Logik und musst diese nicht neu schreiben. Denn im Admintool willst du ja sicher gänzlich andere Views haben und kannst sohin evtl. nicht das selbe ViewModel wie in deiner normalen App nutzen. Damit du nicht alles übertragen musst wandert einfach so viel als möglich in die BL.

    Was wäre das z.b.

    OK, gehen wir davon aus das deine Anwendung einen Login hat. So hast du sicher zumindest ein Command im Viewmodel. Ich nenne diesen Command nun mal DoLoginCommand.
    OK, dann wirdst du sicher noch eine Methode haben welche CheckLoginData benannt wäre.

    Gut. CheckLoginData könnte prüfen ob es den Benutzernamen gibt, ob das Konto auch nicht gesperrt ist, das Passwort nicht abgelaufen ist, das Passwort korrekt ist usw.
    Als rückgabe könnte es ja eine List(Of String) mit den aufgetretenen Fehlern zurückgeben. Ist die auflistung leer gab es keinen Fehler und der User kann eingeloggt werden.
    Anschliessen (sobald der User eingeloggt wird soll aber vieleicht ein Log in eine Datenbank geschrieben werden. Wenn das Passwort aber falsch war soll in der Datenbank der Zähler für "Failed Logins" um eines hochgezählt werden da nach 10 Versuchen das Konto gesperrt werden soll.

    Also haben wir mal geklärt was wir wollen.

    Ich gehe immer so vor das ich zuerst eine art pseudocode im ViewModel schreibe und mir dann einfach die Methoden automatisch von VS erstellen lasse.

    Also schreibe ich im ViewModel mal so was wie (jetzt rein aus der Hüfte):

    VB.NET-Quellcode

    1. Private ReadOnly _blLogin As BL.LoginBl (meine BL Instanz welche ich z.b. im Konstruktor erstelle)
    3. Private Function CheckLogin(user as String, password as String) As Boolean
    4. Dim loginErrors = _blLogin.CheckUserLogin(user,password)
    5. If loginErrors.Any() then
    6. 'Messagebox mit fehlern für User ausgeben
    7. End if
    8. Return Not loginErrors.Any()
    9. End Function
    11. Private Sub DoLoginCommand_Execute
    12. If CheckLogin() Then
    13. 'Login vollziehen. Fenster öffnen oder sonst was.
    14. End If
    15. End Sub


    Kurz und bündig sowie übersichtlich im ViewModel.

    Im Code der BL sieht das z.b. so aus (wieder extremer PsoidoCode):

    VB.NET-Quellcode

    1. Public Function CheckUserLogin(user as String, password as String) As List(Of String)
    2. Dim errorList as New List(Of String)
    4. Dim userInfo As Model.User = GetUserInfo(user).SingleOrDefault()
    6. If userInfo Is Nothing then loginErrors.Add("Unbekannter Benutzername") : WriteDbLog(loginErrors.First) : Return loginErrors
    7. If userInfo.AccountLocked Then loginErrors.Add("Benutzerkonto wurde gesperrt, verständigen Sie den Administrator.") : Return loginErrors
    9. If userInfo.PasswordRunnsOutAt <= DateTime.Now Then
    10. loginErrors.Add("Das Passwort ist abgelaufen, vor dem Login müssen Sie ein neues Passwort vergeben")
    11. End If
    14. If Not userInfo.Password = Helper.EncryptPassword(password) Then
    15. loginErrors.Add("Falsches Passwort")
    16. AddFailedLogins(userInfo.Id)
    17. End If
    19. Return loginErrors
    20. End Function
    22. Private Sub AddFailedLogins(userID as Integer)
    23. 'User von DB holen, Failed Logins beim User hochzählen und Wert in der DB zurückspeichern.
    24. End Sub


    Ich hoffe man versteht was ich meine.

    Grüße
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „Nofear23m“ ()

    Hallo @Akanel,
    ich stand auch vor dem gleichen Problem, komme mit Generic Repository auch nicht klar...
    Ich hab es dann in verschiedene Layer aufgeteilt. Bei mir gibt es den Context (der sollte klar sein, oder?), dann gibt es den DataProvider und die Logic.
    Am Beispeil von @Nofear23m nehme ich hier auch mal den Login.
    Ich hab erst mal eine DataProviderBase Klasse, die quasi den Context erstellt und hält:
    Spoiler anzeigen

    C#-Quellcode

    1. using System;
    2. using System.Linq;
    3. using TimeTracker.Data.Context;
    5. namespace TimeTracker.Data.Provider
    6. {
    7. public class DataProviderBase : IDisposable
    8. {
    9. #region Private Fields
    10. private bool _disposeContext;
    12. internal TimeTrackerContext Context { get; private set; }
    14. #endregion
    16. #region Constructor
    17. public DataProviderBase()
    18. {
    19. CreateContext(new TimeTrackerContext());
    20. _disposeContext = true;
    21. }
    23. internal DataProviderBase(TimeTrackerContext context)
    24. {
    25. CreateContext(context);
    26. _disposeContext = false;
    27. }
    28. #endregion
    30. #region Helper Methods
    31. private void CreateContext(TimeTrackerContext context)
    32. {
    33. Context = context;
    34. Context.ChangeTracker.LazyLoadingEnabled = false;
    35. Context.ChangeTracker.AutoDetectChangesEnabled = false;
    36. Context.Database.EnsureCreated();
    37. }
    39. public bool HasDataBaseEntries()
    40. {
    41. return Context.People.Any();
    42. }
    43. #endregion
    45. #region IDisposable
    46. public void Dispose()
    47. {
    48. if (_disposeContext) Context.Dispose();
    49. }
    50. #endregion
    52. }
    53. }


    Dann habe ich mir ein Interface erstellt, wecles einen Typ T engegen nimmt und die üblichen Metjoden bereit stellt, also Insert, Update, Delete usw.
    Sieht dann so aus:
    Spoiler anzeigen

    C#-Quellcode

    1. using System.Collections.Generic;
    3. namespace TimeTracker.Data.Provider
    4. {
    5. public interface IGenericProvider<T>
    6. {
    7. bool Insert(T item);
    8. bool Update(T item);
    9. bool Delete(T item);
    10. T Get(int id);
    11. List<T> GetList();
    12. }
    13. }

    Nun kann ich für jede Modelklasse einen Provider erstellen, zum Beispiel für den Log in einen PeopleProvider (weil die Klasse Person eine AccessData hat, die die Login Daten hält)

    der PeopleProvider sieht dann so aus:
    Spoiler anzeigen

    C#-Quellcode

    1. using Microsoft.EntityFrameworkCore;
    2. using System.Collections.Generic;
    3. using System.Linq;
    4. using TimeTracker.Model;
    6. namespace TimeTracker.Data.Provider
    7. {
    8. public class PeopleProvider : DataProviderBase, IGenericProvider<Person>
    9. {
    10. public bool Insert(Person item)
    11. {
    12. Context.People.Add(item);
    13. return Context.SaveChanges() > 0; ;
    14. }
    16. public bool Update(Person item)
    17. {
    18. Context.People.Update(item);
    19. return Context.SaveChanges() > 0;
    20. }
    21. public bool Delete(Person item)
    22. {
    23. item.DeletedFlag = true;
    24. Context.People.Update(item);
    25. return Context.SaveChanges() > 0;
    26. }
    28. public Person Get(int id)
    29. {
    30. return Context.People.Include(c => c.Company).Include(l => l.AcessData).Single(p => p.Id == id);
    31. }
    33. public Person Get()
    34. {
    35. return Context.People.Include(c => c.Company).Include(l => l.AcessData).FirstOrDefault(p => !p.DeletedFlag);
    36. }
    38. public List<Person> GetList()
    39. {
    40. return Context.People.ToList();
    41. }
    43. }
    44. }


    so, das sind die Vorarbeiten die ich einmal benötige.
    Im Logic Projekt hab ich nun eine Klasse für die Logic... kann man sicherlich auch in verschiedene Klassen aufteilen, zum Beispiel eine LoginLogic oder RegisterLogic oder so...

    In meiner Logic hab ich dann eine Methode um die ID des Users zurück zu geben wenn er eingeloggt werden kann, kann er nicht eingeloggt werden, wird -1 zurück gegeben was ich dann im ViewModel abfangen kann.
    Die Methode sieht dann so aus:

    C#-Quellcode

    1. /// <summary>
    2. /// gibt die ID der ersten Person aus der Datenbank zurück
    3. /// wenn keine Person in der Datenbank vorhanden ist
    4. /// wird -1 zurück gegeben
    5. /// </summary>
    6. /// <returns></returns>
    7. public static int GetLoggedInPersonId(string user, string pass)
    8. {
    9. using (var prov = new PeopleProvider())
    10. {
    11. var usr = prov.Get();
    12. if (usr.AcessData.UserName == user && usr.AcessData.EncryptedPassword == pass.GetHash())
    13. return usr.Id;
    14. return -1;
    15. }
    16. }


    So, nun zum ViewModel.
    Im Grunde ähnlich wie oben bei Sascha.
    Spoiler anzeigen

    C#-Quellcode

    1. using System.Threading.Tasks;
    2. using System.Windows.Input;
    3. using TimeTracker.Data.Logic;
    5. namespace TimeTracker.ViewModel
    6. {
    7. public class LoginViewModel : ViewModelBase
    8. {
    10. private string _userName;
    11. public string UserName { get => _userName; set => SetValue(ref _userName, value); }
    13. private bool _loginIsRunning;
    14. public bool LoginIsRunning { get => _loginIsRunning; set => SetValue(ref _loginIsRunning, value); }
    16. #region Commands
    18. public ICommand LogInCommand { get; private set; }
    20. public ICommand RegisterCommand { get; private set; }
    22. #endregion
    24. public LoginViewModel()
    25. {
    26. LogInCommand = new RelayCommand(async (parameter) => await LoginAsync(parameter));
    27. }
    29. #region Command Methods
    31. /// <summary>
    32. /// den User einloggen
    33. /// </ summary>
    34. /// <param name = "parameter"> der <see cref = "SecureString" />, der aus der View für das Benutzerkennwort übergeben wurde </ param>
    35. /// <returns> </ returns>
    36. public async Task LoginAsync(object parameter)
    37. {
    38. await RunCommandAsync(() => LoginIsRunning, async () =>
    39. {
    40. var user = UserName;
    41. var pass = (parameter as IHavePassword).SecurePassword.Unsecure();
    42. IoC.Application.CurrentPersonId = TimeTrackerLogic.GetLoggedInPersonId(user,pass);
    43. if (IoC.Application.CurrentPersonId > 0)
    44. {
    45. IoC.Application.GoToPage(ApplicationPageEnum.Main, new MainViewModel(IoC.Application.CurrentPersonId));
    46. }
    47. else
    48. {
    49. await IoC.UI.ShowMessage(new MessageBoxDialogViewModel
    50. {
    51. WindowTitle = "Falsche Zugangsdaten",
    52. Message = "Ihre Zugangsdaten waren nicht korrekt, bitte versuchen Sie es erneut",
    53. ButtonText = "Zurück"
    54. });
    55. }
    56. });
    57. }
    58. #endregion
    59. }
    60. }

    Ich arbeite mit dem nuget Paket ninject. Das stellt mir IoC bereit, womit ich halt Fenster öffnen kann, oder ein bestimmtes ViewModel statisch überall verfügbar habe. Das ist bei mir das ApplicationViewModel. Das AppliucationViewModel hält zum Beispiel die ID des angemeldeten Users. Das VM hat auch Boolean Properties um Menüs anzuzeigen oder auszublenden. Und das VM hat eine Methode GoToPage um die Pages in meiner WPF Anwendung hin und her zu schalten.

    In der LoginAsync Methode rufe ich einmal die Logic auf um an die ID zu kommen. Mehr benötigt es ja zu dem Zeitpunkt nicht.
    Alle anderen Daten rufe ich immer erst dann ab, wenn ich sie brauche, dann eben immer über die Logic.

    Mein Zeiterfassungstool ist noch nicht ganz fertig, paar Kleinigkeiten fehlen noch, vor allem Übersichten und Salden.
    Das Registrieren (wenn die Tabelle Person leer ist wird dies automatisch aufgerufen), das einloggen, das ändern des Kennwortes und der Personen Details, sowie das "Stechen" (Kommen, Gehen, Pause) und die Pausenberechnung funktioniert schon.
    Falls interesse besteht, kann ich es Dir schicken oder hier hoch laden.
    Ich nutze übrigens EFCore mit SQLite und alle Projekte sind .NetCore 3.1.
    "Hier könnte Ihre Werbung stehen..."

    MichaHo schrieb:

    komme mit Generic Repository auch nicht klar..

    Achso? Denn....

    MichaHo schrieb:

    Ich hab es dann in verschiedene Layer aufgeteilt. Bei mir gibt es den Context (der sollte klar sein, oder?), dann gibt es den DataProvider und die Logic.

    Genau das ist ein Generisches Repository und genau das selbe wie in meinem Notes Projekt. Deines ist auch generisch weil ja IGenericProvider<T>.

    Obs nun GenericRepository oder GenericPRovider benannt ist kann ja mala egal sein. Du abstrahierst die EF Methoden um dieses vor der Businesslogik zu verstecken um somit in der Businesslogik keinen Verweis auf EF Core haben zu müssen. Was ja auch der einzige Sinn eines Repositories ist. Obs nun generisch ist oder nicht.

    Im umkehrschluss ergibt dies aber @Akanel und @MichaHo das man auch nur diesen zusätzlichen Layer einführen sollte wenn dieser auch benötigt wird. Ist es ausgeschlossen das später auf eine andere Datenhaltung umgestellt wird oder die BL wo anders verwendet werden muss wo kein EF Core möglich ist (da mittlerweile Cross-Plattform fähig eher unwarscheinlich aber es könnte ja evtl. Konzernrichtlinien verletzen) würde ich den Layer auch schlicht weglassen.

    Grüße
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    Hallo zusammen
    Ich habe mal ein paar versuche unternommen und habe gleich wieder eine Ohrfeige bekommen.
    Ich habe mir mal eine MainWorkspaceBl angelegt, mit folgendem Inhalt.

    VB.NET-Quellcode

    1. Imports HStatistik.Model
    2. Imports StatsContext
    4. Public Class MainWorkspaceBl
    6. Private ReadOnly _ctx As StatsDbContext
    8. Public Sub New()
    9. _ctx = New StatsDbContext
    10. _ctx.SalesStaffs.Add(New SalesStaff With {.Name = "Max Mustermann"})
    11. _ctx.SaveChanges()
    12. End Sub
    14. End Class


    Aufgerufen wird diese im MainWorkspace

    VB.NET-Quellcode

    1. Imports HStatistik.BusinessLogic
    3. Namespace Workspaces
    4. Public Class MainWorkspace
    5. Inherits ViewModelBase
    7. Private _statusWs As StatusWorkspace
    8. Public Property StatusWs() As StatusWorkspace
    9. Get
    10. Return _statusWs
    11. End Get
    12. Set(ByVal value As StatusWorkspace)
    13. _statusWs = value
    14. RaisePropertyChanged()
    15. End Set
    16. End Property
    18. Private _planWs As PlanWorkspace
    19. Public Property PlanWs() As PlanWorkspace
    20. Get
    21. Return _planWs
    22. End Get
    23. Set(ByVal value As PlanWorkspace)
    24. _planWs = value
    25. RaisePropertyChanged()
    26. End Set
    27. End Property
    29. Private ReadOnly mainBl As MainWorkspaceBl
    31. Public Sub New()
    32. Me.mainBl = New MainWorkspaceBl
    33. Me.PlanWs = New PlanWorkspace
    34. Me.StatusWs = New StatusWorkspace
    35. End Sub
    37. End Class
    38. End Namespace


    Wenn ich versuche dies zu kompilieren bekomme ich folgenden Fehler


    Eigentlich hatte ich gedacht mein Model wäre so gewählt das ich einen SalesStaff (Vertriebler) anlegen kann ohne das diese einen Plan haben MÜSSEN.
    Dem scheint aber nicht so wenn ich die Fehlermeldung richtig verstehe.
    Oder liegt der fehler hier ganz wo anders?

    Ziel ist folgendes:
    Ich kann einen Vertriebler anlegen, oder auch mehrere. Dies müssen nicht zwingend einen Plan haben.
    Dann Kann ich einen Plan anlegen und dabei via Combobox einen Vertriebler auswählen.

    Hier noch das Model dazu.

    Spoiler anzeigen

    VB.NET-Quellcode

    1. Imports System.ComponentModel.DataAnnotations
    3. Public Class Plan
    5. <Key>
    6. Public Property Id As Integer
    8. <Required>
    9. Public Property PlanNumber As Integer
    11. <Required(AllowEmptyStrings:=False, ErrorMessage:="Das Feld 'Firma' muss ausgefüllt sein.")>
    12. <MinLength(3, ErrorMessage:="Das Feld 'Firma' muss mindestens 3 Zeichen enthalten.")>
    13. <MaxLength(30, ErrorMessage:="Das Feld 'Firma' darf maximal 30 Zeichen enthalten.")>
    14. Public Property Company As String
    15. Public Property SetOn As Date
    16. Public Property UploadOn As Date
    18. <MaxLength(250, ErrorMessage:="Das Feld 'Kommentar' darf maximal 250 Zeichen enthalten.")>
    19. Public Property Comment As String
    20. Public Property Finished As Boolean
    22. End Class

    VB.NET-Quellcode

    1. Imports System.ComponentModel.DataAnnotations
    3. Public Class SalesStaff
    5. <Key>
    6. Public Property Id As Integer
    8. <Required(AllowEmptyStrings:=False, ErrorMessage:="Das Feld 'Name' muss ausgefüllt sein.")>
    9. <MinLength(3, ErrorMessage:="Das Feld 'Name' muss mindestens 3 Zeichen enthalten.")>
    10. <MaxLength(30, ErrorMessage:="Das Feld 'Name' darf maximal 30 Zeichen enthalten.")>
    11. Public Property Name As String
    12. Public Property PlanId As Integer
    13. Public Property Plans As Plan
    15. End Class
    Rechtschreibfehler betonen den künstlerischen Charakter des Autors.

    Hallo

    Nur zum verständnis:

    Akanel schrieb:

    Ich kann einen Vertriebler anlegen, oder auch mehrere. Dies müssen nicht zwingend einen Plan haben.
    Dann Kann ich einen Plan anlegen und dabei via Combobox einen Vertriebler auswählen.

    Moment. Geht (jetzt das ohne ViewModel irgendwie im Hinterkopf zu haben, rein um die Daten) es jetzt darum das du per "Default" keinen Plan für einen Vertriebler hast oder soll er nicht unbedingt einen bekommen müssen.
    Das ist ein großer Unterschied. Oder andersrum. DARF es im Endeffekt in der Datenbank einen Vertriebler ohne Plan geben.
    Danke daran: Was das Model betrifft muss dies so Designed werden das du KEINE falschen Daten haben/bekommen kannst. Also unabhängig von einem View oder sonstwas muss es so Designed werden das du beim Programmieren was die datenhaltung betrifft im Grunde keine Fehler in der Struktur machen kannst. so zumindest der Idealfall.

    Akanel schrieb:

    Eigentlich hatte ich gedacht mein Model wäre so gewählt das ich einen SalesStaff (Vertriebler) anlegen kann ohne das diese einen Plan haben MÜSSEN.

    Wenn du entweder die PlanId als Eigenschaft rausnimmst oder diese als Nullable definierst funktioniert das auch. Es ist klar. Plans ist Nullable weil Plans ja nothing sein kann. Da es aber der dazugehörige Fremdschlüssel (PlanId) nicht sein kann schlägt dies fehl. Aber was nun genau gewollt ist habe ich oben ja in den Raumgestellt.

    PS: Bitte benenne die Property Plans in Plan um. Plans impliziert das es mehrere geben kann, das führt später in der Anwendung des Property sicher zur verwirrung.

    Grüße
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    Es geht darum das der Vertriebler per "Default" keinen Plan haben muss. Am ende darf es aber keinen Vertriebler ohne Plan geben.
    Vielleicht ist meine Denkweise auch falsch.
    Wenn ich einen Plan anlege muss ich diesen ja einem Vertriebler zuordnen können. Dies geht aber nur wenn der Vertriebler schon da ist. So meine Denkweise.

    Folgendes war der Plan.
    Ich habe eine UI mit 2 Listboxen. Links eine mit allen Vertrieblern und rechts eine mit den dazugehörigen Plänen zum jeweiligen Vertriebler.
    Wenn ich nun einen neuen Plan anlege, muss ich auswählen welcher Vertriebler den erstellt hat. Dieser Vertriebler kann schon x pläne erstellt haben, oder es ist sein erster.

    Kann man das irgendwie nachvollziehen?
    Rechtschreibfehler betonen den künstlerischen Charakter des Autors.

    Hallo

    Jaja, klar ist das nachvollziehbar und eben so wie ich es mir dachte.

    Akanel schrieb:

    Am ende darf es aber keinen Vertriebler ohne Plan geben.

    Siehste, wie ich dachte. Also wir das Model so erstellt das es einen Plan geben MUSS.

    Das ViewModel wird aber so gemacht das es keinen geben muss. Also das Property im ViewModel per Default Nothing ist.
    Aber... der Speichern Button darf eben erst gekickt werden wenn es einen Plan gibt.

    So kannst du dich selbst nicht "vertun" und unabsichtlich vieleicht mal irgendwo vergessen einen Plan für einen Vertriebler zu hinterlegen, weil dir die DB das schlicht nicht zulässt. Du hast dich also abgesichert.
    Merke, mit einer echten MVVM Struktur (im Gegensatz zu den Strukturen die ich oft sehe wo das Model Quasi als ViewModel verwendet wird weil ja so viel einfacher ;-)) kannst du dich auch vor deinen eigenen Fehler schützen und bist eben flexibler, wie ja in diesem Fall das dein ViewModel anders aufgebaut werden kann als es das Model vorgeben würde.

    Ich hoffe es wird nun etwas klarer.

    Grüße
    SAscha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

Facebook
Telefonspende