Renamed IFileManager to IDataAccess

extracted out IDataAccess to be more extensible in case need to swap it out for database access.
renamed all instance to be more appropriate.

Author: mstew20

src/HyperMC/HyperMcView.cs

@@ -26,16 +26,16 @@ public partial class HyperMcView : Form, IViewHost
     {
 
         private readonly IForgeClient _forgeClient;
-        private readonly IFileManager _fileManager;
+        private readonly IDataAccess _dataAccess;
         private readonly IUserSettings _settings;
         private readonly SettingView _settingView;
 
-        public HyperMcView(IForgeClient forgeClient, IFileManager fileManager, IUserSettings settings, SettingView settingView)
+        public HyperMcView(IForgeClient forgeClient, IDataAccess dataAccess, IUserSettings settings, SettingView settingView)
         {
             InitializeComponent();
 
             _forgeClient = forgeClient;
-            _fileManager = fileManager;
+            _dataAccess = dataAccess;
             _settings = settings;
             _settingView = settingView;
 
@@ -46,7 +46,7 @@ public HyperMcView(IForgeClient forgeClient, IFileManager fileManager, IUserSett
         private async void HyperMcView_Load(object sender, EventArgs e)
         {
             SetView(new ControlView(pnl_MainArea));
-            var mods = await _fileManager.ReadFile<ModpackData[]>(_settings.ModPacksFile);
+            var mods = await _dataAccess.LoadData<ModpackData[]>(_settings.ModPacksFile);
             if (mods != null)
             {
                 foreach (var mod in mods)
@@ -127,7 +127,7 @@ private async void ModpacksUpdated(object? sender, NotifyCollectionChangedEventA
             }
 
             SortModpacks();
-            await _fileManager.WriteToFile(_modpacks.ToArray(), _settings.ModPacksFile);
+            await _dataAccess.SaveData(_modpacks.ToArray(), _settings.ModPacksFile);
         }
 
         private ModpackBox CreateModpackBox(ModpackData data)

src/HyperMC/Program.cs

@@ -49,7 +49,7 @@ private static void ConfigureServices(IServiceCollection services)
                     .AddSingleton<IUserSettings, UserSettings>();
 
             services.AddTransient<SettingView>()
-                    .AddTransient<IFileManager, FileManager>();
+                    .AddTransient<IDataAccess, FileDataAccess>();
 
             services.AddForgeClient();
         }

src/HyperMC/Services/FileManager.cs renamed to src/HyperMC/Services/FileDataAccess.cs

@@ -8,9 +8,9 @@
 
 namespace Hypermc.Services
 {
-    public class FileManager : IFileManager
+    public class FileDataAccess : IDataAccess
     {
-        public async Task<T> ReadFile<T>(string file)
+        public async Task<T> LoadData<T>(string file)
         {
             if (File.Exists(file))
             {
@@ -22,10 +22,19 @@ public async Task<T> ReadFile<T>(string file)
             return default(T);
         }
 
-        public async Task WriteToFile<T>(T data, string file)
+        public async Task SaveData<T>(T data, string file)
         {
             string settingsToSave = JsonSerializer.Serialize(data);
             await File.WriteAllTextAsync(file, settingsToSave);
         }
+
+        public Task<List<T>> LoadData<T, U>(string query, U parameters, string connectionStringName)
+        {
+            return null;
+        }
+        public Task SaveData<T>(string query, T parameters, string connectionStringName)
+        {
+            return null;
+        }
     }
 }

src/HyperMC/Services/IDataAccess.cs

@@ -0,0 +1,16 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Hypermc.Services
+{
+    public interface IDataAccess
+    {
+        Task SaveData<T>(T data, string file);
+        Task<T> LoadData<T>(string file);
+        Task<List<T>> LoadData<T, U>(string query, U parameters, string connectionStringName);
+        Task SaveData<T>(string query, T parameters, string connectionStringName);
+    }
+}

src/HyperMC/Services/IFileManager.cs

@@ -15,19 +15,19 @@ public class UserSettings : IUserSettings
     {
         private readonly string _appPath;
         private readonly string _settingsFile;
-        private readonly IFileManager _fileManager;
+        private readonly IDataAccess _dataAccess;
 
         public string MinecraftPath { get; private set; }
         public string ModPacksPath { get; private set; }
         public string ModPacksFile { get; }
 
-        public UserSettings(IFileManager fileManager, IConfiguration config)
+        public UserSettings(IDataAccess dataAccess, IConfiguration config)
         {
             var settings = config.GetSection(nameof(ApplicationSettings)).Get<ApplicationSettings>();
             _appPath = string.Format(settings.AppPath, Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData));
             _settingsFile = string.Format(settings.SettingsFile, _appPath);
 
-            _fileManager = fileManager;
+            _dataAccess = dataAccess;
 
             MinecraftPath = string.Format(settings.DeafultMinecraftPath, Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData));
             ModPacksPath = string.Format(settings.DefaultModPacksPath, MinecraftPath);
@@ -46,7 +46,7 @@ public async Task Initialize()
                 Directory.CreateDirectory(ModPacksPath);
             }
 
-            var settings = await _fileManager.ReadFile<UserSettings>(_settingsFile);
+            var settings = await _dataAccess.LoadData<UserSettings>(_settingsFile);
 
             if (settings != null)
             {
@@ -60,7 +60,7 @@ public async Task UpdateSettings(string mcPath, string modPath)
             MinecraftPath = mcPath;
             ModPacksPath = modPath;
 
-            await _fileManager.WriteToFile(this, _settingsFile);
+            await _dataAccess.SaveData(this, _settingsFile);
         }
     }
 }