Go
Incorrect 'strings.Replace' count argument
Warning
Performance
New
Last modified: 03 December 2024 Reports strings.Replace
calls with the replace count 0
instead of -1
.
The count argument for how many times a string should be replaced should not be 0
as it will not replace anything and make the function call redundant. Use -1
instead.
Example:
a := strings.Replace("SSS", "S", "H", 0) // replaces nothing
b := strings.Replace("SSS", "S", "H", -1) // replaces all S occurrences with H
- By ID
Can be used to locate inspection in e.g. Qodana configuration files, where you can quickly enable or disable it, or adjust its settings.
GoStringsReplaceCount
Inspection Details | |
---|---|
By default bundled with: | |
Can be installed with plugin: | Go, 243.23131 |
Thanks for your feedback!
Was this page helpful?