coffeescript refactoring

Question

Is there any coffeescript specific trickery that would make this look neater:

index = (->
          if segment == 'index'
            return 0
          else if segment == 'inbox'
            return 2

          1
        )()

epidemian · Accepted Answer · 2012-11-28 15:19:24Z

7

Yes, a switch expression:

index = switch segment
  when 'index' then 0
  when 'inbox' then 2
  else 1

answered Nov 28, 2012 at 15:19

epidemian

19.3k3 gold badges64 silver badges73 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Courtney Christensen · Accepted Answer · 2013-03-22 15:51:11Z

1

You could use an inline if ... then ... else statement broken up into multiple lines (for readability) by using the \ character.

index = if segment == 'index' then 0 \
        else if segment == 'inbox' then 2 \
        else 1

This is useful if your conditional logic is too complex for a simple switch block.

answered Mar 22, 2013 at 15:51

9,5956 gold badges50 silver badges56 bronze badges

Jebdm · Accepted Answer · 2013-01-08 08:12:14Z

0

Yes, the CoffeeScript-specific existential operator:

index = {'index': 0, 'inbox': 2}[segment] ? 1

You could also use an inline if statement to get rid of the function call:

index = if segment == 'index' then 0 else if segment == 'inbox' then 2 else 1

But an inline if wouldn't be any harder in straight Javascript:

index = segment == 'index' ? 0 : segment == 'inbox' ? 2 : 1

answered Jan 8, 2013 at 8:12

Jebdm

1