Methode zum Konvertieren von römischen Zahlen in Arabisch druckt nicht die Antwort

Question

Ich erstelle eine Methode, die römische Zahlen in Ganzzahlen konvertiert. Ich habe dies geschrieben:

def roman_to_integer(roman) 
    roman = roman.upcase 

    num_I = roman.count("I").to_i 
    num_V = roman.count("V").to_i 
    num_X = roman.count("X").to_i 
    num_L = roman.count("L").to_i 
    num_C = roman.count("C").to_i 
    num_D = roman.count("D").to_i 
    num_M = roman.count("M").to_i 

    rom_num = { "I" => 1, 
       "V" => 5, 
       "X" => 10, 
       "L" => 50, 
       "C" => 100, 
       "D" => 500, 
       "M" => 1000 } 

    while roman 
     if roman[0] == "M" 
     integer = num_M * rom_num["M"] + num_D * rom_num["D"] + num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "D" 
     integer = num_D * rom_num["D"] + num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "C" 
     integer = num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "L" 
     integer = num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "X" 
     integer = num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "V" 
     integer = num_V * rom_num["V"] + num_I * rom_num["I"] 
     elsif roman[0] == "I" 
     integer = num_I * rom_num["I"] 
     end 
    end 
    return integer 
end 

puts "Tell me any number in Roman System and I will convert it to the Arabic Number!" 
number = gets.chomp 
puts "Here is your number:" 
puts(roman_to_integer(number)) 

Wenn es um die letzte Zeile kommt, wo es aus dem Ergebnis gedruckt werden soll, bekomme ich nur die leere Eingabezeile in der Befehlszeile. Ich verstehe nicht, warum es nicht funktioniert und brauche Hilfe, um herauszufinden, wo der Code falsch ist.

Auch wenn ich

puts number.roman_to_integer 

als letzte Zeile verwenden, erhalte ich

private method "roman_to_integer" called on "":String (NoMethodError) 

Jede Hilfe würde geschätzt.

Answer 1

Ihre Methode nicht zurück, da es in der while roman Block steckt - da Sie nicht über die roman Variable ändern, wird es immer truthy sein, und der Block wird auch weiterhin immer und immer wieder laufen, nie einen Wert zurückgibt :

while roman 
    #... runs forever 
end 

Unabhängig davon Ihre Methode scheint nicht zu tun, was Sie denken, es ist

Answer 2

Entfernen Sie Ihre while Schleife. Ich denke nicht, dass es dem Code etwas hinzufügt. Ich habe auch die Formatierung verbessert und die to_s Methoden entfernt, die überflüssig waren.

def roman_to_integer(roman) 

    roman = roman.upcase 

    # Counts the occurrences of each letter from the input 

    num_I = roman.count("I") 
    num_V = roman.count("V") 
    num_X = roman.count("X") 
    num_L = roman.count("L") 
    num_C = roman.count("C") 
    num_D = roman.count("D") 
    num_M = roman.count("M") 

    # Assigns values to every Roman letter 

    rom_num = { "I" => 1, 
       "V" => 5, 
       "X" => 10, 
       "L" => 50, 
       "C" => 100, 
       "D" => 500, 
       "M" => 1000 } 

    # Multiplies the value of the letter by number of occurrences   


    if roman[0] == "M" 
    integer = num_M * rom_num["M"] + num_D * rom_num["D"] + num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 
    elsif roman[0] == "D" 
    integer = num_D * rom_num["D"] + num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 

    elsif roman[0] == "C" 
    integer = num_C * rom_num["C"] + num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 

    elsif roman[0] == "L" 
    integer = num_L * rom_num["L"] + num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 

    elsif roman[0] == "X" 
    integer = num_X * rom_num["X"] + num_V * rom_num["V"] + num_I * rom_num["I"] 

    elsif roman[0] == "V" 
    integer = num_V * rom_num["V"] + num_I * rom_num["I"] 

    elsif roman[0] == "I" 
    integer = num_I * rom_num["I"] 
    end 

    return integer 
end 

puts "Tell me any number in Roman System and I will convert it to the Arabic Number!" 

number = gets.chomp 
puts "Here is your number:" 
puts roman_to_integer(number) 

Auch puts number.roman_to_integer Arbeit machen Sie würden Ihre Methode der String-Klasse hinzuzufügen, die Sie übrigens nicht tun sollten.