From fbf4d0e41922fdb26f92f35039f3f4032656f725 Mon Sep 17 00:00:00 2001 From: Artyom Date: Tue, 15 Dec 2015 10:51:04 +0200 Subject: [PATCH] Fixed ArgumentOutOfRangeException If the list is empty you get the ArgumentOutOfRangeException when calling flattenString.Remove(flattenString.Length - 1). Also using a StringBuilder is the prefered way to concatenate strings in a loop. --- .../src/main/resources/csharp/ApiClient.mustache | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/modules/swagger-codegen/src/main/resources/csharp/ApiClient.mustache b/modules/swagger-codegen/src/main/resources/csharp/ApiClient.mustache index 707ce9eb560..1ab3c939e4c 100644 --- a/modules/swagger-codegen/src/main/resources/csharp/ApiClient.mustache +++ b/modules/swagger-codegen/src/main/resources/csharp/ApiClient.mustache @@ -163,13 +163,14 @@ namespace {{packageName}}.Client return ((DateTime)obj).ToString (Configuration.DateTimeFormat); else if (obj is IList) { - string flattenString = ""; - string separator = ","; + var flattenedString = new StringBuilder(); foreach (var param in (IList)obj) { - flattenString += param.ToString() + separator; + if (flattenedString.Length > 0) + flattenedString.Append(","); + flattenedString.Append(param); } - return flattenString.Remove(flattenString.Length - 1);; + return flattenedString.ToString(); } else return Convert.ToString (obj);