Add the IMultipartFormDataParser interface

Author: Jericho

Source/HttpMultipartParser/IMultipartFormDataParser.cs

@@ -0,0 +1,25 @@
+using System.Collections.Generic;
+
+namespace HttpMultipartParser
+{
+    /// <summary>
+    ///     Provides methods to parse a
+    ///     <see href="http://www.ietf.org/rfc/rfc2388.txt">
+    ///         <c>multipart/form-data</c>
+    ///     </see>
+    ///     stream into it's parameters and file data.
+    /// </summary>
+    public interface IMultipartFormDataParser
+    {
+        /// <summary>
+        ///     Gets the mapping of parameters parsed files. The name of a given field
+        ///     maps to the parsed file data.
+        /// </summary>
+        IReadOnlyList<FilePart> Files { get; }
+
+        /// <summary>
+        ///     Gets the parameters. Several ParameterParts may share the same name.
+        /// </summary>
+        IReadOnlyList<ParameterPart> Parameters { get; }
+    }
+}

Source/HttpMultipartParser/MultipartFormDataParser.cs

@@ -81,9 +81,9 @@ namespace HttpMultipartParser
     ///     }
     ///   </code>
     /// </example>
-    public class MultipartFormDataParser
+    public class MultipartFormDataParser : IMultipartFormDataParser
     {
-        #region Constants
+        #region Constants and fields
 
         /// <summary>
         ///     The default buffer size.
@@ -95,6 +95,9 @@ public class MultipartFormDataParser
         /// </remarks>
         private const int DefaultBufferSize = 4096;
 
+        private readonly List<FilePart> _files;
+        private readonly List<ParameterPart> _parameters;
+
         #endregion
 
         #region Constructors and Destructors
@@ -104,6 +107,8 @@ public class MultipartFormDataParser
         /// </summary>
         private MultipartFormDataParser()
         {
+            _files = new List<FilePart>();
+            _parameters = new List<ParameterPart>();
         }
 
         #endregion
@@ -114,12 +119,12 @@ private MultipartFormDataParser()
         ///     Gets the mapping of parameters parsed files. The name of a given field
         ///     maps to the parsed file data.
         /// </summary>
-        public List<FilePart> Files { get; private set; }
+        public IReadOnlyList<FilePart> Files => _files.AsReadOnly();
 
         /// <summary>
         ///     Gets the parameters. Several ParameterParts may share the same name.
         /// </summary>
-        public List<ParameterPart> Parameters { get; private set; }
+        public IReadOnlyList<ParameterPart> Parameters => _parameters.AsReadOnly();
 
         #endregion
 
@@ -304,18 +309,15 @@ public IEnumerable<string> GetParameterValues(string name)
         /// </param>
         private void ParseStream(Stream stream, string boundary, Encoding encoding, int binaryBufferSize, string[] binaryMimeTypes)
         {
-            Files = new List<FilePart>();
-            Parameters = new List<ParameterPart>();
-
             var streamingParser = new StreamingMultipartFormDataParser(stream, boundary, encoding ?? Encoding.UTF8, binaryBufferSize, binaryMimeTypes);
-            streamingParser.ParameterHandler += parameterPart => Parameters.Add(parameterPart);
+            streamingParser.ParameterHandler += parameterPart => _parameters.Add(parameterPart);
 
             streamingParser.FileHandler += (name, fileName, type, disposition, buffer, bytes, partNumber, additionalProperties) =>
             {
                 if (partNumber == 0)
                 {
                     // create file with first partNo
-                    Files.Add(new FilePart(name, fileName, Utilities.MemoryStreamManager.GetStream($"{typeof(MultipartFormDataParser).FullName}.{nameof(ParseStream)}"), additionalProperties, type, disposition));
+                    _files.Add(new FilePart(name, fileName, Utilities.MemoryStreamManager.GetStream($"{typeof(MultipartFormDataParser).FullName}.{nameof(ParseStream)}"), additionalProperties, type, disposition));
                 }
 
                 Files[Files.Count - 1].Data.Write(buffer, 0, bytes);
@@ -352,18 +354,15 @@ private void ParseStream(Stream stream, string boundary, Encoding encoding, int
         /// </param>
         private async Task ParseStreamAsync(Stream stream, string boundary, Encoding encoding, int binaryBufferSize, string[] binaryMimeTypes)
         {
-            Files = new List<FilePart>();
-            Parameters = new List<ParameterPart>();
-
             var streamingParser = new StreamingMultipartFormDataParser(stream, boundary, encoding ?? Encoding.UTF8, binaryBufferSize, binaryMimeTypes);
-            streamingParser.ParameterHandler += parameterPart => Parameters.Add(parameterPart);
+            streamingParser.ParameterHandler += parameterPart => _parameters.Add(parameterPart);
 
             streamingParser.FileHandler += (name, fileName, type, disposition, buffer, bytes, partNumber, additionalProperties) =>
             {
                 if (partNumber == 0)
                 {
                     // create file with first partNo
-                    Files.Add(new FilePart(name, fileName, Utilities.MemoryStreamManager.GetStream($"{typeof(MultipartFormDataParser).FullName}.{nameof(ParseStreamAsync)}"), additionalProperties, type, disposition));
+                    _files.Add(new FilePart(name, fileName, Utilities.MemoryStreamManager.GetStream($"{typeof(MultipartFormDataParser).FullName}.{nameof(ParseStreamAsync)}"), additionalProperties, type, disposition));
                 }
 
                 Files[Files.Count - 1].Data.Write(buffer, 0, bytes);