* Code improvements

* Logging improvements

Author: CodeDead

MemPlus/Classes/GUI/StyleManager.cs

@@ -3,7 +3,7 @@
 using System.Windows.Media;
 using Syncfusion.Windows.Shared;
 
-namespace MemPlus.Classes
+namespace MemPlus.Classes.GUI
 {
     /// <summary>
     /// Static class to change the style of an object

MemPlus/Classes/RAM/RamController.cs

@@ -9,10 +9,11 @@
 
 namespace MemPlus.Classes.RAM
 {
+    /// <inheritdoc />
     /// <summary>
     /// Sealed class containing methods and interaction logic in terms of RAM
     /// </summary>
-    internal sealed class RamController
+    internal sealed class RamController : IDisposable
     {
         #region Variables
         /// <summary>
@@ -102,6 +103,11 @@ internal RamController(Dispatcher dispatcher, SfCircularGauge gauge, Label lblTo
             _logController.AddLog(new ApplicationLog("Done initializing RamController"));
         }
 
+        ~RamController()
+        {
+            _ramTimer?.Dispose();
+        }
+
         /// <summary>
         /// Enable RAM usage monitoring
         /// </summary>
@@ -200,5 +206,10 @@ private void UpdateRamUsage()
 
             _logController.AddLog(new ApplicationLog("Finished updating RAM usage"));
         }
+
+        public void Dispose()
+        {
+            _ramTimer?.Dispose();
+        }
     }
 }

MemPlus/Classes/RAM/RamOptimizer.cs

@@ -89,7 +89,7 @@ internal sealed class RamOptimizer
         /// <param name="lpName">A pointer to a null-terminated string that specifies the name of the privilege, as defined in the Winnt.h header file. For example, this parameter could specify the constant, SE_SECURITY_NAME, or its corresponding string, "SeSecurityPrivilege"</param>
         /// <param name="pluid">A pointer to a variable that receives the LUID by which the privilege is known on the system specified by the lpSystemName parameter.</param>
         /// <returns>If the function succeeds, the function returns nonzero. Otherwise, return value will be zero</returns>
-        [DllImport("advapi32.dll", SetLastError = true)]
+        [DllImport("advapi32.dll", SetLastError = true, CharSet = CharSet.Unicode)]
         private static extern bool LookupPrivilegeValue(string lpSystemName, string lpName, ref long pluid);
         /// <summary>
         /// Enables or disables privileges in the specified access token. Enabling or disabling privileges in an access token requires TOKEN_ADJUST_PRIVILEGES access

MemPlus/Windows/LogWindow.xaml

@@ -23,13 +23,14 @@
                 </GridView>
             </ListView.View>
         </ListView>
+
         <Grid Grid.Row="1">
             <Grid.ColumnDefinitions>
                 <ColumnDefinition></ColumnDefinition>
                 <ColumnDefinition></ColumnDefinition>
             </Grid.ColumnDefinitions>
 
-            <Button MinHeight="25" Margin="5" Content="Export" Click="BtnExport_OnClick" />
+            <Button Grid.Column="0" MinHeight="25" Margin="5" Content="Export" Click="BtnExport_OnClick" />
             <Button Grid.Column="1" MinHeight="25" Margin="5" Content="Clear logs" Click="BtnClear_OnClick" />
         </Grid>
     </Grid>

MemPlus/Windows/LogWindow.xaml.cs

@@ -4,7 +4,7 @@
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Controls.Primitives;
-using MemPlus.Classes;
+using MemPlus.Classes.GUI;
 using MemPlus.Classes.LOG;
 
 namespace MemPlus.Windows

MemPlus/Windows/MainWindow.xaml.cs

@@ -1,7 +1,7 @@
 using System;
 using System.Windows;
 using System.Windows.Media;
-using MemPlus.Classes;
+using MemPlus.Classes.GUI;
 using MemPlus.Classes.LOG;
 using MemPlus.Classes.RAM;
 
@@ -70,17 +70,21 @@ private async void BtnClearMemory_OnClick(object sender, RoutedEventArgs e)
                 double ramSavings = _ramController.RamSavings / 1024 / 1024;
                 if (ramSavings < 0)
                 {
-                    MessageBox.Show("Looks like your RAM usage has increased with " + Math.Abs(ramSavings).ToString("F2") + " MB!", "MemPlus", MessageBoxButton.OK, MessageBoxImage.Information);
+                    ramSavings = Math.Abs(ramSavings);
+                    _logController.AddLog(new ApplicationLog("RAM usage increase: " + ramSavings.ToString("F2") + " MB"));
+                    MessageBox.Show("Looks like your RAM usage has increased with " + ramSavings.ToString("F2") + " MB!", "MemPlus", MessageBoxButton.OK, MessageBoxImage.Information);
                 }
                 else
                 {
+                    _logController.AddLog(new ApplicationLog("RAM usage decrease: " + ramSavings.ToString("F2") + " MB"));
                     MessageBox.Show("You saved " + ramSavings.ToString("F2") + " MB of RAM!", "MemPlus", MessageBoxButton.OK, MessageBoxImage.Information);
                 }
 
                 BtnClearMemory.IsEnabled = true;
             }
             catch (Exception ex)
             {
+                _logController.AddLog(new ApplicationLog(ex.Message));
                 MessageBox.Show(ex.Message, "MemPlus", MessageBoxButton.OK, MessageBoxImage.Error);
             }
 
@@ -120,6 +124,7 @@ private void HomePageMenuItem_OnClick(object sender, RoutedEventArgs e)
             }
             catch (Exception ex)
             {
+                _logController.AddLog(new ApplicationLog(ex.Message));
                 MessageBox.Show(ex.Message, "MemPlus", MessageBoxButton.OK, MessageBoxImage.Error);
             }
         }