From 89900d4b1c174cbc4f610670b7e0e766fef56038 Mon Sep 17 00:00:00 2001 From: JeremyCaney Date: Wed, 8 Dec 2021 12:53:29 -0800 Subject: [PATCH] CA1847: Prefer `Contains(char)` when evaluating a single character --- OnTopic.AspNetCore.Mvc/TopicViewResultExecutor.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/OnTopic.AspNetCore.Mvc/TopicViewResultExecutor.cs b/OnTopic.AspNetCore.Mvc/TopicViewResultExecutor.cs index 5e259ac2..dc2be3e0 100644 --- a/OnTopic.AspNetCore.Mvc/TopicViewResultExecutor.cs +++ b/OnTopic.AspNetCore.Mvc/TopicViewResultExecutor.cs @@ -114,10 +114,10 @@ public ViewEngineResult FindView(ActionContext actionContext, TopicViewResult vi if (!(view?.Success ?? false) && requestContext.Headers.ContainsKey("Accept")) { foreach (var header in requestContext.Headers["Accept"]) { var value = header.Replace("+", "-", StringComparison.Ordinal); - if (value.Contains("/", StringComparison.Ordinal)) { + if (value.Contains('/', StringComparison.Ordinal)) { value = value[(value.IndexOf("/", StringComparison.Ordinal)+1)..]; } - if (value.Contains(";", StringComparison.Ordinal)) { + if (value.Contains(';', StringComparison.Ordinal)) { value = value[..(value.IndexOf(";", StringComparison.Ordinal))]; } if (value is not null) {