Skip to main content
3 de 3
se añadieron 159 caracteres en el cuerpo
LuisC
  • 365
  • 1
  • 8

recomendaciones para el buen uso de querys

Estoy haciendo uso de un query para la busqueda de registros, tengo dos tablas, una es "venta" y la otra es "detallevet"(esta hace referencia a los detalles de la venta) mediante este query procedo a hacer la busqueda de las ventas en su modelo correspondient

  def self.busqueda_general(params)
  query = joins("left outer join detallevet on venta.Documento=detallevet.Docto and venta.RutaId=detallevet.RutaId")
                     .where("(venta.RutaId = :rutaId or :rutaId = '') AND (detallevet.Articulo = :articulo or :articulo = '')  AND (venta.CodCliente = :codcliente or :codcliente = '')",{rutaId: params[:search], articulo: params[:search3], codcliente: params[:search2]})
                     .distinct
  end

luego llamo dicho metodo en el controller en un metodo llamado busqueda_ventas

  def busqueda_ventas
   @detallevet = Detalleve.all
    @ventas = Vent.busqueda_general(params)

    respond_to do |format|
    format.js
    end
  end

las ventas estan relacionadas a los detalleventa pero no por un campo venta_id que los identifique, sino por medio de dos campos, uno llamado documento y otro RutaId en ambas tablas, ambas combinaciones deben de ser iguales para relacionarse. en mi vista estoy ordenando todas mis ventas con sus detalles respectivamente, para hacer esto en metodo del controller busque todas los detalle de esta forma @detallevet = Detalleve.all y en la vista hice un render de la siguiente manera

<%= render `@detallevet.where(:Docto => vent.Documento, :RutaId => vent.RutaId) %>

al igual que para el mostrado de datos:

 <td><%=vent.IVA%></td>
        <td><%[email protected](:Docto => vent.Documento, :RutaId => vent.RutaId ).sum(:DescMon)%></td>
        <td><%=vent.TOTAL%></td>
        <td><%[email protected](:Docto => vent.Documento, :RutaId => vent.RutaId).count%></td>

        <td><%[email protected](:Docto => vent.Documento, :RutaId => vent.RutaId).sum(:Pza)%></td>

pero me parece que estoy haciendo uso de malas practicas colocando ese tipo de consultas en la vista, y de ser asi de que forma podria mejorarlo?

CODIGO ACTUALIZADO:

  has_many :detallevet, -> (vent) { where(Docto: vent.Documento, RutaId: vent.RutaId) },
           class_name: 'Detalleve'
LuisC
  • 365
  • 1
  • 8