Saltar al contenido

Se puede simplificar código de select case?


Recommended Posts

publicado

Saludos amigos del foro, ahora me he puesto a pensar que puede simplificarse un código que uso para borrar el contenido de ciertas columnas. Si en el texbox1 coloco el 5 se borra todo el contenido de esa columna, en el rango indicado. Pero me parece que todo ese código es muy largo. 

Tengo 20 columnas en cada uno lleva informacion que en cierto tiempo puedo decidir borrar todo el contenido de una de las columnas.

Expongo el código para saber si es posible encontrar un código mas corto que haga los mismo. Pro ejemplo veo que el mensaje puede ser solo 1 y cada vez lo que cambia es el numero de columna igualmente ocurre cuando se decide cancelar el borrado, y así por el estilo:

El codigo:

Private Sub CmdBorrarCol_Click()
Dim nCol As Integer
 If TextBox1 = "" Then Exit Sub
nCol = TextBox1.Value

Select Case nCol

Case 1
 If MsgBox("Se va a borrar todo el contenido de la columna 1, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
     Range("C6:C77").ClearContents
    TextBox1 = ""
    
    Case 2
     If MsgBox("Se va a borrar todo el contenido de la columna 2, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("D6:D77").ClearContents
        TextBox1 = ""
    Case 3
     If MsgBox("Se va a borrar todo el contenido de la columna 3, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("E6:E77").ClearContents
        TextBox1 = ""
    Case 4
     If MsgBox("Se va a borrar todo el contenido de la columna 4, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("F6:F77").ClearContents
        TextBox1 = ""
    Case 5
     If MsgBox("Se va a borrar todo el contenido de la columna 5, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("G6:G77").ClearContents
        TextBox1 = ""

    Case 6
     If MsgBox("Se va a borrar todo el contenido de la columna 6, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("H6:H77").ClearContents
        TextBox1 = ""

    Case 7
     If MsgBox("Se va a borrar todo el contenido de la columna 7, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("I6:I77").ClearContents
        TextBox1 = ""
    Case 8
     If MsgBox("Se va a borrar todo el contenido de la columna 8, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("J6:J77").ClearContents
        TextBox1 = ""
    Case 9
     If MsgBox("Se va a borrar todo el contenido de la columna 9, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("K6:K77").ClearContents
        TextBox1 = ""
    Case 10
     If MsgBox("Se va a borrar todo el contenido de la columna 10, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("L6:L77").ClearContents
        TextBox1 = ""
Case 11
 If MsgBox("Se va a borrar todo el contenido de la columna 11, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
     Range("M6:M77").ClearContents
    TextBox1 = ""
    
    Case 12
     If MsgBox("Se va a borrar todo el contenido de la columna 12, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("N6:N77").ClearContents
        TextBox1 = ""
    Case 13
     If MsgBox("Se va a borrar todo el contenido de la columna 13, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("O6:O77").ClearContents
        TextBox1 = ""
    Case 14
     If MsgBox("Se va a borrar todo el contenido de la columna 14, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("P6:P77").ClearContents
        TextBox1 = ""
    Case 15
     If MsgBox("Se va a borrar todo el contenido de la columna 15, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("Q6:Q77").ClearContents
        TextBox1 = ""

    Case 16
     If MsgBox("Se va a borrar todo el contenido de la columna 16, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("R6:R77").ClearContents
        TextBox1 = ""

    Case 17
     If MsgBox("Se va a borrar todo el contenido de la columna 17, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("S6:S77").ClearContents
        TextBox1 = ""
    Case 18
     If MsgBox("Se va a borrar todo el contenido de la columna 18, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("T6:T77").ClearContents
        TextBox1 = ""
    Case 19
     If MsgBox("Se va a borrar todo el contenido de la columna 19, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("U6:U77").ClearContents
        TextBox1 = ""
    Case 20
     If MsgBox("Se va a borrar todo el contenido de la columna 20, Presione Aceptar para continuar.", vbOKCancel, "Excel123") = vbCancel Then
        MsgBox "Has cancelado el borrado de la columna"
        Exit Sub
    End If
    'MsgBox "Se procede a borrar."
    Range("V6:V77").ClearContents
        TextBox1 = ""
End Select
End Sub

Agradezco su gentil apoyo en colaborar en este tema

publicado

(p.ej.)

Private Sub CmdBorrarCol_Click()
  Dim nCol As Integer, rangoBorrar As String
  If TextBox1 = "" Then Exit Sub Else nCol = TextBox1.Value
  Select Case nCol
    Case 1 To 20
      rangoBorrar = Range("b6:b77").Offset(, nCol).Address
      If MsgBox("Se va a borrar el contenido de la columna " & nCol & vbCr & _
        "Presiona ""Aceptar"" para continuar", vbOKCancel, "") = vbCancel Then
          MsgBox "Operacion cancelada por el usuario"
          Exit Sub
      End If
      Range(rangoBorrar).ClearContents
      TextBox1 = ""
    Case Else
      MsgBox "El valor introducido en el control de texto NO es válido !!!"
  End Select
End Sub

saludos,

hector.

publicado

Ya veo!! La clave offset, .....a partir de la columna B se ubica automáticamente en la posición nCol que se le haya puesto en el textbox1 que lo toma como rango a borrar en la variable rangoborrar. En caso de introducir un valor fuera del 1-20 me advierte y termina la instrucción, esto ultimo no lo tenia en el código subido y si que es importante.

Efectivamente simplificado

Gracias por este apoyo. Quedo perfecto y he podido usar este modo para corregir otros códigos similares.

Tema solucionado

  • 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

  • Current Donation Goals

    • Raised 0.00 EUR of 130.00 EUR target
  • 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.