Ruby using each_with_index

Question

I'd like this method to circle through each item in the array of names katz_deli and use puts to display the name and its index. However, the output is just the first name in the array with its index.

My code:

def line (katz_deli)
  if katz_deli.count > 1
    katz_deli.each_with_index {|name, index| puts "The line is currently: #{index +1}. #{name}" }
  else
    puts "The line is currently empty."
  end
end

I want my output to be "The line is currently: 1. Logan 2. Avi 3. Spencer" But I'm getting "The line is currently: 1. Logan." Thanks!

are you sure that you have more than one element in your array? if yes, please provide an example of your array's data — Long Nguyen
– Long Nguyen, Commented Nov 20, 2015 at 19:48
I cannot reproduce your problem. When I call line([ "Logan", "Avi", "Spencer" ]) I get three lines of output: The line is currently: 1. Logan, The line is currently: 2. Avi, and The line is currently: 3. Spencer. — Jordan Running
– Jordan Running, Commented Nov 20, 2015 at 19:49
The array is katz_deli = ["Logan", "Avi", "Spencer"]. I see. Is there a way to get this get this all on one line---i.e for the output to be exactly The line is currently: 1. Logan 2. Avi 3. Spencer I'm doing this for a course and I think that's what the instructors are looking for. Thanks! — Keren
– Keren, Commented Nov 20, 2015 at 20:04

Dmitry Sokurenko · Accepted Answer · 2015-11-20 20:30:52Z

3

You can build the output string at first, and puts it once it is ready:

input = ["Logan", "Avi", "Spencer"]

def line (katz_deli)
  if katz_deli.count > 1
    output = "The line is currently:"
    katz_deli.each_with_index do |name, index|
      output << " #{index +1}. #{name}"
    end
    puts output
  else
    puts "The line is currently empty."
  end
end

line(input)

answered Nov 20, 2015 at 20:30

Dmitry Sokurenko

6,1324 gold badges34 silver badges54 bronze badges

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

Comments

Long Nguyen · Accepted Answer · 2015-11-20 20:27:19Z

1

def line (katz_deli)
  if katz_deli.count > 1
    print "The line is currently:"
    katz_deli.each_with_index {|name, index|  print " #{index +1}. #{name}" }
  else
    puts "The line is currently empty."
  end
end

answered Nov 20, 2015 at 20:27

Long Nguyen

11.3k2 gold badges21 silver badges25 bronze badges

Collectives™ on Stack Overflow

Ruby using each_with_index

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related