@yuriy_tymchuk
What if Clippy Would Criticize Your Code?
R AE E LV
Sometimes I just popup for no reason at all. Like now.
Code Quality
How easy it is tounderstandmodifytest
the software
ISO/IEC, ISO/IEC 9126. Software Engineering – Product quality 6.5. ISO/IEC, 2001
(maintainability)
Code Review
A. Bacchelli, C. Bird. Expectations, outcomes, and challenges of modern code review. In Proceedings of ICSE’13, pp. 712–721, IEEE, 2013
Visual Design Inspection
Y. Tymchuk, A. Mocci, and M. Lanza. Code Review: Veni, ViDI, Vici. In Proceedings of SANER’15, pp. 151-160, IEEE, 2015
Y. Tymchuk, A. Mocci, and M. Lanza. Code Review: Veni, ViDI, Vici. In Proceedings of SANER’15, pp. 151-160, IEEE, 2015 Y. Tymchuk, A. Mocci, and M. Lanza. Vidi: The Visual Design Inspector. In Proceedings of ICSE’15, to be published, IEEE, 2015
ViDI
Pharo 4: patches / months ≈ patches/day1726 12 5Pharo 5: patches / months ≈ patches/day1560 7.5 7
packages 400 classes 5.5k methods 90k
Pharo 4: patches / months ≈ patches/day1726 12 5Pharo 5: patches / months ≈ patches/day1560 7.5 7
packages 400 classes 5.5k methods 90k
Pharo 4: patches / months ≈ patches/day1726 12 5Pharo 5: patches / months ≈ patches/day1560 7.5 7
packages 400 classes 5.5k methods 90k
Critics Browser QualityAssistantau
to"x
ban
igno
re
ban
igno
rera
tiona
le
highli
ght
diff
auto"x
ban
ban i
n s
cope
31%
10%31%
7%3%
17%
48%
14%
21%
17% Don’t know QANegativeSlightly NegativeNeutralSlightly PositivePositive
removal of “Missing yourself”
addition of “notNilDo: -> notNil:”
QualityAssistant-In!uenced Changes
38%
24%
14%
3%3%
17%
48%
14%
21%
17%
“!nd QualityAssistant very useful”
“!nd introduction of a new rule in"uenced by QualityAssistant to be positive”
Developers Like QualityAssistant
38%
24%
14%
3%3%
17%
48%
14%
21%
17%
“!nd QualityAssistant very useful”
“!nd introduction of a new rule in"uenced by QualityAssistant to be positive”
Developers Like QualityAssistant