Какой самый лучший способ дезинфицировать destroy_all - Рельсы

Question

У меня есть следующий контроллер в Rails:

class FooController < ApplicationController 
    def delete_foo(bar): 
     Foo.destroy_all("foo = '#{@bar}'") 

Является

Foo.destroy_all("foo = ?", @bar) 

всегда действует?

Answer 1

destroy_all работает над отношением. Почему бы не сделать

Foo.where(foo: bar).destroy_all 

Answer 2

Foo.destroy_all("foo = ?", @bar), Это недействительно.

С apidoc, мы найдем:

destroy_all(conditions = nil) public 

destroy_all метод принимает только один аргумент, аргумент может быть строка, массив или хэш. Вы не можете передать два аргумента.

Таким образом, вы можете написать так:

Foo.destroy_all("foo = #{@bar}") 
Foo.destroy_all(foo: @bar) 
Foo.where(foo: @bar).destroy_all