Ruby method unable to pass Rspec test

Question

I am trying to create a method that takes in a user input. It turns that user input into an integer, it then subtracts one from the user input. It also returns -1 if the user input is not a number. However the tests throws an error.

  describe '#input_to_index' do

    it 'converts a user_input to an integer' do
      user_input = "1"

      expect(input_to_index(user_input)).to be_a(Fixnum)
    end

    it 'subtracts 1 from the user_input' do
      user_input = "6"

      expect(input_to_index(user_input)).to be(5)
    end

    it 'returns -1 for strings without integers' do
      user_input = "invalid"

      expect(input_to_index(user_input)).to be(-1)
    end

  end

Here is my method:

def input_to_index(user_input)
  user_input = user_input.to_i
  user_input = user_input - 1
  return -1 if !user_input.is_a? Numeric
end

Simple Lime · Accepted Answer · 2017-08-03 04:17:29Z

3

It's because you're only returning something if !user_input.is_a?(Numeric) and you've already cast user_input to an integer.

-1 if false # => nil
-1 if true # => -1

So that last line in the method returns nil because the condition is never going to be met.

"a".to_i # => 0
"a".to_i.is_a?(Numeric) # => true
("a".to_i - 1).is_a?(Numeric) # => true

You don't even need that last line at all and things will work fine:

def input_to_index(user_input)
  user_input = user_input.to_i
  user_input = user_input - 1
end
input_to_index("1") # => 0
input_to_index("6") # => 5
input_to_index("invalid") # => -1

and more succinctly:

def input_to_index(user_input)
  user_input.to_i - 1
end
input_to_index("1") # => 0
input_to_index("6") # => 5
input_to_index("invalid") # => -1

edited Aug 3, 2017 at 4:17

answered Aug 3, 2017 at 4:12

Simple Lime

11.1k2 gold badges20 silver badges32 bronze badges

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

Comments

Asdrubal · Accepted Answer · 2017-08-03 10:52:49Z

1

I'm sure there is a more eloquent way to do this, but you could do this:

def input_to_index(user_input)
  user_input = user_input.to_i
  user_input = user_input - 1
  if !user_input.is_a? Numeric
    -1
  else
    user_input
  end
end

EDIT

This might be a more eloquent way to do it:

def input_to_index(user_input)
  user_input = user_input.to_i - 1
  !user_input.is_a?(Numeric) ? -1 : user_input
end

Below is the most eloquent way to do that:

def input_to_index(user_input)
  user_input.to_i - 1
end

Credit: Simple Lime

edited Aug 3, 2017 at 10:52

answered Aug 3, 2017 at 4:08

Asdrubal

2,4814 gold badges31 silver badges38 bronze badges

2 Comments

Holger Just Over a year ago

After user_input = user_input.to_i, it is always an Integer, hence a Numeric. The if branch will thus never match. You are always going to the else branch.

Holger Just Over a year ago

What I wanted to say was that at the point where you check it, user_input can never be anything else than a Numeric. There is thus no point in checking.

Collectives™ on Stack Overflow

Ruby method unable to pass Rspec test

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related