Заглавные буквы книг без .titleize

Question

Мне нужно загладить каждое слово в названии книги для моего кода. У меня возникают проблемы с итерацией каждого элемента массива и заглавной буквы первой буквы. Я действительно не вижу, что я делаю неправильно. Любая помощь будет принята с благодарностью.

class Title 
    attr_accessor :string 

    def initialize(string) 
    @string = string 
    end 

    def fix 
    @string.downcase! 
    new_array = [] 
    if @string.include?(" ") 
     new_array << @string.split(" ") 
     new_array.map {|word| word.capitalize} 
     new_array.join(" ") 
    else 
     @string.capitalize! 
    end 
    end 
end 

Характеристики:

describe "Title" do 
    describe "fix" do 
    it "capitalizes the first letter of each word" do 
     expect(Title.new("the great gatsby").fix).to eq("The Great Gatsby") 
    end 
    it "works for words with mixed cases" do 
     expect(Title.new("liTTle reD Riding hOOD").fix).to eq("Little Red Riding Hood") 
    end 
    it "downcases articles" do 
     expect(Title.new("The lord of the rings").fix).to eq("The Lord of the Rings") 
     expect(Title.new("The sword And The stone").fix).to eq("The Sword and the Stone") 
     expect(Title.new("the portrait of a lady").fix).to eq("The Portrait of a Lady") 
    end 
    it "works for strings with all uppercase characters" do 
     expect(Title.new("THE SWORD AND THE STONE").fix).to eq("The Sword and the Stone") 
    end 
    end 
end 

Answer 1

Вы должны избавиться от

new_array = [] 

Вам не нужно инициализировать массив для split. Вы создаете массив внутри массива.

Answer 2

Ваша функция исправления должны быть:

def fix 
    new_array = @string.split(" ") 
    new_array.map {|word| word.capitalize}.join(" ") 
end 

Там нет необходимости downcase в начале, потому что капитализировать downcases строку, а затем заглавной первую букву.

Также этот код не имеет значения, имеет ли строка только одно слово или более, поэтому если это не нужно.

И последнее, что вы инициализировали массив, а затем вставляете другой массив внутри, используя < <, вы должны просто назначить то, что вы получаете из раскола, потому что оно уже является массивом.