Fixed serializer problem with the last value of each row

Author: LucaMozzo

CSVSerializer.NET.CS/Deserializer.cs

@@ -1,22 +1,15 @@
 using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 using System.IO;
 
 namespace CSVSerializer
 {
     public class Deserializer
     {
-        private List<String> Headers;
-        private List<Row> Rows;
         private StreamReader File;
 
         public Deserializer(String FilePath)
         {
-            File = new StreamReader(FilePath);
-
+                File = new StreamReader(FilePath); //may throw an exception if the file is not found, to be handled by the final user
         }
         public Deserializer(byte[] buffer)
         {
@@ -32,43 +25,53 @@ public Document Deserialize()
                 Row row = new Row();
                 char[] line = File.ReadLine().ToCharArray();
                 char[] buffer = new char[line.Length]; //temporary buffer for composing values
+                int bufferIndex = 0;
                 for (int i = 0; i < line.Length; i++)
                 {
-                    int bufferIndex = 0;
                     if (line[i] == '\"')
                     {
-                        int closingQuoteIndex = IndexOf(line, '\"', i + 1);
+                        int closingQuoteIndex = IndexOf(line, '\"', i + 1); //copies all the stuff between the double quotes in the buffer
                         for (int a = i + 1; a < closingQuoteIndex; a++)
                         {
                             buffer[bufferIndex] = line[a];
                             ++bufferIndex;
                         }
-                        i = closingQuoteIndex;
+                        i = closingQuoteIndex; //the index is now the closing double quote, next loop it will be the ,
                     }           
-                    else if (line[i] == ',')
+                    else if (line[i] == ',') //when it sees a comma, it dumps the content of the buffer in the row
                     {
-                        String tmp = "";
-                        foreach (char c in buffer)
-                            tmp += c;
-                        Value value = new Value(tmp);
-                        row.AddValue(value);
+                        DumpBuffer(row, buffer);
+                        buffer = new char[line.Length]; //clears the array after dumping values
+                        bufferIndex = 0;
                     }
                     else
                     {
                         buffer[bufferIndex] = line[i];
                         ++bufferIndex;
                     }
                 }
-                if (index == 0)
+                DumpBuffer(row, buffer); //the last column must be added
+                if (index == 0) //the first row is the header
                     doc.SetHeader(row);
                 else
                     doc.AddRow(row);
                 ++index;
             }
             return doc;
         }
+        //for a better reusability I created this method to avoid redundancy
+        private static void DumpBuffer(Row row, char[] buffer)
+        {
+            String tmp = "";
+            foreach (char c in buffer)
+                if (c != '\0')
+                    tmp += c;
+                else break; //otherwise keeps looping through the empty slot of the buffer
+            Value value = new Value(tmp);
+            row.AddValue(value);
+        }
 
-        //returns the index of the first occurrence of the spacified element
+        //returns the index of the first occurrence of the specified element
         private int IndexOf(char[] array, char element, int startingIndex = 0)
         {
             for(int i = startingIndex; i < array.Length; i++)