r = "FBFB".split("").map do |char|
if char == "F"
return 1
end
if char == "B"
return 2
end
end
puts r.inspect
I get nothing printed to console. What am I doing wrong?
2 Answers
r = "FBFB".split("").map do |char|
if char == "F"
1
elsif char == "B"
2
end
end
puts r.inspect
Comments
Turns out you cannot use return inside of a code block, but you should use next (source).
7 Comments
Sergio Tulentsev
IMHO, you shouldn't use
next here. Could, but shouldn't. I'd personally do what's described in zswqa's answer.
Evgeni Petrov
Why? You give no explanation.
Sergio Tulentsev
Code is easier to read when it has only one exit point instead of several.
Sergio Tulentsev
Your comment contradicts itself. In absence of explicit returns (or equivalent statements, like
next in this case), the body will always be executed in full and return the value of the last evaluated expression. That's what I call "one exit point". Always the last expression. And yes, that is precisely why I like that code.Sergio Tulentsev
Nope, I mean the whole method body. Anyhow, this is getting us nowhere, so I'll stop. Using
next here works, even if it isn't idiomatic ruby. |
returnand correct if-else statement