Add support for nested serialization of arrays (#64)

Define nested serialization options for arrays.  This allows complex object serialization to produce something better than "System.String[]" all over the place.

CSVFile.nuspec

@@ -2,7 +2,7 @@
 <package >
 	<metadata>
 		<id>CSVFile</id>
-		<version>3.1.2</version>
+		<version>3.1.3</version>
 		<title>CSVFile</title>
 		<authors>Ted Spence</authors>
 		<owners>Ted Spence</owners>
@@ -15,10 +15,10 @@
 		<releaseNotes>
 			July 18, 2023
 
-			* Fix issue with inconsistent handling of embedded newlines in the streaming version of the reader
+			* Add serialization options for arrays
 		</releaseNotes>
 		<readme>docs/README.md</readme>
-		<copyright>Copyright 2006 - 2023</copyright>
+		<copyright>Copyright 2006 - 2024</copyright>
     	<tags>fast csv parser serialization deserialization streaming async</tags>
 		<repository type="git" url="https://github.com/tspence/csharp-csv-reader" />
 		<dependencies>

src/CSV.cs

@@ -4,6 +4,7 @@
  * Home page: https://github.com/tspence/csharp-csv-reader
  */
 using System;
+using System.Collections;
 using System.Collections.Generic;
 using System.IO;
 using System.Text;
@@ -339,7 +340,7 @@ internal static string RemoveByteOrderMarker(string rawString)
         /// <param name="riskyChars"></param>
         /// <param name="forceQualifierTypes"></param>
         /// <returns></returns>
-        internal static string ItemsToCsv(IEnumerable<object> items, CSVSettings settings, char[] riskyChars, Dictionary<Type, int> forceQualifierTypes)
+        internal static string ItemsToCsv(IEnumerable items, CSVSettings settings, char[] riskyChars, Dictionary<Type, int> forceQualifierTypes)
         {
             var sb = new StringBuilder();
             foreach (var item in items)
@@ -355,12 +356,67 @@ internal static string ItemsToCsv(IEnumerable<object> items, CSVSettings setting
                     continue;
                 }
 
-                // Is this a date time?
+                // Special cases for other types of serialization
                 string s;
+                var itemType = item.GetType();
+                var interfaces = itemType.GetInterfaces();
+                bool isEnumerable = false;
+                if (itemType != typeof(string))
+                {
+                    foreach (var itemInterface in interfaces)
+                    {
+                        if (itemInterface == typeof(IEnumerable))
+                        {
+                            isEnumerable = true;
+                        }
+                    }
+                }
+
                 if (item is DateTime)
                 {
                     s = ((DateTime)item).ToString(settings.DateTimeFormat);
                 }
+                else if (isEnumerable)
+                {
+                    IEnumerable enumerable = item as IEnumerable;
+                    s = string.Empty;
+                    switch (settings.NestedArrayBehavior)
+                    {
+                        case ArrayOptions.ToString:
+                            s = item.ToString();
+                            break;
+                        case ArrayOptions.CountItems:
+                            // from https://stackoverflow.com/questions/3546051/how-to-invoke-system-linq-enumerable-count-on-ienumerablet-using-reflection
+                            if (enumerable != null)
+                            {
+                                int enumerableCount = 0;
+                                var iter = enumerable.GetEnumerator();
+                                using (iter as IDisposable)
+                                {
+                                    while (iter.MoveNext())
+                                    {
+                                        enumerableCount++;
+                                    }
+                                }
+
+                                s = enumerableCount.ToString();
+                            }
+
+                            break;
+                        case ArrayOptions.TreatAsNull:
+                            if (settings.AllowNull)
+                            {
+                                s = settings.NullToken;
+                            }
+                            break;
+                        case ArrayOptions.RecursiveSerialization:
+                            if (enumerable != null)
+                            {
+                                s = ItemsToCsv(enumerable, settings, riskyChars, forceQualifierTypes);
+                            }
+                            break;
+                    }
+                }
                 else
                 {
                     s = item.ToString();
@@ -399,7 +455,10 @@ internal static string ItemsToCsv(IEnumerable<object> items, CSVSettings setting
             }
 
             // Subtract the trailing delimiter so we don't inadvertently add an empty column at the end
-            sb.Length -= 1;
+            if (sb.Length > 0)
+            {
+                sb.Length -= 1;
+            }
             return sb.ToString();
         }
 

src/CSVSettings.cs

@@ -9,6 +9,32 @@
 
 namespace CSVFile
 {
+    /// <summary>
+    /// Defines the behavior of CSV serialization when a nested array is encountered
+    /// </summary>
+    public enum ArrayOptions
+    {
+        /// <summary>
+        /// Use built-in string conversion, which renders arrays as `MyObject[]` 
+        /// </summary>
+        ToString,
+
+        /// <summary>
+        /// Convert any array columns that are array types into nulls (either blanks or null tokens)
+        /// </summary>
+        TreatAsNull,
+
+        /// <summary>
+        /// Render the number of items in the array
+        /// </summary>
+        CountItems,
+
+        /// <summary>
+        /// Serialize child objects recursively using the same settings
+        /// </summary>
+        RecursiveSerialization,
+    }
+
     /// <summary>
     /// Settings to configure how a CSV file is parsed
     /// </summary>
@@ -133,11 +159,16 @@ public class CSVSettings
         /// </summary>
         public string DateTimeFormat { get; set; } = "o";
 
+        /// <summary>
+        /// The behavior to use when serializing a column of an array type
+        /// </summary>
+        public ArrayOptions NestedArrayBehavior = ArrayOptions.TreatAsNull;
+
         /// <summary>
         /// Standard comma-separated value (CSV) file settings
         /// </summary>
         public static readonly CSVSettings CSV = new CSVSettings();
-
+        
         /// <summary>
         /// Standard comma-separated value (CSV) file settings that permit rendering of NULL values
         /// </summary>

tests/SerializationTest.cs

@@ -44,6 +44,16 @@ public class TestClassTwo
             public EnumTestType? ThirdColumn;
         }
 
+        public class TestClassThree
+        {
+            public string Name { get; set; }
+            public string[] StringArray { get; set; }
+            public List<int> IntList { get; set; }
+            public IEnumerable<bool> BoolEnumerable { get; set; }
+            public List<Guid> GuidList { get; set; }
+            public List<Guid> NullableList { get; set; }
+        }
+
         [Test]
         public void TestObjectSerialization()
         {
@@ -125,6 +135,57 @@ public void TestNullSerialization()
             }
         }
 
+        /// <summary>
+        /// Arrays and child objects aren't well suited for complex serialization within a CSV file.
+        /// However, we have options:
+        /// * ToString just converts it to "MyClass[]"
+        /// * CountItems just produces the number of elements in the array
+        /// </summary>
+        [Test]
+        public void TestArraySerialization()
+        {
+            var list = new List<TestClassThree>();
+            list.Add(new TestClassThree()
+            {
+                Name = "Test",
+                StringArray = new [] { "a", "b", "c"},
+                IntList = new List<int> { 1, 2, 3 },
+                BoolEnumerable = new [] { true, false, true, false },
+                GuidList = new List<Guid>(),
+            });
+
+            // Serialize to a CSV string using ToString
+            // This was the default behavior in CSVFile 3.1.2 and earlier - it's pretty ugly!
+            var options = new CSVSettings()
+            {
+                HeaderRowIncluded = true,
+                NestedArrayBehavior = ArrayOptions.ToString,
+                NullToken = "NULL",
+                AllowNull = true,
+            };
+            var toStringCsv = CSV.Serialize(list, options);
+            Assert.AreEqual($"Name,StringArray,IntList,BoolEnumerable,GuidList,NullableList{Environment.NewLine}"
+                + $"Test,System.String[],System.Collections.Generic.List`1[System.Int32],System.Boolean[],System.Collections.Generic.List`1[System.Guid],NULL{Environment.NewLine}", toStringCsv);
+
+            // Serialize to a CSV string using counts
+            options.NestedArrayBehavior = ArrayOptions.CountItems;
+            var countItemsCsv = CSV.Serialize(list, options);
+            Assert.AreEqual($"Name,StringArray,IntList,BoolEnumerable,GuidList,NullableList{Environment.NewLine}"
+                + $"Test,3,3,4,0,NULL{Environment.NewLine}", countItemsCsv);
+
+            // Serialize to a CSV string using counts
+            options.NestedArrayBehavior = ArrayOptions.TreatAsNull;
+            var ignoreArraysCsv = CSV.Serialize(list, options);
+            Assert.AreEqual($"Name,StringArray,IntList,BoolEnumerable,GuidList,NullableList{Environment.NewLine}"
+                + $"Test,NULL,NULL,NULL,NULL,NULL{Environment.NewLine}", ignoreArraysCsv);
+
+            // And now for the magic: Recursive serialization!
+            options.NestedArrayBehavior = ArrayOptions.RecursiveSerialization;
+            var recursiveCsv = CSV.Serialize(list, options);
+            Assert.AreEqual($"Name,StringArray,IntList,BoolEnumerable,GuidList,NullableList{Environment.NewLine}"
+                            + $"Test,\"a,b,c\",\"1,2,3\",\"True,False,True,False\",,NULL{Environment.NewLine}", recursiveCsv);
+        }
+
         [Test]
         public void TestCaseInsensitiveDeserializer()
         {