Created
November 24, 2011 12:18
-
-
Save rodrigopinto/1391227 to your computer and use it in GitHub Desktop.
A long journey of day at the office
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
### 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 |
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
Então. Hoje sei como você se sentia. Vejo coisas muito mais bizarras que isso aqui e quando eu refatoro, levo esporro. rs
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
Não pode usar um IN nesse caso?
Algo +/- assim: