Slide 1

Slide 1 text

@sandimetz Mar 2014 sandi metz all the little things Friday, March 7, 14

Slide 2

Slide 2 text

@sandimetz Mar 2014 Want better apps? Friday, March 7, 14

Slide 3

Slide 3 text

@sandimetz Mar 2014 make smaller things Friday, March 7, 14

Slide 4

Slide 4 text

@sandimetz Mar 2014 The Gilded Rose Kata Friday, March 7, 14

Slide 5

Slide 5 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        #  ...    end end Friday, March 7, 14

Slide 6

Slide 6 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        #  ...    end end Friday, March 7, 14

Slide 7

Slide 7 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        #  ...    end end Friday, March 7, 14

Slide 8

Slide 8 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        #  ...    end end Friday, March 7, 14

Slide 9

Slide 9 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        #  21  more  lines    def  tick        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 10

Slide 10 text

@sandimetz Mar 2014    def  tick        #  22  previous  lines        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end    def  tick        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 11

Slide 11 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Friday, March 7, 14

Slide 12

Slide 12 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 44.7 0 25 50 big conditional Complexity (Flog) Friday, March 7, 14

Slide 13

Slide 13 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 44.7 0 25 50 big conditional Complexity (Flog) 50 Friday, March 7, 14

Slide 14

Slide 14 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 44.7 0 25 50 big conditional Complexity (Flog) 45 tick 50 Friday, March 7, 14

Slide 15

Slide 15 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Friday, March 7, 14

Slide 16

Slide 16 text

@sandimetz Mar 2014 Friday, March 7, 14

Slide 17

Slide 17 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Friday, March 7, 14

Slide 18

Slide 18 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Squint Test Friday, March 7, 14

Slide 19

Slide 19 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end changes in shape Friday, March 7, 14

Slide 20

Slide 20 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 2 Friday, March 7, 14

Slide 21

Slide 21 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end changes in color Friday, March 7, 14

Slide 22

Slide 22 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Friday, March 7, 14

Slide 23

Slide 23 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end Friday, March 7, 14

Slide 24

Slide 24 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 16 if statements Friday, March 7, 14

Slide 25

Slide 25 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 16 if statements 7 != Friday, March 7, 14

Slide 26

Slide 26 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 16 if statements 7 != 2 != with && Friday, March 7, 14

Slide 27

Slide 27 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 16 if statements 7 != 2 != with && 3 magic strings Friday, March 7, 14

Slide 28

Slide 28 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end 16 if statements 7 != 2 != with && 3 magic strings ? magic numbers Friday, March 7, 14

Slide 29

Slide 29 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end magic strings Friday, March 7, 14

Slide 30

Slide 30 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end ‘Aged  Brie’ magic strings Friday, March 7, 14

Slide 31

Slide 31 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end ‘Aged  Brie’ ‘Sulfuras,  Hand  of  Ragnaros’ magic strings Friday, March 7, 14

Slide 32

Slide 32 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end ‘Aged  Brie’ ‘Sulfuras,  Hand  of  Ragnaros’ ‘Backstage  passes    to  a  TAFKAL80ETC  concert’ magic strings Friday, March 7, 14

Slide 33

Slide 33 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end it has tests Friday, March 7, 14

Slide 34

Slide 34 text

@sandimetz Mar 2014 #  Running: S..S.S...S..........S....S..... Fabulous  run  in  0.002800s,  11071.4286  runs/s,  17857.1429  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 35

Slide 35 text

@sandimetz Mar 2014 #  Running: S..S.S...S..........S....S..... Fabulous  run  in  0.002800s,  11071.4286  runs/s,  17857.1429  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 36

Slide 36 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date Friday, March 7, 14

Slide 37

Slide 37 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date brie Friday, March 7, 14

Slide 38

Slide 38 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date brie sulfuras Friday, March 7, 14

Slide 39

Slide 39 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date brie sulfuras backstage  pass Friday, March 7, 14

Slide 40

Slide 40 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date brie sulfuras backstage  pass normal Friday, March 7, 14

Slide 41

Slide 41 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound let’s look at a test Friday, March 7, 14

Slide 42

Slide 42 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 43

Slide 43 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 44

Slide 44 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 45

Slide 45 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound name Friday, March 7, 14

Slide 46

Slide 46 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound name quality Friday, March 7, 14

Slide 47

Slide 47 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound name quality days_remaining Friday, March 7, 14

Slide 48

Slide 48 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 49

Slide 49 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 50

Slide 50 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 51

Slide 51 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 52

Slide 52 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound Friday, March 7, 14

Slide 53

Slide 53 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound reduced by 1 Friday, March 7, 14

Slide 54

Slide 54 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end Friday, March 7, 14

Slide 55

Slide 55 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end 6 skipped tests Friday, March 7, 14

Slide 56

Slide 56 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end Friday, March 7, 14

Slide 57

Slide 57 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end Friday, March 7, 14

Slide 58

Slide 58 text

@sandimetz Mar 2014 Friday, March 7, 14

Slide 59

Slide 59 text

@sandimetz Mar 2014 Task: Friday, March 7, 14

Slide 60

Slide 60 text

@sandimetz Mar 2014 Task: Implement Conjured Friday, March 7, 14

Slide 61

Slide 61 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end Friday, March 7, 14

Slide 62

Slide 62 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end I tried Friday, March 7, 14

Slide 63

Slide 63 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end but failed Friday, March 7, 14

Slide 64

Slide 64 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_conjured_item_before_sell_date    def  test_conjured_item_at_zero_quality    def  test_conjured_item_on_sell_date    def  test_conjured_item_on_sell_date_at_zero_quality    def  test_conjured_item_after_sell_date    def  test_conjured_item_after_sell_date_at_zero_quality end miserably Friday, March 7, 14

Slide 65

Slide 65 text

@sandimetz Mar 2014 the pattern failed me Friday, March 7, 14

Slide 66

Slide 66 text

@sandimetz Mar 2014 Friday, March 7, 14

Slide 67

Slide 67 text

@sandimetz Mar 2014 Task: Friday, March 7, 14

Slide 68

Slide 68 text

@sandimetz Mar 2014 Task: Refactor code so I can Implement Conjured Friday, March 7, 14

Slide 69

Slide 69 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test    def  test_normal_item_before_sell_date    def  test_normal_item_on_sell_date    def  test_normal_item_after_sell_date    def  test_normal_item_of_zero_quality normal Friday, March 7, 14

Slide 70

Slide 70 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        #  ....    def  tick        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 71

Slide 71 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        #  ....    def  tick        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 72

Slide 72 text

@sandimetz Mar 2014    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end    def  tick        if  name  ==  'normal'            return        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 73

Slide 73 text

@sandimetz Mar 2014    def  tick        if  name  ==  'normal'            return        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end    def  tick        if  name  ==  'normal'            return        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end end Friday, March 7, 14

Slide 74

Slide 74 text

@sandimetz Mar 2014 #  Running: .S...S...S..SFF....F.....SS.F.. Fabulous  run  in  0.020340s,  1524.0905  runs/s,  2310.7178  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:37]: Expected:  4    Actual:  5    2)  Failure: GildedRoseTest#test_normal_item_before_sell_date  [gilded_rose_test.rb:12]: Expected:  9    Actual:  10    3)  Failure: GildedRoseTest#test_normal_item_on_sell_date  [gilded_rose_test.rb:20]: Expected:  8    Actual:  10    4)  Failure: GildedRoseTest#test_normal_item_after_sell_date  [gilded_rose_test.rb:28]: Expected:  8    Actual:  10 31  runs,  47  assertions,  4  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 75

Slide 75 text

@sandimetz Mar 2014 #  Running: .S...S...S..SFF....F.....SS.F.. Fabulous  run  in  0.020340s,  1524.0905  runs/s,  2310.7178  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:37]: Expected:  4    Actual:  5    2)  Failure: GildedRoseTest#test_normal_item_before_sell_date  [gilded_rose_test.rb:12]: Expected:  9    Actual:  10    3)  Failure: GildedRoseTest#test_normal_item_on_sell_date  [gilded_rose_test.rb:20]: Expected:  8    Actual:  10    4)  Failure: GildedRoseTest#test_normal_item_after_sell_date  [gilded_rose_test.rb:28]: Expected:  8    Actual:  10 31  runs,  47  assertions,  4  failures,  0  errors,  6  skips 4 failures Friday, March 7, 14

Slide 76

Slide 76 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end end Friday, March 7, 14

Slide 77

Slide 77 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end    def  normal_tick    end end Friday, March 7, 14

Slide 78

Slide 78 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return  normal_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end    def  normal_tick    end end Friday, March 7, 14

Slide 79

Slide 79 text

@sandimetz Mar 2014 #  Running: .S...S...S..SFF....F.....SS.F.. Fabulous  run  in  0.020340s,  1524.0905  runs/s,  2310.7178  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:37]: Expected:  4    Actual:  5    2)  Failure: GildedRoseTest#test_normal_item_before_sell_date  [gilded_rose_test.rb:12]: Expected:  9    Actual:  10    3)  Failure: GildedRoseTest#test_normal_item_on_sell_date  [gilded_rose_test.rb:20]: Expected:  8    Actual:  10    4)  Failure: GildedRoseTest#test_normal_item_after_sell_date  [gilded_rose_test.rb:28]: Expected:  8    Actual:  10 31  runs,  47  assertions,  4  failures,  0  errors,  6  skips 4 failures Friday, March 7, 14

Slide 80

Slide 80 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end Friday, March 7, 14

Slide 81

Slide 81 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick    end end Friday, March 7, 14

Slide 82

Slide 82 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick    end end Friday, March 7, 14

Slide 83

Slide 83 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 84

Slide 84 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 85

Slide 85 text

@sandimetz Mar 2014    def  test_normal_item_before_sell_date        item  =  GildedRose.new('normal',  10,  5)        item.tick        assert_equal  9,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 86

Slide 86 text

@sandimetz Mar 2014 #  Running: .....S...FS....FS.S...FS....S.. Fabulous  run  in  0.029178s,  1062.4443  runs/s,  1713.6199  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:38]: Expected:  0    Actual:  -­‐1    2)  Failure: GildedRoseTest#test_normal_item_after_sell_date  [gilded_rose_test.rb:30]: Expected:  8    Actual:  9    3)  Failure: GildedRoseTest#test_normal_item_on_sell_date  [gilded_rose_test.rb:22]: Expected:  8    Actual:  9 31  runs,  50  assertions,  3  failures,  0  errors,  6  skips 3 failures Friday, March 7, 14

Slide 87

Slide 87 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 88

Slide 88 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 89

Slide 89 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 90

Slide 90 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick            @quality  -­‐=  1        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 91

Slide 91 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        if  @days_remaining  >  0            @quality  -­‐=  1        end        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 92

Slide 92 text

@sandimetz Mar 2014    def  test_normal_item_on_sell_date        item  =  GildedRose.new('normal',  10,  0)        item.tick        assert_equal    8,  item.quality        assert_equal  -­‐1,  item.days_remaining    end class  GildedRose    #  ...    def  normal_tick        if  @days_remaining  >  0            @quality  -­‐=  1        end        if  @days_remaining  <=  0            @quality  -­‐=  2        end        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 93

Slide 93 text

@sandimetz Mar 2014 #  Running: ...S.S.....S..S..SF.....S...... Fabulous  run  in  0.019924s,  1555.9125  runs/s,  2459.3455  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:37]: Expected:  0    Actual:  -­‐1 31  runs,  49  assertions,  1  failures,  0  errors,  6  skips 1 failure ?? Friday, March 7, 14

Slide 94

Slide 94 text

@sandimetz Mar 2014 #  Running: ...S.S.....S..S..SF.....S...... Fabulous  run  in  0.019924s,  1555.9125  runs/s,  2459.3455  assertions/s.    1)  Failure: GildedRoseTest#test_normal_item_of_zero_quality  [gilded_rose_test.rb:37]: Expected:  0    Actual:  -­‐1 31  runs,  49  assertions,  1  failures,  0  errors,  6  skips 1 failure ?? Friday, March 7, 14

Slide 95

Slide 95 text

@sandimetz Mar 2014    def  test_normal_item_of_zero_quality        item  =  GildedRose.new('normal',  0,  5)        item.tick        assert_equal  0,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose      #  ...    def  normal_tick        if  @days_remaining  >  0            @quality  -­‐=  1        end        if  @days_remaining  <=  0            @quality  -­‐=  2        end        @days_remaining  -­‐=  1    end   end Friday, March 7, 14

Slide 96

Slide 96 text

@sandimetz Mar 2014    def  test_normal_item_of_zero_quality        item  =  GildedRose.new('normal',  0,  5)        item.tick        assert_equal  0,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose      #  ...    def  normal_tick        if  @days_remaining  >  0            @quality  -­‐=  1        end        if  @days_remaining  <=  0            @quality  -­‐=  2        end        @days_remaining  -­‐=  1    end   end Friday, March 7, 14

Slide 97

Slide 97 text

@sandimetz Mar 2014    def  test_normal_item_of_zero_quality        item  =  GildedRose.new('normal',  0,  5)        item.tick        assert_equal  0,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose      #  ...    def  normal_tick        if  @days_remaining  >  0            @quality  -­‐=  1        end        if  @days_remaining  <=  0            @quality  -­‐=  2        end        @days_remaining  -­‐=  1    end   end Friday, March 7, 14

Slide 98

Slide 98 text

@sandimetz Mar 2014    def  test_normal_item_of_zero_quality        item  =  GildedRose.new('normal',  0,  5)        item.tick        assert_equal  0,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose      #  ...    def  normal_tick            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        @days_remaining  -­‐=  1    end   end Friday, March 7, 14

Slide 99

Slide 99 text

@sandimetz Mar 2014    def  test_normal_item_of_zero_quality        item  =  GildedRose.new('normal',  0,  5)        item.tick        assert_equal  0,  item.quality        assert_equal  4,  item.days_remaining    end class  GildedRose      #  ...    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end        @days_remaining  -­‐=  1    end end Friday, March 7, 14

Slide 100

Slide 100 text

@sandimetz Mar 2014 #  Running: S..S.S...S..........S....S..... Fabulous  run  in  0.002800s,  11071.4286  runs/s,  17857.1429  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 101

Slide 101 text

@sandimetz Mar 2014 refactor under green Friday, March 7, 14

Slide 102

Slide 102 text

@sandimetz Mar 2014    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end        @days_remaining  -­‐=  1    end Friday, March 7, 14

Slide 103

Slide 103 text

@sandimetz Mar 2014    def  normal_tick    end    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end        @days_remaining  -­‐=  1    end Friday, March 7, 14

Slide 104

Slide 104 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1    end    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end        @days_remaining  -­‐=  1    end Friday, March 7, 14

Slide 105

Slide 105 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1    end    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end    end Friday, March 7, 14

Slide 106

Slide 106 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0    end    def  normal_tick        if  @quality  !=  0            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end        end    end Friday, March 7, 14

Slide 107

Slide 107 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0    end    def  normal_tick            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end    end Friday, March 7, 14

Slide 108

Slide 108 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1    end    def  normal_tick            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end    end Friday, March 7, 14

Slide 109

Slide 109 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  normal_tick            if  @days_remaining  >  0                @quality  -­‐=  1            end            if  @days_remaining  <=  0                @quality  -­‐=  2            end    end Friday, March 7, 14

Slide 110

Slide 110 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  normal_tick    end Friday, March 7, 14

Slide 111

Slide 111 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end Friday, March 7, 14

Slide 112

Slide 112 text

@sandimetz Mar 2014 #  Running: S..S.S...S..........S....S..... Fabulous  run  in  0.002800s,  11071.4286  runs/s,  17857.1429  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 113

Slide 113 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return  normal_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end end Reprise Friday, March 7, 14

Slide 114

Slide 114 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return  normal_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end end Friday, March 7, 14

Slide 115

Slide 115 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return  normal_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end end Friday, March 7, 14

Slide 116

Slide 116 text

@sandimetz Mar 2014 class  GildedRose    def  tick        if  name  ==  'normal'            return  normal_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    #  ...  40  more  lines    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end end normal Friday, March 7, 14

Slide 117

Slide 117 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test #  def  test_normal  ...  etc    def  test_brie_before_sell_date    def  test_brie_before_sell_date_with_max_quality    def  test_brie_on_sell_date    def  test_brie_on_sell_date_near_max_quality    def  test_brie_on_sell_date_with_max_quality    def  test_brie_after_sell_date    def  test_brie_after_sell_date_with_max_quality #    def  test_sulfuras    ...  etc #    def  test_backstage  ...  etc end brie Friday, March 7, 14

Slide 118

Slide 118 text

@sandimetz Mar 2014    def  tick      if  name  ==  'normal'            return  normal_tick        end        #  ...    end Friday, March 7, 14

Slide 119

Slide 119 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        end        #  ...    end Friday, March 7, 14

Slide 120

Slide 120 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end Friday, March 7, 14

Slide 121

Slide 121 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end    def  brie_tick    end Friday, March 7, 14

Slide 122

Slide 122 text

@sandimetz Mar 2014 #  Running: .F...F.....S.S.F..S.FF.S.F.FS.S Fabulous  run  in  0.027410s,  1130.9741  runs/s,  1678.2196  assertions/s.    1)  Failure: GildedRoseTest#test_brie_on_sell_date_with_max_quality  [gilded_rose_test.rb:78]: Expected:  4    Actual:  5    2)  Failure: GildedRoseTest#test_brie_on_sell_date_near_max_quality  [gilded_rose_test.rb:69]: Expected:  50    Actual:  49    3)  Failure: GildedRoseTest#test_brie_on_sell_date  [gilded_rose_test.rb:61]: Expected:  12    Actual:  10    4)  Failure: GildedRoseTest#test_brie_after_sell_date_with_max_quality  [gilded_rose_test.rb:94]: Expected:  -­‐11    Actual:  -­‐10    5)  Failure: GildedRoseTest#test_brie_before_sell_date_with_max_quality  [gilded_rose_test.rb:54]: Expected:  6    Actual:  7 7 failures Friday, March 7, 14

Slide 123

Slide 123 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end    def  brie_tick    end Friday, March 7, 14

Slide 124

Slide 124 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end    def  brie_tick    end fast forward... Friday, March 7, 14

Slide 125

Slide 125 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end Friday, March 7, 14

Slide 126

Slide 126 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        end        #  ...    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end brie Friday, March 7, 14

Slide 127

Slide 127 text

@sandimetz Mar 2014 #  Running: ...S......S.....SS.....SS...... Fabulous  run  in  0.003862s,  8026.9291  runs/s,  12946.6598  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 128

Slide 128 text

@sandimetz Mar 2014  def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end Similarity Friday, March 7, 14

Slide 129

Slide 129 text

@sandimetz Mar 2014  def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end Friday, March 7, 14

Slide 130

Slide 130 text

@sandimetz Mar 2014 duplication is far cheaper than the wrong abstraction Friday, March 7, 14

Slide 131

Slide 131 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test #  def  test_normal  ...  etc #  def  test_brie      ...  etc    def  test_sulfuras_before_sell_date    def  test_sulfuras_on_sell_date    def  test_sulfuras_after_sell_date #  def  test_backstage  ...  etc end sulfuras Friday, March 7, 14

Slide 132

Slide 132 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        end        #  ...    end    def  sulfuras_tick    end Friday, March 7, 14

Slide 133

Slide 133 text

@sandimetz Mar 2014 #  Running: ...S......S.....SS.....SS...... Fabulous  run  in  0.003862s,  8026.9291  runs/s,  12946.6598  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips 0 failures ?!? Friday, March 7, 14

Slide 134

Slide 134 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        end        #  ...    end    def  sulfuras_tick    end Friday, March 7, 14

Slide 135

Slide 135 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        end        #  ...    end    def  sulfuras_tick    end nothing changes Friday, March 7, 14

Slide 136

Slide 136 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        end        #  ...    end    def  sulfuras_tick    end sulfuras Friday, March 7, 14

Slide 137

Slide 137 text

@sandimetz Mar 2014 class  GildedRoseTest  <  Minitest::Test #  def  test_normal      ...  etc #  def  test_brie          ...  etc #  def  test_sulfuras  ...  etc    def  test_backstage_pass_long_before_sell_date    def  test_backstage_pass_medium_close_to_sell_date_upper_bound    def  test_backstage_pass_medium_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_medium_close_to_sell_date_lower_bound    def  test_backstage_pass_medium_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_upper_bound    def  test_backstage_pass_very_close_to_sell_date_upper_bound_at_max_quality    def  test_backstage_pass_very_close_to_sell_date_lower_bound    def  test_backstage_pass_very_close_to_sell_date_lower_bound_at_max_quality    def  test_backstage_pass_on_sell_date    def  test_backstage_pass_after_sell_date end backstage Friday, March 7, 14

Slide 138

Slide 138 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'        when  'Aged  Brie'        when  'Sulfuras,  Hand  of  Ragnaros'        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        #  ...    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end backstage Friday, March 7, 14

Slide 139

Slide 139 text

@sandimetz Mar 2014 #  Running: ...S......S.....SS.....SS...... Fabulous  run  in  0.003472s,  8928.5714  runs/s,  14400.9217  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 140

Slide 140 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 141

Slide 141 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 142

Slide 142 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 143

Slide 143 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 144

Slide 144 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 145

Slide 145 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 146

Slide 146 text

@sandimetz Mar 2014    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  c            #  ...    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick    def  sulfuras_tick    def  backstage_tick class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remai    end    def  tick        case  name        when  'normal'            return  normal_tick        when  'Aged  Brie'            return  brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            return  sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            return  backstage_tick        end        if  name  !=  'Aged  Brie'  &&  name  !=  'Backstage  passes  to  a  TAF            if  quality  >  0                if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  quality  <  50                  @quality  +=  1                if  name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  days_remaining  <  11                        if  quality  <  50                            @quality  +=  1                        end                    end                    if  days_remaining  <  6                        if  quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  days_remaining  <  0            if  name  !=  "Aged  Brie"                if  name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  quality  >  0                        if  name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  quality  -­‐  quality                end            else                if  quality  <  50                    @quality  +=  1                end            end        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Friday, March 7, 14

Slide 147

Slide 147 text

@sandimetz Mar 2014 #  Running: ...S......S.....SS.....SS...... Fabulous  run  in  0.003472s,  8928.5714  runs/s,  14400.9217  assertions/s. 31  runs,  50  assertions,  0  failures,  0  errors,  6  skips Friday, March 7, 14

Slide 148

Slide 148 text

@sandimetz Mar 2014 small methods are simple Friday, March 7, 14

Slide 149

Slide 149 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end small methods Friday, March 7, 14

Slide 150

Slide 150 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end end big conditional small methods Friday, March 7, 14

Slide 151

Slide 151 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 25 50 big conditional Total Complexity 50 Friday, March 7, 14

Slide 152

Slide 152 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 25 50 big conditional small methods tick normal brie backstage other 40 Total Complexity 50 Friday, March 7, 14

Slide 153

Slide 153 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 25 50 big conditional small methods tick normal brie backstage other 12 40 45 Total Complexity 50 Friday, March 7, 14

Slide 154

Slide 154 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end Complexity 50 big conditional Friday, March 7, 14

Slide 155

Slide 155 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 45 90 Complexity 50 big conditional Friday, March 7, 14

Slide 156

Slide 156 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 45 90 Complexity 50 84 40 small methods big conditional 10 Refactorings Friday, March 7, 14

Slide 157

Slide 157 text

@sandimetz Mar 2014 Task: Implement Conjured Friday, March 7, 14

Slide 158

Slide 158 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end Friday, March 7, 14

Slide 159

Slide 159 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick Friday, March 7, 14

Slide 160

Slide 160 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick Friday, March 7, 14

Slide 161

Slide 161 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick def  sulfuras_tick Friday, March 7, 14

Slide 162

Slide 162 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick def  sulfuras_tick def  backstage_tick Friday, March 7, 14

Slide 163

Slide 163 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick def  sulfuras_tick def  backstage_tick def  conjured_tick Friday, March 7, 14

Slide 164

Slide 164 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick def  sulfuras_tick def  backstage_tick def  conjured_tick ??? Friday, March 7, 14

Slide 165

Slide 165 text

@sandimetz Mar 2014 the pattern still fails me Friday, March 7, 14

Slide 166

Slide 166 text

@sandimetz Mar 2014 open / closed Friday, March 7, 14

Slide 167

Slide 167 text

@sandimetz Mar 2014 open for extension Friday, March 7, 14

Slide 168

Slide 168 text

@sandimetz Mar 2014 open for extension closed for modification Friday, March 7, 14

Slide 169

Slide 169 text

@sandimetz Mar 2014    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end def  normal_tick def  brie_tick def  sulfuras_tick def  backstage_tick Friday, March 7, 14

Slide 170

Slide 170 text

@sandimetz Mar 2014 they’re screaming to get out Friday, March 7, 14

Slide 171

Slide 171 text

@sandimetz Mar 2014 GildedRose Friday, March 7, 14

Slide 172

Slide 172 text

@sandimetz Mar 2014 brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 173

Slide 173 text

@sandimetz Mar 2014 brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 174

Slide 174 text

@sandimetz Mar 2014 Normal brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 175

Slide 175 text

@sandimetz Mar 2014 tick Normal brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 176

Slide 176 text

@sandimetz Mar 2014 tick Normal brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 177

Slide 177 text

@sandimetz Mar 2014 tick Normal brie_tick backstage_tick sulfuras_tick normal_tick GildedRose Friday, March 7, 14

Slide 178

Slide 178 text

@sandimetz Mar 2014 tick Normal brie_tick backstage_tick sulfuras_tick GildedRose Friday, March 7, 14

Slide 179

Slide 179 text

@sandimetz Mar 2014 tick Normal GildedRose brie_tick backstage_tick sulfuras_tick Friday, March 7, 14

Slide 180

Slide 180 text

@sandimetz Mar 2014 tick Normal GildedRose brie_tick backstage_tick sulfuras_tick Friday, March 7, 14

Slide 181

Slide 181 text

@sandimetz Mar 2014        def  normal_tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end Friday, March 7, 14

Slide 182

Slide 182 text

@sandimetz Mar 2014        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end Friday, March 7, 14

Slide 183

Slide 183 text

@sandimetz Mar 2014    class  Normal        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 184

Slide 184 text

@sandimetz Mar 2014    class  Normal        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 185

Slide 185 text

@sandimetz Mar 2014    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 186

Slide 186 text

@sandimetz Mar 2014        class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 187

Slide 187 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick    end    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 188

Slide 188 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)    end    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 189

Slide 189 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 190

Slide 190 text

@sandimetz Mar 2014 tick Normal GildedRose brie_tick backstage_tick sulfuras_tick Friday, March 7, 14

Slide 191

Slide 191 text

@sandimetz Mar 2014 normal is an object Friday, March 7, 14

Slide 192

Slide 192 text

@sandimetz Mar 2014 brie/sulfuras/backstage are not Friday, March 7, 14

Slide 193

Slide 193 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end Friday, March 7, 14

Slide 194

Slide 194 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end Friday, March 7, 14

Slide 195

Slide 195 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  quality    end    def  days_remaining    end Friday, March 7, 14

Slide 196

Slide 196 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  quality        return  item.quality  if  item    end    def  days_remaining    end Friday, March 7, 14

Slide 197

Slide 197 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  quality        return  item.quality  if  item        @quality    end    def  days_remaining    end Friday, March 7, 14

Slide 198

Slide 198 text

@sandimetz Mar 2014 class  GildedRose      def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  quality        return  item.quality  if  item        @quality    end    def  days_remaining        return  item.days_remaining  if  item        @days_remaining    end Friday, March 7, 14

Slide 199

Slide 199 text

@sandimetz Mar 2014 brie Friday, March 7, 14

Slide 200

Slide 200 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end Friday, March 7, 14

Slide 201

Slide 201 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    class  Brie    end Friday, March 7, 14

Slide 202

Slide 202 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    class  Brie        def  tick        end    end Friday, March 7, 14

Slide 203

Slide 203 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick    end    class  Brie        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 204

Slide 204 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 205

Slide 205 text

@sandimetz Mar 2014 class  GildedRose    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 206

Slide 206 text

@sandimetz Mar 2014 sulfuras Friday, March 7, 14

Slide 207

Slide 207 text

@sandimetz Mar 2014 class  GildedRose    def  sulfuras_tick    end Friday, March 7, 14

Slide 208

Slide 208 text

@sandimetz Mar 2014 class  GildedRose    def  sulfuras_tick    end    class  Sulfuras        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end Friday, March 7, 14

Slide 209

Slide 209 text

@sandimetz Mar 2014 class  GildedRose    def  sulfuras_tick    end    class  Sulfuras        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end Friday, March 7, 14

Slide 210

Slide 210 text

@sandimetz Mar 2014 class  GildedRose    def  sulfuras_tick        @item  =  Sulfuras.new(quality,  days_remaining)        item.tick    end    class  Sulfuras        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end Friday, March 7, 14

Slide 211

Slide 211 text

@sandimetz Mar 2014 backstage Friday, March 7, 14

Slide 212

Slide 212 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end Friday, March 7, 14

Slide 213

Slide 213 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end    class  Backstage        def  tick        end Friday, March 7, 14

Slide 214

Slide 214 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick    end    class  Backstage        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end Friday, March 7, 14

Slide 215

Slide 215 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick    end    class  Backstage        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end Friday, March 7, 14

Slide 216

Slide 216 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick    end    class  Backstage        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end Friday, March 7, 14

Slide 217

Slide 217 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick    end    class  Backstage        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end Friday, March 7, 14

Slide 218

Slide 218 text

@sandimetz Mar 2014 class  GildedRose    def  backstage_tick        @item  =  Backstage.new(quality,  days_remaining)        item.tick    end    class  Backstage        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end Friday, March 7, 14

Slide 219

Slide 219 text

@sandimetz Mar 2014 normal/brie/sulfuras/backstage are now objects Friday, March 7, 14

Slide 220

Slide 220 text

@sandimetz Mar 2014 GildedRose tick Normal tick Brie tick Sulfuras attr_reader  :item tick Backstage attr_reader    :quality,    :days_remaining Friday, March 7, 14

Slide 221

Slide 221 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 222

Slide 222 text

@sandimetz Mar 2014 rewind Friday, March 7, 14

Slide 223

Slide 223 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 224

Slide 224 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 225

Slide 225 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @item  =  Normal.new(quality,  days_remaining)        item.tick    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 226

Slide 226 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 227

Slide 227 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 228

Slide 228 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 229

Slide 229 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'        when  'Sulfuras,  Hand  of  Ragnaros'        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'        end    end    def  brie_tick        @item  =  Brie.new(quality,  days_remaining)        item.tick    end    #  etc Friday, March 7, 14

Slide 230

Slide 230 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 231

Slide 231 text

@sandimetz Mar 2014 abstract away the duplication Friday, March 7, 14

Slide 232

Slide 232 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 233

Slide 233 text

@sandimetz Mar 2014 class  GildedRose    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 234

Slide 234 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 235

Slide 235 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 236

Slide 236 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 237

Slide 237 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 238

Slide 238 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???.new(quality,  days_remaining)    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 239

Slide 239 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???.new(quality,  days_remaining)    end    def  tick        case  name        when  'normal'            @item  =  Normal.new(quality,  days_remaining)            item.tick        when  'Aged  Brie'            @item  =  Brie.new(quality,  days_remaining)            item.tick        when  'Sulfuras,  Hand  of  Ragnaros'            @item  =  Sulfuras.new(quality,  days_remaining)            item.tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            @item  =  Backstage.new(quality,  days_remaining)            item.tick        end    end Friday, March 7, 14

Slide 240

Slide 240 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???.new(quality,  days_remaining)    end    def        case  name        when  'normal'                            Normal        when  'Aged  Brie'                            Brie        when  'Sulfuras,  Hand  of  Ragnaros'                            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'                            Backstage        end Friday, March 7, 14

Slide 241

Slide 241 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???.new(quality,  days_remaining)    end    def        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 242

Slide 242 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  ???.new(quality,  days_remaining)    end    def  klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 243

Slide 243 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 244

Slide 244 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 245

Slide 245 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 246

Slide 246 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end Friday, March 7, 14

Slide 247

Slide 247 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end Friday, March 7, 14

Slide 248

Slide 248 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        return  item.quality  if  item        @quality    end    def  days_remaining        return  item.days_remaining  if  item        @days_remaining    end Friday, March 7, 14

Slide 249

Slide 249 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        return  item.quality  if  item        @quality    end    def  days_remaining        return  item.days_remaining  if  item        @days_remaining    end Friday, March 7, 14

Slide 250

Slide 250 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        return  item.quality  if  item    end    def  days_remaining        return  item.days_remaining  if  item    end Friday, March 7, 14

Slide 251

Slide 251 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        return  item.quality  if  item    end    def  days_remaining        return  item.days_remaining  if  item    end Friday, March 7, 14

Slide 252

Slide 252 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        return  item.quality  if  item    end    def  days_remaining        return  item.days_remaining  if  item    end Friday, March 7, 14

Slide 253

Slide 253 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end Friday, March 7, 14

Slide 254

Slide 254 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 255

Slide 255 text

@sandimetz Mar 2014 item is a role Friday, March 7, 14

Slide 256

Slide 256 text

@sandimetz Mar 2014 GildedRose tick Normal tick Brie tick Sulfuras attr_reader  :item tick Backstage attr_reader    :quality,    :days_remaining Friday, March 7, 14

Slide 257

Slide 257 text

@sandimetz Mar 2014 attr_reader  :item GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 258

Slide 258 text

@sandimetz Mar 2014 attr_reader  :item ItemRole def  tick   def  quality def  days_remaining GildedRose Friday, March 7, 14

Slide 259

Slide 259 text

@sandimetz Mar 2014 attr_reader  :item def  tick end def  quality end def  days_remaining end ItemRole def  tick   def  quality def  days_remaining GildedRose Friday, March 7, 14

Slide 260

Slide 260 text

@sandimetz Mar 2014 attr_reader  :item def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 261

Slide 261 text

@sandimetz Mar 2014 attr_reader  :item def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 262

Slide 262 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 263

Slide 263 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item attr_reader  :rose GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 264

Slide 264 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item attr_reader  :rose def  doit    rose.tick    rose.quality    rose.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 265

Slide 265 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item attr_reader  :rose def  doit    rose.tick    rose.quality    rose.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 266

Slide 266 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item attr_reader  :rose def  doit    rose.tick    rose.quality    rose.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 267

Slide 267 text

@sandimetz Mar 2014 Foo def  tick    item.tick end def  quality    item.quality end def  days_remaining    item.days_remaining end attr_reader  :item attr_reader  :rose def  doit    rose.tick    rose.quality    rose.days_remaining end GildedRose ItemRole def  tick   def  quality def  days_remaining Middleman Friday, March 7, 14

Slide 268

Slide 268 text

@sandimetz Mar 2014 Foo attr_reader  :item attr_reader  :rose GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 269

Slide 269 text

@sandimetz Mar 2014 Foo attr_reader  :item attr_reader  :rose def  getit    @rose  =          GildedRose.new(            ‘normal’,  ...)     end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 270

Slide 270 text

@sandimetz Mar 2014 Foo def  initialize(name,...)    @item  =        klass_for(name).          new(              quality,              days_remaining) end def  klass_for(name)    case  name    when  'normal'        Normal attr_reader  :item attr_reader  :rose def  getit    @rose  =          GildedRose.new(            ‘normal’,  ...)     end GildedRose ItemRole def  tick   def  quality def  days_remaining Friday, March 7, 14

Slide 271

Slide 271 text

@sandimetz Mar 2014 Foo def  initialize(name,...)    @item  =        klass_for(name).          new(              quality,              days_remaining) end def  klass_for(name)    case  name    when  'normal'        Normal attr_reader  :item attr_reader  :rose def  getit    @rose  =          GildedRose.new(            ‘normal’,  ...)     end GildedRose ItemRole def  tick   def  quality def  days_remaining Item Factory Friday, March 7, 14

Slide 272

Slide 272 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 273

Slide 273 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 274

Slide 274 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 275

Slide 275 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  initialize(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 276

Slide 276 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 277

Slide 277 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 278

Slide 278 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 279

Slide 279 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        @item  =  klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 280

Slide 280 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 281

Slide 281 text

@sandimetz Mar 2014 module  GildedRose    attr_reader  :item    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 282

Slide 282 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 283

Slide 283 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 284

Slide 284 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 285

Slide 285 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Middleman Friday, March 7, 14

Slide 286

Slide 286 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    def  tick        item.tick    end    def  quality        item.quality    end    def  days_remaining        item.days_remaining    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Middleman Friday, March 7, 14

Slide 287

Slide 287 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 288

Slide 288 text

@sandimetz Mar 2014 Foo def  self.for(name,...) end attr_reader  :rose def  getit    @rose  =          GildedRose.for(            ‘normal’,  ...)     end GildedRose Friday, March 7, 14

Slide 289

Slide 289 text

@sandimetz Mar 2014 Foo attr_reader  :rose def  getit    @rose  =          GildedRose.for(            ‘normal’,  ...)     end ItemRole def  tick   def  quality def  days_remaining def  self.for(name,...) end GildedRose Friday, March 7, 14

Slide 290

Slide 290 text

@sandimetz Mar 2014 extract common code Friday, March 7, 14

Slide 291

Slide 291 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 292

Slide 292 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        #  ...    end    class  Normal    class  Brie    class  Sulfuras    class  Backstage end Friday, March 7, 14

Slide 293

Slide 293 text

@sandimetz Mar 2014 module  GildedRose    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end Friday, March 7, 14

Slide 294

Slide 294 text

@sandimetz Mar 2014 module  GildedRose    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick          #  ...        end    end Friday, March 7, 14

Slide 295

Slide 295 text

@sandimetz Mar 2014 module  GildedRose    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick          #  ...        end    end Friday, March 7, 14

Slide 296

Slide 296 text

@sandimetz Mar 2014 module  GildedRose    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick          #  ...        end    end Friday, March 7, 14

Slide 297

Slide 297 text

@sandimetz Mar 2014 module  GildedRose    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick Friday, March 7, 14

Slide 298

Slide 298 text

@sandimetz Mar 2014 module  GildedRose    class  Item    end    class  Normal        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick Friday, March 7, 14

Slide 299

Slide 299 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick Friday, March 7, 14

Slide 300

Slide 300 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick Friday, March 7, 14

Slide 301

Slide 301 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick          #  ...        end    end Friday, March 7, 14

Slide 302

Slide 302 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick          #  ...        end    end Friday, March 7, 14

Slide 303

Slide 303 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end Friday, March 7, 14

Slide 304

Slide 304 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end    class  Backstage  <  Item        def  tick Friday, March 7, 14

Slide 305

Slide 305 text

@sandimetz Mar 2014 inheritance is not evil Friday, March 7, 14

Slide 306

Slide 306 text

@sandimetz Mar 2014 but programmers sometimes are Friday, March 7, 14

Slide 307

Slide 307 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end    class  Backstage  <  Item        def  tick Friday, March 7, 14

Slide 308

Slide 308 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end    class  Backstage  <  Item        def  tick Item    quality    days_remaining Friday, March 7, 14

Slide 309

Slide 309 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end    class  Backstage  <  Item        def  tick Item    quality    days_remaining Normal Brie Sulfuras Backstage    quality    days_remaining    tick Friday, March 7, 14

Slide 310

Slide 310 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end    class  Backstage  <  Item        def  tick Item    quality    days_remaining Normal Brie Sulfuras Backstage    quality    days_remaining    tick Friday, March 7, 14

Slide 311

Slide 311 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 312

Slide 312 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 313

Slide 313 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            #  ...        end    end    class  Brie  <  Item        def  tick          #  ...        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 314

Slide 314 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 315

Slide 315 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 316

Slide 316 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end Friday, March 7, 14

Slide 317

Slide 317 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 318

Slide 318 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Sulfuras        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 319

Slide 319 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 320

Slide 320 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    class  Sulfuras  <  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 321

Slide 321 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 322

Slide 322 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 323

Slide 323 text

@sandimetz Mar 2014 module  GildedRose    class  Item        def  tick        end    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end Friday, March 7, 14

Slide 324

Slide 324 text

@sandimetz Mar 2014 extract configuration Friday, March 7, 14

Slide 325

Slide 325 text

@sandimetz Mar 2014 module  GildedRose    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end end Friday, March 7, 14

Slide 326

Slide 326 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end end Friday, March 7, 14

Slide 327

Slide 327 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        klass_for(name).new(quality,  days_remaining)    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end end Friday, March 7, 14

Slide 328

Slide 328 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        klass_for(name).            new(quality,  days_remaining)    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end end Friday, March 7, 14

Slide 329

Slide 329 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).            new(quality,  days_remaining)    end    def  self.klass_for(name)        case  name        when  'normal'            Normal        when  'Aged  Brie'            Brie        when  'Sulfuras,  Hand  of  Ragnaros'            Item        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            Backstage        end    end end Friday, March 7, 14

Slide 330

Slide 330 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).            new(quality,  days_remaining)    end end Friday, March 7, 14

Slide 331

Slide 331 text

@sandimetz Mar 2014 small objects Friday, March 7, 14

Slide 332

Slide 332 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,      end end Friday, March 7, 14

Slide 333

Slide 333 text

@sandimetz Mar 2014 module  GildedRose    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,      end end Friday, March 7, 14

Slide 334

Slide 334 text

@sandimetz Mar 2014 module  GildedRose    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).            new(quality,  days_remaining)    end end module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,      end end Friday, March 7, 14

Slide 335

Slide 335 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remainin    end end small objects Friday, March 7, 14

Slide 336

Slide 336 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remainin    end end small objects class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        if  @name  !=  'Aged  Brie'  &&  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC            if  @quality  >  0                if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                    @quality  -­‐=  1                end            end        else            if  @quality  <  50                  @quality  +=  1                if  @name  ==  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @days_remaining  <  11                        if  @quality  <  50                            @quality  +=  1                        end                    end                    if  @days_remaining  <  6                        if  @quality  <  50                            @quality  +=  1                        end                    end                end            end        end        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'            @days_remaining  -­‐=  1        end        if  @days_remaining  <  0            if  @name  !=  "Aged  Brie"                if  @name  !=  'Backstage  passes  to  a  TAFKAL80ETC  concert'                    if  @quality  >  0                        if  @name  !=  'Sulfuras,  Hand  of  Ragnaros'                            @quality  -­‐=  1                        end                    end                else                    @quality  =  @quality  -­‐  @quality                end            else                if  @quality  <  50                    @quality  +=  1                end            end        end    end end big conditional Friday, March 7, 14

Slide 337

Slide 337 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remainin    end end small methods class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end small objects Friday, March 7, 14

Slide 338

Slide 338 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 big conditional small methods small objects tick/for normal brie backstage other Total Complexity 40 50 Friday, March 7, 14

Slide 339

Slide 339 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 big conditional small methods small objects tick/for normal brie backstage other 33 40 50 Total Complexity Friday, March 7, 14

Slide 340

Slide 340 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end big conditional small methods small objects 50 84 40 Total Complexity Friday, March 7, 14

Slide 341

Slide 341 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 45 90 big conditional small methods small objects 50 84 40 Total Complexity Friday, March 7, 14

Slide 342

Slide 342 text

@sandimetz Mar 2014 class  GildedRose    attr_reader  :name,  :quality,  :days_remaining    def  initialize(name,  quality,  days_remaining)        @name,  @quality,  @days_remaining  =  name,  quality,  days_remaining    end    def  tick        case  name        when  'normal'            normal_tick        when  'Aged  Brie'            brie_tick        when  'Sulfuras,  Hand  of  Ragnaros'            sulfuras_tick        when  'Backstage  passes  to  a  TAFKAL80ETC  concert'            backstage_tick        end    end    def  normal_tick        @days_remaining  -­‐=  1        return  if  @quality  ==  0        @quality  -­‐=  1        @quality  -­‐=  1  if  @days_remaining  <=  0    end    def  brie_tick        @days_remaining  -­‐=  1        return  if  @quality  >=  50        @quality  +=  1        @quality  +=  1  if  @days_remaining  <=  0    end    def  sulfuras_tick    end    def  backstage_tick        @days_remaining  -­‐=  1        return                            if  @quality  >=  50        return  @quality  =  0  if  @days_remaining  <  0        @quality  +=  1        @quality  +=  1  if  @days_remaining  <  10        @quality  +=  1  if  @days_remaining  <  5    end end 0 45 90 big conditional small methods small objects 50 84 40 33 86 Total Complexity Friday, March 7, 14

Slide 343

Slide 343 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 big cond small meth small objs tick/for normal brie backstage other 33 Complexity 40 50 Friday, March 7, 14

Slide 344

Slide 344 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose small meth small objs tick/for normal brie backstage other 33 Complexity 40 50 Friday, March 7, 14

Slide 345

Slide 345 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose GildedRose(2) small objs tick/for normal brie backstage other 33 Complexity 40 50 Friday, March 7, 14

Slide 346

Slide 346 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose GildedRose(2) many objs tick/for normal brie backstage other 33 Complexity 40 50 Friday, March 7, 14

Slide 347

Slide 347 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose GildedRose(2) many objs tick/for normal brie backstage other 33 Complexity 40 50 Friday, March 7, 14

Slide 348

Slide 348 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose GildedRose(2) many objs Backstage tick/for normal brie backstage other 33 Complexity 40 50 12 Friday, March 7, 14

Slide 349

Slide 349 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose GildedRose(2) many objs Backstage average tick/for normal brie backstage other 33 Complexity 40 50 7 12 Friday, March 7, 14

Slide 350

Slide 350 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose Backstage Complexity 50 12 Friday, March 7, 14

Slide 351

Slide 351 text

@sandimetz Mar 2014 module  GildedRose    class  Item        attr_reader  :quality,  :days_remaining        def  initialize(quality,  days_remaining)            @quality,  @days_remaining  =  quality,  days_remaining        end        def  tick        end    end    class  Normal  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  1            @quality  -­‐=  1  if  @days_remaining  <=  0        end    end    class  Brie  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  >=  50            @quality  +=  1            @quality  +=  1  if  @days_remaining  <=  0        end    end    class  Backstage  <  Item        def  tick            @days_remaining  -­‐=  1            return                            if  @quality  >=  50            return  @quality  =  0  if  @days_remaining  <  0            @quality  +=  1            @quality  +=  1  if  @days_remaining  <  10            @quality  +=  1  if  @days_remaining  <  5        end    end    DEFAULT_CLASS  =  Item    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage  }    def  self.for(name,  quality,  days_remaining)        (SPECIALIZED_CLASSES[name]  ||  DEFAULT_CLASS).new(quality,  days_remaining)    end end 0 25 50 GildedRose Backstage Complexity 50 12 75% Friday, March 7, 14

Slide 352

Slide 352 text

@sandimetz Mar 2014 Task: Implement Conjured Friday, March 7, 14

Slide 353

Slide 353 text

@sandimetz Mar 2014 module  GildedRose Friday, March 7, 14

Slide 354

Slide 354 text

@sandimetz Mar 2014 module  GildedRose    class  Conjured  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  2            @quality  -­‐=  2  if  @days_remaining  <=  0        end    end Friday, March 7, 14

Slide 355

Slide 355 text

@sandimetz Mar 2014 module  GildedRose    class  Conjured  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  2            @quality  -­‐=  2  if  @days_remaining  <=  0        end    end    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage} Friday, March 7, 14

Slide 356

Slide 356 text

@sandimetz Mar 2014 module  GildedRose    class  Conjured  <  Item        def  tick            @days_remaining  -­‐=  1            return  if  @quality  ==  0            @quality  -­‐=  2            @quality  -­‐=  2  if  @days_remaining  <=  0        end    end    SPECIALIZED_CLASSES  =  {        'normal'                                                                        =>  Normal,        'Aged  Brie'                                                                  =>  Brie,        'Backstage  passes  to  a  TAFKAL80ETC  concert'  =>  Backstage,        'Conjured  Mana  Cake'                                                =>  Conjured  } Friday, March 7, 14

Slide 357

Slide 357 text

@sandimetz Mar 2014 #  Running: ............................... Fabulous  run  in  0.003917s,  7914.2201  runs/s,  15828.4401  assertions/s. 31  runs,  62  assertions,  0  failures,  0  errors,  0  skips Friday, March 7, 14

Slide 358

Slide 358 text

@sandimetz Mar 2014 Summary Friday, March 7, 14

Slide 359

Slide 359 text

@sandimetz Mar 2014 duplication is far cheaper than the wrong abstraction Friday, March 7, 14

Slide 360

Slide 360 text

@sandimetz Mar 2014 reach for open / closed Friday, March 7, 14

Slide 361

Slide 361 text

@sandimetz Mar 2014 small methods are simple Friday, March 7, 14

Slide 362

Slide 362 text

@sandimetz Mar 2014 small objects are simple Friday, March 7, 14

Slide 363

Slide 363 text

@sandimetz Mar 2014 refactor through complexity Friday, March 7, 14

Slide 364

Slide 364 text

@sandimetz Mar 2014 refactor into simplicity Friday, March 7, 14

Slide 365

Slide 365 text

@sandimetz Mar 2014 Friday, March 7, 14

Slide 366

Slide 366 text

@sandimetz Mar 2014 Thanks Friday, March 7, 14

Slide 367

Slide 367 text

@sandimetz Mar 2014 Thanks Jim Friday, March 7, 14

Slide 368

Slide 368 text

@sandimetz Mar 2014 Questions? Friday, March 7, 14

Slide 369

Slide 369 text

http://poodr.com Sandi Metz @sandimetz @sandimetz Mar 2014 Friday, March 7, 14