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

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

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

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

ハッシュ

ハッシュは、高速にデータ検索を行うアルゴリズムのことです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

解決済

1回答

1732閲覧

delete_ifメソッドの使い方

no1knows

総合スコア3365

Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

ハッシュ

ハッシュは、高速にデータ検索を行うアルゴリズムのことです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

0クリップ

投稿2020/08/30 14:39

配列に格納されたハッシュの1つがblankのとき、その要素から削除したいと思っています。
作ったコードが冗長なので、delete_ifメソッドを使えばシンプルな形にできるかと思ったのですが、うまく使えなかったので教えていただけないでしょうか?


作ったコード

Ruby

1n = 0 2array_del = [] 3 4clap_params.each do 5 if clap_params[n][:claps_count].to_i.zero? # 0または''を判断 6 array_del << n 7 end 8 n += 1 9end 10 11array_del.sort.reverse.each do |del| 12 clap_params.delete_at(del) 13end

具体的には・・・配列clap_paramsの要素のハッシュの1つがclaps_count==""なら、その要素はclap_paramsから削除するという形です。
例えば下記のような場合だと

clap_params = [ <ActionController::Parameters {"post_id"=>"15", "claps_count"=>""} permitted: true>, <ActionController::Parameters {"post_id"=>"16", "claps_count"=>"1"} permitted: true>, <ActionController::Parameters {"post_id"=>"17", "claps_count"=>""} permitted: true>, <ActionController::Parameters {"post_id"=>"18", "claps_count"=>"1"} permitted: true> ]

以下のような形にしたいです。

clap_params = [ <ActionController::Parameters {"post_id"=>"16", "claps_count"=>"1"} permitted: true>, <ActionController::Parameters {"post_id"=>"18", "claps_count"=>"1"} permitted: true> ]

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

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

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

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

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

guest

回答1

0

ベストアンサー

ruby

1require 'active_support' 2require 'active_support/core_ext/object/blank' 3 4 5clap_params = [ 6{:"post_id"=>"15", :"claps_count"=>""}, 7{:"post_id"=>"16", :"claps_count"=>"1"}, 8{:"post_id"=>"17", :"claps_count"=>""}, 9{:"post_id"=>"18", :"claps_count"=>"1"} 10] 11 12pp clap_params.delete_if{|param| param[:claps_count].blank?}

投稿2020/08/30 16:14

asm

総合スコア15149

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

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

no1knows

2020/08/31 01:36

いつもありがとうございます!!! delete_ifの中のハッシュをどのように扱えばよいかわからなかったのですが、param[:claps_cout]と指定してからブランクかどうかを調べれば良かったのですね。 スッキリしたコードで可読性が良くなりました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.37%

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

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

質問する

関連した質問