ruby on rails: how to delete element by column value at model array

Question

Model Cafe has meal_available, which has boolean type.

When searching cafe, I want to re-search by category checkbox:

<form action="/cafe/cafesearch" method="GET">
<label for="chk1"><input type="checkbox" id="chk1" name="meal_available" value=true>meal_available</label>
<input type="submit" value="submit">
</form>

At view, pass checkbox value through form tag, and at controller, I tried like this:

coffee = Cafe.all
if params[:meal_available] == true
  @cafe = coffee.delete_if{|x| x.meal_available == false}
end

It didn't work. How can I delete element by model column in array?

sghosh968 · Accepted Answer · 2016-02-14 08:02:16Z

1

You can just put in a scope in Cafe model to get the cafes with meal_available, more information on ActiveRecord scopes here

In cafe model :

class Cafe < ActiveRecord::Base
  scope :meals_available, -> { where(meal_available: true) }
end

In controller :

@cafes = Cafe.meals_available

thats it, thanks.

answered Feb 14, 2016 at 8:02

sghosh968

6017 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Ilya Over a year ago

it is not answer on question. Question is How delete element?

sghosh968 Over a year ago

@Ilya if you read the question and code properly again you will get it right. Sangwoo Park is trying to search through the records in the controller and only wants to get the cafes with meal_available: true and scopes is a better options for that.

sghosh968 Over a year ago

delete was just to reject the cafes with meal_available: false scopes would work better in these situation

Ilya Over a year ago

Scope is a good solution, but you did not answer on main question. Author attempts to delete records from database.

Linus Oleander Over a year ago

I must agree with @sghosh968 here. OP is asking for deleting items in an array, not the database. Manipulating the database on each page reload on a search action (which is triggered by a GET request) doesn't make any sense. The only reason OP accepted the above answer is b/c it has the same outcome as asked for. +1

Ilya · Accepted Answer · 2016-02-14 09:05:35Z

0

You can use ActiveRecord methods for it only:

if params[:meal_available] == true
  @cafe = Cafe.where(meal_available: false)
end

In this case coffee = Cafe.all is redundant.

coffee = Cafe.all
if params[:meal_available] == true
  @cafe = coffee.select { |x| x.meal_available == false }
end

edited Feb 14, 2016 at 9:05

answered Feb 14, 2016 at 7:31

Ilya

13.5k5 gold badges41 silver badges56 bronze badges

Collectives™ on Stack Overflow

ruby on rails: how to delete element by column value at model array

2 Answers 2

5 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related