Merge pull request #14 from TechieGuy12/cleanup

TechieGuy12 · web-flow · commit 90c0cfef38ca · 2022-02-03T09:36:39.000-05:00
Cleanup
diff --git a/FileWatcher/Configuration/Actions/Action.cs b/FileWatcher/Configuration/Actions/Action.cs
@@ -14,14 +14,6 @@ namespace TE.FileWatcher.Configuration.Actions
     /// </summary>
     public class Action : RunnableBase
     {
-        // The placeholders for the destination path
-        //private Dictionary<string, string> _destinationPlaceholders 
-        //    = new Dictionary<string, string>();
-
-        // The placeholders for the source path
-        //private Dictionary<string, string> _sourcePlaceholders
-        //    = new Dictionary<string, string>();
-
         /// <summary>
         /// The type of action to perform.
         /// </summary>
@@ -118,7 +110,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
                         {
                             if (string.IsNullOrWhiteSpace(destination))
                             {
-                                Logger.WriteLine($"The file '{source}' could not be copied because the destination file was not specified.");
+                                Logger.WriteLine(
+                                    $"The file '{source}' could not be copied because the destination file was not specified.",
+                                    LogLevel.ERROR);
                                 return;
                             }
 
@@ -127,7 +121,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
                         }
                         else
                         {
-                            Logger.WriteLine($"The file '{source}' could not be copied because the path was not valid, the file doesn't exists, or it was in use.");
+                            Logger.WriteLine(
+                                $"The file '{source}' could not be copied because the path was not valid, the file doesn't exists, or it was in use.",
+                                LogLevel.ERROR);
                         }
                         break;
                     case ActionType.Move:
@@ -144,7 +140,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
                         }
                         else
                         {
-                            Logger.WriteLine($"The file '{source}' could not be moved because the path was not valid, the file doesn't exists, or it was in use.");
+                            Logger.WriteLine(
+                                $"The file '{source}' could not be moved because the path was not valid, the file doesn't exists, or it was in use.",
+                                LogLevel.ERROR);
                         }
                         break;
                     case ActionType.Delete:
@@ -155,15 +153,17 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
                         }
                         else
                         {
-                            Logger.WriteLine($"The file '{source}' could not be deleted because the path was not valid, the file doesn't exists, or it was in use.");
+                            Logger.WriteLine(
+                                $"The file '{source}' could not be deleted because the path was not valid, the file doesn't exists, or it was in use.", 
+                                LogLevel.ERROR);
                         }
                         break;
                 }
             }
             catch (Exception ex)
             {
                 string message = (ex.InnerException == null) ? ex.Message : ex.InnerException.Message;
-                Logger.WriteLine($"Could not {Type.ToString().ToLower()} file {source}. Reason: {message}");
+                Logger.WriteLine($"Could not {Type.ToString().ToLower()} file {source}. Reason: {message}", LogLevel.ERROR);
                 return;
             }
         }
diff --git a/FileWatcher/Configuration/Commands/Command.cs b/FileWatcher/Configuration/Commands/Command.cs
@@ -77,13 +77,16 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
 
             if (string.IsNullOrWhiteSpace(commandPath))
             {
-                Logger.WriteLine($"The command was not provided. Command was not run.");
+                Logger.WriteLine($"The command was not provided. Command was not run.",
+                    LogLevel.ERROR);
                 return;
             }
 
             if (!File.Exists(commandPath))
             {
-                Logger.WriteLine($"The command '{commandPath}' was not found. Command was not run.");
+                Logger.WriteLine(
+                    $"The command '{commandPath}' was not found. Command was not run.",
+                    LogLevel.ERROR);
                 return;
             }
 
@@ -111,7 +114,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)
             }
             catch (Exception ex)
             {
-                Logger.WriteLine($"Could not run the command '{commandPath} {arguments}'. Reason: {ex.Message}");
+                Logger.WriteLine(
+                    $"Could not run the command '{commandPath} {arguments}'. Reason: {ex.Message}",
+                    LogLevel.ERROR);
             }
         }
 
@@ -148,7 +153,9 @@ private void Execute()
                     }
                     else
                     {
-                        Logger.WriteLine($"The command '{startInfo.FileName}' was not found. Command was not run.");
+                        Logger.WriteLine(
+                            $"The command '{startInfo.FileName}' was not found. Command was not run.",
+                            LogLevel.ERROR);
 
                         // Execute the next process in the queue
                         Execute();
@@ -159,11 +166,15 @@ private void Execute()
             {
                 if (_process != null)
                 {
-                    Logger.WriteLine($"Could not run the command '{_process.StartInfo.FileName} {_process.StartInfo.Arguments}'. Reason: {ex.Message}");
+                    Logger.WriteLine(
+                        $"Could not run the command '{_process.StartInfo.FileName} {_process.StartInfo.Arguments}'. Reason: {ex.Message}",
+                        LogLevel.ERROR);
                 }
                 else
                 {
-                    Logger.WriteLine($"Could not run the command. Reason: {ex.Message}");
+                    Logger.WriteLine(
+                        $"Could not run the command. Reason: {ex.Message}",
+                        LogLevel.ERROR);
                 }
             }
         }
diff --git a/FileWatcher/Configuration/Logging.cs b/FileWatcher/Configuration/Logging.cs
@@ -15,9 +15,25 @@ namespace TE.FileWatcher.Configuration
     /// </summary>
     public class Logging
     {
+        /// <summary>
+        /// The default log size.
+        /// </summary>
+        public const int DEFAULT_LOG_SIZE = 5;
+
+        /// <summary>
+        /// The default log number.
+        /// </summary>
+        public const int DEFAULT_LOG_NUMBER = 10;
+
         // The log path
         private string? _logPath;
 
+        // The size of the log file
+        private int _logSize = DEFAULT_LOG_SIZE;
+
+        // The number of log files to retain
+        private int _logNumber = DEFAULT_LOG_NUMBER;
+
         /// <summary>
         /// Gets or sets the path of the log file.
         /// </summary>
@@ -39,13 +55,33 @@ public string? LogPath
         /// backed up and a new log file is created.
         /// </summary>
         [XmlElement("size")]
-        public int Size { get; set; }
+        public int Size
+        {
+            get
+            {
+                return _logSize;
+            }
+            set
+            {
+                _logSize = value > 0 ? value : DEFAULT_LOG_SIZE;
+            }
+        }
 
         /// <summary>
         /// Gets or sets the number of log file to retain.
         /// </summary>
         [XmlElement("number")]
-        public int Number { get; set; }
+        public int Number
+        { 
+            get
+            {
+                return _logNumber;
+            }
+            set
+            {
+                _logNumber = value > 0 ? value : DEFAULT_LOG_NUMBER;
+            }
+        }
 
         /// <summary>
         /// Initializes an instance of the <see cref="Logging"/> class.
@@ -62,8 +98,8 @@ public string? LogPath
         public Logging() 
         {
             LogPath = Path.Combine(Path.GetTempPath(), Logger.DEFAULT_LOG_NAME);
-            Size = 5;
-            Number = 10;
+            Size = DEFAULT_LOG_SIZE;
+            Number = DEFAULT_LOG_NUMBER;
         }
     }
 }
diff --git a/FileWatcher/Configuration/Notifications/Notifications.cs b/FileWatcher/Configuration/Notifications/Notifications.cs
@@ -99,13 +99,17 @@ private async void OnElapsed(object? source, ElapsedEventArgs e)
                     foreach (Exception ex in aex.Flatten().InnerExceptions)
                     {
                         Logger.WriteLine(ex.Message, LogLevel.ERROR);
-                        Logger.WriteLine($"StackTrace:{Environment.NewLine}{ex.StackTrace}");
+                        Logger.WriteLine(
+                            $"StackTrace:{Environment.NewLine}{ex.StackTrace}",
+                            LogLevel.ERROR);
                     }
                 }
                 catch (NullReferenceException ex)
                 {
                     Logger.WriteLine(ex.Message, LogLevel.ERROR);
-                    Logger.WriteLine($"StackTrace:{Environment.NewLine}{ex.StackTrace}");
+                    Logger.WriteLine(
+                        $"StackTrace:{Environment.NewLine}{ex.StackTrace}",
+                        LogLevel.ERROR);
                 }
             }
         }
diff --git a/FileWatcher/Configuration/RunnableBase.cs b/FileWatcher/Configuration/RunnableBase.cs
@@ -130,7 +130,7 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)
         /// </returns>
         private static string? GetRelativePath(string watchPath, string fullPath)
         {
-            string? relativeFullPath = IO.Path.GetDirectoryName(fullPath);
+            string? relativeFullPath = Path.GetDirectoryName(fullPath);
             if (relativeFullPath == null)
             {
                 return null;
@@ -150,12 +150,12 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)
         /// </returns>
         private static string? GetFilename(string fullPath, bool includeExtension)
         {
-            if (string.IsNullOrEmpty(fullPath) || !IO.File.Exists(fullPath))
+            if (string.IsNullOrEmpty(fullPath) || !File.Exists(fullPath))
             {
                 return null;
             }
 
-            return includeExtension ? IO.Path.GetFileNameWithoutExtension(fullPath) : IO.Path.GetFileName(fullPath);
+            return includeExtension ? Path.GetFileNameWithoutExtension(fullPath) : Path.GetFileName(fullPath);
         }
 
         /// <summary>
@@ -169,12 +169,12 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)
         /// </returns>
         private static string? GetFileExtension(string fullPath)
         {
-            if (string.IsNullOrEmpty(fullPath) || !IO.File.Exists(fullPath))
+            if (string.IsNullOrEmpty(fullPath) || !File.Exists(fullPath))
             {
                 return null;
             }
 
-            return IO.Path.GetExtension(fullPath);
+            return Path.GetExtension(fullPath);
         }
     }
 }
diff --git a/FileWatcher/Configuration/Watch.cs b/FileWatcher/Configuration/Watch.cs
@@ -173,7 +173,9 @@ private void CreateFileSystemWatcher()
         {
             if (string.IsNullOrWhiteSpace(Path))
             {
-                Logger.WriteLine("The path to watch was not specified.");
+                Logger.WriteLine(
+                    "The path to watch was not specified.",
+                    LogLevel.ERROR);
                 return;
             }
 
@@ -461,7 +463,7 @@ private static void NotAccessibleError(FileSystemWatcher source, ErrorEventArgs
                 catch
                 {
                     source.EnableRaisingEvents = false;
-                    System.Threading.Thread.Sleep(iTimeOut);
+                    Thread.Sleep(iTimeOut);
                 }
             }
 
diff --git a/FileWatcher/Configuration/Watches.cs b/FileWatcher/Configuration/Watches.cs
@@ -31,8 +31,9 @@ public class Watches
         /// </summary>
         public void Start()
         {
-            if (WatchList == null)
+            if (WatchList == null || WatchList.Count <= 0)
             {
+                Logger.WriteLine("No watches were specified.", LogLevel.ERROR);
                 return;
             }
 
diff --git a/FileWatcher/Configuration/XmlFile.cs b/FileWatcher/Configuration/XmlFile.cs
@@ -11,6 +11,9 @@
 
 namespace TE.FileWatcher.Configuration
 {
+    /// <summary>
+    /// The XML configuration file.
+    /// </summary>
     public class XmlFile : IConfigurationFile
     {
         // The default configuration file name
@@ -47,7 +50,7 @@ public XmlFile(string path, string name)
         /// The folder path.
         /// </param>
         /// <returns>
-        /// The folder path of the files, otherwise null.
+        /// The folder path of the files, otherwise <c>null</c>.
         /// </returns>
         private static string? GetFolderPath(string? path)
         {
@@ -85,7 +88,7 @@ public XmlFile(string path, string name)
         /// The name of the configuration file.
         /// </param>
         /// <returns>
-        /// The full path to the configuration file, otherwise null.
+        /// The full path to the configuration file, otherwise <c>null</c>.
         /// </returns>
         private static string? GetFullPath(string path, string name)
         {
@@ -126,7 +129,7 @@ public XmlFile(string path, string name)
         /// </summary>
         /// <returns>
         /// A <see cref="Watches"/> object if the file was read successfully,
-        /// otherwise null.
+        /// otherwise <c>null</c>.
         /// </returns>
         [RequiresUnreferencedCode("Calls XmlSerializer")]
         public Watches? Read()
diff --git a/FileWatcher/FileSystem/Directory.cs b/FileWatcher/FileSystem/Directory.cs
@@ -18,11 +18,20 @@ public static class Directory
 		/// <param name="path">
 		/// The path that includes the folder structure to create.
 		/// </param>
+		/// <exception cref="ArgumentNullException">
+		/// Thrown when an argument is null or empty.
+		/// </exception>
+		/// <exception cref="FileWatcherException">
+		/// Thrown when the directory could not be created.
+		/// </exception>
+		/// <exception cref="NullReferenceException">
+		/// Thrown when the directory from the path is null.
+		/// </exception>
         public static void Create(string path)
         {
 			if (string.IsNullOrWhiteSpace(path))
             {
-				return;
+				throw new ArgumentNullException(nameof(path));
             }
 
 			string? folders = Path.GetDirectoryName(path);
@@ -34,8 +43,16 @@ public static void Create(string path)
 				if (!IO.Directory.Exists(folders))
 				{
 					IO.Directory.CreateDirectory(folders);
+					if (!IO.Directory.Exists(folders))
+                    {
+						throw new FileWatcherException($"The directory {folders} could not be created.");
+                    }
 				}
 			}
+			else
+            {
+				throw new NullReferenceException("The directory path could not be determined.");
+            }
 		}
 
 		/// <summary>
diff --git a/FileWatcher/FileSystem/File.cs b/FileWatcher/FileSystem/File.cs
@@ -182,7 +182,7 @@ public static void Copy(string source, string destination, bool verify)
         /// <returns>
         /// <c>true</c> if the file is valid, otherwise <c>false</c>.
         /// </returns>
-        public static bool IsValid(string? path)
+        public static bool IsValid(string path)
         {
             if (string.IsNullOrWhiteSpace(path))
             {
diff --git a/FileWatcher/FileWatcher.csproj b/FileWatcher/FileWatcher.csproj
diff --git a/FileWatcher/Logging/Logger.cs b/FileWatcher/Logging/Logger.cs
diff --git a/FileWatcher/Logging/Message.cs b/FileWatcher/Logging/Message.cs
diff --git a/FileWatcher/Program.cs b/FileWatcher/Program.cs

Original file line number	Diff line number	Diff line change
`@@ -14,14 +14,6 @@ namespace TE.FileWatcher.Configuration.Actions`
`14`	`14`	`/// </summary>`
`15`	`15`	`public class Action : RunnableBase`
`16`	`16`	`{`
`17`		`- // The placeholders for the destination path`
`18`		`- //private Dictionary<string, string> _destinationPlaceholders`
`19`		`- // = new Dictionary<string, string>();`
`20`		`-`
`21`		`- // The placeholders for the source path`
`22`		`- //private Dictionary<string, string> _sourcePlaceholders`
`23`		`- // = new Dictionary<string, string>();`
`24`		`-`
`25`	`17`	`/// <summary>`
`26`	`18`	`/// The type of action to perform.`
`27`	`19`	`/// </summary>`
`@@ -118,7 +110,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`118`	`110`	`{`
`119`	`111`	`if (string.IsNullOrWhiteSpace(destination))`
`120`	`112`	`{`
`121`		`- Logger.WriteLine($"The file '{source}' could not be copied because the destination file was not specified.");`
	`113`	`+ Logger.WriteLine(`
	`114`	`+ $"The file '{source}' could not be copied because the destination file was not specified.",`
	`115`	`+ LogLevel.ERROR);`
`122`	`116`	`return;`
`123`	`117`	`}`
`124`	`118`
`@@ -127,7 +121,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`127`	`121`	`}`
`128`	`122`	`else`
`129`	`123`	`{`
`130`		`- Logger.WriteLine($"The file '{source}' could not be copied because the path was not valid, the file doesn't exists, or it was in use.");`
	`124`	`+ Logger.WriteLine(`
	`125`	`+ $"The file '{source}' could not be copied because the path was not valid, the file doesn't exists, or it was in use.",`
	`126`	`+ LogLevel.ERROR);`
`131`	`127`	`}`
`132`	`128`	`break;`
`133`	`129`	`case ActionType.Move:`
`@@ -144,7 +140,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`144`	`140`	`}`
`145`	`141`	`else`
`146`	`142`	`{`
`147`		`- Logger.WriteLine($"The file '{source}' could not be moved because the path was not valid, the file doesn't exists, or it was in use.");`
	`143`	`+ Logger.WriteLine(`
	`144`	`+ $"The file '{source}' could not be moved because the path was not valid, the file doesn't exists, or it was in use.",`
	`145`	`+ LogLevel.ERROR);`
`148`	`146`	`}`
`149`	`147`	`break;`
`150`	`148`	`case ActionType.Delete:`
`@@ -155,15 +153,17 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`155`	`153`	`}`
`156`	`154`	`else`
`157`	`155`	`{`
`158`		`- Logger.WriteLine($"The file '{source}' could not be deleted because the path was not valid, the file doesn't exists, or it was in use.");`
	`156`	`+ Logger.WriteLine(`
	`157`	`+ $"The file '{source}' could not be deleted because the path was not valid, the file doesn't exists, or it was in use.",`
	`158`	`+ LogLevel.ERROR);`
`159`	`159`	`}`
`160`	`160`	`break;`
`161`	`161`	`}`
`162`	`162`	`}`
`163`	`163`	`catch (Exception ex)`
`164`	`164`	`{`
`165`	`165`	`string message = (ex.InnerException == null) ? ex.Message : ex.InnerException.Message;`
`166`		`- Logger.WriteLine($"Could not {Type.ToString().ToLower()} file {source}. Reason: {message}");`
	`166`	`+ Logger.WriteLine($"Could not {Type.ToString().ToLower()} file {source}. Reason: {message}", LogLevel.ERROR);`
`167`	`167`	`return;`
`168`	`168`	`}`
`169`	`169`	`}`
Original file line number	Diff line number	Diff line change
`@@ -77,13 +77,16 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`77`	`77`
`78`	`78`	`if (string.IsNullOrWhiteSpace(commandPath))`
`79`	`79`	`{`
`80`		`- Logger.WriteLine($"The command was not provided. Command was not run.");`
	`80`	`+ Logger.WriteLine($"The command was not provided. Command was not run.",`
	`81`	`+ LogLevel.ERROR);`
`81`	`82`	`return;`
`82`	`83`	`}`
`83`	`84`
`84`	`85`	`if (!File.Exists(commandPath))`
`85`	`86`	`{`
`86`		`- Logger.WriteLine($"The command '{commandPath}' was not found. Command was not run.");`
	`87`	`+ Logger.WriteLine(`
	`88`	`+ $"The command '{commandPath}' was not found. Command was not run.",`
	`89`	`+ LogLevel.ERROR);`
`87`	`90`	`return;`
`88`	`91`	`}`
`89`	`92`
`@@ -111,7 +114,9 @@ public override void Run(string watchPath, string fullPath, TriggerType trigger)`
`111`	`114`	`}`
`112`	`115`	`catch (Exception ex)`
`113`	`116`	`{`
`114`		`- Logger.WriteLine($"Could not run the command '{commandPath} {arguments}'. Reason: {ex.Message}");`
	`117`	`+ Logger.WriteLine(`
	`118`	`+ $"Could not run the command '{commandPath} {arguments}'. Reason: {ex.Message}",`
	`119`	`+ LogLevel.ERROR);`
`115`	`120`	`}`
`116`	`121`	`}`
`117`	`122`
`@@ -148,7 +153,9 @@ private void Execute()`
`148`	`153`	`}`
`149`	`154`	`else`
`150`	`155`	`{`
`151`		`- Logger.WriteLine($"The command '{startInfo.FileName}' was not found. Command was not run.");`
	`156`	`+ Logger.WriteLine(`
	`157`	`+ $"The command '{startInfo.FileName}' was not found. Command was not run.",`
	`158`	`+ LogLevel.ERROR);`
`152`	`159`
`153`	`160`	`// Execute the next process in the queue`
`154`	`161`	`Execute();`
`@@ -159,11 +166,15 @@ private void Execute()`
`159`	`166`	`{`
`160`	`167`	`if (_process != null)`
`161`	`168`	`{`
`162`		`- Logger.WriteLine($"Could not run the command '{_process.StartInfo.FileName} {_process.StartInfo.Arguments}'. Reason: {ex.Message}");`
	`169`	`+ Logger.WriteLine(`
	`170`	`+ $"Could not run the command '{_process.StartInfo.FileName} {_process.StartInfo.Arguments}'. Reason: {ex.Message}",`
	`171`	`+ LogLevel.ERROR);`
`163`	`172`	`}`
`164`	`173`	`else`
`165`	`174`	`{`
`166`		`- Logger.WriteLine($"Could not run the command. Reason: {ex.Message}");`
	`175`	`+ Logger.WriteLine(`
	`176`	`+ $"Could not run the command. Reason: {ex.Message}",`
	`177`	`+ LogLevel.ERROR);`
`167`	`178`	`}`
`168`	`179`	`}`
`169`	`180`	`}`
Original file line number	Diff line number	Diff line change
`@@ -99,13 +99,17 @@ private async void OnElapsed(object? source, ElapsedEventArgs e)`
`99`	`99`	`foreach (Exception ex in aex.Flatten().InnerExceptions)`
`100`	`100`	`{`
`101`	`101`	`Logger.WriteLine(ex.Message, LogLevel.ERROR);`
`102`		`- Logger.WriteLine($"StackTrace:{Environment.NewLine}{ex.StackTrace}");`
	`102`	`+ Logger.WriteLine(`
	`103`	`+ $"StackTrace:{Environment.NewLine}{ex.StackTrace}",`
	`104`	`+ LogLevel.ERROR);`
`103`	`105`	`}`
`104`	`106`	`}`
`105`	`107`	`catch (NullReferenceException ex)`
`106`	`108`	`{`
`107`	`109`	`Logger.WriteLine(ex.Message, LogLevel.ERROR);`
`108`		`- Logger.WriteLine($"StackTrace:{Environment.NewLine}{ex.StackTrace}");`
	`110`	`+ Logger.WriteLine(`
	`111`	`+ $"StackTrace:{Environment.NewLine}{ex.StackTrace}",`
	`112`	`+ LogLevel.ERROR);`
`109`	`113`	`}`
`110`	`114`	`}`
`111`	`115`	`}`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)`
`130`	`130`	`/// </returns>`
`131`	`131`	`private static string? GetRelativePath(string watchPath, string fullPath)`
`132`	`132`	`{`
`133`		`- string? relativeFullPath = IO.Path.GetDirectoryName(fullPath);`
	`133`	`+ string? relativeFullPath = Path.GetDirectoryName(fullPath);`
`134`	`134`	`if (relativeFullPath == null)`
`135`	`135`	`{`
`136`	`136`	`return null;`
`@@ -150,12 +150,12 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)`
`150`	`150`	`/// </returns>`
`151`	`151`	`private static string? GetFilename(string fullPath, bool includeExtension)`
`152`	`152`	`{`
`153`		`- if (string.IsNullOrEmpty(fullPath) \|\| !IO.File.Exists(fullPath))`
	`153`	`+ if (string.IsNullOrEmpty(fullPath) \|\| !File.Exists(fullPath))`
`154`	`154`	`{`
`155`	`155`	`return null;`
`156`	`156`	`}`
`157`	`157`
`158`		`- return includeExtension ? IO.Path.GetFileNameWithoutExtension(fullPath) : IO.Path.GetFileName(fullPath);`
	`158`	`+ return includeExtension ? Path.GetFileNameWithoutExtension(fullPath) : Path.GetFileName(fullPath);`
`159`	`159`	`}`
`160`	`160`
`161`	`161`	`/// <summary>`
`@@ -169,12 +169,12 @@ private static string GetRelativeFullPath(string watchPath, string fullPath)`
`169`	`169`	`/// </returns>`
`170`	`170`	`private static string? GetFileExtension(string fullPath)`
`171`	`171`	`{`
`172`		`- if (string.IsNullOrEmpty(fullPath) \|\| !IO.File.Exists(fullPath))`
	`172`	`+ if (string.IsNullOrEmpty(fullPath) \|\| !File.Exists(fullPath))`
`173`	`173`	`{`
`174`	`174`	`return null;`
`175`	`175`	`}`
`176`	`176`
`177`		`- return IO.Path.GetExtension(fullPath);`
	`177`	`+ return Path.GetExtension(fullPath);`
`178`	`178`	`}`
`179`	`179`	`}`
`180`	`180`	`}`
Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,9 @@ private void CreateFileSystemWatcher()`
`173`	`173`	`{`
`174`	`174`	`if (string.IsNullOrWhiteSpace(Path))`
`175`	`175`	`{`
`176`		`- Logger.WriteLine("The path to watch was not specified.");`
	`176`	`+ Logger.WriteLine(`
	`177`	`+ "The path to watch was not specified.",`
	`178`	`+ LogLevel.ERROR);`
`177`	`179`	`return;`
`178`	`180`	`}`
`179`	`181`
`@@ -461,7 +463,7 @@ private static void NotAccessibleError(FileSystemWatcher source, ErrorEventArgs`
`461`	`463`	`catch`
`462`	`464`	`{`
`463`	`465`	`source.EnableRaisingEvents = false;`
`464`		`- System.Threading.Thread.Sleep(iTimeOut);`
	`466`	`+ Thread.Sleep(iTimeOut);`
`465`	`467`	`}`
`466`	`468`	`}`
`467`	`469`
Original file line number	Diff line number	Diff line change
`@@ -31,8 +31,9 @@ public class Watches`
`31`	`31`	`/// </summary>`
`32`	`32`	`public void Start()`
`33`	`33`	`{`
`34`		`- if (WatchList == null)`
	`34`	`+ if (WatchList == null \|\| WatchList.Count <= 0)`
`35`	`35`	`{`
	`36`	`+ Logger.WriteLine("No watches were specified.", LogLevel.ERROR);`
`36`	`37`	`return;`
`37`	`38`	`}`
`38`	`39`
Original file line number	Diff line number	Diff line change
`@@ -18,11 +18,20 @@ public static class Directory`
`18`	`18`	`/// <param name="path">`
`19`	`19`	`/// The path that includes the folder structure to create.`
`20`	`20`	`/// </param>`
	`21`	`+ /// <exception cref="ArgumentNullException">`
	`22`	`+ /// Thrown when an argument is null or empty.`
	`23`	`+ /// </exception>`
	`24`	`+ /// <exception cref="FileWatcherException">`
	`25`	`+ /// Thrown when the directory could not be created.`
	`26`	`+ /// </exception>`
	`27`	`+ /// <exception cref="NullReferenceException">`
	`28`	`+ /// Thrown when the directory from the path is null.`
	`29`	`+ /// </exception>`
`21`	`30`	`public static void Create(string path)`
`22`	`31`	`{`
`23`	`32`	`if (string.IsNullOrWhiteSpace(path))`
`24`	`33`	`{`
`25`		`- return;`
	`34`	`+ throw new ArgumentNullException(nameof(path));`
`26`	`35`	`}`
`27`	`36`
`28`	`37`	`string? folders = Path.GetDirectoryName(path);`
`@@ -34,8 +43,16 @@ public static void Create(string path)`
`34`	`43`	`if (!IO.Directory.Exists(folders))`
`35`	`44`	`{`
`36`	`45`	`IO.Directory.CreateDirectory(folders);`
	`46`	`+ if (!IO.Directory.Exists(folders))`
	`47`	`+ {`
	`48`	`+ throw new FileWatcherException($"The directory {folders} could not be created.");`
	`49`	`+ }`
`37`	`50`	`}`
`38`	`51`	`}`
	`52`	`+ else`
	`53`	`+ {`
	`54`	`+ throw new NullReferenceException("The directory path could not be determined.");`
	`55`	`+ }`
`39`	`56`	`}`
`40`	`57`
`41`	`58`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ public static void Copy(string source, string destination, bool verify)`
`182`	`182`	`/// <returns>`
`183`	`183`	`/// <c>true</c> if the file is valid, otherwise <c>false</c>.`
`184`	`184`	`/// </returns>`
`185`		`- public static bool IsValid(string? path)`
	`185`	`+ public static bool IsValid(string path)`
`186`	`186`	`{`
`187`	`187`	`if (string.IsNullOrWhiteSpace(path))`
`188`	`188`	`{`