Combine two checks using TryGetValue() for efficiency

MaggieKimani1 · MaggieKimani1 · commit 9b9113028a37 · 2023-08-22T19:37:53.000+03:00
diff --git a/src/Microsoft.OpenApi.Readers/V2/OpenApiResponseDeserializer.cs b/src/Microsoft.OpenApi.Readers/V2/OpenApiResponseDeserializer.cs
@@ -92,12 +92,12 @@ private static void ProcessProduces(MapNode mapNode, OpenApiResponse response, P
                 foreach (var produce in produces)
                 {
 
-                    if (response.Content.ContainsKey(produce) && response.Content[produce] != null)
+                    if (response.Content.TryGetValue(produce, out var produceValue)) 
                     {
                         if (schema != null)
                         {
-                            response.Content[produce].Schema = schema;
-                            ProcessAnyFields(mapNode, response.Content[produce], _mediaTypeAnyFields);
+                            produceValue.Schema = schema;
+                            ProcessAnyFields(mapNode, produceValue, _mediaTypeAnyFields);
                         }
                     }
                     else

Original file line number	Diff line number	Diff line change
`@@ -92,12 +92,12 @@ private static void ProcessProduces(MapNode mapNode, OpenApiResponse response, P`
`92`	`92`	`foreach (var produce in produces)`
`93`	`93`	`{`
`94`	`94`
`95`		`- if (response.Content.ContainsKey(produce) && response.Content[produce] != null)`
	`95`	`+ if (response.Content.TryGetValue(produce, out var produceValue))`
`96`	`96`	`{`
`97`	`97`	`if (schema != null)`
`98`	`98`	`{`
`99`		`- response.Content[produce].Schema = schema;`
`100`		`- ProcessAnyFields(mapNode, response.Content[produce], _mediaTypeAnyFields);`
	`99`	`+ produceValue.Schema = schema;`
	`100`	`+ ProcessAnyFields(mapNode, produceValue, _mediaTypeAnyFields);`
`101`	`101`	`}`
`102`	`102`	`}`
`103`	`103`	`else`