Question

I'm working through the TestFirst.org tutorials and have gotten an error message I can't untangle:

 Failure/Error: repeat("hello").should == "hello hello"
 TypeError:
   String can't be coerced into Fixnum
 # ./03_simon_says/simon_says.rb:13:in `+'
 # ./03_simon_says/simon_says.rb:13:in `block in repeat'
 # ./03_simon_says/simon_says.rb:12:in `times'
 # ./03_simon_says/simon_says.rb:12:in `repeat'
 # ./03_simon_says/simon_says_spec.rb:39:in `block (3 levels) in <top (required)>'

Here is the code those errors are talking about ("def repeat" is line 09)

def repeat (say, how_many=2)
    repetition = say
    how_many = how_many-1

    how_many.times do |repetition|
        repetition = repetition + " " + say
    end

    return repetition
end

And here is the rake test that set it off:

it "should repeat a number of times" do
  repeat("hello", 3).should == "hello hello hello"
end

I understand that the error message is about trying to use a string like a numeric value but I can't see how or where that is happening

Was it helpful?

Solution

The below is the problem source

repetition = repetition + " " + say
#              ^ this is a Fixnum

In the line repetition + " " + say, you are trying to do a concatenation between a Fixnum and String instance, which caused the error String can't be coerced into Fixnum.

2.1.2 :001 > 1 + ""
TypeError: String can't be coerced into Fixnum
        from (irb):1:in `+'
        from (irb):1
        from /home/arup/.rvm/rubies/ruby-2.1.2/bin/irb:11:in `<main>'
2.1.2 :002 >

Your code can be written as :

#!/usr/bin/env ruby

def repeat (say, how_many = 1)
  ("#{say} " * how_many).strip
end

In my test_spec.rb file :-

require_relative "../test.rb"

describe "#repeat" do
  it "returns 'hello' 3 times" do
    expect(repeat('hello', 3)).to eq('hello hello hello')
  end
end

Lets run the test :-

arup@linux-wzza:~/Ruby> rspec spec/test_spec.rb
.

Finished in 0.00129 seconds (files took 0.1323 seconds to load)
1 example, 0 failures
arup@linux-wzza:~/Ruby>

update

repetition = say
how_many = how_many-1
  how_many.times do |repetition|

If you think, repetition declared outside of the block and inside the block are same, you are completely wrong. They are different, as they created in 2 different scopes. See the below example :-

var = 2
2.times { |var| var = 10 } # shadowing outer local variable - var
var # => 2
Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top