Code cleanup

SewerStammGen.DAL/Class1.cs

@@ -1,7 +0,0 @@
-namespace SewerStammGen.DAL
-{
-    public class Class1
-    {
-
-    }
-}

SewerStammGen.DAL/Services/AuftraggeberDataService.cs

@@ -9,33 +9,45 @@ using System.Threading.Tasks;
 
 namespace SewerStammGen.DAL.Services
 {
-    public class AuftraggeberDataService : IAuftraggeberDataService
+    public class AuftraggeberDataService : PostgresqlDataService, IAuftraggeberDataService 
     {
-        string connString = "Host = localhost; Database = SewerGen; Username = SewerGen; Password = SewerGen";
+        /*
-        NpgsqlDataSource dataSource;
+         * CREATE TABLE public."Auftraggebers"
-        NpgsqlConnection? conn = null;
+(
-        public AuftraggeberDataService()
+    "Id" serial,
-        {
+    "Name" text,
-            var dataSourceBuilder = new NpgsqlDataSourceBuilder(connString);
+    "Strasse" text,
-            dataSource = dataSourceBuilder.Build();
+    "Ort" text,
+    "Postleitzahl" text,
+    "Ansprechpartner" text,
+    "Telefonnummer" text,
+    PRIMARY KEY ("Id")
+);
 
-            //OpenConnection();
+ALTER TABLE IF EXISTS public."Auftraggebers"
-            conn = dataSource.OpenConnection();
+    OWNER to "SewerGen";
+        */
+        public AuftraggeberDataService() : base("public.\"Auftraggebers\"") { }
         
-        }
+        public async Task<Auftraggeber> Create(Auftraggeber entity)
-        public Task<Auftraggeber> Create(Auftraggeber entity)
         {
-            throw new NotImplementedException();
+            string command = "INSERT INTO public.\"Auftraggebers\" (\"Name\", \"Strasse\", \"Ort\", \"Postleitzahl\", \"Ansprechpartner\", \"Telefonnummer\") " +
-        }
+                "VALUES(@1,@2,@3,@4,@5,@6) RETURNING \"Id\"";
 
-        public Auftraggeber CreateNonAsync(Auftraggeber entity)
+            await using (var cmd = new NpgsqlCommand(command, conn))
-        {
+            {
-            throw new NotImplementedException();
+                cmd.Parameters.AddWithValue("1", entity.Name);
-        }
+                cmd.Parameters.AddWithValue("2", entity.Strasse);
+                cmd.Parameters.AddWithValue("3", entity.Ort);
+                cmd.Parameters.AddWithValue("4", entity.Postleitzahl);
+                cmd.Parameters.AddWithValue("5", entity.Ansprechpartner);
+                cmd.Parameters.AddWithValue("6", entity.Telefonnummer);
                 
-        public Task<bool> Delete(int id)
+                using var reader = await cmd.ExecuteReaderAsync();
-        {
+                reader.Read();
-            throw new NotImplementedException();
+                entity.Id = reader.GetInt32(0);
+            }
+            return entity;
         }
 
         public async Task<Auftraggeber> Get(int id)
@@ -48,9 +60,7 @@ namespace SewerStammGen.DAL.Services
                 reader.Read();
                 result = parseAuftraggeber(reader);
             }
-            
             return result;       
-                    
         }
 
         private Auftraggeber parseAuftraggeber(NpgsqlDataReader reader)
@@ -81,7 +91,7 @@ namespace SewerStammGen.DAL.Services
             return result;
         }
 
-        public Task<Auftraggeber> Update(int id, Auftraggeber entity)
+        public Task<Auftraggeber> Update(Auftraggeber entity)
         {
             throw new NotImplementedException();
         }

SewerStammGen.DAL/Services/PostgresqlDataService.cs

@@ -0,0 +1,49 @@
+using Npgsql;
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SewerStammGen.DAL.Services
+{
+    public class PostgresqlDataService : IDisposable
+    {
+        string connString = "Host = localhost; Database = SewerGen; Username = SewerGen; Password = SewerGen";
+        NpgsqlDataSource dataSource;
+        protected NpgsqlConnection? conn = null;
+        private string tableName = "";
+
+        public PostgresqlDataService(string tableName)
+        {
+            var dataSourceBuilder = new NpgsqlDataSourceBuilder(connString);
+            dataSource = dataSourceBuilder.Build();
+
+            conn = dataSource.OpenConnection();
+            this.tableName = tableName;
+
+        }
+
+
+        public virtual async Task<bool> Delete(int id)
+        {
+            string command = $"DELETE "+tableName+" WHERE \"Id\" = @1";
+            await using (var cmd = new NpgsqlCommand(command, conn))
+            {
+                cmd.Parameters.AddWithValue("1", id);
+                int res = await cmd.ExecuteNonQueryAsync();
+            }
+            return true;
+        }
+
+        public void Dispose()
+        {
+            if(conn != null && conn.State == System.Data.ConnectionState.Open)
+            {
+                conn.Close();
+                conn.Dispose();
+            }
+        }
+    }
+}

SewerStammGen.DAL/Services/ProjektDataService.cs

@@ -8,27 +8,15 @@ using System.Threading.Tasks;
 using Npgsql;
 using SewerStammGen.Shared.Domain;
 using SewerStammGen.Shared.Enum;
+using System.Diagnostics;
 
 namespace SewerStammGen.DAL.Services
 {
-    public class ProjektDataService : IProjektDataService
+    public class ProjektDataService : PostgresqlDataService, IProjektDataService
     {
-        string connString = "Host = localhost; Database = SewerGen; Username = SewerGen; Password = SewerGen";
+        public ProjektDataService() : base("public.\"Projekte\"")
-        NpgsqlDataSource dataSource;
-        NpgsqlConnection? conn = null;
-        public ProjektDataService()
         {
-            var dataSourceBuilder = new NpgsqlDataSourceBuilder(connString);
-            dataSource = dataSourceBuilder.Build();
             
-            //OpenConnection();
-            conn = dataSource.OpenConnection();
-            
-        }
-
-        private async void OpenConnection()
-        {
-            conn = await dataSource.OpenConnectionAsync();
         }
 
         public async Task<Projekt> Create(Projekt entity)
@@ -52,15 +40,6 @@ namespace SewerStammGen.DAL.Services
             return entity;
         }
 
-        public Projekt CreateNonAsync(Projekt entity)
-        {
-            throw new NotImplementedException();
-        }
-
-        public Task<bool> Delete(int id)
-        {
-            throw new NotImplementedException();
-        }
 
         public async Task<Projekt> Get(int id)
         {
@@ -77,17 +56,17 @@ namespace SewerStammGen.DAL.Services
             return result;
         }
 
-        private Projekt? parseProjekt(NpgsqlDataReader reader)
+        private Projekt parseProjekt(NpgsqlDataReader reader)
         {
             return new Projekt()
             {
                 Id = reader.GetInt32(0),
-                Projektname = reader.GetString(1),
+                Projektname = reader.IsDBNull(1) ? "" : reader.GetString(1),
-                Erstelldatum = reader.GetString(2),
+                Erstelldatum = reader.IsDBNull(2) ? "" : reader.GetString(2),
-                Strasse = reader.GetString(3),
+                Strasse = reader.IsDBNull(3) ? "" : reader.GetString(3),
-                Ort = reader.GetString(4),
+                Ort = reader.IsDBNull(4) ? "" : reader.GetString(4),
-                ExportType = (EExportType)reader.GetInt32(5),
+                ExportType = reader.IsDBNull(5) ? EExportType.XML2013 : (EExportType)reader.GetInt32(5),
-                Kodierungssystem = (EKodierungssystem)reader.GetInt32(6)
+                Kodierungssystem = reader.IsDBNull(6) ? EKodierungssystem.EN13508_2_2011_DWA_M_192_2 : (EKodierungssystem)reader.GetInt32(6)
             };
         }
 
@@ -111,9 +90,45 @@ namespace SewerStammGen.DAL.Services
             return result;
         }
 
-        public Task<Projekt> Update(int id, Projekt entity)
+        public async Task<Projekt> Update(Projekt entity)
         {
-            throw new NotImplementedException();
+            string command = "UPDATE public.\"Projekte\" SET \"Projektname\"=@1, \"Erstelldatum\"=@2, \"Strasse\"=@3, \"Ort\"=@4, \"ExportType\"=@5, \"Kodierungssystem\"=@6 WHERE \"Id\" = @8";
+            using(var cmd = new NpgsqlCommand(command, conn))
+            {
+                cmd.Parameters.AddWithValue("1", entity.Projektname);
+                cmd.Parameters.AddWithValue("2", entity.Erstelldatum);
+                cmd.Parameters.AddWithValue("3", entity.Strasse);
+                cmd.Parameters.AddWithValue("4", entity.Ort);
+                cmd.Parameters.AddWithValue("5", (int)entity.ExportType);
+                cmd.Parameters.AddWithValue("6", (int)entity.Kodierungssystem);
+                cmd.Parameters.AddWithValue("8", entity.Id);
+                int res = await cmd.ExecuteNonQueryAsync();
+                Trace.WriteLine(res);
+            }
+            return entity;
         }
     }
 }
+
+/*
+ * CREATE TABLE public."Projekte"
+(
+    "Id" serial,
+    "Projektname" text,
+    "Erstelldatum" text,
+    "Strasse" text,
+    "Ort" text,
+    "ExportType" bigint,
+    "Kodierungssystem" bigint,
+    "AuftraggeberId" bigint,
+    PRIMARY KEY ("Id"),
+    CONSTRAINT "Auftraggeber" FOREIGN KEY ("AuftraggeberId")
+        REFERENCES public."Auftraggebers" ("Id") MATCH SIMPLE
+        ON UPDATE NO ACTION
+        ON DELETE NO ACTION
+        NOT VALID
+);
+
+ALTER TABLE IF EXISTS public."Projekte"
+    OWNER to "SewerGen";
+*/

SewerStammGen/Commands/HaltungEditSaveCommand.cs

@@ -15,23 +15,23 @@ namespace SewerStammGen.WPF.Commands
     {
         private HaltungEditViewModel haltungEditViewModel;
         private IDataService<Kanal> kanalDataService;
-        private ISchachtService schachtService;
         
-        public HaltungEditSaveCommand(HaltungEditViewModel haltungEditViewModel,  ISchachtService schachtService)
+
+        public HaltungEditSaveCommand(HaltungEditViewModel haltungEditViewModel)
         {
             this.haltungEditViewModel = haltungEditViewModel;
             //this.kanalDataService = kanalDataService;
-            this.schachtService = schachtService;
+            
         }
 
         public override async Task ExecuteAsync(object? parameter)
         {
             if(haltungEditViewModel._oberePunkt != haltungEditViewModel.Model.StartSchacht.Objektbezeichnung)
             {
-                Schacht s = await schachtService.FindSchachtByNameAndProjektID(haltungEditViewModel._oberePunkt, haltungEditViewModel.Model.Projekt.Id);
+                //Schacht s = await schachtService.FindSchachtByNameAndProjektID(haltungEditViewModel._oberePunkt, haltungEditViewModel.Model.Projekt.Id);
-                haltungEditViewModel.Model.StartSchacht = s;
+                //haltungEditViewModel.Model.StartSchacht = s;
             }
-            haltungEditViewModel.Model = await kanalDataService.Update(haltungEditViewModel.Model.Id, haltungEditViewModel.Model);
+            //haltungEditViewModel.Model = await kanalDataService.Update(haltungEditViewModel.Model.Id, haltungEditViewModel.Model);
             Trace.WriteLine("Daten gespeichert");
         }
     }

SewerStammGen/Commands/SchachtAddCommand.cs

@@ -16,21 +16,20 @@ namespace SewerStammGen.WPF.Commands
         private readonly IActualState actualState;
         private readonly IRenavigator renavigator;
         private readonly IDataService<Projekt> projektService;
-        private readonly ISchachtService schachtService;
         
 
-        public SchachtAddCommand(IDataService<Projekt> projektService, IActualState actualState, IRenavigator renavigator, ISchachtService schachtService)
+        public SchachtAddCommand(IDataService<Projekt> projektService, IActualState actualState, IRenavigator renavigator)
         {
             this.actualState = actualState;
             this.renavigator = renavigator;
             this.projektService = projektService;
-            this.schachtService = schachtService;
+            
         }
 
         public override async Task ExecuteAsync(object? parameter)
         {
             Projekt aktuelleProjekt = await projektService.Get(actualState.ProjektID);
-            Schacht schacht = await schachtService.CreateSchacht(aktuelleProjekt);
+           // Schacht schacht = await schachtService.CreateSchacht(aktuelleProjekt);
 
         }
     }

SewerStammGen/HostBuilders/AddServicesHostBuilderExtensions.cs

@@ -2,7 +2,6 @@
 using Microsoft.Extensions.Hosting;
 using SewerStammGen.DAL.Services;
 using SewerStammGen.Shared.Contracts;
-using SewerStammGen.Shared.Services;
 using SewerStammGen.WPF.Interface.Navigator;
 using SewerStammGen.WPF.ViewModel;
 using SewerStammGen.WPF.ViewModel.State.Navigation;
@@ -28,7 +27,7 @@ namespace SewerStammGen.HostBuilders
                 services.AddSingleton<IProjektDataService, ProjektDataService>();
                 services.AddSingleton<IAuftraggeberDataService, AuftraggeberDataService>();
                
-                services.AddSingleton<ISchachtService, SchachtService>();
+                
             });
             return host;
         }

SewerStammGen/HostBuilders/AddViewModelsHostBuilderExtensions.cs

@@ -48,8 +48,8 @@ namespace SewerStammGen.HostBuilders
                         services.GetRequiredService<ISchachtDataService>(),
                         services.GetRequiredService<IDataService<Projekt>>(),
                         services.GetRequiredService<ViewModelDelegateRenavigator<ManholeEditViewModel>>(),
-                        services.GetRequiredService<IActualState>(),
+                        services.GetRequiredService<IActualState>()
-                        services.GetRequiredService<ISchachtService>()
+                        
                         );
                 });
 
@@ -76,7 +76,7 @@ namespace SewerStammGen.HostBuilders
                 {
                     return () => new HaltungEditViewModel(
                         
-                        services.GetRequiredService<ISchachtService>(),
+                        
                         services.GetRequiredService<IActualState>()
                         );
                 });

SewerStammGen/ViewModel/Haltung/HaltungEditViewModel.cs

@@ -18,7 +18,7 @@ namespace SewerStammGen.WPF.ViewModel
     {
         private readonly IActualState _actualState;
         //private readonly IHaltungDataService _kanalDataService;
-        private readonly ISchachtService _schachtService;
+        
         private Kanal _model;
 
         public Kanal Model
@@ -113,14 +113,14 @@ namespace SewerStammGen.WPF.ViewModel
         public ICommand Speichern { get; set; }
 
         public HaltungEditViewModel(
-            ISchachtService schachtService,
+            
             IActualState actualState)
         {
             _actualState = actualState;
             //_kanalDataService = kanalDataService;
-            _schachtService = schachtService;
+            
             _model = new Kanal();
-            Speichern = new HaltungEditSaveCommand(this, schachtService);
+            Speichern = new HaltungEditSaveCommand(this);
 
             LoadModel();
         

SewerStammGen/ViewModel/Projekt/ProjektEditViewModel.cs

@@ -109,7 +109,7 @@ namespace SewerStammGen.WPF.ViewModel
             }
             else
             {
-                _dataService.Update(_model.Id, _model);
+                _dataService.Update(_model);
             }
             _renavigator.Renavigate();
         }

SewerStammGen/ViewModel/Projekt/ProjektListViewModel.cs

@@ -61,7 +61,9 @@ namespace SewerStammGen.WPF.ViewModel
         private async void LoadProjekte()
         {
             var projects = await genericDataService.GetAll();
+
             InitCollection(_projekte, projects);
+            
         }
 
         private void InitCollection(ObservableCollection<Projekt> projekte, IEnumerable<Projekt> projects)

SewerStammGen/ViewModel/Schacht/ManholeEditViewModel.cs

@@ -109,7 +109,7 @@ namespace SewerStammGen.WPF.ViewModel
 
         private void SaveSchacht()
         {
-            _schachtDataService.Update(_model.Id, _model);
+            //_schachtDataService.Update(_model.Id, _model);
         }
 
         private async void LoadModel()

SewerStammGen/ViewModel/Schacht/ManholeListViewModel.cs

@@ -19,7 +19,7 @@ namespace SewerStammGen.WPF.ViewModel
         private ISchachtDataService _schachtDataService;
         private readonly ObservableCollection<Schacht> _schaechte;
         private readonly IActualState _actualState;
-        private readonly ISchachtService _schachtService;
+        
         public ObservableCollection<Schacht> Schaechte { get => _schaechte; }
 
         public Schacht? SelectedSchacht { get; set; }
@@ -29,15 +29,15 @@ namespace SewerStammGen.WPF.ViewModel
         public ICommand DeleteSchachtCommand { get; set; }
 
 
-        public ManholeListViewModel(ISchachtDataService schachtDataService, IDataService<Projekt> projektService,IRenavigator renavigator ,IActualState actualState, ISchachtService schachtService)
+        public ManholeListViewModel(ISchachtDataService schachtDataService, IDataService<Projekt> projektService,IRenavigator renavigator ,IActualState actualState)
         {
             _schachtDataService = schachtDataService;
             _actualState = actualState;
 
             _schaechte = new ObservableCollection<Schacht>();
-            _schachtService = schachtService;
             
-            AddSchachtCommand = new SchachtAddCommand(projektService, actualState,renavigator,schachtService);
+
+            AddSchachtCommand = new SchachtAddCommand(projektService, actualState,renavigator);
             EditSchachtCommand = new SchachtEditCommand(schachtDataService, actualState, renavigator,this);
             DeleteSchachtCommand = new SchachtDeleteCommand(schachtDataService, actualState, renavigator, this);
 
@@ -46,8 +46,8 @@ namespace SewerStammGen.WPF.ViewModel
 
         private async void LoadSchaechte()
         {
-            var schaechte = await _schachtDataService.GetAll(_actualState.ProjektID);
+            //var schaechte = await _schachtDataService.GetAll(_actualState.ProjektID);
-            InitCollection(_schaechte, schaechte);
+            //InitCollection(_schaechte, schaechte);
         }
 
         private void InitCollection(ObservableCollection<Schacht> dest, IEnumerable<Schacht> source)

Shared/Contracts/IDataService.cs

@@ -6,13 +6,13 @@ using System.Threading.Tasks;
 
 namespace Shared.Contracts
 {
-    public interface IDataService<T>
+    public interface IDataService<T>: IDisposable
     {
         Task<IEnumerable<T>> GetAll();
         Task<T> Get(int id);
         Task<T> Create(T entity);
-        Task<T> Update(int id,T entity);
+        Task<T> Update(T entity);
         Task<bool> Delete(int id);
-        T CreateNonAsync(T entity);
+        //T CreateNonAsync(T entity);
     }
 }

Shared/Contracts/ISchachtDataService.cs

@@ -11,7 +11,6 @@ namespace SewerStammGen.Shared.Contracts
 {
     public interface ISchachtDataService : IDataService<Schacht>
     {
-        Task<IEnumerable<Schacht>> GetAll(int projektID);
+        
-        Task<Schacht> GetSchachtByNameAndProjekt(string name, int projektID);
     }
 }

Shared/Contracts/ISchachtService.cs

@@ -1,16 +0,0 @@
-
-using SewerStammGen.Shared.Domain;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
-namespace SewerStammGen.Shared.Contracts
-{
-    public interface ISchachtService
-    {
-        Task<Schacht> CreateSchacht(Projekt proj);
-        Task<Schacht> FindSchachtByNameAndProjektID(string name, int projektID);
-    }
-}

Shared/Domain/Auftraggeber.cs

@@ -8,11 +8,11 @@ namespace SewerStammGen.Shared.Domain
 {
     public class Auftraggeber : DBObject
     {
-        public string? Name { get; set; }
+        public string Name { get; set; } = String.Empty;
-        public string? Strasse { get; set; } 
+        public string Strasse { get; set; } = String.Empty;
-        public string? Ort { get; set; }
+        public string Ort { get; set; } = String.Empty;
-        public string? Postleitzahl { get; set; }
+        public string Postleitzahl { get; set; } = String.Empty;
-        public string? Ansprechpartner { get; set; }
+        public string Ansprechpartner { get; set; } = String.Empty;
-        public string? Telefonnummer { get; set; }
+        public string Telefonnummer { get; set; } = String.Empty;
     }
 }

Shared/Domain/Kanal.cs

@@ -8,13 +8,13 @@ namespace SewerStammGen.Shared.Domain
 {
     public class Kanal : DBObject
     {
-        public string? Objektbezeichnung { get; set; }
+        public string Objektbezeichnung { get; set; } = String.Empty;
         public Schacht? StartSchacht { get; set; }
         public Schacht? EndSchacht { get; set; }
         public int DN { get; set; }
-        public string? Material { get; set; }
+        public string Material { get; set; } = String.Empty;
         public decimal Haltungslaenge { get; set; }
-        public virtual Projekt? Projekt { get; set; }
+        public Projekt? Projekt { get; set; }
         public EEntwaeserung Entwaesserung { get; set; }
 
     }

Shared/Domain/Projekt.cs

@@ -10,10 +10,10 @@ namespace SewerStammGen.Shared.Domain
 {
     public class Projekt : DBObject
     {
-        public string? Projektname { get; set; }
+        public string Projektname { get; set; } = String.Empty;
-        public string? Erstelldatum { get; set; }
+        public string Erstelldatum { get; set; } = String.Empty;
-        public string? Strasse { get;set; }
+        public string Strasse { get; set; } = String.Empty;
-        public string? Ort { get; set; }
+        public string Ort { get; set; } = String.Empty;
         public Auftraggeber Auftraggeber { get; set; }
         public EExportType ExportType { get; set; }
         public EKodierungssystem Kodierungssystem { get; set; }

Shared/Domain/Schacht.cs

@@ -9,21 +9,12 @@ namespace SewerStammGen.Shared.Domain
 {
     public class Schacht : DBObject
     {
-        public string? Objektbezeichnung { get; set; }
+        public string Objektbezeichnung { get; set; } = String.Empty;
-
-        [Column(TypeName = "decimal(18,4)")]
         public decimal RechtsWert { get; set; }
-
-        [Column(TypeName = "decimal(18,4)")]
         public decimal HochWert { get; set; }
-
-        [Column(TypeName = "decimal(18,4)")]
         public decimal SohlHoehe { get; set; }
-
-        [Column(TypeName = "decimal(18,4)")]
         public decimal DeckelHoehe { get; set; }
-
+        public Projekt? Projekt { get; set; }
-        public virtual Projekt? Projekt { get; set; }
         public EEntwaeserung Entwaesserung { get; set; }
     }
 

Shared/Services/SchachtService.cs

@@ -1,44 +0,0 @@
-using SewerStammGen.Shared.Contracts;
-using SewerStammGen.Shared.Domain;
-using Shared.Contracts;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
-namespace SewerStammGen.Shared.Services
-{
-    public class SchachtService : ISchachtService
-    {
-        private readonly IDataService<Projekt> _projectService;
-        private readonly ISchachtDataService _schachtDataService;
-
-        public SchachtService(
-            IDataService<Projekt> projectService,
-            ISchachtDataService schachtDataService
-            )
-        {
-            _projectService = projectService;
-            _schachtDataService = schachtDataService;
-        }
-
-        public async Task<Schacht> CreateSchacht(Projekt proj)
-        {
-            Schacht schacht = new Schacht() { 
-                Projekt = proj
-            };
-            proj.Schaechte.Add(schacht);
-            await _projectService.Update(proj.Id, proj);
-            return schacht;
-        }
-
-        public async Task<Schacht> FindSchachtByNameAndProjektID(string name, int projektID)
-        {
-            Schacht result = await _schachtDataService.GetSchachtByNameAndProjekt(name, projektID);
-            return result;
-            
-            
-        }
-    }
-}

Shared/SewerStammGen.Shared.csproj

@@ -6,4 +6,8 @@
     <Nullable>enable</Nullable>
   </PropertyGroup>
 
+  <ItemGroup>
+    <Folder Include="Services\" />
+  </ItemGroup>
+
 </Project>