Prefer Contains() over IndexOf() >= 0

JeremyCaney · JeremyCaney · commit ebb917cfd51b · 2020-11-24T13:30:18.000-08:00
diff --git a/OnTopic.Editor.AspNetCore/Components/FilePathViewComponent.cs b/OnTopic.Editor.AspNetCore/Components/FilePathViewComponent.cs
@@ -226,7 +226,7 @@ public string GetPath(string attributeKey, FilePathAttributeTopicViewModel optio
       /*------------------------------------------------------------------------------------------------------------------------
       | Replace path slashes with backslashes if the resulting file path value uses a UNC or basic file path format
       \-----------------------------------------------------------------------------------------------------------------------*/
-      if (filePath.IndexOf("\\", StringComparison.InvariantCulture) >= 0) {
+      if (filePath.Contains("\\", StringComparison.InvariantCulture)) {
         filePath                        = filePath.Replace("/", "\\");
       }
 

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ public string GetPath(string attributeKey, FilePathAttributeTopicViewModel optio`
`226`	`226`	`/*------------------------------------------------------------------------------------------------------------------------`
`227`	`227`	`\| Replace path slashes with backslashes if the resulting file path value uses a UNC or basic file path format`
`228`	`228`	`\-----------------------------------------------------------------------------------------------------------------------*/`
`229`		`- if (filePath.IndexOf("\\", StringComparison.InvariantCulture) >= 0) {`
	`229`	`+ if (filePath.Contains("\\", StringComparison.InvariantCulture)) {`
`230`	`230`	`filePath = filePath.Replace("/", "\\");`
`231`	`231`	`}`
`232`	`232`