Saltar al contenido

Error de compilación...


RubenDario

Recommended Posts

publicado

Viendo el error y el libro, no tienes una hoja llamada Hoja1, se llama "sumatorias", ademas no puedes referirte directamente a una hoja por su nombre en el código. En su lugar, debes usar la colección Worksheets o Sheets.

Algo así:

With Worksheets("sumatorias")

Haz la Prueba, esto debería ayudarte en base a lo expuesto en el tema.

Saludines.

publicado

Tu libro y código editado.

Option Explicit
Sub Billetes()
'
' Billetes Macro
'
' Acceso directo: Ctrl+Mayús+L
'
    With Worksheets("sumatorias")
        .Range("B1:B13").ClearContents
        .Range("A1").Select
    End With
End Sub
Sub BorraColumnas()
'
' BorraColumnas Macro
'
' Acceso directo: Ctrl+Mayús+K
'
    With Worksheets("sumatorias")
        .Range("E1:E21,G1:G21,I1:I21,K1:K21,M1:M21").ClearContents
        .Range("A1").Select
    End With
End Sub

 

SumasVarias - Ayuda Excel.xlsm

publicado

Hola

Estás usando de forma incorrecta Selection ya que al ser una propiedad del objeto Application, basta y sobra con señalar con Select, de forma previa, el ojbeto, y listo, no es necesario volver a hacer referencia a dicho objeto, es decir, solo quita esos puntos delante de tus Selection y listo. Ojo, es correcto hacer referencia al nombre del módulo de la hoja, que en ese caso sí es Hoja1, no es necesario y/o obligatorio hacer referencia al nombre de la hoja, que sí es "sumatorías" (aunque también se puede usar); o sea, son correctas ambas formas, es decir, nuestro amigo no estaba cometiendo un error al uasar el nombre del módulo de la hoja. Saludos a ambos. 

publicado

Al margen de los consejos ya mostrados y ciñiéndome estrictamente a tu consulta, sustituye .Selection por Selection, o sea elimina el punto de .Selection y listo.

publicado

Muchísimas gracias estimados @Israel Cassales @Abraham Valencia y a ti @Antoni que siempre me has dado soluciones que hasta lo sigo usando con gran utilidad !!

Aprendo mucho de todos ustedes.

Verán, cuando tengo otra hoja en el mismo libro en el cual también necesito hacer correr esas mismas macros me salta otro error de compilación.

¿Cómo hago para que el ámbito solo tenga efecto en la hoja actual? 

¿Debo usar acaso:

Private Sub Worksheet_Activate()

End Sub ?

Algo así... 😨😨

Gracias muchachos, siempre !

Rubén

Captura de pantalla 2023-11-21 103156.png

SumasVarias.xlsm

publicado

Claro, sale un error porque ni los módulos de las hojas, ni los nombres de las hojas son iguales (y no pueden ser iguales) y tú estás intentando usar mismas macros, en donde figura una hoja en particular, para diferentes hojas. Usa "ActiveSheet" en lugar del nombre del módulo de la hoja.

PD: En general se recomienda no usar Select, Selection o ActiveSheet

publicado
hace 1 hora, Abraham Valencia dijo:

Claro, sale un error porque ni los módulos de las hojas, ni los nombres de las hojas son iguales (y no pueden ser iguales) y tú estás intentando usar mismas macros, en donde figura una hoja en particular, para diferentes hojas. Usa "ActiveSheet" en lugar del nombre del módulo de la hoja.

PD: En general se recomienda no usar Select, Selection o ActiveSheet

Esta respuesta es más corta y la he leído entera, jajaja 😉 Un abrazo.

publicado
hace 22 horas, Abraham Valencia dijo:

Claro, sale un error porque ni los módulos de las hojas, ni los nombres de las hojas son iguales (y no pueden ser iguales) y tú estás intentando usar mismas macros, en donde figura una hoja en particular, para diferentes hojas. Usa "ActiveSheet" en lugar del nombre del módulo de la hoja.

PD: En general se recomienda no usar Select, Selection o ActiveSheet

Gracias Abraham por tu amable respuesta.

Me dices que use "ActiveSheet" pero en el postdata me dices que no es recomendable usar "ActiveSheet".

En realidad yo quiero hacer una macro para cada hoja (de ser posible eso). Entonces debo cambiar "With Hoja1" por "With Hoja2". ¿O tener un solo código usando "ActiveSheet" ?

Desde ya agradezco tu ayuda.

Gracias.

Rubén

publicado

Resumiendo:

Sub Billetes()
'
' Billetes Macro
'
' Acceso directo: Ctrl+Mayús+L
'
    With ActiveSheet
        .Range("B1:B13").ClearContents
        .Range("A1").Select
    End With
End Sub
'---------------------------------
Sub BorraColumnas()
'
' BorraColumnas Macro
'
' Acceso directo: Ctrl+Mayús+K
'
    With ActiveSheet
        .Range("E1:E21,G1:G21,I1:I21,K1:K21,M1:M21").ClearContents
        .Range("A1").Select
    End With
End Sub

O también y más sencillo:

Sub Billetes()
'
' Billetes Macro
'
' Acceso directo: Ctrl+Mayús+L
'   
        Range("B1:B13").ClearContents
        Range("A1").Select
End Sub
Sub BorraColumnas()
'
' BorraColumnas Macro
'
' Acceso directo: Ctrl+Mayús+K
'
        Range("E1:E21,G1:G21,I1:I21,K1:K21,M1:M21").ClearContents
        Range("A1").Select
End Sub

 

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.