What if Clippy Would Criticize Your Code? (benevol2015)
1 like498 views
The document discusses the concept of a code review assistant, inspired by Clippy, that critiques code quality focusing on maintainability and ease of understanding based on various standards such as ISO/IEC 9126. It highlights research on modern code reviews and introduces tools like static analyzers and visual design inspections that can help identify bugs and improve code quality. Additionally, surveys indicate that developers find such quality assistants useful in influencing positive changes in coding practices.
What if Clippy Would Criticize Your Code? (benevol2015)
- 1. What if Clippy Would Criticize Your Code? Yuriy TymchukR AE E LV
- 2. @yuriy_tymchuk What if Clippy Would Criticize Your Code? R AE E LV Sometimes I just popup for no reason at all. Like now.
- 3. @yuriy_tymchuk What if Clippy Would Criticize Your Code? R AE E LV Criticize Your Code?
- 4. Code Quality
- 5. Code Quality How easy it is to understand modify test the software ISO/IEC, ISO/IEC 9126. Software Engineering – Product quality 6.5. ISO/IEC, 2001 (maintainability)
- 6. Code Review A. Bacchelli, C. Bird. Expectations, outcomes, and challenges of modern code review. In Proceedings of ICSE’13, pp. 712–721, IEEE, 2013
- 10. Quality Rules Source Code Static Analyzer
- 12. Find Bugs
- 13. 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
- 14. 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
- 17. Pharo 4: patches / months ≈ patches/day1726 12 5 Pharo 5: patches / months ≈ patches/day1560 7.5 7 packages400 classes5.5k methods90k
- 18. ViDI is …
- 20. Yes, I will help you to write code!
- 21. Static Analyzer Interface FindBugs IntelliJ Code Analysis
- 22. Static Analyzer Interface FindBugs IntelliJ Code Analysis standalone/on demand embedded/intrusive
- 23. Static Analyzer Interface Critics Browser standalone/on demand embedded/intrusive in Pharo
- 24. Critics Browser
- 25. Critics Browser Survey 46 participants
- 28. QualityAssistant
- 29. QualityAssistant in Code Browser
- 30. QualityAssistant in Code Browser
- 33. Pharo 4: patches / months ≈ patches/day1726 12 5 Pharo 5: patches / months ≈ patches/day1560 7.5 7 packages400 classes5.5k methods90k
- 34. Pharo 4: patches / months ≈ patches/day1726 12 5 Pharo 5: patches / months ≈ patches/day1560 7.5 7 packages400 classes5.5k methods90k
- 37. Browser Inspector Spotter QualityAssistant Survey
- 46. 31% 10% 31% 7% 3% 17% 48% 14% 21% 17% Don’t know QA Negative Slightly Negative Neutral Slightly Positive Positive removal of “Missing yourself” addition of “notNilDo: -> notNil:” QualityAssistant-In!uenced Changes
- 48. Developers Like QualityAssistant 38% 24% 14% 3% 3% 17% “!nd QualityAssistant very useful”
- 49. 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
- 50. 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
- 51. @yuriy_tymchuk 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