Ruby include? with YAML array

Question

I have a config YAML file with the following array:

EMAILS: ['[email protected]', '[email protected]', '[email protected]']

And then I want to search inside this YAML array for a particular string, say "[email protected]".

I use the following ruby code:

if CONFIG[:EMAILS].include?("[email protected]")
  return true
else
  return false
end

However, this always returns false.

I can PUTS the CONFIG[:EMAIL] array fine.

Can anyone suggest what I'm doing wrong.

Your ruby code assumes that the YAML has already been loaded into a hash. How did that happen? — Mori
– Mori, Commented Nov 8, 2012 at 16:07
Can you post the output of CONFIG[:EMAILS] please? and how you parse the yaml file? — MrYoshiji
– MrYoshiji, Commented Nov 8, 2012 at 16:11

Thilo · Accepted Answer · 2012-11-08 16:12:03Z

2

You're using a symbol, while the YAML hash uses strings as keys. This should work:

if CONFIG['EMAILS'].include?("[email protected]")
  return true
else
  return false
end

Which, by the way, is equivalent to simply

return CONFIG['EMAILS'].include?("[email protected]")

answered Nov 8, 2012 at 16:12

Thilo

17.8k5 gold badges70 silver badges86 bronze badges

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

4 Comments

tadman Over a year ago

The return part is completely redundant if you have this as your last line of a method. Presumably this line would be given the previous behavior was to exit regardless of the condition.

tadman Over a year ago

In both cases it would exit the function, so if there was code after that it wouldn't have been executed.

Thilo Over a year ago

Sure. It's still a big "if" and therefore not equivalent in all circumstances.

tadman Over a year ago

I was pretty clear that if that was the last line, then the return is not required. This is basic Ruby behavior. It is not controversial. Under all circumstances where it's the last line, it is equivalent.

Collectives™ on Stack Overflow

Ruby include? with YAML array

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related