create conditions in model

rubyonrails-talk
1

Hello guys,

I trying to create the conditions of my search, but I have some troubles.

This is the method I’m trying to create in my model.

def self.searchadv(title, place, category, date)
    !title.blank? ? conditions = ['title LIKE ?', "%#{title}%"] : conditions = []
    if conditions
!place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] :  conditions << []
        !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] :  conditions << []
        !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] :  conditions << []
    else
        !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] :  conditions << []
        !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] :  conditions << []
        !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] :  conditions << []
    end
    find(:all, :conditions => conditions)
end

I get this error

wrong number of bind variables (4 for 1) in: title LIKE ?

if I delete this:

if conditions
!place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] :  conditions << []
    !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] :  conditions << []
    !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] :  conditions << []
else
    !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] :  conditions << []
    !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] :  conditions << []
    !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] :  conditions << []
end

Everything works great, but I need this other options in order to create my search and I don’t undertand why the error is in the “LiKE”

Does anyone could help me please?

Thanks in advance!

2

Hello guys,

I trying to create the conditions of my search, but I have some troubles.

This is the method I'm trying to create in my model.

def self.searchadv(title, place, category, date) !title.blank? ? conditions = ['title LIKE ?', "%#{title}%"] : conditions =

This (and the remainder) would probably be more readable if you used present? rather than !blank?

if conditions
    \!place\.blank? ? conditions &lt;&lt; \[&#39; AND place LIKE ?&#39;, &quot;%\#\{place\}%&quot;\] :  conditions &lt;&lt; \[\]

I think you are confusing << with concat. This code appends a single item to conditions (that is a 0 or 2 item array) which isn't what you intended. I'd probably write this as

conditions += [' and ...', "%#{place}" if place.present?

More readable (I think) would be

fragments = bind_variables =

if title.present?   fragments << 'title LIKE ?'   bind_variables << title end

If place.present?   fragments << 'category like ?'   bind_variables << %#{place}%"   ... end

conditions = [fragments.join(' AND ')] + bind_variables

Which removes some of the duplication you currently have. You might also want to look at something like squeel, which extends active record to make it easier to write this sort of query.

Fred

3

def self.searchadv(title, place, category, date)

Colour me doubtful, but since you're passing in category and date, why are you using place in your category and date subclauses?

        !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] : conditions <<

I really like the way Fred puts it together by joining the fragments into the where clause.

4

Hello guys,

I trying to create the conditions of my search, but I have some troubles.

This is the method I'm trying to create in my model.

def self.searchadv(title, place, category, date)     !title.blank? ? conditions = ['title LIKE ?', "%#{title}%"] : conditions =     if conditions         !place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] : conditions <<     else         !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] : conditions <<     end     find(:all, :conditions => conditions)end

I get this error

wrong number of bind variables (4 for 1) in: title LIKE ?

if I delete this:

if conditions     !place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] : conditions << else     !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] : conditions << end

Everything works great, but I need this other options in order to create my search and I don't undertand why the error is in the "LiKE"

Does anyone could help me please?

Thanks in advance!

you can also do it like this

def self.searchadv(title, place, category, date)     klass = scoped

    klass = klass.where(conditions for title here) if title.present? klass = klass.where(conditions for place here) if place.present? klass = klass.where(conditions for category here) if category.present?    klass = klass.where(conditions for date here) if date.present?

    scopedend

This is using rails 3. But you can still use this for rails 2.3 (i'm not sure what version they introduced this), you just have to change the where calls to scoped

5

Hello guys,

I trying to create the conditions of my search, but I have some troubles.

This is the method I'm trying to create in my model.

def self.searchadv(title, place, category, date)     !title.blank? ? conditions = ['title LIKE ?', "%#{title}%"] : conditions =     if conditions         !place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] : conditions <<     else         !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] : conditions <<         !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] : conditions <<     end     find(:all, :conditions => conditions)end

I get this error

wrong number of bind variables (4 for 1) in: title LIKE ?

if I delete this:

if conditions     !place.blank? ? conditions << [' AND place LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' AND category LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' AND date LIKE ?', "%#{place}%"] : conditions << else     !place.blank? ? conditions << [' place LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' category LIKE ?', "%#{place}%"] : conditions <<     !place.blank? ? conditions << [' date LIKE ?', "%#{place}%"] : conditions << end

Everything works great, but I need this other options in order to create my search and I don't undertand why the error is in the "LiKE"

Does anyone could help me please?

Thanks in advance!

you can also do it like this

def self.searchadv(title, place, category, date)

    klass = scoped

    klass = klass.where(conditions for title here) if title.present? klass = klass.where(conditions for place here) if place.present? klass = klass.where(conditions for category here) if category.present? klass = klass.where(conditions for date here) if date.present?

    scopedend

This is using rails 3. But you can still use this for rails 2.3 (i'm not sure what version they introduced this), you just have to change the where calls to scoped

the last line that calls scoped should be klass. my bad.