HashとArrayの使い方がわからず悩んでいます...

Question

Ruby on Railsを先週から勉強はじめました。
以下のコードでわからないとこがあるので教えて下さい...。宜しくお願いします。

①『item.csv』というcvsデータを読み込む。
→csvデータの中には【商品コード】【商品名】【金額】が書いている。

※csvデータ

A1,たい焼き,500
A2,たこ焼き,300
A3,やきとり,100
B1,ビール,600
B2,焼きそば,300

②コマンドライン引数で、【商品コード】とマッチしていれば、以下のように表示する。

※実行結果【理想】

ruby Cart.rb A1 A2 A5 B2 →

【商品コード】A1【商品名】たい焼き【金額】500円
【商品コード】A2【商品名】たこ焼き【金額】300円
商品がありません。
【商品コード】B2【商品名】ビール【金額】600円

とコマンドライン引数の要素を一つづつ判別して、trueかfalseか調べたいのですが、実行すると、

※実行結果【現実】

ruby Cart.rb A1 A5 →

[商品コード] A1 [商品名] たい焼き [金額] 500円
商品がありません。
商品がありません。
商品がありません。
商品がありません。
商品がありません。
商品がありません。
商品がありません。
商品がありません。
商品がありません。

と表示されてしまいます。readFileの中でループ処理を2回しているのが原因だと思うんですが、
その解決がわからないので、もしわかればおしえていただきたいです。
宜しくお願いします。

ITEM = "item.csv"

class Cart
	def initialize(code,name,value)
		@code = code
		@name = name
		@value = value
	end

	def toString
		return  "[商品コード] #{@code} [商品名] #{@name} [金額] #{@value}円"
	end
end


def readFile
	goods = File.read(ITEM)
	goods.each_line do |line|
		good = line.split(",")
		cart = Cart.new(good[0],good[1],good[2])
		for i in 0..ARGV.length
		    hash = {good[0] => cart.toString()}
			if ARGV[i] == good[0]
				puts hash[ARGV[i]]
			else
				puts "商品がありません。"
            end
        end
    end
end


def exec
	readFile()
end


exec()

コードが読みにくいかもしれませんが、よろしくお願いします。

Comments

[iwamoto_takaaki]

実行してみて調べる回答者のために　item.csv　の中身と、入力したコマンドラインも載せてください。
ちなみにコードやファイルの行頭と行末を「’’’」で囲むとコードが四角く囲まれ読みやすくなります。

[fujimaru]

ご指摘ありがとうございました。変更いたしました。引き続きお願いします。

Answer 1

「コマンドライン引数1つずつに対して、ファイルにあるかどうか調べる」
じゃなくて、
「ファイルの1行ずつについて、コマンドライン引数にあるかどうか調べる」
になっているので、表示される結果はファイルの行数と同じになります。コマンドラインにないファイルの行については、「商品がありません。」と出ます。

という説明で分かりますでしょうか？

Answer 2

fujimaru さんの回答コードを少し書きなおしてみました。
ccc.rb

class Goods
  ITEM_FILE = 'item.csv'

  def read_csv
    csv = []
    File.read(ITEM_FILE).each_line do |line|
      csv << line.chop.split(',')
    end
    csv # 配列
  end

  def items_hash
    hash = {}
    csv = read_csv
    # 配列からハッシュを作る
    csv.each do |line|
      hash[line[0]] = { name: line[1], price: line[2].to_i }
    end
    hash
  end

  def exec(argv)
    hash = items_hash
    argv.each do |code|
      data = hash[code]
      if data
        puts "【商品コード】#{code} 【商品名】#{data[:name]} 【金額】#{data[:price]}"
      else
        puts "商品がありません。(#{code})"
      end
    end
  end
end

goods = Goods.new
goods.exec(ARGV)

実行結果

$ ruby ccc.rb A1 A2 A5 B2
【商品コード】A1 【商品名】たい焼き 【金額】500
【商品コード】A2 【商品名】たこ焼き 【金額】300
商品がありません。(A5)
【商品コード】B2 【商品名】焼きそば 【金額】300

Comments

[fujimaru]

お！なるほど！僕は普段Javaを使っているのでJava的なコードになってしまうんですが、こんな書き方もあるんですね。とても参考になりました。ありがとうございます！

[fujimaru]

hash = items_hash
argv.each do |code|
data = hash[code]
if data
puts "【商品コード】#{code} 【商品名】#{data[:name]} 【金額】#{data[:price]}"
else
puts "商品がありません。(#{code})"
end

ここの部分は
data = nil => else
という分岐になるんですか？

[katoy]

if data は if data != nil としてもよいです。

[katoy]

もっと ruby らしく書くことは可能です。
いろいろな書き方を考えてみるとよいと思います。

Answer 3

以下のコードで解決できました。

やはりfor文を2回回していた箇所でエラーが出ていたので、
for文を分割して処理しました。

class Cart

	def initialize(code,name,capa)
		@code = code
		@name = name
		@capa = capa
	end

	def toString
		return  "[商品コード] #{@code} [商品名] #{@name} [金額] #{@value}円"
	end

end


class Goods

	Item = "item.csv"

	def readFile
		result = []
		goods = File.read(ITEM)
		goods.each_line do |line|
			good = line.split(",")
			ware = Cart.new(good[0],good[1],good[2])
			result << ware.toString
		end
		return result
	end


	def readCode
		code = []
		goods = File.read(ITEM)
		goods.each_line do |line|
			good = line.split(",")
			code << good[0]
		end
		return code
	end
	
	
	def getHash
		hash = {}
		for i in 0..readCode().length
			hash.store(readCode()[i], readFile()[i])
		end
		return hash
	end


	def argv
		for i in 0..ARGV.length-1
			if getHash()[ARGV[i]] != nil
				puts getHash()[ARGV[i]]
			else
				puts "商品がありません。"
			end
		end
	end


	def exec
		argv()
	end

	goods = Goods.new()
	goods.exec()

end