azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SetterBackingFieldAssignment: minor refactoring

Browse Source
This commit is contained in:
Mikhail Glukhikh
2018-10-23 12:35:35 +03:00
parent 29cc727c5a
commit fcc6395b14
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/src/org/jetbrains/kotlin/idea/inspections/SetterBackingFieldAssignmentInspection.kt
+12 -4
View File
@@ -34,13 +34,17 @@ class SetterBackingFieldAssignmentInspection : AbstractKotlinInspection(), Clean
if (bodyExpression.anyDescendantOfType<KtExpression> {
when (it) {
is KtBinaryExpression ->
it.left?.text == KtTokens.FIELD_KEYWORD.value && it.operationToken in assignmentOperators
it.left.isBackingFieldReference(property) && it.operationToken in assignmentOperators
is KtUnaryExpression ->
it.baseExpression?.text == KtTokens.FIELD_KEYWORD.value && it.operationToken in incrementAndDecrementOperators
it.baseExpression.isBackingFieldReference(property) && it.operationToken in incrementAndDecrementOperators
is KtCallExpression ->
it.valueArguments.any { arg ->
arg.text == parameter?.text
&& arg.getArgumentExpression().getResolvedCall(accessorContext)?.resultingDescriptor == parameterDescriptor
arg.text == parameter?.text && run {
val argumentResultingDescriptor =
arg.getArgumentExpression().getResolvedCall(accessorContext)?.resultingDescriptor
argumentResultingDescriptor == parameterDescriptor
}
}
else -> false
}
@@ -54,6 +58,10 @@ class SetterBackingFieldAssignmentInspection : AbstractKotlinInspection(), Clean
)
})
}
private fun KtExpression?.isBackingFieldReference(property: KtProperty) = with(SuspiciousVarPropertyInspection) {
this@isBackingFieldReference != null && isBackingFieldReference(property)
}
}
private val assignmentOperators = listOf(KtTokens.EQ, KtTokens.PLUSEQ, KtTokens.MINUSEQ, KtTokens.MULTEQ, KtTokens.DIVEQ, KtTokens.PERCEQ)