Gilding the Rose: Refactoring Legacy Code

Slide 1

Slide 1 text

Gilding the Rose: Refactoring Legacy Code Randy Coulman Principal Software Engineer http://randycoulman.com @randycoulman randycoulman

Slide 2

Slide 2 text

Welcome to our new job! Enhance the inventory management system for the Gilded Rose Inn

Slide 3

Slide 3 text

The System

Slide 4

Slide 4 text

The Spec Can we trust it?

Slide 5

Slide 5 text

Once the sell-by date has passed, quality degrades twice as fast The quality of an item is never negative "Aged Brie" actually increases in quality the older it gets The quality of an item is never more than 50 "Sulfuras", being a legendary item, never has to be sold or decreases in quality "Backstage passes", like aged brie, increases in quality as its sell-by date approaches; quality increases by 2 when there are 10 days or less and by 3 when there are 5 days or less, but quality drops to 0 after the concert.

Slide 6

Slide 6 text

Constraint The I t e m class and @ i t e m s property are off limits - no collective code ownership

Slide 7

Slide 7 text

Our Task Support Conjured items Conjured items degrade in quality twice as fast as normal items.

Slide 8

Slide 8 text

The Code

Slide 9

Slide 9 text

c l a s s I t e m a t t r _ a c c e s s o r : n a m e , : s e l l _ i n , : q u a l i t y d e f i n i t i a l i z e ( n a m e , s e l l _ i n , q u a l i t y ) @ n a m e = n a m e @ s e l l _ i n = s e l l _ i n @ q u a l i t y = q u a l i t y e n d e n d

Slide 10

Slide 10 text

r e q u i r e ' . / i t e m . r b ' c l a s s G i l d e d R o s e @ i t e m s = [ ] d e f i n i t i a l i z e @ i t e m s = [ ] @ i t e m s < < I t e m . n e w ( " + 5 D e x t e r i t y V e s t " , 1 0 , 2 0 ) @ i t e m s < < I t e m . n e w ( " A g e d B r i e " , 2 , 0 ) @ i t e m s < < I t e m . n e w ( " E l i x i r o f t h e M o n g o o s e " , 5 , 7 ) @ i t e m s < < I t e m . n e w ( " S u l f u r a s , H a n d o f R a g n a r o s " , 0 , 8 0 ) @ i t e m s < < I t e m . n e w ( " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " , 1 5 , 2 0 ) @ i t e m s < < I t e m . n e w ( " C o n j u r e d M a n a C a k e " , 3 , 6 ) e n d d e f u p d a t e _ q u a l i t y # . . . e n d e n d

Slide 11

Slide 11 text

http://www.kategreene.net/do-not-look-at-laser-with-remaining-eye/

Slide 12

Slide 12 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i f ( @ i t e m s [ i ] . n a m e ! = " A g e d B r i e " & & @ i t e m s [ i ] . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( @ i t e m s [ i ] . q u a l i t y > 0 ) i f ( @ i t e m s [ i ] . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y - 1 e n d e n d e l s e i f ( @ i t e m s [ i ] . q u a l i t y < 5 0 ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y + 1 i f ( @ i t e m s [ i ] . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( @ i t e m s [ i ] . s e l l _ i n < 1 1 ) i f ( @ i t e m s [ i ] . q u a l i t y < 5 0 ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y + 1 e n d e n d i f ( @ i t e m s [ i ] . s e l l _ i n < 6 ) i f ( @ i t e m s [ i ] . q u a l i t y < 5 0 ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y + 1 e n d e n d e n d e n d e n d # . . . c o n t i n u e d . . .

Slide 13

Slide 13 text

# . . . c o n t i n u e d . . . i f ( @ i t e m s [ i ] . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) @ i t e m s [ i ] . s e l l _ i n = @ i t e m s [ i ] . s e l l _ i n - 1 ; e n d i f ( @ i t e m s [ i ] . s e l l _ i n < 0 ) i f ( @ i t e m s [ i ] . n a m e ! = " A g e d B r i e " ) i f ( @ i t e m s [ i ] . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( @ i t e m s [ i ] . q u a l i t y > 0 ) i f ( @ i t e m s [ i ] . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y - 1 e n d e n d e l s e @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y - @ i t e m s [ i ] . q u a l i t y e n d e l s e i f ( @ i t e m s [ i ] . q u a l i t y < 5 0 ) @ i t e m s [ i ] . q u a l i t y = @ i t e m s [ i ] . q u a l i t y + 1 e n d e n d e n d e n d e n d

Slide 14

Slide 14 text

The Tests

Slide 15

Slide 15 text

. . .

Slide 16

Slide 16 text

https://twitter.com/staszv/status/507374076252282881/photo/1

Slide 17

Slide 17 text

Our Strategy

Slide 18

Slide 18 text

No content

Slide 19

Slide 19 text

https://www.facebook.com/pages/The-Official-Unofficial-Rocket-City-Rednecks/250397118342321?sk=notes

Slide 20

Slide 20 text

Don't trust the spec (yet)

Slide 21

Slide 21 text

Fight the urge to rewrite

Slide 22

Slide 22 text

The Boy Scout Rule "Leave the campground cleaner than you found it"

Slide 23

Slide 23 text

Baby Steps Lots of safe, tiny refactorings add up to big changes

Slide 24

Slide 24 text

Don't Boil The Ocean Stay focused!

Slide 25

Slide 25 text

For each desired change, make the change easy (warning: this may be hard), then make the easy change. --Kent Beck (https://twitter.com/KentBeck/status/250733358307500032)

Slide 26

Slide 26 text

Conjured Items Are we there yet?

Slide 27

Slide 27 text

Slide 28

Slide 28 text

When you start a new programming job you have to walk right up to the biggest function in the yard and refactor it in front of everyone. --@supermoof (https://twitter.com/supermoof/status/478322429257715712)

Slide 29

Slide 29 text

Reduce Noise

Slide 30

Slide 30 text

Slide 31

Slide 31 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i t e m = @ i t e m s [ i ] i f ( i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( i t e m . q u a l i t y > 0 ) i f ( i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) i t e m . q u a l i t y = i t e m . q u a l i t y - 1 e n d e n d e l s e i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 i f ( i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( i t e m . s e l l _ i n < 1 1 ) i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d i f ( i t e m . s e l l _ i n < 6 ) i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d e n d e n d e n d # . . . c o n t i n u e d . . .

Slide 32

Slide 32 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i t e m = @ i t e m s [ i ] # . . . 4 0 + l i n e s o f c o d e u s i n g i t e m . . . e n d e n d

Slide 33

Slide 33 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i t e m = @ i t e m s [ i ] u p d a t e ( i t e m ) e n d e n d d e f u p d a t e ( i t e m ) # . . . 4 0 + l i n e s o f c o d e u s i n g i t e m . . . e n d

Slide 34

Slide 34 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i t e m = @ i t e m s [ i ] u p d a t e ( i t e m ) e n d e n d

Slide 35

Slide 35 text

Idiomatic Code is More Readable

Slide 36

Slide 36 text

d e f u p d a t e _ q u a l i t y f o r i i n 0 . . ( @ i t e m s . s i z e - 1 ) i t e m = @ i t e m s [ i ] u p d a t e ( i t e m ) e n d e n d

Slide 37

Slide 37 text

d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | u p d a t e ( i t e m ) e n d e n d

Slide 38

Slide 38 text

d e f u p d a t e ( i t e m ) i f ( i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( i t e m . q u a l i t y > 0 ) i f ( i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) i t e m . q u a l i t y = i t e m . q u a l i t y - 1 e n d e n d e l s e i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 i f ( i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) i f ( i t e m . s e l l _ i n < 1 1 ) i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d i f ( i t e m . s e l l _ i n < 6 ) i f ( i t e m . q u a l i t y < 5 0 ) i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d e n d e n d e n d i f ( i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " ) i t e m . s e l l _ i n = i t e m . s e l l _ i n - 1 ; e n d # . . . 1 7 m o r e l i n e s . . . e n d

Slide 39

Slide 39 text

d e f u p d a t e ( i t e m ) i f i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . q u a l i t y > 0 i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . q u a l i t y = i t e m . q u a l i t y - 1 e n d e n d e l s e i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y = i t e m . q u a l i t y + 1 i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . s e l l _ i n < 1 1 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d i f i t e m . s e l l _ i n < 6 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y = i t e m . q u a l i t y + 1 e n d e n d e n d e n d e n d i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . s e l l _ i n = i t e m . s e l l _ i n - 1 ; e n d # . . . 1 7 m o r e l i n e s . . . e n d

Slide 40

Slide 40 text

Slide 41

Slide 41 text

d e f u p d a t e ( i t e m ) i f i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . q u a l i t y > 0 i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . q u a l i t y - = 1 e n d e n d e l s e i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . s e l l _ i n < 1 1 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d i f i t e m . s e l l _ i n < 6 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . s e l l _ i n - = 1 e n d # . . . 1 7 m o r e l i n e s . . . e n d

Slide 42

Slide 42 text

Conjured Items Are we there yet?

Slide 43

Slide 43 text

d e f u p d a t e ( i t e m ) i f i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . q u a l i t y > 0 i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . q u a l i t y - = 1 e n d e n d e l s e i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . s e l l _ i n < 1 1 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d i f i t e m . s e l l _ i n < 6 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . s e l l _ i n - = 1 e n d # . . . 1 7 m o r e l i n e s . . . e n d

Slide 44

Slide 44 text

Eliminate Feature Envy

Slide 45

Slide 45 text

d e f u p d a t e ( i t e m ) i f i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . q u a l i t y > 0 i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . q u a l i t y - = 1 e n d e n d e l s e i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . s e l l _ i n < 1 1 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d i f i t e m . s e l l _ i n < 6 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . s e l l _ i n - = 1 e n d # . . . c o n t i n u e d . . . e n d

Slide 46

Slide 46 text

Constraint The I t e m class and @ i t e m s property are off limits - no collective code ownership

Slide 47

Slide 47 text

c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | u p d a t e ( i t e m ) e n d e n d d e f u p d a t e ( i t e m ) # . . . 4 3 l i n e s u s i n g i t e m . . . e n d e n d

Slide 48

Slide 48 text

r e q u i r e ' d e l e g a t e ' c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r e n d c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | u p d a t e ( I t e m W r a p p e r . n e w ( i t e m ) ) e n d e n d d e f u p d a t e ( i t e m ) # . . . 4 3 l i n e s u s i n g i t e m . . . e n d e n d

Slide 49

Slide 49 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f u p d a t e i t e m = s e l f # . . . 4 3 l i n e s u s i n g i t e m . . . e n d e n d c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | u p d a t e ( I t e m W r a p p e r . n e w ( i t e m ) ) e n d e n d d e f u p d a t e ( i t e m ) i t e m . u p d a t e e n d e n d

Slide 50

Slide 50 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f u p d a t e i t e m = s e l f # . . . 4 3 l i n e s u s i n g i t e m . . . e n d e n d c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | I t e m W r a p p e r . n e w ( i t e m ) . u p d a t e e n d e n d e n d

Slide 51

Slide 51 text

d e f u p d a t e i t e m = s e l f i f i t e m . n a m e ! = " A g e d B r i e " & & i t e m . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . q u a l i t y > 0 i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . q u a l i t y - = 1 e n d e n d e l s e i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f i t e m . s e l l _ i n < 1 1 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d i f i t e m . s e l l _ i n < 6 i f i t e m . q u a l i t y < 5 0 i t e m . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f i t e m . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i t e m . s e l l _ i n - = 1 e n d # . . . c o n t i n u e d . . . e n d

Slide 52

Slide 52 text

d e f u p d a t e i f s e l f . n a m e ! = " A g e d B r i e " & & s e l f . n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l f . q u a l i t y > 0 i f s e l f . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . q u a l i t y - = 1 e n d e n d e l s e i f s e l f . q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 i f s e l f . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l f . s e l l _ i n < 1 1 i f s e l f . q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d i f s e l f . s e l l _ i n < 6 i f s e l f . q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f s e l f . n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d # . . . c o n t i n u e d . . . e n d

Slide 53

Slide 53 text

d e f u p d a t e i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f q u a l i t y > 0 i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . q u a l i t y - = 1 e n d e n d e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d i f s e l l _ i n < 6 i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d e n d e n d e n d i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d # . . . c o n t i n u e d . . . e n d

Slide 54

Slide 54 text

Conjured Items Are we there yet?

Slide 55

Slide 55 text

Slide 56

Slide 56 text

Remove Duplication

Slide 57

Slide 57 text

d e f u p d a t e i f # . . . # . . . e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 i f # . . . i f # . . . i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d i f # . . . i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d e n d e n d e n d # . . . i f # . . . i f # . . . # . . . e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d e n d e n d

Slide 58

Slide 58 text

d e f u p d a t e i f # . . . # . . . e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 i f # . . . i f # . . . i n c r e a s e _ q u a l i t y e n d i f # . . . i n c r e a s e _ q u a l i t y e n d e n d e n d e n d # . . . i f # . . . i f # . . . # . . . e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d

Slide 59

Slide 59 text

d e f u p d a t e i f # . . . # . . . e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 6 i n c r e a s e _ q u a l i t y e n d e n d e n d e n d # . . . e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d

Slide 60

Slide 60 text

d e f u p d a t e i f # . . . # . . . e l s e i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 6 i n c r e a s e _ q u a l i t y e n d e n d e n d # . . . e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d

Slide 61

Slide 61 text

d e f u p d a t e i f # . . . # . . . e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 6 i n c r e a s e _ q u a l i t y e n d e n d e n d # . . . e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d

Slide 62

Slide 62 text

d e f u p d a t e i f # . . . i f q u a l i t y > 0 i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . q u a l i t y - = 1 e n d e n d e l s e # . . . e n d # . . . i f # . . . i f # . . . i f # . . . i f q u a l i t y > 0 i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . q u a l i t y - = 1 e n d e n d e l s e # . . . e n d e l s e # . . . e n d e n d e n d

Slide 63

Slide 63 text

d e f u p d a t e i f # . . . i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i f q u a l i t y > 0 s e l f . q u a l i t y - = 1 e n d e n d e l s e # . . . e n d # . . . i f # . . . i f # . . . i f # . . . i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " i f q u a l i t y > 0 s e l f . q u a l i t y - = 1 e n d e n d e l s e # . . . e n d e l s e # . . . e n d e n d e n d

Slide 64

Slide 64 text

d e f u p d a t e i f # . . . i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e # . . . e n d # . . . i f # . . . i f # . . . i f # . . . i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e # . . . e n d e l s e # . . . e n d e n d e n d d e f d e c r e a s e _ q u a l i t y i f q u a l i t y > 0 s e l f . q u a l i t y - = 1 e n d e n d

Slide 65

Slide 65 text

Conjured Items Are we there yet?

Slide 66

Slide 66 text

d e f u p d a t e i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 6 i n c r e a s e _ q u a l i t y e n d e n d e n d i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d i f s e l l _ i n < 0 i f n a m e ! = " A g e d B r i e " i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 67

Slide 67 text

Group Similar Tasks

Slide 68

Slide 68 text

Slide 69

Slide 69 text

d e f u p d a t e i f # . . . # . . . e l s e # . . . i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 1 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 6 i n c r e a s e _ q u a l i t y e n d e n d e n d i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d i f s e l l _ i n < 0 # . . . e n d e n d

Slide 70

Slide 70 text

d e f u p d a t e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d i f # . . . # . . . e l s e # . . . i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d i f s e l l _ i n < 0 # . . . e n d e n d

Slide 71

Slide 71 text

d e f u p d a t e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d i f s e l l _ i n < 0 i f n a m e ! = " A g e d B r i e " i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 72

Slide 72 text

d e f u p d a t e a g e i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d i f s e l l _ i n < 0 i f n a m e ! = " A g e d B r i e " i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d d e f a g e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d e n d

Slide 73

Slide 73 text

d e f u p d a t e a g e u p d a t e _ q u a l i t y e n d d e f u p d a t e _ q u a l i t y i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d i f s e l l _ i n < 0 i f n a m e ! = " A g e d B r i e " i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 74

Slide 74 text

Conjured Items Are we there yet?

Slide 75

Slide 75 text

d e f u p d a t e _ q u a l i t y i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d i f s e l l _ i n < 0 i f n a m e ! = " A g e d B r i e " i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 76

Slide 76 text

Simplify Conditionals

Slide 77

Slide 77 text

Slide 78

Slide 78 text

i f n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e s e l f . q u a l i t y - = q u a l i t y e n d

Slide 79

Slide 79 text

i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d

Slide 80

Slide 80 text

i f n a m e ! = " A g e d B r i e " i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d e l s e i n c r e a s e _ q u a l i t y e n d

Slide 81

Slide 81 text

i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s e i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 82

Slide 82 text

i f n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e l s e i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 83

Slide 83 text

DeMorgan's Laws not (A and B) ==> (not A) or (not B) not (A or B) ==> (not A) and (not B)

Slide 84

Slide 84 text

n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t "

Slide 85

Slide 85 text

n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " # N e g a t e : ! ( n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " )

Slide 86

Slide 86 text

n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " # N e g a t e : ! ( n a m e ! = " A g e d B r i e " & & n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " ) # D e M o r g a n : n o t ( A a n d B ) = = > ( n o t A ) o r ( n o t B ) ! ( n a m e ! = " A g e d B r i e " ) | | ! ( n a m e ! = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " )

Slide 87

Slide 87 text

Slide 88

Slide 88 text

Slide 89

Slide 89 text

i f n a m e = = " A g e d B r i e " | | n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e l s e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d

Slide 90

Slide 90 text

d e f u p d a t e _ q u a l i t y i f # . . . # . . . e l s e i f # . . . # . . . e n d e n d i f # . . . i f # . . . # . . . e l s e i f # . . . # . . . e l s e i f # . . . # . . . e n d e n d e n d e n d e n d

Slide 91

Slide 91 text

d e f u p d a t e _ q u a l i t y i f # . . . # . . . e l s i f # . . . # . . . e n d i f # . . . i f # . . . # . . . e l s i f # . . . # . . . e l s i f # . . . # . . . e n d e n d e n d

Slide 92

Slide 92 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " | | n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e l s i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 93

Slide 93 text

d e f u p d a t e _ q u a l i t y i f # . . . # . . . e l s i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d i f # . . . i f # . . . # . . . e l s i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 94

Slide 94 text

d e f u p d a t e _ q u a l i t y r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " i f # . . . # . . . e l s e d e c r e a s e _ q u a l i t y e n d i f # . . . i f # . . . # . . . e l s e d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 95

Slide 95 text

d e f u p d a t e a g e u p d a t e _ q u a l i t y e n d d e f a g e i f n a m e ! = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d e n d d e f u p d a t e _ q u a l i t y r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " # . . . 2 2 l i n e s . . . e n d

Slide 96

Slide 96 text

d e f u p d a t e a g e u p d a t e _ q u a l i t y e n d d e f a g e r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " s e l f . s e l l _ i n - = 1 e n d d e f u p d a t e _ q u a l i t y r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " # . . . 2 2 l i n e s . . . e n d

Slide 97

Slide 97 text

d e f u p d a t e r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " a g e u p d a t e _ q u a l i t y e n d d e f a g e s e l f . s e l l _ i n - = 1 e n d d e f u p d a t e _ q u a l i t y # . . . 2 2 l i n e s . . . e n d

Slide 98

Slide 98 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " | | n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e n d e l s e d e c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s e d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 99

Slide 99 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e l s e d e c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e l s e d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 100

Slide 100 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d e l s e d e c r e a s e _ q u a l i t y i f s e l l _ i n < 0 d e c r e a s e _ q u a l i t y e n d e n d i f s e l l _ i n < 0 i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y - = q u a l i t y e n d e n d e n d

Slide 101

Slide 101 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s e d e c r e a s e _ q u a l i t y i f s e l l _ i n < 0 d e c r e a s e _ q u a l i t y e n d e n d i f s e l l _ i n < 0 i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y e n d e n d e n d

Slide 102

Slide 102 text

Slide 103

Slide 103 text

Conjured Items Are we there yet?

Slide 104

Slide 104 text

Slide 105

Slide 105 text

Yes!

Slide 106

Slide 106 text

Make the Easy Change

Slide 107

Slide 107 text

Slide 108

Slide 108 text

Our Task Support Conjured items Conjured items degrade in quality twice as fast as normal items.

Slide 109

Slide 109 text

Slide 110

Slide 110 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 0 i n c r e a s e _ q u a l i t y e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " d e c r e a s e _ q u a l i t y d e c r e a s e _ q u a l i t y i f s e l l _ i n < 0 d e c r e a s e _ q u a l i t y d e c r e a s e _ q u a l i t y e n d e l s e d e c r e a s e _ q u a l i t y i f s e l l _ i n < 0 d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 111

Slide 111 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 0 i n c r e a s e _ q u a l i t y e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " s e l f . q u a l i t y - = 2 # F a i l s t e s t s - w e c a n ' t d o t h i s i f s e l l _ i n < 0 s e l f . q u a l i t y - = 2 # F a i l s t e s t s - w e c a n ' t d o t h i s e n d e l s e d e c r e a s e _ q u a l i t y i f s e l l _ i n < 0 d e c r e a s e _ q u a l i t y e n d e n d e n d

Slide 112

Slide 112 text

Slide 113

Slide 113 text

Express Important Ideas

Slide 114

Slide 114 text

c l a s s I t e m W r a p p e r d e f d e c r e a s e _ q u a l i t y i f q u a l i t y > 0 s e l f . q u a l i t y - = 1 e n d e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d e n d

Slide 115

Slide 115 text

The Spec ... The quality of an item is never negative ... The quality of an item is never more than 50 ...

Slide 116

Slide 116 text

Slide 117

Slide 117 text

c l a s s I t e m W r a p p e r d e f d e c r e a s e _ q u a l i t y i f q u a l i t y > 0 s e l f . q u a l i t y - = 1 e n d e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d d e f q u a l i t y = ( n e w _ q u a l i t y ) s u p e r e n d e n d

Slide 118

Slide 118 text

c l a s s I t e m W r a p p e r d e f d e c r e a s e _ q u a l i t y s e l f . q u a l i t y - = 1 e n d d e f i n c r e a s e _ q u a l i t y i f q u a l i t y < 5 0 s e l f . q u a l i t y + = 1 e n d e n d d e f q u a l i t y = ( n e w _ q u a l i t y ) n e w _ q u a l i t y = 0 i f n e w _ q u a l i t y < 0 s u p e r ( n e w _ q u a l i t y ) e n d e n d

Slide 119

Slide 119 text

c l a s s I t e m W r a p p e r d e f d e c r e a s e _ q u a l i t y s e l f . q u a l i t y - = 1 e n d d e f i n c r e a s e _ q u a l i t y s e l f . q u a l i t y + = 1 e n d d e f q u a l i t y = ( n e w _ q u a l i t y ) n e w _ q u a l i t y = 0 i f n e w _ q u a l i t y < 0 n e w _ q u a l i t y = 5 0 i f n e w _ q u a l i t y > 5 0 s u p e r ( n e w _ q u a l i t y ) e n d e n d

Slide 120

Slide 120 text

Slide 121

Slide 121 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 0 i n c r e a s e _ q u a l i t y e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " i n c r e a s e _ q u a l i t y i f s e l l _ i n < 1 0 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 5 i n c r e a s e _ q u a l i t y e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " s e l f . q u a l i t y - = 1 s e l f . q u a l i t y - = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 1 s e l f . q u a l i t y - = 1 e n d e l s e s e l f . q u a l i t y - = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 1 e n d e n d e n d d e f i n c r e a s e _ q u a l i t y s e l f . q u a l i t y + = 1 e n d

Slide 122

Slide 122 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " s e l f . q u a l i t y + = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y + = 1 e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y + = 1 i f s e l l _ i n < 1 0 s e l f . q u a l i t y + = 1 e n d i f s e l l _ i n < 5 s e l f . q u a l i t y + = 1 e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " s e l f . q u a l i t y - = 1 s e l f . q u a l i t y - = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 1 s e l f . q u a l i t y - = 1 e n d e l s e s e l f . q u a l i t y - = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 1 e n d e n d e n d

Slide 123

Slide 123 text

d e f u p d a t e _ q u a l i t y i f n a m e = = " A g e d B r i e " s e l f . q u a l i t y + = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y + = 1 e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " s e l f . q u a l i t y + = 1 i f s e l l _ i n < 1 0 s e l f . q u a l i t y + = 1 e n d i f s e l l _ i n < 5 s e l f . q u a l i t y + = 1 e n d i f s e l l _ i n < 0 s e l f . q u a l i t y - = q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " s e l f . q u a l i t y - = 2 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 2 e n d e l s e s e l f . q u a l i t y - = 1 i f s e l l _ i n < 0 s e l f . q u a l i t y - = 1 e n d e n d e n d

Slide 124

Slide 124 text

d e f u p d a t e _ q u a l i t y s e l f . q u a l i t y + = q u a l i t y _ a d j u s t m e n t e n d d e f q u a l i t y _ a d j u s t m e n t i f n a m e = = " A g e d B r i e " a d j u s t m e n t = 1 i f s e l l _ i n < 0 a d j u s t m e n t = 2 e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " a d j u s t m e n t = 1 i f s e l l _ i n < 1 0 a d j u s t m e n t = 2 e n d i f s e l l _ i n < 5 a d j u s t m e n t = 3 e n d i f s e l l _ i n < 0 a d j u s t m e n t = - q u a l i t y e n d e l s i f n a m e = = " C o n j u r e d M a n a C a k e " a d j u s t m e n t = - 2 i f s e l l _ i n < 0 a d j u s t m e n t = - 4 e n d e l s e a d j u s t m e n t = - 1 i f s e l l _ i n < 0 a d j u s t m e n t = - 2 e n d e n d a d j u s t m e n t e n d

Slide 125

Slide 125 text

Replace Conditional With Polymorphism

Slide 126

Slide 126 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r # . . . e n d c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | I t e m W r a p p e r . n e w ( i t e m ) . u p d a t e e n d e n d e n d

Slide 127

Slide 127 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) n e w ( i t e m ) e n d # . . . e n d c l a s s G i l d e d R o s e d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | I t e m W r a p p e r . w r a p ( i t e m ) . u p d a t e e n d e n d e n d

Slide 128

Slide 128 text

c l a s s I t e m W r a p p e r d e f s e l f . w r a p ( i t e m ) n e w ( i t e m ) e n d d e f q u a l i t y _ a d j u s t m e n t i f n a m e = = " A g e d B r i e " a d j u s t m e n t = 1 i f s e l l _ i n < 0 a d j u s t m e n t = 2 e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " # . . . e l s i f n a m e = = " C o n j u r e d M a n a C a k e " # . . . e l s e # . . . e n d a d j u s t m e n t e n d e n d

Slide 129

Slide 129 text

c l a s s I t e m W r a p p e r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f q u a l i t y _ a d j u s t m e n t i f n a m e = = " A g e d B r i e " a d j u s t m e n t = 1 i f s e l l _ i n < 0 a d j u s t m e n t = 2 e n d e l s i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " # . . . e l s i f n a m e = = " C o n j u r e d M a n a C a k e " # . . . e l s e # . . . e n d a d j u s t m e n t e n d e n d c l a s s A g e d B r i e < I t e m W r a p p e r e n d

Slide 130

Slide 130 text

c l a s s I t e m W r a p p e r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f q u a l i t y _ a d j u s t m e n t i f n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " # . . . e l s i f n a m e = = " C o n j u r e d M a n a C a k e " # . . . e l s e # . . . e n d a d j u s t m e n t e n d e n d c l a s s A g e d B r i e < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 0 a d j u s t m e n t = 2 e n d a d j u s t m e n t e n d e n d

Slide 131

Slide 131 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f q u a l i t y _ a d j u s t m e n t i f n a m e = = " C o n j u r e d M a n a C a k e " # . . . e l s e # . . . e n d a d j u s t m e n t e n d e n d c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 1 0 a d j u s t m e n t = 2 e n d i f s e l l _ i n < 5 a d j u s t m e n t = 3 e n d i f s e l l _ i n < 0 a d j u s t m e n t = - q u a l i t y e n d a d j u s t m e n t e n d e n d

Slide 132

Slide 132 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s i f i t e m . n a m e = = " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f q u a l i t y _ a d j u s t m e n t # . . . a d j u s t m e n t e n d e n d c l a s s C o n j u r e d I t e m < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = - 2 i f s e l l _ i n < 0 a d j u s t m e n t = - 4 e n d a d j u s t m e n t e n d e n d

Slide 133

Slide 133 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s i f i t e m . n a m e = = " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f u p d a t e r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " a g e u p d a t e _ q u a l i t y e n d e n d

Slide 134

Slide 134 text

The Spec ... "Sulfuras", being a legendary item, never has to be sold or decreases in quality

Slide 135

Slide 135 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s i f i t e m . n a m e = = " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) e l s i f i t e m . n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " L e g e n d a r y I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f u p d a t e r e t u r n i f n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " a g e u p d a t e _ q u a l i t y e n d e n d c l a s s L e g e n d a r y I t e m < I t e m W r a p p e r e n d

Slide 136

Slide 136 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s i f i t e m . n a m e = = " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) e l s i f i t e m . n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " L e g e n d a r y I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d d e f u p d a t e a g e u p d a t e _ q u a l i t y e n d e n d c l a s s L e g e n d a r y I t e m < I t e m W r a p p e r d e f u p d a t e # N o t h i n g t o d o - L e g e n d a r y i t e m s d o n ' t c h a n g e e n d e n d

Slide 137

Slide 137 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) i f i t e m . n a m e = = " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) e l s i f i t e m . n a m e = = " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) e l s i f i t e m . n a m e = = " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) e l s i f i t e m . n a m e = = " S u l f u r a s , H a n d o f R a g n a r o s " L e g e n d a r y I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d e n d

Slide 138

Slide 138 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f s e l f . w r a p ( i t e m ) c a s e i t e m . n a m e w h e n " A g e d B r i e " A g e d B r i e . n e w ( i t e m ) w h e n " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " B a c k s t a g e P a s s . n e w ( i t e m ) w h e n " C o n j u r e d M a n a C a k e " C o n j u r e d I t e m . n e w ( i t e m ) w h e n " S u l f u r a s , H a n d o f R a g n a r o s " L e g e n d a r y I t e m . n e w ( i t e m ) e l s e n e w ( i t e m ) e n d e n d e n d

Slide 139

Slide 139 text

Express Domain Concepts

Slide 140

Slide 140 text

The Spec Once the sell-by date has passed, quality degrades twice as fast

Slide 141

Slide 141 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = - 1 i f s e l l _ i n < 0 a d j u s t m e n t = - 2 e n d a d j u s t m e n t e n d e n d

Slide 142

Slide 142 text

c l a s s I t e m W r a p p e r < S i m p l e D e l e g a t o r d e f q u a l i t y _ a d j u s t m e n t i f s e l l _ i n < 0 p a s t _ d a t e _ a d j u s t m e n t e l s e n o r m a l _ a d j u s t m e n t e n d e n d d e f n o r m a l _ a d j u s t m e n t - 1 e n d d e f p a s t _ d a t e _ a d j u s t m e n t 2 * n o r m a l _ a d j u s t m e n t e n d e n d

Slide 143

Slide 143 text

The Spec ... "Aged Brie" actually increases in quality the older it gets

Slide 144

Slide 144 text

c l a s s A g e d B r i e < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 0 a d j u s t m e n t = 2 e n d a d j u s t m e n t e n d e n d

Slide 145

Slide 145 text

c l a s s A g e d B r i e < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t - s u p e r e n d e n d

Slide 146

Slide 146 text

The Spec ... Conjured items degrade in quality twice as fast as normal items.

Slide 147

Slide 147 text

c l a s s C o n j u r e d I t e m < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = - 2 i f s e l l _ i n < 0 a d j u s t m e n t = - 4 e n d a d j u s t m e n t e n d e n d

Slide 148

Slide 148 text

c l a s s C o n j u r e d I t e m < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t 2 * s u p e r e n d e n d

Slide 149

Slide 149 text

The Spec ... "Backstage passes" ... quality drops to 0 after the concert.

Slide 150

Slide 150 text

c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f q u a l i t y _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 1 0 a d j u s t m e n t = 2 e n d i f s e l l _ i n < 5 a d j u s t m e n t = 3 e n d i f s e l l _ i n < 0 a d j u s t m e n t = - q u a l i t y e n d a d j u s t m e n t e n d e n d

Slide 151

Slide 151 text

c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 1 0 a d j u s t m e n t = 2 e n d i f s e l l _ i n < 5 a d j u s t m e n t = 3 e n d a d j u s t m e n t e n d d e f p a s t _ d a t e _ a d j u s t m e n t - q u a l i t y e n d e n d

Slide 152

Slide 152 text

c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t a d j u s t m e n t = 1 i f s e l l _ i n < 1 0 a d j u s t m e n t = 2 e n d i f s e l l _ i n < 5 a d j u s t m e n t = 3 e n d a d j u s t m e n t e n d e n d

Slide 153

Slide 153 text

c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t i f s e l l _ i n < 5 3 e l s i f s e l l _ i n < 1 0 2 e l s e 1 e n d e n d e n d

Slide 154

Slide 154 text

The Spec ... "Backstage passes" ... quality increases by 2 when there are 10 days or less and by 3 when there are 5 days or less, ...

Slide 155

Slide 155 text

c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t i f s e l l _ i n < 5 3 e l s i f s e l l _ i n < 1 0 2 e l s e 1 e n d e n d e n d

Slide 156

Slide 156 text

Don't Blindly Fix Bugs Sometimes people or other systems rely on the current behavior.

Slide 157

Slide 157 text

Final Code

Slide 158

Slide 158 text

Slide 159

Slide 159 text

Slide 160

Slide 160 text

Flog 179.5: flog total 155.1: GildedRose#update_quality

Slide 161

Slide 161 text

80 Micro-Refactorings Later...

Slide 162

Slide 162 text

r e q u i r e ' . / i t e m . r b ' r e q u i r e ' d e l e g a t e ' c l a s s G i l d e d R o s e @ i t e m s = [ ] d e f i n i t i a l i z e @ i t e m s = [ ] @ i t e m s < < I t e m . n e w ( " + 5 D e x t e r i t y V e s t " , 1 0 , 2 0 ) @ i t e m s < < I t e m . n e w ( " A g e d B r i e " , 2 , 0 ) @ i t e m s < < I t e m . n e w ( " E l i x i r o f t h e M o n g o o s e " , 5 , 7 ) @ i t e m s < < I t e m . n e w ( " S u l f u r a s , H a n d o f R a g n a r o s " , 0 , 8 0 ) @ i t e m s < < I t e m . n e w ( " B a c k s t a g e p a s s e s t o a T A F K A L 8 0 E T C c o n c e r t " , 1 5 , 2 0 ) @ i t e m s < < I t e m . n e w ( " C o n j u r e d M a n a C a k e " , 3 , 6 ) e n d d e f u p d a t e _ q u a l i t y @ i t e m s . e a c h d o | i t e m | I t e m W r a p p e r . w r a p ( i t e m ) . u p d a t e e n d e n d e n d

Slide 163

Slide 163 text

Slide 164

Slide 164 text

# . . . c o n t i n u e d . . . d e f u p d a t e a g e u p d a t e _ q u a l i t y e n d d e f a g e s e l f . s e l l _ i n - = 1 e n d # . . . c o n t i n u e d . . .

Slide 165

Slide 165 text

# . . . c o n t i n u e d . . . d e f u p d a t e _ q u a l i t y s e l f . q u a l i t y + = q u a l i t y _ a d j u s t m e n t e n d d e f q u a l i t y _ a d j u s t m e n t i f s e l l _ i n < 0 p a s t _ d a t e _ a d j u s t m e n t e l s e n o r m a l _ a d j u s t m e n t e n d e n d d e f n o r m a l _ a d j u s t m e n t - 1 e n d d e f p a s t _ d a t e _ a d j u s t m e n t 2 * n o r m a l _ a d j u s t m e n t e n d d e f q u a l i t y = ( n e w _ q u a l i t y ) n e w _ q u a l i t y = 0 i f n e w _ q u a l i t y < 0 n e w _ q u a l i t y = 5 0 i f n e w _ q u a l i t y > 5 0 s u p e r ( n e w _ q u a l i t y ) e n d e n d

Slide 166

Slide 166 text

c l a s s A g e d B r i e < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t - s u p e r e n d e n d c l a s s B a c k s t a g e P a s s < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t i f s e l l _ i n < 5 3 e l s i f s e l l _ i n < 1 0 2 e l s e 1 e n d e n d d e f p a s t _ d a t e _ a d j u s t m e n t - q u a l i t y e n d e n d c l a s s C o n j u r e d I t e m < I t e m W r a p p e r d e f n o r m a l _ a d j u s t m e n t 2 * s u p e r e n d e n d c l a s s L e g e n d a r y I t e m < I t e m W r a p p e r d e f u p d a t e # N o t h i n g t o d o - L e g e n d a r y i t e m s d o n ' t c h a n g e e n d e n d

Slide 167

Slide 167 text

Flog 67.1: flog total 18.2: GildedRose#initialize 9.5: ItemWrapper::wrap

Slide 168

Slide 168 text

Takeaways

Slide 169

Slide 169 text

Slow and Steady Wins the Race

Slide 170

Slide 170 text

Make the Change Easy, Then Make the Easy Change

Slide 171

Slide 171 text

Make Simple Mechanical Changes First

Slide 172

Slide 172 text

Reduce Noise

Slide 173

Slide 173 text

Remove Duplication

Slide 174

Slide 174 text

Express Domain Concepts and Important Ideas

Slide 175

Slide 175 text

Don't Mindlessly Fix Bugs

Slide 176

Slide 176 text

https://github.com/randycoulman/GildedRose

Slide 177

Slide 177 text

Acknowledgements Key Technology (http://www.key.net/) Zeal (@codingzeal) Kent Beck (@kentbeck)

Slide 178

Slide 178 text

References Gilded Rose Kata Working Effectively With Legacy Code - Michael Feathers Refactoring - Martin Fowler

Slide 179

Slide 179 text

Questions?

Slide 180

Slide 180 text

Randy Coulman http://speakerrate.com/randycoulman https://speakerdeck.com/randycoulman http://randycoulman.com @randycoulman randycoulman