1

Trabajo con C#, Visual Studio 2019 Tengo la siguiente query

using (var context = new BusinessContext())
        {
            var result = from ov in context.OrdenVentas
                         join c in context.Clientes on ov.ClienteId equals c.ClienteId
                         where ov.Estado == EstadoOrdenVenta.Emitido &&
                             (filter.Codigo == null || ov.CodigoNumerico.Contains(filter.Codigo))
                             &&
                             (!filter.ClienteId.HasValue || ov.ClienteId.Equals(filter.ClienteId.Value))
                             &&
                             (filter.FechaEmision == null || ov.FechaEmision >= filter.FechaEmision)
                             &&
                             (filter.FechaCaducidad == null || ov.FechaCaducidad <= filter.FechaCaducidad)
                         select
                             new UniversalExtend()
                             {
                                 Id = ov.OrdenVentaId,
                                 NumeroComprobante = ov.NumeroComprobante,
                                 Descripcion = c.RazonSocial,
                                 FechaEmision = ov.FechaEmision,
                                 FechaCaducidad = ov.FechaCaducidad,
                                 EstadoId = ov.Estado,
                                 Total = ov.Total
                             };
            return result.ToList();
        }

En el filtro debo agregarle un parámetro de estado el cual es emitido.

using (var context = new BusinessContext())
        {
            var result = from ov in context.OrdenVentas
                         join c in context.Clientes on ov.ClienteId equals c.ClienteId
                         where
                             (filter.Codigo == null || ov.CodigoNumerico.Contains(filter.Codigo) && ov.Estado == EstadoOrdenVenta.Emitido)
                             &&
                             (!filter.ClienteId.HasValue || ov.ClienteId.Equals(filter.ClienteId.Value) && ov.Estado == EstadoOrdenVenta.Emitido)
                             &&
                             (filter.FechaEmision == null || ov.FechaEmision >= filter.FechaEmision && ov.Estado == EstadoOrdenVenta.Emitido)
                             &&
                             (filter.FechaCaducidad == null || ov.FechaCaducidad <= filter.FechaCaducidad && ov.Estado == EstadoOrdenVenta.Emitido)
                         select
                             new UniversalExtend()
                             {
                                 Id = ov.OrdenVentaId,
                                 NumeroComprobante = ov.NumeroComprobante,
                                 Descripcion = c.RazonSocial,
                                 FechaEmision = ov.FechaEmision,
                                 FechaCaducidad = ov.FechaCaducidad,
                                 EstadoId = ov.Estado,
                                 Total = ov.Total
                             };
            return result.ToList();
        }

Pero si no le mando nada me trae a todos los emitidos, si es que no le mando nada no me debe de mostrar nada.

¿Como debería hacerlo?

Saludos!

8
  • Has probado con parámetros opcionales? Commented el 17 jun. 2019 a las 21:13
  • @PauloUrbanoRivera Es lo que tengo Commented el 17 jun. 2019 a las 21:22
  • ¿Qué es lo opuesto (o quizás sería mejor decir complementario) a ||? Porque el comportamiento que describes es justo el opuesto del que usas para los parámetros opcionales.
    – SJuan76
    Commented el 17 jun. 2019 a las 21:23
  • Por cierto, creo que has confundido los dos bloques de código (el primero debería ser el "antiguo" y el segundo el "nuevo" que has probado añadiendo Emitido)
    – SJuan76
    Commented el 17 jun. 2019 a las 21:26
  • Lo que pasa es que indicarle que tu parámetro es == null y utilizar la condición || estas automáticamente trayendo todo. Si tu filtro de estado no viene vacío se aplica sobre la base total de los registros. Commented el 17 jun. 2019 a las 21:27

2 respuestas 2

1

Sinceramente no entiendo muy bien tu pregunta, pero el titulo me da entender que es lo que necesitas.

Lo que deseas es que si una variable contiene un valor agregar el where a tu consulta SQL? si esto es así puedes guiarte del siguiente ejemplo.

    public async Task<List<Movements>> GetMovementsAsync(string idClient)
    {
        using (var db = new LITEFACTEntities())
        {
            var query =
                 (from mv in db.Movements
                  select mv);
            //Query es de tipo IQueryable, esto permite hacer mas consultas
            //generando asi nuevas consultas SQL

            if (!string.IsNullOrEmpty(idClient))
                query = query.Where(p => p.IdClient == idClient);
            //Si se cumple la condicion agrega un where a nuestra consulta SQL

            return
                await query.ToListAsync();
            //Esto ejecuta la sentencia sql almacenada en la variable query.
        }
    }

Lo que hace el código es que si me envían un idClient válido, devuelvo los movimientos de ese cliente, caso contrario se devuelve los movimientos de todos los clientes.

1
  • Adicionalmente a lo que busca en la pregunta, esta es una muy buena practica cuando se quiere hace múltiples filtros a una consulta
    – Elcainas
    Commented el 17 jun. 2019 a las 22:35
0

Lo que veo es que el nuevo filtro debe ir al final y no en cada bloque de condición porque sino necesariamente todos debería resultar en true para que pueda darte registros:

using (var context = new BusinessContext())
        {
            var result = from ov in context.OrdenVentas
                         join c in context.Clientes on ov.ClienteId equals c.ClienteId
                         where
                             (filter.Codigo == null || ov.CodigoNumerico.Contains(filter.Codigo))
                             &&
                             (!filter.ClienteId.HasValue || ov.ClienteId.Equals(filter.ClienteId.Value))
                             &&
                             (filter.FechaEmision == null || ov.FechaEmision >= filter.FechaEmision)
                             &&
                             (filter.FechaCaducidad == null || ov.FechaCaducidad <= filter.FechaCaducidad)
                             &&
                             ov.Estado == EstadoOrdenVenta.Emitido
                         select
                             new UniversalExtend()
                             {
                                 Id = ov.OrdenVentaId,
                                 NumeroComprobante = ov.NumeroComprobante,
                                 Descripcion = c.RazonSocial,
                                 FechaEmision = ov.FechaEmision,
                                 FechaCaducidad = ov.FechaCaducidad,
                                 EstadoId = ov.Estado,
                                 Total = ov.Total
                             };
            return result.ToList();
        }
2
  • De esa manera también he probado y de esa manera siempre me traera todos los emitidos yo solo quiero que me muestre los emitidos cuando se cumpla un filtro. Commented el 17 jun. 2019 a las 22:18
  • Puedes probarlo nuevamente, ya que al poner en AND el estado en la ultima condición los registros solo depende de las propiedades de tu variable filter, si las envías en null sus propiedades, te traerá todos los registros con estado emitido sin restricciones adicionales
    – Elcainas
    Commented el 17 jun. 2019 a las 22:29

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.