Refactor StundePage to MVVM with DI

Jugenddienst Stunden/Interfaces/IAlertService.cs

@@ -5,7 +5,7 @@ using System.Text;
 using System.Threading.Tasks;
 
 namespace Jugenddienst_Stunden.Interfaces; 
-internal interface IAlertService {
+public interface IAlertService {
 	event EventHandler<string> AlertRaised;
 	void Raise(string message);
 }

Jugenddienst Stunden/MauiProgram.cs

@@ -114,6 +114,8 @@ public static class MauiProgram {
 		// DI: Views/ViewModels
 		builder.Services.AddTransient<ViewModels.StundenViewModel>();
 		builder.Services.AddTransient<Views.StundenPage>();
+		builder.Services.AddTransient<ViewModels.StundeViewModel>();
+		builder.Services.AddTransient<Views.StundePage>();
 		builder.Services.AddTransient<ViewModels.LoginViewModel>();
 		builder.Services.AddTransient<Views.LoginPage>();
 

Jugenddienst Stunden/ViewModels/StundeViewModel.cs

@@ -77,33 +77,16 @@ public partial class StundeViewModel : ObservableObject, IQueryAttributable {
 	//public ICommand LoadDataCommand { get; private set; }
 
 
-	public StundeViewModel() : this(GetServiceOrCreate()) {
+	public StundeViewModel(IHoursService hoursService, IAlertService alertService) {
-		LoadSettingsAsync();
-	}
-
-	private static IHoursService GetServiceOrCreate() {
-		// Fallback-Konstruktion, falls DI nicht injiziert wurde (z. B. im Designer)
-		var http = new HttpClient();
-		var options = new Infrastructure.ApiOptions { BaseUrl = GlobalVar.ApiUrl, Timeout = TimeSpan.FromSeconds(15) };
-		var tokenProvider = new GlobalVarTokenProvider();
-		var api = new ApiClient(http, options, tokenProvider, new PreferencesAppSettings());
-		var repo = new HoursRepository(api);
-		var validator = new HoursValidator();
-		return new HoursService(repo, validator);
-	}
-
-	internal StundeViewModel(IHoursService hoursService) {
 		_hoursService = hoursService;
 		SaveCommand = new AsyncRelayCommand(Save);
-		//DeleteCommand = new AsyncRelayCommand(Delete);
 		DeleteConfirmCommand = new Command(async () => await DeleteConfirm());
-	}
 
-	// DI-Konstruktor, der den globalen Alert-Service abonniert und Alerts an das ViewModel weiterreicht.
-	internal StundeViewModel(IHoursService hoursService, IAlertService alertService) : this(hoursService) {
 		if (alertService is not null) {
 			alertService.AlertRaised += (s, msg) => AlertEvent?.Invoke(this, msg);
 		}
+
+		LoadSettingsAsync();
 	}
 
 	private async void LoadSettingsAsync() {
@@ -125,7 +108,7 @@ public partial class StundeViewModel : ObservableObject, IQueryAttributable {
 	async Task Save() {
 		bool exceptionOccurred = false;
 		bool proceed = true;
-		
+
 		//Arbeitszeit sollte nicht null sein
 		if (DayTime.TimeSpanVon == DayTime.TimeSpanBis && DayTime.FreistellungAktiv.Name == null) {
 			proceed = false;
@@ -203,45 +186,37 @@ public partial class StundeViewModel : ObservableObject, IQueryAttributable {
 			//DateTime heute = DateTime.Now;
 			try {
 				var entry = await _hoursService.GetEntryAsync(Convert.ToInt32(query["load"]));
-				// var settings = await _hoursService.GetSettingsAsync();
-				// GlobalVar.Settings = settings;
-				// GemeindeAktivSet = settings.GemeindeAktivSet;
-				// ProjektAktivSet = settings.ProjektAktivSet;
 
 				DayTime = entry;
 				DayTime.TimeSpanVon = entry.Begin.ToTimeSpan();
 				DayTime.TimeSpanBis = entry.End.ToTimeSpan();
 
-				// OptionsGemeinde = settings.Gemeinden ?? new List<Gemeinde>();
-				// OptionsProjekt = settings.Projekte ?? new List<Projekt>();
-				// OptionsFreistellung = settings.Freistellungen ?? new List<Freistellung>();
-
 				DayTime.GemeindeAktiv = OptionsGemeinde.FirstOrDefault(Gemeinde => Gemeinde.Id == DayTime.Gemeinde) ??
-				                        new Gemeinde();
+										new Gemeinde();
 				DayTime.ProjektAktiv = OptionsProjekt.FirstOrDefault(Projekt => Projekt.Id == DayTime.Projekt) ??
-				                       new Projekt();
+									   new Projekt();
 				DayTime.FreistellungAktiv =
 					OptionsFreistellung.FirstOrDefault(Freistellung => Freistellung.Id == DayTime.Free) ??
 					new Freistellung();
 
-				//Evtl. noch die anderen Zeiten des gleichen Tages holen
-				var day = await _hoursService.GetDayWithSettingsAsync(DayTime.Day);
-				DayTimes = day.dayTimes;
 				OnPropertyChanged(nameof(DayTime));
-				OnPropertyChanged(nameof(DayTimes));
+				if (System.String.IsNullOrEmpty(DayTime.Description)) {
+					InfoEvent?.Invoke(this, "Eintrag hat keinen Beschreibungstext");
+				}
+
+				SubTitle = DayTime.Day.ToString("dddd, d. MMMM yyyy");
+				OnPropertyChanged(nameof(SubTitle));
+
+				FreistellungEnabled = !DayTime.Approved;
 			} catch (Exception e) {
 				AlertEvent?.Invoke(this, e.Message);
 			} finally {
+				//Evtl. noch die anderen Zeiten des gleichen Tages holen
+				var day = await _hoursService.GetDayWithSettingsAsync(DayTime.Day);
+				DayTimes = day.dayTimes;
+				OnPropertyChanged(nameof(DayTimes));
 			}
 
-			if (System.String.IsNullOrEmpty(DayTime.Description)) {
-				InfoEvent?.Invoke(this, "Eintrag hat keinen Beschreibungstext");
-			}
-
-			SubTitle = DayTime.Day.ToString("dddd, d. MMMM yyyy");
-			OnPropertyChanged(nameof(SubTitle));
-
-			FreistellungEnabled = !DayTime.Approved;
 			//OnPropertyChanged(nameof(DayTime));
 		} else if (query.ContainsKey("date")) {
 			Title = "Neuer Eintrag";

Jugenddienst Stunden/Views/StundePage.xaml

@@ -8,10 +8,6 @@
              x:Class="Jugenddienst_Stunden.Views.StundePage"
              Title="{Binding Title}">
 
-    <ContentPage.BindingContext>
-        <models:StundeViewModel />
-    </ContentPage.BindingContext>
-
     <ContentPage.Resources>
         <ResourceDictionary>
             <conv:IntBoolConverter x:Key="IntBoolConverter" />

Jugenddienst Stunden/Views/StundePage.xaml.cs

@@ -13,14 +13,13 @@ public partial class StundePage : ContentPage {
 	/// <summary>
 	/// CTOR
 	/// </summary>
-	public StundePage() {
+	public StundePage(StundeViewModel vm) {
 		InitializeComponent();
 
-		if (BindingContext is StundeViewModel vm) {
+		BindingContext = vm;
-			vm.AlertEvent += Vm_AlertEvent;
+		vm.AlertEvent += Vm_AlertEvent;
-			vm.InfoEvent += Vm_InfoEvent;
+		vm.InfoEvent += Vm_InfoEvent;
-			vm.ConfirmEvent += ShowConfirm;
+		vm.ConfirmEvent += ShowConfirm;
-		}
 	}
 
 	private void Vm_AlertEvent(object? sender, string e) {