reduce the amount of memory allocations (#134)

DAXGRID · Aug 12, 2024 · ff7df55 · ff7df55
1 parent 18357ed
commit ff7df55
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/OpenFTTH.APIGateway/Reporting/CustomerTerminationReport.cs b/OpenFTTH.APIGateway/Reporting/CustomerTerminationReport.cs
@@ -327,16 +327,13 @@ private List<string> GetCsvLinesFromTraceResult(List<InstallationTraceResultLine
             if (traces.Count == 0)
                 return new List<string>() { "no installations found" };
 
-            var resultCsvLines = new List<string>();
+            var resultCsvLines = new List<string>(traces.Count);
 
-            var firstObject = traces.First();
-
-            var myType = firstObject.GetType();
-            var props = new List<PropertyInfo>(myType.GetProperties().Reverse());
+            var myType = traces.First().GetType();
 
             var csvHeader = "";
 
-            foreach (PropertyInfo prop in props)
+            foreach (PropertyInfo prop in myType.GetProperties().Reverse())
             {
                 if (csvHeader != "")
                     csvHeader += ";";
@@ -352,7 +349,7 @@ private List<string> GetCsvLinesFromTraceResult(List<InstallationTraceResultLine
 
                 bool first = true;
 
-                foreach (PropertyInfo prop in props)
+                foreach (PropertyInfo prop in myType.GetProperties().Reverse())
                 {
                     if (!first)
                         csvLine += ";";