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

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

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

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

Q&A

解決済

3回答

1600閲覧

もう少しスマートに書きたいです(配列の最大値のindexを返す)

torisan3

総合スコア50

Ruby

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

0グッド

0クリップ

投稿2020/07/17 15:05

やりたい事は、配列(要素数4固定)の値の中で最大値のindex値を取得したいです。

ruby

1ha1 = [200, 250, 350 ,220] 2p ha1.index(ha1.max) #=>2

通常時ならこれで事足りるのですが、仕様上最大値を持っているindexが複数ある場合があります。
複数ある場合は、もう一つの配列ha2(ha1と同要素数:4)を見て最小値を持つindexを返したいです。
(※ha2の配列は同一値が入ることはありません。すべてユニークです。)

ruby

1#ha1で最大なのは420なので3を返す。ha2は無視。 2ha1 = [200, 250, 350 ,420] 3ha2 = [1, 4, 3 ,2] 4-------------------------------------- 5#ha1で最大なのは100(indexは0と2)。 6#ha2[0](1)とha2[2](3)を比較するとha2[0]の方が小さいので0を返す。 7ha1 = [100, 20, 100 ,0] 8ha2 = [1, 2, 3 ,4] 9-------------------------------------- 10#ha1の値がすべて同じ場合もあり得る。 11#ha2の中で1が一番小さいのでindexは2を返す。 12ha1 = [200, 200, 200 ,200] 13ha2 = [2, 4, 1 ,3]

現状のプログラム

ruby

1 ha1 = [100, 20, 100 ,100] 2 ha2 = [3, 2, 1 ,4] 3 tmp = [] 4 for j in 0..3 5 tmp.push(j) if ha1[j] == ha1.max 6 end 7 8 ind = tmp[0] 9 for j in 1..tmp.size - 1 10 ind = tmp[j] if ha2[ind] > ha2[tmp[j]] 11 end 12 p ind #=>2

値がユニークならたった1行で済む事なので、もう少しスマートに書けないかと思っています。

急いではいないので作業の息抜き時にでも回答していただければと思います。
よろしくお願いいたします。

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

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

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

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

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

guest

回答3

0

ベストアンサー

Enumerable#each_with_index が便利です

ruby

1hal = [100, 200, 50, 150] 2p hal.each_with_index.max # => [200, 1] 3 4ha1 = [100, 20, 100 ,0] 5ha2 = [1, 2, 3 ,4] 6 7p ha1.zip(ha2.map(&:-@)).each_with_index.max[1] # => 0 8 9ha1 = [100, 20, 100 ,0] 10ha2 = [3, 2, 1 ,4] 11 12p ha1.zip(ha2.map(&:-@)).each_with_index.max[1] # => 2

投稿2020/07/17 17:20

maai

総合スコア463

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

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

torisan3

2020/07/18 02:34

each_with_indexは存在を知りませんでした。 index用の配列を作成しなくてよいのがスマートだと思いました。 回答ありがとうございました!
guest

0

Ruby

1ha1.zip(ha2.map(&:-@), ha1.size.times).max[2]

これでどうでしょう。
ha1.zip(ha2, ha1.size.times)[[ha1[0], ha2[0], 0], [ha1[1], ha2[1], 1], ...]というような配列を生成しています。
それをmaxすると、Array#<=>は要素を前から順に比較していく(辞書式)ので、ha1を優先し、ha1が同じならha2を比較し、それが同じならインデックスが大きい方をとる、というふうにしています。
普通のindexはインデックスが小さい方をとるので、そうしたければ↓のようになります。

Ruby

1ha1.size - ha1.zip(ha2.map(&:-@), ha1.size.downto(1)).max[2]

投稿2020/07/17 15:42

編集2020/07/17 17:54
magf

総合スコア212

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

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

torisan3

2020/07/18 02:35

ha2.map(&:-@)に最初戸惑ったのですが、ようするに値の正負を入れ替えるということですね。 maxの比較判定方法は知りませんでした。 回答ありがとうございました!
guest

0

Ruby

1ind = (0...ha1.size).zip(ha1,ha2).max_by{|i,x,y| [x,-y]}[0]

投稿2020/07/17 15:41

otn

総合スコア84423

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

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

torisan3

2020/07/18 02:39

max_byが二次元配列にも使えるのは知りませんでした。 書き方も、命令を知らなくてもなんとなく想像できるのがいいですね。 回答ありがとうございました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問