Bug я не могу определить

Question

Я побежал этот кусок кода раньше:

require "awesome_print" 
require "rexml/document" 
require "debugger" 

include REXML 

class Scrapper 
    attr_reader :data 

    def initialize 
    file = File.new("./cia-1996.xml") 
    @data = REXML::Document.new(file) 
    end 

    def get_country_inflation 
    inflation_hash = {} 
    XPath.match(data, "//country").map { |element| 
     inflation_hash[element.attributes["name"]] = element.attributes["inflation"].to_i} 
     nested_array = inflation_hash.to_a 
     sorted_array = nested_array.sort_by {|country, inflation_value| inflation_value}.reverse 
     puts "The countries with the highest inflation indexes in 1996 were:" 
     first_five = sorted_array.first(5) 
     first_five.each do |item| 
     puts "#{item[0]}, with an inflation index of #{item[1]}" 
     end 
    end 
    end 
end 


sample = Scrapper.new 
sample.get_country_inflation 

После внесения некоторых изменений, теперь я получаю сообщение об ошибке

economics_challenge.rb: 36: ошибка синтаксиса, неожиданный keyword_end, ожидающий конец ввода

Не могли бы вы дать мне указания относительно того, где может произойти ошибка/опечатка (вначале играли в нее какое-то время и хотели бы получить обратную связь от ne w набор глаз).

Большое вам спасибо!

Edit: поэтому я сделал изменения предложил, но я получил несколько сообщений об ошибках:

economics_challenge.rb:26: syntax error, unexpected tSTRING_DEND, expecting keyword_end 
economics_challenge.rb:29: syntax error, unexpected tSTRING_DEND, expecting '}' 
...flation_value| inflation_value}.reverse 
...        ^
economics_challenge.rb:35: syntax error, unexpected keyword_end, expecting '}' 
economics_challenge.rb:46: syntax error, unexpected end-of-input, expecting '}' 

линии 26 относится к 2-й линии в куске кода ниже: кусок кода (и я думаю, что это где исходная задача):

XPath.match(data, "//country").map do |element| 
     inflation_hash[element.attributes["name"]] = element.attributes["inflation"].to_i} 
end 

линия 29 является:

sorted_array = nested_array.sort_by {|country, inflation_value| inflation_value}.reverse 

Я попытаюсь исправить ошибку в 29, вызвав reverse на отсортированном массиве и сохранив это для переменной.

Линия 35 представляет собой конец заявление, и нет никакой линии 46.

Любые советы?

Спасибо!

2nd Редактировать: Вау! Я не могу поверить, что не понял, что я не закончил многие вещи. С этого момента я буду придерживаться окончательного синтаксиса do and.

Благодарим вас за то, что вы очень помогли мне ... очень цените это!

Answer 1

Проблема в том, у вас есть один дополнительный end

Как @david-grayson государства имели свои отступы были правильными, вы, возможно, заметили его.

Этому, конечно, предоставляется код, как вы его представили.Возможно, это не так, хотя сообщение об ошибке соответствует найденной проблеме.

Вот код с отступом, некоторые изменения стиля, и никаких синтаксических ошибок:

require 'awesome_print' 
require 'rexml/document' 
require 'debugger' 

include REXML 

class Scrapper 
    attr_reader :data 

    def initialize 
    file = File.new('./cia-1996.xml') 
    @data = REXML::Document.new(file) 
    end 

    def get_country_inflation 
    inflation_hash = {} 
    XPath.match(data, '//country').map do |element| 
     inflation_hash[element.attributes['name']] = element.attributes['inflation'].to_i 
    end 
    nested_array = inflation_hash.to_a 
    sorted_array = nested_array.sort_by do |country, inflation_value| 
     inflation_value 
    end.reverse 
    puts 'The countries with the highest inflation indexes in 1996 were:' 
    first_five = sorted_array.first(5) 
    first_five.each do |item| 
     puts "#{item[0]}, with an inflation index of #{item[1]}" 
    end 
    end 
end 

sample = Scrapper.new 
sample.get_country_inflation 

Answer 2

Ваш отступы перепутались начиная здесь:

XPath.match(data, "//country").map { |element| 
    inflation_hash[element.attributes["name"]] = element.attributes["inflation"].to_i} 
    nested_array = inflation_hash.to_a 

Последняя строка этого отрывка должна быть отступа на один уровень, поскольку блок вы прошли на «карте» было прекращено на второй строке правой скобкой ,

Попробуйте исправить это и все после него.

Кроме того, здесь приведены советы: всегда записывайте многострочные блоки с использованием do и end и размещайте end на своей собственной линии. Тогда вы могли бы:

XPath.match(data, "//country").map do |element| 
    inflation_hash[element.attributes["name"]] = element.attributes["inflation"].to_i 
end 
nested_array = inflation_hash.to_a