Added initialization method, remove duplicate code

Author: TheNicker

src/LogExpert/Classes/Log/LogfileReader.cs

@@ -58,73 +58,77 @@ public class LogfileReader : IAutoLogLineColumnizerCallback
         #endregion
 
         #region cTor
-
         public LogfileReader(string fileName, EncodingOptions encodingOptions, bool multiFile, int bufferCount, int linesPerBuffer, MultiFileOptions multiFileOptions)
         {
             if (fileName == null)
             {
-                return;
+                throw new ArgumentNullException(nameof(fileName));
             }
 
             _fileName = fileName;
-            EncodingOptions = encodingOptions;
-            IsMultiFile = multiFile;
             _MAX_BUFFERS = bufferCount;
             _MAX_LINES_PER_BUFFER = linesPerBuffer;
             _multiFileOptions = multiFileOptions;
             _logLineFx = GetLogLineInternal;
-            InitLruBuffers();
-
-            if (multiFile)
-            {
-                ILogFileInfo info = GetLogFileInfo(fileName);
-                RolloverFilenameHandler rolloverHandler = new(info, _multiFileOptions);
-                LinkedList<string> nameList = rolloverHandler.GetNameList();
-
-                ILogFileInfo fileInfo = null;
-                foreach (string name in nameList)
-                {
-                    fileInfo = AddFile(name);
-                }
-
-                _watchedILogFileInfo = fileInfo; // last added file in the list is the watched file
-            }
-            else
-            {
-                _watchedILogFileInfo = AddFile(fileName);
-            }
+            IsMultiFile = multiFile;
 
-            StartGCThread();
+            InitializeReader(encodingOptions, null);
         }
 
         public LogfileReader(string[] fileNames, EncodingOptions encodingOptions, int bufferCount, int linesPerBuffer, MultiFileOptions multiFileOptions)
         {
             if (fileNames == null || fileNames.Length < 1)
             {
-                return;
+                throw new ArgumentNullException(nameof(fileNames));
             }
 
-            EncodingOptions = encodingOptions;
-            IsMultiFile = true;
+            _fileName = fileNames[0];
             _MAX_BUFFERS = bufferCount;
             _MAX_LINES_PER_BUFFER = linesPerBuffer;
             _multiFileOptions = multiFileOptions;
             _logLineFx = GetLogLineInternal;
+            IsMultiFile = true;
+
+            InitializeReader(encodingOptions, fileNames);
+        }
 
+        private void InitializeReader(EncodingOptions encodingOptions, string[]? additionalFiles)
+        {
+            EncodingOptions = encodingOptions;
             InitLruBuffers();
 
-            ILogFileInfo fileInfo = null;
-            foreach (string name in fileNames)
+            if (additionalFiles != null)
             {
-                fileInfo = AddFile(name);
+                // Handle array constructor case
+                ILogFileInfo fileInfo = null;
+                foreach (string name in additionalFiles)
+                {
+                    fileInfo = AddFile(name);
+                }
+                _watchedILogFileInfo = fileInfo;
             }
+            else if (IsMultiFile)
+            {
+                // Handle single file multiFile case
+                ILogFileInfo info = GetLogFileInfo(_fileName);
+                RolloverFilenameHandler rolloverHandler = new(info, _multiFileOptions);
+                LinkedList<string> nameList = rolloverHandler.GetNameList();
 
-            _watchedILogFileInfo = fileInfo;
-            _fileName = fileInfo.FullName;
+                ILogFileInfo fileInfo = null;
+                foreach (string name in nameList)
+                {
+                    fileInfo = AddFile(name);
+                }
+                _watchedILogFileInfo = fileInfo; // last added file in the list is the watched file
+            }
+            else
+            {
+                // Handle single file case
+                _watchedILogFileInfo = AddFile(_fileName);
+            }
 
             StartGCThread();
         }
-
         #endregion
 
         #region Delegates