17

Saw a strange case come up, trying to figure out what is happening here:

> def test
>   p yield
> end
=> nil
> test { 1 }
1
=> 1
> p test { 1 }
1
1
=> 1
> p test do
>   1
> end
LocalJumpError: no block given (yield)
Brian Armstrong
  • 19,707
  • 17
  • 115
  • 144

2 Answers2

22

The parser recognizes this

p test do
  1
end

as this

p(test) do
  1
end

The block is passed to p, not test. Therefore, yield can't yield and raises that error.

Alex Wayne
  • 178,991
  • 47
  • 309
  • 337
Sergio Tulentsev
  • 226,338
  • 43
  • 373
  • 367
17

do and {} to denote blocks attached to methods are not completely interchangeable.

p test do
  1
end

Precedence is screwing with you. This is actually this:

p(test()) do
  1
end

So the block is getting passed to p, not test.

{} has higher precedence than do, and so binds more tightly to the syntactically closer method. This is also true for other ruby keywords that have symbolic equivalents, such as and/&& and or/||, which is why the symbols are usually recommended over the words.

Sergio Tulentsev
  • 226,338
  • 43
  • 373
  • 367
Alex Wayne
  • 178,991
  • 47
  • 309
  • 337