16

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,437
  • 17
  • 111
  • 143

2 Answers2

23

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
  • 162,909
  • 46
  • 287
  • 312
Sergio Tulentsev
  • 219,187
  • 42
  • 361
  • 354
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
  • 219,187
  • 42
  • 361
  • 354
Alex Wayne
  • 162,909
  • 46
  • 287
  • 312