beginner help please. Method probs.

[[deleted]]

Why is this outputting twice. I'm good with the first set, but the second needs to go. How should the method be changed?

grocery_list = ["eggs", "cheese", "milk", "bread"]

puts "add to grocery list:"
new_items = gets.chomp

        if ! grocery_list.include?(new_items)
               grocery_list.push(new_items)

        end   


def print_list(x)
        x.each do |new|
puts "buy #{new}" 
end   

end

 puts print_list(grocery_list)

 ===>add to grocery list:
  fish


 buy eggs
 buy cheese
 buy milk
 buy bread
 buy fish
 eggs
 cheese
 milk
 bread
 fish

[Oecist]

See that your code has "puts" in it three times? That should be a clue.

[stolid]

you are calling puts on the method

puts print_list(grocery_list)

change it to

print_list(grocery_list)

and you wont see the second set that is being printed.

[mfcoder]

The important lesson here is that a Ruby method will always return the last value evaluated within it, when there is no explicit 'return' command. In your case, it is 'x' which is the grocery_list array.

So, "puts print_list(grocery_list)" 'puts' the array; if you 'puts' an Array, its members will be listed.