gitlabEdit

chart-mixedText - NKS0047

Use TextTools.SplitNewLines()

This analyzer provides the following strings:

Context
String

Error List

Caller uses .Split("\n") instead of TextTools.SplitNewLines()

Suggestion Box

Use TextTools.SplitNewLines() instead of .Split("\n")

Description

TextTools.SplitNewLines() simplifies the readability of the split by new lines function.


Extended Description

This code analyzer detects the usage of Split("\n") from the String class found in the System namespace.

Splitting by new lines is a good way to get the lines from your string. However, the complexity needs to be reduced so that more readability is achieved. As a result, SplitNewLines() is here to do it in the simplest way possible.


Analysis Comparison

To get a brief insight about how this analyzer works, compare the two code blocks shown to you below:

Before the fix

Somewhere in your mod code...
public static void MyFunction()
{
    string var = "Hello\nWorld!";
    var split = var.Split("\n");
}

After the fix

After the fix (alternate)

Last updated