Tweet
Tweet

Slide 1

Slide 1 text

What if Clippy Would Criticize Your Code? Yuriy Tymchuk R A E E L V

Slide 2

Slide 2 text

@yuriy_tymchuk What if Clippy Would Criticize Your Code? R A E E L V Sometimes I just popup for no reason at all. Like now.

Slide 3

Slide 3 text

@yuriy_tymchuk What if Clippy Would Criticize Your Code? R A E E L V Criticize Your Code?

Slide 4

Slide 4 text

Code Quality

Slide 5

Slide 5 text

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)

Slide 6

Slide 6 text

Code Review A. Bacchelli, C. Bird. Expectations, outcomes, and challenges of modern code review. In Proceedings of ICSE’13, pp. 712–721, IEEE, 2013

Slide 7

Slide 7 text

Static Analyzer

Slide 8

Slide 8 text

Static Analyzer

Slide 9

Slide 9 text

Quality Rules Static Analyzer

Slide 10

Slide 10 text

Quality Rules Source Code Static Analyzer

Slide 11

Slide 11 text

Quality Rules Critics Static Analyzer

Slide 12

Slide 12 text

Find Bugs

Slide 13

Slide 13 text

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

Slide 14

Slide 14 text

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

Slide 15

Slide 15 text

No content

Slide 16

Slide 16 text

packages 400 classes 5.5k methods 90k

Slide 17

Slide 17 text

Pharo 4: patches / months ≈ patches/day 1726 12 5 Pharo 5: patches / months ≈ patches/day 1560 7.5 7 packages 400 classes 5.5k methods 90k

Slide 18

Slide 18 text

ViDI is …

Slide 19

Slide 19 text

Disconnected from Development

Slide 20

Slide 20 text

Yes, I will help you
 to write code!

Slide 21

Slide 21 text

Static Analyzer Interface FindBugs IntelliJ Code Analysis

Slide 22

Slide 22 text

Static Analyzer Interface FindBugs IntelliJ Code Analysis standalone/on demand embedded/intrusive

Slide 23

Slide 23 text

Static Analyzer Interface Critics Browser standalone/on demand embedded/intrusive in Pharo

Slide 24

Slide 24 text

Critics Browser

Slide 25

Slide 25 text

Critics Browser Survey 46 participants

Slide 26

Slide 26 text

Critics Browser Survey

Slide 27

Slide 27 text

Critics Browser Survey

Slide 28

Slide 28 text

QualityAssistant

Slide 29

Slide 29 text

QualityAssistant in Code Browser

Slide 30

Slide 30 text

QualityAssistant in Code Browser

Slide 31

Slide 31 text

QualityAssistant in Inspector

Slide 32

Slide 32 text

QualityAssistant in Spotter

Slide 33

Slide 33 text

Pharo 4: patches / months ≈ patches/day 1726 12 5 Pharo 5: patches / months ≈ patches/day 1560 7.5 7 packages 400 classes 5.5k methods 90k

Slide 34

Slide 34 text

Pharo 4: patches / months ≈ patches/day 1726 12 5 Pharo 5: patches / months ≈ patches/day 1560 7.5 7 packages 400 classes 5.5k methods 90k

Slide 35

Slide 35 text

QualityAssistant Survey 29 participants

Slide 36

Slide 36 text

QualityAssistant Survey

Slide 37

Slide 37 text

Browser Inspector Spotter QualityAssistant Survey

Slide 38

Slide 38 text

QualityAssistant Survey

Slide 39

Slide 39 text

Critics Browser QualityAssistant

Slide 40

Slide 40 text

Critics Browser QualityAssistant

Slide 41

Slide 41 text

Critics Browser QualityAssistant

Slide 42

Slide 42 text

Critics Browser QualityAssistant

Slide 43

Slide 43 text

Critics Browser QualityAssistant

Slide 44

Slide 44 text

Critics Browser QualityAssistant auto"x ban
 ignore ban
 ignore rationale highlight diff auto"x ban ban in
 scope

Slide 45

Slide 45 text

No content

Slide 46

Slide 46 text

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

Slide 47

Slide 47 text

Developers Like QualityAssistant

Slide 48

Slide 48 text

Developers Like QualityAssistant 38% 24% 14% 3% 3% 17% “!nd QualityAssistant very useful”

Slide 49

Slide 49 text

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

Slide 50

Slide 50 text

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

Slide 51

Slide 51 text

@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