Skip to content

Instantly share code, notes, and snippets.

@rodrigopinto
Created November 24, 2011 12:18
Show Gist options
  • Save rodrigopinto/1391227 to your computer and use it in GitHub Desktop.
Save rodrigopinto/1391227 to your computer and use it in GitHub Desktop.
A long journey of day at the office
### THIS IS PART OF A QUERY
def conditions_for_regioncode(region_ids)
regions = []
region_ids.each do |region|
regions << "reg.regioncode = '#{region}'"
end
"(#{regions.join(' OR ')})"
end
@tapajos
Copy link

tapajos commented Nov 24, 2011

Não pode usar um IN nesse caso?

Algo +/- assim:

where(["reg.regioncode in (?)", region_ids])

@rodrigopinto
Copy link
Author

Vixxi claro que pode, esse foi o código que encontrei para refatorar.
Só para constar:

  • isto está em um controller
  • o outro método que o chama, tmb está no controller
  • e dentro da action tem um find_by_sql de 60 linhas
  • na view tinha uma outra query relaciona a core da funcionalida.

Isso só para o começo! Não sabe da missa a metáde!

hoje estou resmungando só para não guardar magoa KKKKKKKK

@fbazzarella
Copy link

Então. Hoje sei como você se sentia. Vejo coisas muito mais bizarras que isso aqui e quando eu refatoro, levo esporro. rs

@rrmartins
Copy link

Estou pensando em começar a juntar as coisas bizarras daqui, e mandar para o Planeta Bizarro! LOL kkkk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment