Rubyでデータを反復処理する方法は？

Question

私はかなりプログラミングに新しいし、私のコードのいくつかのヘルプ/フィードバックが必要です。 私の目標は、正常に動作している私のデータを掻き集め、そのデータをユーザに番号付きリストで表示することです。私はこのデータを表示するのが難しいです。私のプログラムは私のメソッドを単純にスキップしてエラーを戻すことはありません。どんなヘルプ/フィードバックにも事前に感謝しています！

class BestPlaces::Places 
    attr_accessor :name, :population, :places 
    @@places = [] 

    def self.list_places 
    # puts "this is inside list places" 
    self.scrape_places 
    end 

     def self.scrape_places 
     doc = Nokogiri::HTML(open("https://nomadlist.com/best-cities-to-live")) 
      places = doc.search("div.text h2.itemName").text 
      rank = doc.search("div.rank").text 

      places.collect{|e| e.text.strip} 
       puts "you are now in title" 
       @@places << self.scrape_places 
       puts "#{rank}. #{places}" 
      end 
      end 
     end 

CLI Page: 
class BestPlaces::CLI 

    def list_places 
    puts "Welcome to the best places on Earth!" 
    puts @places = BestPlaces::Places.list_places 
    end 

    def call 
    list_places 
    menu 
    goodbye 
    end 
end 

Answer 1

あり、このコードで対処することができ、いくつかのものがありますが、まずは、リワークを見てみましょう：

require 'nokogiri' 
require 'open-uri' 

module BestPlaces 

    class Places 
    attr_accessor :name, :population, :places 

    def initialize 
     @places = [] 
    end 

    def scrape_places 
     doc = Nokogiri::HTML(open("https://nomadlist.com/best-cities-to-live")) 
     places = doc.search("div.text h2.itemName") 
     ranks = doc.search("div.rank") 
     places.each{|e| @places << e.text.strip} 
     puts "you are now in title" 
     @places.each do |place| 
     i = @places.index(place) 
     puts "#{ranks[i].text}. #{place}" 
     end 
    end 

end 

class CLI 

    def list_places 
    puts "Welcome to the best places on Earth!" 
    BestPlaces::Places.scrape_places 
    end 

    def call 
    list_places 
    menu 
    goodbye 
    end 

end 

end 

あなたが不完全なモジュール/クラスのセットアップになりそうだものを持っています。一つは、上記のようになどを呼び出すことができます。

bp = BestPlaces::Places.new 
bp.scrape_places 

@@変数は、我々は場所、クラス内でアクセスする必要のある値を保持する代わりに@placesを使用することができ、不要だっ配置します。また、nokogiriは検索結果に.textメソッドを使用するときに文字列オブジェクトを返します。つまり、配列のように繰り返し処理することはできません。私はこれが役立つことを願っています