rspec/rules/S1317/java/rule.adoc

43 lines
1019 B
Plaintext
Raw Normal View History

== Why is this an issue?
2021-04-28 16:49:39 +02:00
Instantiating a ``++StringBuilder++`` or a ``++StringBuffer++`` with a character is misleading because most Java developers would expect the character to be the initial value of the ``++StringBuffer++``.
What actually happens is that the int representation of the character is used to determine the initial size of the ``++StringBuffer++``.
=== Noncompliant code example
2021-04-28 16:49:39 +02:00
2022-02-04 17:28:24 +01:00
[source,java]
2021-04-28 16:49:39 +02:00
----
StringBuffer foo = new StringBuffer('x'); //equivalent to StringBuffer foo = new StringBuffer(120);
----
=== Compliant solution
2021-04-28 16:49:39 +02:00
2022-02-04 17:28:24 +01:00
[source,java]
2021-04-28 16:49:39 +02:00
----
StringBuffer foo = new StringBuffer("x");
----
ifdef::env-github,rspecator-view[]
'''
== Implementation Specification
(visible only on this page)
=== Message
Replace the constructor character parameter 'X' with string parameter "X".
'''
== Comments And Links
(visible only on this page)
=== on 15 Oct 2013, 08:54:16 Freddy Mallet wrote:
Is implemented by \http://jira.codehaus.org/browse/SONARJAVA-359
endif::env-github,rspecator-view[]