rspec/rules/S1117/swift/rule.adoc

19 lines
494 B
Plaintext
Raw Normal View History

:message: Rename "${elementName}" which has the same name as the (field|case) declared at line {}.
== Why is this an issue?
2020-06-30 12:47:33 +02:00
Shadowing fields or `enum case`s with a local variable is a bad practice that reduces code readability: It makes it confusing to know whether the field or the variable is being used.
2020-06-30 12:47:33 +02:00
2022-02-04 17:28:24 +01:00
[source,swift]
2020-06-30 12:47:33 +02:00
----
public class Foo {
public var myField:Int = 0
public func doSomething() {
var myField = 0 /// Noncompliant
2020-06-30 12:47:33 +02:00
...
}
}
----
include::../rspecator.adoc[]