Reducir código vba en proyecto.

Para Dante Amor

Hola con respecto a mi proyecto de aplicación VAM creo que el código que llevo creado se podría reducir bastante y me gustaría le echaras un vistazo y me dijeras como podría hacerlo.

1 respuesta

Respuesta
1

H o l a:

Cuando tienes las mismas tareas en varios controles, lo puedes reducir creando un procedimiento general, por ejemplo, de esto que tienes:

Private Sub Op_Limpieza_Diaria_Click()
If Op_Limpieza_Diaria.Value = True Then
     Cmb_Denominacion1.Value = ""
     Cmb_Denominacion2.Value = ""
     Cmb_Denominacion3.Value = ""
     Cmb_Denominacion4.Value = ""
     Cmb_Denominacion5.Value = ""
     Cmb_Denominacion6.Value = ""
     Cmb_Denominacion1.Visible = True
     Cmb_Denominacion2.Visible = True
     Cmb_Denominacion3.Visible = True
     Cmb_Denominacion4.Visible = True
     Cmb_Denominacion5.Visible = True
     Cmb_Denominacion6.Visible = True
     Cmb_Denominacion1.List() = Range("Limpieza_Diaria").Value
     Cmb_Denominacion2.List() = Range("Limpieza_Diaria").Value
     Cmb_Denominacion3.List() = Range("Limpieza_Diaria").Value
     Cmb_Denominacion4.List() = Range("Limpieza_Diaria").Value
     Cmb_Denominacion5.List() = Range("Limpieza_Diaria").Value
     Cmb_Denominacion6.List() = Range("Limpieza_Diaria").Value
     End If
End Sub
Private Sub Op_Limpieza_Vam_Click()
If Op_Limpieza_Vam.Value = True Then
     Cmb_Denominacion1.Value = ""
     Cmb_Denominacion2.Value = ""
     Cmb_Denominacion3.Value = ""
     Cmb_Denominacion4.Value = ""
     Cmb_Denominacion5.Value = ""
     Cmb_Denominacion6.Value = ""
     Cmb_Denominacion1.Visible = True
     Cmb_Denominacion2.Visible = True
     Cmb_Denominacion3.Visible = True
     Cmb_Denominacion4.Visible = True
     Cmb_Denominacion5.Visible = True
     Cmb_Denominacion6.Visible = True
     Cmb_Denominacion1.List() = Range("Limpieza_Vam").Value
     Cmb_Denominacion2.List() = Range("Limpieza_Vam").Value
     Cmb_Denominacion3.List() = Range("Limpieza_Vam").Value
     Cmb_Denominacion4.List() = Range("Limpieza_Vam").Value
     Cmb_Denominacion5.List() = Range("Limpieza_Vam").Value
     Cmb_Denominacion6.List() = Range("Limpieza_Vam").Value
     End If
End Sub
Private Sub Op_Oficina_Diaria_Click()
If Op_Oficina_Diaria.Value = True Then
     Cmb_Denominacion1.Value = ""
     Cmb_Denominacion2.Value = ""
     Cmb_Denominacion3.Value = ""
     Cmb_Denominacion4.Value = ""
     Cmb_Denominacion5.Value = ""
     Cmb_Denominacion6.Value = ""
     Cmb_Denominacion1.Visible = True
     Cmb_Denominacion2.Visible = True
     Cmb_Denominacion3.Visible = True
     Cmb_Denominacion4.Visible = True
     Cmb_Denominacion5.Visible = True
     Cmb_Denominacion6.Visible = True
     Cmb_Denominacion1.List() = Range("Oficina_Diaria").Value
     Cmb_Denominacion2.List() = Range("Oficina_Diaria").Value
     Cmb_Denominacion3.List() = Range("Oficina_Diaria").Value
     Cmb_Denominacion4.List() = Range("Oficina_Diaria").Value
     Cmb_Denominacion5.List() = Range("Oficina_Diaria").Value
     Cmb_Denominacion6.List() = Range("Oficina_Diaria").Value
     End If
End Sub
Private Sub Op_Oficina_Vam_Click()
If Op_Oficina_Vam.Value = True Then
     Cmb_Denominacion1.Value = ""
     Cmb_Denominacion2.Value = ""
     Cmb_Denominacion3.Value = ""
     Cmb_Denominacion4.Value = ""
     Cmb_Denominacion5.Value = ""
     Cmb_Denominacion6.Value = ""
     Cmb_Denominacion1.Visible = True
     Cmb_Denominacion2.Visible = True
     Cmb_Denominacion3.Visible = True
     Cmb_Denominacion4.Visible = True
     Cmb_Denominacion5.Visible = True
     Cmb_Denominacion6.Visible = True
     Cmb_Denominacion1.List() = Range("Oficina_Vam").Value
     Cmb_Denominacion2.List() = Range("Oficina_Vam").Value
     Cmb_Denominacion3.List() = Range("Oficina_Vam").Value
     Cmb_Denominacion4.List() = Range("Oficina_Vam").Value
     Cmb_Denominacion5.List() = Range("Oficina_Vam").Value
     Cmb_Denominacion6.List() = Range("Oficina_Vam").Value
     End If
End Sub
Private Sub Op_Otros_Click()
If Op_Otros.Value = True Then
     Cmb_Denominacion1.Value = ""
     Cmb_Denominacion2.Value = ""
     Cmb_Denominacion3.Value = ""
     Cmb_Denominacion4.Value = ""
     Cmb_Denominacion5.Value = ""
     Cmb_Denominacion6.Value = ""
     Cmb_Denominacion1.Visible = False
     Cmb_Denominacion2.Visible = False
     Cmb_Denominacion3.Visible = False
     Cmb_Denominacion4.Visible = False
     Cmb_Denominacion5.Visible = False
     Cmb_Denominacion6.Visible = False
     Txt_Denominacion1.Visible = True
     Txt_Denominacion2.Visible = True
     Txt_Denominacion3.Visible = True
     Txt_Denominacion4.Visible = True
     Txt_Denominacion5.Visible = True
     Txt_Denominacion6.Visible = True
     End If
End Sub

Podría quedar así:

Private Sub OP_LIMPIEZA_DIARIA_Click()
'Por.Dante Amor
    rango = "LIMPIEZA_DIARIA"
    carga rango, True
End Sub
Private Sub OP_LIMPIEZA_VAM_Click()
    rango = "LIMPIEZA_VAM"
    carga rango, True
End Sub
Private Sub OP_OFICINA_DIARIA_Click()
    rango = "OFICINA_DIARIA"
    carga rango, True
End Sub
Private Sub OP_OFICINA_VAM_Click()
    rango = "OFICINA_VAM"
    carga rango, True
End Sub
Private Sub OP_OTROS_Click()
    carga "OFICINA_VAM", False
End Sub
Sub carga(rng, vis)
'Por.Dante Amor
    For i = 1 To 6
        Me.Controls("CMB_DENOMINACION" & i).Value = ""
        Me.Controls("CMB_DENOMINACION" & i).Visible = vis
        Me.Controls("CMB_DENOMINACION" & i).List() = Range(rng).Value
    Next
End Sub

sal u dos

Añade tu respuesta

Haz clic para o

Más respuestas relacionadas

Detectar cambios en tabla dinámica

Tengo una hoja donde se van actualizando datos mediante una tabla dinámica. ¿Cómo detecto cuando se producen cambios (sólo en la parte de la tabla dinámica) para ejecutar otra macro en la misma hoja?
1 respuesta

Macro con combinación de teclas

Acudo aquí buscando ayuda, ¿Cómo crear una macro que genere una combinación de teclas? NO se trata de asignar una macro a una combinación de teclas, es todo lo contrario. El problema radica en que tengo una macro que rellena ciertas celdas con...
1 respuesta

Pasar datos de listbox a hoja con decimales

Como están estimados maestros y maestras, una vez mas solicitando vuestra experta ayuda. Tengo un formulario que busca datos entre 2 fechas desde 2 textbox, carga en listbox lo filtrado, en la columna 2 del listbox carga valores con formato de 2...
1 respuesta

Una macro que de aviso sobre vencimiento de una cedula de identidad

Tengo una hoja de calculo de 6 columnas en donde la columna A es fecha de vencimiento, las columnas B, C, y D son apellidos paterno, Materno u nombres, la columna E es el numero de cedula de identidad Con Formato ##.###.###-# la columna F es lo...
1 respuesta

Error decimales en hoja de excel

Tengo un albarán de varios productos que no vienen desglosado el iva En una hoja Excel introduzco el precio, con la formula sumar el precio lo desgloso en base imponible y el iva paracque me dé el precio del producto pero cuando sumo el total de...
4 respuestas

Filtrar información y luego pasar a otro libro

Tengo una macro que copia y pega del libro llamado Origen y Hoja1, dónde este está cerrado, al libro Destino y Hoja 2 de la siguiente manera: Sub importar() Application.ScreenUpdating = False Application.Calculation = xlCalculationManual...
1 respuesta

Código VBA para ocultar formulario

Tengo un formulario para colocar USUARIO y CONTRASEÑA, lo cual lo hace bie, pero requiero que una vez se introduzca los datos correctos, el formulario se oculte y me redirecciona u "hoja1" Intenté ocultarlos con esta instrucción Private Sub...
2 respuestas

Diferentes Formato de Fecha en Hoja de Calculo

Pregunta a Elsa Matilde Estoy trabajando en un formulario VB donde los 2 TextBox veo que la fecha tiene formato 01/11/2024, pero cuando pasa los datos desde el formulario a la hoja de trabajo una columna se refleja 01/11/24 y la otra columna...
1 respuesta

Como insertar de forma separada registros en diferentes columnas en una misma fila

Estoy tratando de realizar un formulario de registro de asistencia que registre la hora de entrada, la hora de inicio y fin del cafe, la hora de inicio y fin del amuerzo y la hora de la salida. Hay unos botones de opción que seleccionan el tipo de...
1 respuesta

Macro para copiar valor de texbox a hoja2

Necesito un poco de ayuda con una macro estoy aprendiendo mi duda es la siguiente tengo una hoja llamada Principal utilizo un botón de comando para llamar a userform1 el cual tiene varios texbox numerados 1,2,3,4,5,6,7,8,9,10,11 aquí utilizo solo...
1 respuesta