HighFile-scopeC# Plug & play

Ensure `StringBuilder` Data Is Used

Ensure that StringBuilder instances are converted to strings using ToString() before use to prevent memory wastage.

Why this matters

Creating a StringBuilder without calling ToString() results in unused objects, wasting memory and reducing performance.

Bad vs. good

Side-by-side examples engineers can pattern-match during review.

Avoid

public void DoSomething(List<string> strings) {
  var sb = new StringBuilder();  // Noncompliant
  sb.Append("Got: ");
  foreach(var str in strings) {
    sb.Append(str).Append(", ");
    // ...
  }
}

Prefer

public void DoSomething(List<string> strings) {
  foreach(var str in strings) {
    // ...
  }
}

Related rules

From the same buckets as this rule.

Critical
Avoid Duplicating String Literals
Store repeated string literals in constants or variables to improve maintainability and reduce redundancy.
duplication-complexitymaintainability
Critical
Avoid Empty Methods
Detect methods that have an empty body. If a method is empty, ensure that a comment is present explaining why it remains unimplemented.
maintainabilityreadability-refactor
Critical
Avoid modifying built-in prototypes
Detect modifications to built-in prototypes. Modifying built-in objects can lead to compatibility issues and unexpected behavior. Recommend creating helper functions instead.
maintainabilitysecurity-hardening

Why this matters

Bad vs. good

Related rules

Avoid Duplicating String Literals

Avoid Empty Methods

Avoid modifying built-in prototypes

Why this matters

Bad vs. good

Related rules

Avoid Duplicating String Literals

Avoid Empty Methods

Avoid modifying built-in prototypes