質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Q&A

解決済

3回答

298閲覧

Ruby 配列間でのvalueの計算

parapa

総合スコア17

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

0グッド

0クリップ

投稿2018/04/04 19:35

前提・実現したいこと

ゲームに参加するにはメダルが必要です。
必要メダルと所持メダルの配列があります。
keyはメダルの種類。valueはその数です。

足りていない場合はエラーを出力し、
足りている場合は所持メダル数を必要メダル数だけ減らしたいです。

配列間で一致するkeyのvalue同士を計算する方法はないでしょうか。

該当のソースコード

Ruby

1request = {"gold"=>"1", "silver"=>"3"} #必要メダル 2possession = {"gold"=>"5", "silver"=>"15", "bronze"=>"30"} #所持メダル

目標

Ruby

1possession = {"gold"=>"4", "silver"=>"12", "bronze"=>"30"} #所持メダル

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

何故値を数値じゃなくて文字列で保持するのでしょうか?

Ruby

1request = {"gold"=>"1", "silver"=>"3"} #必要メダル 2possession = {"gold"=>"5", "silver"=>"15", "bronze"=>"30"} #所持メダル 3 4possession = possession.merge(request) { |key, pos, req| (pos.to_i-req.to_i).to_s} 5 6p possession

おそらく大丈夫だと思いますが、もしrequestだけにあるキーがあれば、そのデータも結果に含まれます。そういう可能性があるなら、mergeの前にチェックしてください。

#追記
エラー処理を忘れました

Ruby

1request = {"gold"=>"1", "silver"=>"3"} #必要メダル 2possession = {"gold"=>"5", "silver"=>"15", "bronze"=>"30"} #所持メダル 3 4possession = possession.merge(request) do |key, pos, req| 5 rest = pos.to_i-req.to_i 6 if rest < 0 7 エラー処理 8 end 9 rest.to_s 10end 11 12p possession

あと、「配列間」じゃなくて「ハッシュ間」ですね。

投稿2018/04/05 00:55

編集2018/04/05 01:00
otn

総合スコア84423

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

parapa

2018/04/06 20:46

回答ありがとうございます。 文字列の保持する理由はなかったです。また、ハッシュでしたすみません。
guest

0

x001.rb

ruby

1request = {gold: 1, silver: 3} 2possession = {gold: 5, silver: 15, bronze: 30} 3 4x = possession.each_with_object({}){|(k, v), mem| mem[k] = v - request[k].to_i} 5if x.all?{|k, v| v >= 0} 6 possession = x 7else 8 puts "メダル不足" 9 x.select{|k, v| v < 0}.each{|k, v| puts "#{k } #{-v}"} 10end 11p possession 12 13request = {bronze: 40} 14x = possession.each_with_object({}){|(k, v), mem| mem[k] = v - request[k].to_i} 15if x.all?{|k, v| v >= 0} 16 possession = x 17else 18 puts "メダル不足" 19 x.select{|k, v| v < 0}.each{|k, v| puts "#{k } #{-v}"} 20end 21p possession

実行結果
イメージ説明

投稿2018/04/07 00:13

katoy

総合スコア22324

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

Ruby

1request = {"gold"=>"1", "silver"=>"3"} #必要メダル 2possession = {"gold"=>"5", "silver"=>"15", "bronze"=>"30"} #所持メダル 3 4request.each do |k,v| 5 result = x[k].to_i - v.to_i 6 if result < 0 7 puts "less money!!" 8 return 9 end 10 possession[k] = result.to_s 11end

追記:
ご指摘を受け、修正してみ、、、ようとしましたが、possessionを新しいオブジェクトに入れるにはやっぱりmerge使うのが一番きれいなので、やめときます。

投稿2018/04/04 23:57

編集2018/04/05 02:09
kazto

総合スコア7196

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

otn

2018/04/05 01:51

マイナスエラー時の要件が不明なのであれですが、直接更新してしまうと、エラーで中断したときに、更新済の値と更新前の値が混在するので、新規のハッシュに入れた方がおそらく良いと思います。
kazto

2018/04/05 01:54

ごもっともです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問