Saltar al contenido

Macro para resumir datos


Recommended Posts

publicado

Hola,
Tengo un libro que uso como base de datos en un hotel. En la Hoja1 registro todos los datos relativos a reservas, entradas, salidas, facturación, etc., y en diferentes hojas resumo algunos datos de la BD conforme a determinados criterios para obtener distintos tipos de estadísticas.
Inicialmente usé fórmulas para los resúmenes, pero el proceso de cálculo ralentizaba el funcionamiento del libro, por ejemplo al abrirlo.
Entonces decidí grabar las fórmulas en una macro en combinación con el evento Worksheet_SelectionChange(ByVal Target As Range), lo cual agilizó considerablemente el proceso pero me obliga a clickar/seleccionar la fecha en el calendario para que los datos de esa fecha se actualicen.

Solicito vuestro asesoramiento para encontrar una macro que me permita conseguir que los datos se actualicen en las hojas de resumen de forma automática al registrar los datos en la BD.

Adjunto archivo de ejemplo.
Gracias.

REGISTRO.rar

publicado

No entiendo que quieres hacer.
creo que este código debería estar dentro de un for each que recorra todas las filas, parecido al que pones más abajo

    If IsDate(Range("A" & Target.Row)) Then
        Range("C" & Target.Row) = "=COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ent"",Hoja1!C4,""ind"")+COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ocu"",Hoja1!C4,""ind"")"
        Range("D" & Target.Row) = "=COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ent"",Hoja1!C4,""dbl"")+COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ocu"",Hoja1!C4,""dbl"")"
        Range("E" & Target.Row) = "=COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ent"",Hoja1!C4,""tpl"")+COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ocu"",Hoja1!C4,""tpl"")"
        Range("F" & Target.Row) = "=SUMIFS(Hoja1!C6,Hoja1!C1,RC1,Hoja1!C3,""<>res"")-SUMIFS(Hoja1!C6,Hoja1!C1,RC1,Hoja1!C3,""clx"")"
        Range("G" & Target.Row) = "=SUMIFS(Hoja1!C7,Hoja1!C1,RC1,Hoja1!C3,""<>res"")-SUMIFS(Hoja1!C7,Hoja1!C1,RC1,Hoja1!C3,""clx"")"
        Range("H" & Target.Row) = "=SUMIFS(Hoja1!C8,Hoja1!C1,RC1,Hoja1!C3,""<>res"")-SUMIFS(Hoja1!C8,Hoja1!C1,RC1,Hoja1!C3,""clx"")"
        Range("I" & Target.Row) = "=SUMIFS(Hoja1!C10,Hoja1!C1,RC1,Hoja1!C3,""<>res"")-SUMIFS(Hoja1!C10,Hoja1!C1,RC1,Hoja1!C3,""clx"")"
        Range("J" & Target.Row) = "=SUMIFS(Hoja1!C11,Hoja1!C1,RC1,Hoja1!C3,""<>res"")-SUMIFS(Hoja1!C11,Hoja1!C1,RC1,Hoja1!C3,""clx"")"
    End If

Yo traería los valores de la hoja1 y haría las operaciones en el código de visual basic en lugar de pegar esas fórmulas, eso andaría 10 veces más rápido.   

 

 

 

publicado
Cita
Hace 36 minutos , seba64 dijo:

Yo traería los valores de la hoja1 y haría las operaciones en el código de visual basic en lugar de pegar esas fórmulas, eso andaría 10 veces más rápido.

 

Gracias seba64, ¿me puedes hacer una línea de código de ejemplo de lo que sugieres?

publicado

¿que es lo que deberían hacer las funciones?

"=COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ent"",Hoja1!C4,""ind"")+COUNTIFS(Hoja1!C1,RC1,Hoja1!C3,""ocu"",Hoja1!C4,""ind"")"
publicado

mmm...

No llego a entender lo que hace tu código, no se si lo entiendes tú. Parece que busca celdas protegidas o celdas agrupadas, no se. Quitando todo lo que no entiendo, yo veo algo como esto

Private Sub Worksheet_SelectionChange(ByVal Target As Range)
    hoja = "Hoja3"
    f = Sheets(hoja).Cells(Sheets(hoja).Rows.Count, 1).End(xlUp).Row
    Application.ScreenUpdating = False
    For i = 2 To f
    Produccion Range("A" & i)
    Debug.Print i
    Next i
    Application.ScreenUpdating = True
End Sub

Se demora por las funciones que pusiste. Deberías tratar de usar código visual basic y no funciones de excel. el debug es para ver el progreso, lo puedes quitar.

Suerte :rolleyes:

 

publicado

Tu código no solo no lo entiendo, seba64, sino que además en las pruebas que he realizado siempre me bloquea el ordenador.

publicado

El código es tuyo, si no entiendes el mío me da a pensar que tampoco entiendes el tuyo. Y no se pega está trabajando pero esas funciones que usas son lentisimas. Debí haberte dicho que miraras la pantalla de visual basic y no la hoja. Ahí se muestra la línea en que va, y termina en 380. Te pregunte que hacen esas funciones, porque de verdad no lo entendía.

Lo que tiene tu código es que si seleccionas una o varias celdas, sólo va a tomar la que cliqueaste primero.

Lo que yo hice fue un bucle que recorre todas las fechas de la hoja3, ya que según entendí tu problema era tener que seleccionarlas una a una,  Los códigos no son tan diferentes, sólo quite las instrucciones que como te mencione antes parecen ser para quitar la protección o para desagrupar celdas

Private Sub Worksheet_SelectionChange(ByVal Target As Range)
    Application.ScreenUpdating = False
    If Target.Rows.OutlineLevel < 3 Then Exit Sub                          <!--esto no se que hace -->
    uf = Range("A" & Rows.Count).End(xlUp).Row                             <!--a esto yo le llamo f en lugar de uf -->
    col = "A2:A" & uf
    If Not Intersect(Target, Range(col)) Is Nothing Then Produccion Target <!--Si no esta protegida lo manda a produccion-->
    Application.ScreenUpdating = True
End Sub

 

Private Sub Worksheet_SelectionChange(ByVal Target As Range)
    f = Sheets("Hoja3").Cells(Sheets(hoja).Rows.Count, 1).End(xlUp).Row  <!-- Cuenta las filas con datos-->
    Application.ScreenUpdating = False
    For i = 2 To f					 <!-- Empieza un ciclo para recorrer todas las celdas -->
    Produccion Range("A" & i)		 <!-- Manda a produccion cada fechas, es decir una a una-->
    Debug.Print i					 <!-- Esto muestra en visual basic en que línea va (lo puedes quitar) -->
    Next i							 <!-- Acá le indica que siga con la siguiente-->
    Application.ScreenUpdating = True
End Sub

No se que no entiendes porque el código es casi el mismo, con la diferencia de que resuelve tu problema que era:

Cita

me obliga a clickar/seleccionar la fecha en el calendario para que los datos de esa fecha se actualicen.

Bueno ahora podrás apreciar lo lentas que son tus funciones.

 

publicado

Siempre es de agradecer una información tan detallada. Y sobre todo que tomes tu tiempo para intentar ayudarme con este tema.

Dices que el problema es que uso unas funciones muy lentas, y de lo que no sé no discuto.

Lo que si sé es que busco alternativas que mejoren mis opciones y como comprenderás, que se cuelgue el equipo no es una mejora.

Gracias.

publicado

No sé si funciona o no, después de un rato con el equipo colgado tuve que llamar al administrador de tareas para detener el proceso y poder trabajar con el ordenador.

  • Silvia bloqueó este tema

Archivado

Este tema está ahora archivado y está cerrado a más respuestas.

  • 109 ¿Te parecen útiles los tips de las funciones? (ver tema completo)

    1. 1. ¿Te parecen útiles los tips de las funciones?


      • No
      • Ni me he fijado en ellos

  • Ayúdanos a mejorar la comunidad

    • Donaciones recibidas este mes: 0.00 EUR
      Objetivo: 130.00 EUR
  • Archivos

  • Estadísticas de descargas

    • Archivos
      188
    • Comentarios
      98
    • Revisiones
      29

    Más información sobre "Cambios en el Control Horario"
    Última descarga
    Por pegones1

    4    1

  • Crear macros Excel

  • Mensajes

    • Hola, veo que tienes 365, así que esta forma funcionará   Almacen.xlsx
    • Buenos días  @LeandroA espero estes bien Tengo un caso idéntico al planteado en la siguiente pregunta: Sin embargo, a diferencia de quien planteo originalmente la pregunta al correr el código no obtengo ningún resultado podrían ayudarme a resolver este inconveniente y que al hacer click en el Botón Guardar (CommandButton3) del Formulario RCS (frmrcs) el archivo pdf quede configurado con orientación vertical, márgenes superior, inferior, derecho e izquierdo = 1 y en página tamaño carta. Si acaso influye uso Microsoft Excel LTSC MSO (versión 2209 Compilación16.0.1.15629.20200) de 64 bits Mucho le sabre agradecer la ayuda que me pueda dar  RCS PRUEBA - copia.xlsm
    • @JSDJSDCon gusto mi estimado Para la opción 1: Sub Surtirhastadondealcanse() Dim ws As Worksheet Set ws = ThisWorkbook.Sheets(1) Dim filaInicio As Integer: filaInicio = 4 Dim filaFin As Integer: filaFin = 7 Dim colInventario As Integer: colInventario = 2 Dim colSolicitudesInicio As Integer: colSolicitudesInicio = 4 ' Columna C Dim colResultadoInicio As Integer: colResultadoInicio = 9 ' Columna I Dim colTotalSurtido As Integer: colTotalSurtido = 12 ' Columna L Dim colFinalInventario As Integer: colFinalInventario = 13 ' Columna M Dim numClientes As Integer: numClientes = 3 Dim fila As Integer, i As Integer For fila = filaInicio To filaFin Dim inventario As Double inventario = Val(ws.Cells(fila, colInventario).Value) Dim solicitudes(1 To 3) As Double Dim surtido(1 To 3) As Variant Dim totalSurtido As Double: totalSurtido = 0 ' Leer solicitudes For i = 1 To numClientes If IsNumeric(ws.Cells(fila, colSolicitudesInicio + i - 1).Value) Then solicitudes(i) = CDbl(ws.Cells(fila, colSolicitudesInicio + i - 1).Value) Else solicitudes(i) = 0 End If surtido(i) = "POR FALTA STOCK" Next i ' Surtir de acuerdo al inventario disponible For i = 1 To numClientes If solicitudes(i) > 0 Then If inventario >= solicitudes(i) Then surtido(i) = solicitudes(i) inventario = inventario - solicitudes(i) totalSurtido = totalSurtido + solicitudes(i) ElseIf inventario > 0 Then surtido(i) = inventario totalSurtido = totalSurtido + inventario inventario = 0 Else surtido(i) = "POR FALTA STOCK" End If End If Next i ' Escribir resultados en las columnas correspondientes para cada cliente For i = 1 To numClientes With ws.Cells(fila, colResultadoInicio + i - 1) If surtido(i) = "POR FALTA STOCK" Then .Value = surtido(i) .Font.Color = vbRed Else .Value = surtido(i) .Font.Color = vbBlack End If End With Next i ' Escribir total surtido y existencia final ws.Cells(fila, colTotalSurtido).Value = totalSurtido ws.Cells(fila, colFinalInventario).Value = inventario Next fila MsgBox "Resultado surtido cargado con éxito...", vbInformation End Sub Para la opción 2:   Sub surtirenpartesiguales() Dim ws As Worksheet Set ws = ThisWorkbook.Sheets(1) Dim filaInicio As Integer: filaInicio = 13 Dim filaFin As Integer: filaFin = 16 Dim colInventario As Integer: colInventario = 2 Dim colSolicitudesInicio As Integer: colSolicitudesInicio = 4 ' Columna C Dim colResultadoInicio As Integer: colResultadoInicio = 9 ' Columna I Dim colTotalSurtido As Integer: colTotalSurtido = 12 ' Columna L Dim colFinalInventario As Integer: colFinalInventario = 13 ' Columna M Dim numClientes As Integer: numClientes = 3 Dim fila As Integer, i As Integer For fila = filaInicio To filaFin Dim inventario As Double inventario = Val(ws.Cells(fila, colInventario).Value) Dim solicitudes(1 To 3) As Double Dim surtido(1 To 3) As Variant Dim totalSurtido As Double: totalSurtido = 0 Dim totalPedido As Double: totalPedido = 0 ' Leer solicitudes For i = 1 To numClientes If IsNumeric(ws.Cells(fila, colSolicitudesInicio + i - 1).Value) Then solicitudes(i) = CDbl(ws.Cells(fila, colSolicitudesInicio + i - 1).Value) totalPedido = totalPedido + solicitudes(i) Else solicitudes(i) = 0 End If surtido(i) = 0 Next i ' Si hay suficiente inventario, surtir lo que el cliente pide If inventario >= totalPedido Then For i = 1 To numClientes If solicitudes(i) > 0 And inventario >= solicitudes(i) Then surtido(i) = solicitudes(i) inventario = inventario - solicitudes(i) totalSurtido = totalSurtido + solicitudes(i) End If Next i Else ' Reparto base igualitario Dim baseSurtido As Long baseSurtido = Int(inventario / numClientes) For i = 1 To numClientes If solicitudes(i) > 0 Then If solicitudes(i) <= baseSurtido Then surtido(i) = solicitudes(i) inventario = inventario - solicitudes(i) totalSurtido = totalSurtido + solicitudes(i) Else surtido(i) = baseSurtido inventario = inventario - baseSurtido totalSurtido = totalSurtido + baseSurtido End If End If Next i ' Repartir sobrante restante uno por uno, respetando lo pedido Do While inventario > 0 For i = 1 To numClientes If surtido(i) < solicitudes(i) Then surtido(i) = surtido(i) + 1 totalSurtido = totalSurtido + 1 inventario = inventario - 1 If inventario = 0 Then Exit For End If Next i Loop End If ' Escribir resultados en las columnas correspondientes para cada cliente For i = 1 To numClientes With ws.Cells(fila, colResultadoInicio + i - 1) If surtido(i) = 0 Then .Value = "POR FALTA STOCK" .Font.Color = vbRed Else .Value = surtido(i) .Font.Color = vbBlack End If End With Next i ' Escribir total surtido y existencia final ws.Cells(fila, colTotalSurtido).Value = totalSurtido ws.Cells(fila, colFinalInventario).Value = inventario Next fila MsgBox "Resultado surtido cargado con éxito...", vbInformation End Sub Saludos, Diego
    • Buenos dias.  Estoy trabajando en una hoja para poder llevar un control de un pequeño almacén.  Tengo un pedido con varias líneas y "lotes" y necesito sacar las ubicaciones que coincidan con la referencia y lote que pone en el pedido. El problema viene cuando tengo la misma referencia y mismo lote en ubicaciones diferentes y necesito sacar la información en columnas diferentes. No se si  me he explicado bien, pero creo que con el ejemplo adjunto se entiende mejor. Agradecería mucho si me pudieran ayudar  Libro1.xlsx
    • Exelente solución mil gracias 
  • Visualizado recientemente

    • No hay usuarios registrado para ver esta página.
×
×
  • Crear nuevo...

Información importante

Echa un vistazo a nuestra política de cookies para ayudarte a tener una mejor experiencia de navegación. Puedes ajustar aquí la configuración. Pulsa el botón Aceptar, si estás de acuerdo.