Merge branch 'patch-2' of github.com:lechu445/CsvHelper into lechu445…

…-patch-2
JoshClose · Oct 5, 2022 · 62dd759 · 62dd759
2 parents e05b843 + 1525c4a
commit 62dd759
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/CsvHelper/CsvReader.cs b/src/CsvHelper/CsvReader.cs
@@ -185,7 +185,7 @@ protected virtual void ValidateHeader(ClassMap map, List<InvalidHeader> invalidH
 				}
 				else
 				{
-					var index = GetFieldIndex(parameter.Data.Names.ToArray(), parameter.Data.NameIndex, true);
+					var index = GetFieldIndex(parameter.Data.Names, parameter.Data.NameIndex, true);
 					var isValid = index != -1 || parameter.Data.IsOptional;
 					if (!isValid)
 					{
@@ -1210,6 +1210,11 @@ public virtual int GetFieldIndex(string name, int index = 0, bool isTryGet = fal
 
 		/// <inheritdoc/>
 		public virtual int GetFieldIndex(string[] names, int index = 0, bool isTryGet = false, bool isOptional = false)
+		{
+			return GetFieldIndex((IEnumerable<string>)names, index, isTryGet, isOptional);
+		}
+
+		internal int GetFieldIndex(IEnumerable<string> names, int index = 0, bool isTryGet = false, bool isOptional = false)
 		{
 			if (names == null)
 			{
@@ -1236,9 +1241,9 @@ public virtual int GetFieldIndex(string[] names, int index = 0, bool isTryGet =
 
 			// Check all possible names for this field.
 			string name = null;
-			for (var i = 0; i < names.Length; i++)
+			int i = 0;
+			foreach (var n in names)
 			{
-				var n = names[i];
 				// Get the list of indexes for this name.
 				var args = new PrepareHeaderForMatchArgs(n, i);
 				var fieldName = prepareHeaderForMatch(args);
@@ -1247,6 +1252,7 @@ public virtual int GetFieldIndex(string[] names, int index = 0, bool isTryGet =
 					name = fieldName;
 					break;
 				}
+				i++;
 			}
 
 			// Check if the index position exists.
@@ -1255,7 +1261,7 @@ public virtual int GetFieldIndex(string[] names, int index = 0, bool isTryGet =
 				// It doesn't exist. The field is missing.
 				if (!isTryGet && !isOptional)
 				{
-					var args = new MissingFieldFoundArgs(names, index, context);
+					var args = new MissingFieldFoundArgs(names as string[] ?? names.ToArray(), index, context);
 					missingFieldFound?.Invoke(args);
 				}
 

diff --git a/src/CsvHelper/Expressions/ExpressionManager.cs b/src/CsvHelper/Expressions/ExpressionManager.cs
@@ -104,7 +104,7 @@ public virtual void CreateConstructorArgumentExpressionsForMapping(ClassMap map,
 					if (reader.Configuration.HasHeaderRecord && (parameterMap.Data.IsNameSet || !parameterMap.Data.IsIndexSet))
 					{
 						// Use name.
-						index = reader.GetFieldIndex(parameterMap.Data.Names.ToArray(), parameterMap.Data.NameIndex, parameterMap.Data.IsOptional);
+						index = reader.GetFieldIndex(parameterMap.Data.Names, parameterMap.Data.NameIndex, parameterMap.Data.IsOptional);
 						if (index == -1)
 						{
 							if (parameterMap.Data.IsDefaultSet || parameterMap.Data.IsOptional)
@@ -228,7 +228,7 @@ public virtual void CreateMemberAssignmentsForMapping(ClassMap mapping, List<Mem
 			if (reader.Configuration.HasHeaderRecord && (memberMap.Data.IsNameSet || !memberMap.Data.IsIndexSet))
 			{
 				// Use the name.
-				index = reader.GetFieldIndex(memberMap.Data.Names.ToArray(), memberMap.Data.NameIndex, memberMap.Data.IsOptional);
+				index = reader.GetFieldIndex(memberMap.Data.Names, memberMap.Data.NameIndex, memberMap.Data.IsOptional);
 				if (index == -1)
 				{
 					if (memberMap.Data.IsDefaultSet)