Question

fields = CSV.parse(File.open(filename).first)[0]
fields.each_with_index do |field, i|
  fields[i] = field.downcase
end

I want to get the first line from the line, parse it as CSV and make each element lowercase.

This code seems too redundant to me. Any suggestions?

Answer 1

You can make the looping stuff a bit more concise if you wish:

fields.map!(&:downcase)

or even:

fields = CSV.parse(File.open(filename).first)[0].map(&:downcase)

I think you’re leaving a file handle hanging there too so you might want to try something like:

fields = []
File.open(filename) do |f|
  fields = CSV.parse(f.readline)[0].map(&:downcase)
end