2

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'

1 respuesta 1

1

Creo que el principal problema de tu implementación (aparte de no estar siguiendo el patrón MVC como corresponde haciendo la consulta en la vista), es que por cada venta estás realizando 3 consultas a la base de datos, provocando 3N + 1 consultas, siendo N la cantidad de ventas a mostrar (Ej: Si tu consulta @ventas = Vent.busqueda_general(params) te está retornando 20 registros, para mostrar toda la info en la vista terminaste realizando 61 consultas a la base de datos en vez de tan solo una).
Yo aprovecharía el join que estás realizando en busqueda_general y agregaría todos esos sum y count que hiciste en la vista a la consulta (fíjate en el select):

  def self.busqueda_general(params)
    select("venta.*, COUNT(detallevet.*) AS venta_count,
            SUM(detallevet.DescMon) AS sum_DescMon,
            SUM(detallevet.Pza) AS sum_Pza")
      .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])
      .group(:id)
      .distinct
  end

Luego en la vista llamaría las sumatorias y contador de cada venta, basándome en el alias que puse con AS dentro del select:

<td><%=vent.IVA%></td>
<td><%=vent.sum_DescMon)%></td>
<td><%=vent.TOTAL%></td>
<td><%=vent.venta_count%></td>
<td><%=vent.sum_Pza)%></td>

Para acceder a los detalles de una venta, simplemente pon un has_many con condiciones:

has_many :detalles, -> (vent) { where(Docto: vent.Documento, RutaId: vent.RutaId) },
         class_name: 'Detalleve'

luego llamas a:

vent.detalles
17
  • gracias por responder. intento colocarlo tal cual lo publicaste pero me salta un error en la terminal en la consulta, es el siguiente: ActionView::Template::Error (TinyTds::Error: Incorrect syntax near ' * '.: EXEC sp_executesql N'SELECT DISTINCT venta.*, COUNT(detallevet.*) AS venta_count, SUM(detallevet.DescMon) AS sum_DescMon, SUM(detallevet.Pza) AS sum_Pza FROM [venta] LEFT OUTER JOIN detallevet ON venta.Documento=detallevet.Docto AND venta.RutaId=detallevet.RutaId WHERE ((venta.RutaId = N''172'' OR N''172'' = '''')
    – LuisC
    Commented el 23 dic. 2016 a las 21:23
  • @LuisC la verdad ni idea tengo de sql server, ya que nunca lo he usado, pero al parecer la sintaxis que pongas manualmente debe ir encerrada entre corchetes, algo como select("[venta].*, COUNT([detallevet].*) AS venta_count, tal como sale en las consultas de esta respuesta Commented el 23 dic. 2016 a las 23:39
  • gracias Alter, lo cambie de la forma en que me mencionaste y ademas le cambie a comillas simples y funciono, pero ahora me manda un error en el select, estoy intentando varias combinaciones para tratar de hacerlo funcionar, y he notado que si dejo el [venta].* no da dicho error
    – LuisC
    Commented el 24 dic. 2016 a las 1:45
  • pero si ademas agrego los metodos count con la otra tabla me da el siguiente error: Column 'venta.Id' is invalid in the select list because it is not contained in either an aggregate function or the GROUP BY clause.: EXEC sp_executesql N'SELECT [venta].*, SUM([detallevet].DescMon) AS [sum_DescMon] FROM [venta] left outer join detallevet ON venta.Documento=detallevet.Docto'):
    – LuisC
    Commented el 24 dic. 2016 a las 1:45
  • 1
    perfecto ya publique mi otra duda, muchisimas gracias Alter por ayudarme a corregir mi codigo, no tienes idea de cuanto me ha servido tu ayuda, muchisimas gracias amigo.
    – LuisC
    Commented el 28 dic. 2016 a las 15:26

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.