8
¿Es posible crear algo más limpio de esta consulta dinámica:Creación de consultas dinámicas en función de parámetro pasado en los carriles 3
@photos = Photo.within(100, :origin => [params[:latitude], params[:longitude]]) unless (params[:latitude].nil? || params[:longitude].nil?)
if @photos.nil? then
conditions = String.new
values = Array.new
params.each_key do |key|
if key == 'since_id' then
conditions << " AND " unless conditions.length == 0
conditions << "id > ?"
values << params[key]
elsif key == 'user_id' then
conditions << " AND " unless conditions.length == 0
conditions << "user_id = ?"
values << params[key]
elsif key == 'id' then
conditions << " AND " unless conditions.length == 0
conditions << "id = ?"
values << params[key]
end
end
values.insert(0, conditions)
@photos = Photo.limit(15).order("created_at DESC").where(values) unless values.nil?
end
Sí, lo es. Dado que no tendría sentido tener un ID como params y un since_id (deben ser mutuamente excluyentes) –
La consulta que genero está bien. Lo que quiero hacer es hacerlo más bonito. Creo que es engorroso como lo planteé. Estoy buscando alternativas que lo hagan más fácil de mantener –
si tengo todo lo correcto, solo 'id' o' since_id' está presente. ¿Para qué es 'Photo.within'? – Bohdan