Ruby function with an if statement

Question

I have this following function but somehow it doesn't get inside the if statement. This is my gist on line 68

I'm trying to evaluate if an object.country name has the same name as "example_of_a_country_name"

def find_population_of_country(liste_pays, country_name)
  given_country_population = 0
  liste_pays.each do |n| 
    if (n.country.eql?(country_name.upcase))
      given_country_population = n.population

      #I'm trying to see if it's output something here
      puts country_name.upcase
    end
  return given_country_population
  end
end

Sherwyn Goh · Accepted Answer · 2014-01-24 02:22:09Z

1

can you type this within your function and tell me what you get?:

def find_population_of_country(liste_pays, country_name)
  liste_pays.select {|n| n.country.eql?(country_name.upcase) }
 end
end

answered Jan 24, 2014 at 2:22

Sherwyn Goh

1,3521 gold badge10 silver badges19 bronze badges

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

3 Comments

Sherwyn Goh Over a year ago

It would also help if you could specify how the models are related.

Papouche Guinslyzinho Over a year ago

thks Sherwyn goh! First now this function works. I'll be able to continue. Second: It's not a model. I'm trying to learn classes in Ruby by redoing all the assigment that I had in Java. Thks!

Sherwyn Goh Over a year ago

ahh right, my bad didn't see the lack of a 'rails' tag. Happy it works, have fun coding.

jeremycole · Accepted Answer · 2014-01-24 02:30:12Z

1

The problem is your line:

return given_country_population

This should be moved one line up, inside the if ... end block for the name comparison, otherwise, it's returning even for non-matches.

answered Jan 24, 2014 at 2:30

jeremycole

2,78114 silver badges15 bronze badges

1 Comment

Papouche Guinslyzinho Over a year ago

thks @jeremycole! I was thinking how .select works but not .each ??? thanks for the tip.

Collectives™ on Stack Overflow

Ruby function with an if statement

2 Answers 2

3 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related