Solicitud revisión código por inconveniente en variable

Para Dante Amor

Hola buenas tardes:

Nuevamente recurro a su generosidad y excelente dominio del tema, para solicitarle su amable colaboración con la revisión del código para el formulario Informe_por_Cliente, ya que estoy teniendo dificultades cuando la base de datos supera determinada cantidad de registros, creo que es porque hay algunas variables de tipo long que me limitan.

Al correo le envíe el archivo, el siguiente es el código:

'Informe por Cliente/Destino
Private Sub ComboBox1_Enter()
Dim i As Double
Dim final As Double
Dim tareas As String
'Muestra Razon social del proveedor
For i = 2 To 30000
If Hoja3.Cells(i, 4) = "" Then 'Columna D Hoja Salidas
final = i - 1
Exit For
End If
Next
'Muestra Razon social del proveedor
  For i = 2 To final
  tareas = Hoja3.Cells(i, 4)
  ComboBox1.AddItem (tareas)
  Next
End Sub
Private Sub CommandButton1_Click()
Application.ScreenUpdating = False
Dim nuevo As Object
Dim i As Integer
Dim H As Integer
Dim L As Integer
Dim m As Integer
Dim j As Integer
Dim T As Integer
Dim FINALTOTAL As Integer
Dim final As Integer
Dim FINAL2 As Integer
Dim ORIGEN As String
Dim SALDO As Double
Dim VALOR As String
Dim CONTAR As Double
Dim CONTAR1 As Double
Set nuevo = Workbooks.Add
nuevo.Activate
ORIGEN = ActiveWorkbook.Name
For i = 1 To 30000
'SALIDAS
If Hoja3.Cells(i, 4) = "" Then 'Columna D Hoja salidas
final = i - 1
Exit For
End If
Next
VALOR = Informe_por_Cliente.ComboBox1
' SALIDAS
CONTAR = 10
' ASIGNAR VALORES PARA EL INFORME
Application.Workbooks(ORIGEN).Worksheets(1).Cells(1, 1) = "INFORME SALIDAS POR CLIENTE/DESTINO"
Application.Workbooks(ORIGEN).Worksheets(1).Cells(4, 2) = VALOR 'Descripcion del Cliente
For L = 1 To 30000
If Hoja3.Cells(L, 5) = VALOR Then
Application.Workbooks(ORIGEN).Worksheets(1).Cells(4, 4) = Hoja3.Cells(L, 4) 'Razon social del proveedor
Exit For
End If
Next
For j = 1 To final   'la varible guardó última fila de hoja salidas
If Hoja3.Cells(j, 4) = VALOR Then
CONTAR = CONTAR + 1
'funciona salidas
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 2) = Hoja3.Cells(j, 1) 'Codigo Item
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 3) = Hoja3.Cells(j, 2) 'Descricpion Item
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 4) = Hoja3.Cells(j, 3) 'Cantidad
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 5) = Hoja3.Cells(j, 5) 'Fecha Salida
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 6) = Hoja3.Cells(j, 6) 'No de documento
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 7) = Hoja3.Cells(j, 7) 'Valor
End If
Next

1 respuesta

Respuesta
1

Envíame tu archivo para revisar las macros.

Mi correo [email protected]

En el asunto del correo escribe tu nombre de usuario “Gilber Fabian Tobar Diaz”

S a l u d o s . D a n t e   A m o r

Te anexo el código para encontrar la última fila 

'Informe por Cliente/Destino
Private Sub ComboBox1_Enter()
Dim i As Double
Dim final As Double
Dim tareas As String
'Muestra Razon social del proveedor
'For i = 2 To 30000
'If Hoja3.Cells(i, 4) = "" Then 'Columna D Hoja Salidas
'final = i - 1
'Exit For
'End If
'Next
final = Hoja3.Range("D" & Rows.Count).End(xlUp).Row
'Muestra Razon social del proveedor
  For i = 2 To final
  tareas = Hoja3.Cells(i, 4)
  ComboBox1.AddItem (tareas)
  Next
End Sub
Private Sub CommandButton1_Click()
Application.ScreenUpdating = False
Dim nuevo As Object
Dim i As Integer
Dim H As Integer
Dim L As Integer
Dim m As Integer
Dim j As Integer
Dim T As Integer
Dim FINALTOTAL As Integer
Dim final As Integer
Dim FINAL2 As Integer
Dim ORIGEN As String
Dim SALDO As Double
Dim VALOR As String
Dim CONTAR As Double
Dim CONTAR1 As Double
Set nuevo = Workbooks.Add
nuevo.Activate
ORIGEN = ActiveWorkbook.Name
'For i = 1 To 30000
''SALIDAS
'If Hoja3.Cells(i, 4) = "" Then 'Columna D Hoja salidas
'final = i - 1
'Exit For
'End If
'Next
final = Hoja3.Range("D" & Rows.Count).End(xlUp).Row
VALOR = Informe_por_Cliente.ComboBox1
' SALIDAS
CONTAR = 10
' ASIGNAR VALORES PARA EL INFORME
Application.Workbooks(ORIGEN).Worksheets(1).Cells(1, 1) = "INFORME SALIDAS POR CLIENTE/DESTINO"
Application.Workbooks(ORIGEN).Worksheets(1).Cells(4, 2) = VALOR 'Descripcion del Cliente
For L = 1 To final
If Hoja3.Cells(L, 5) = VALOR Then
Application.Workbooks(ORIGEN).Worksheets(1).Cells(4, 4) = Hoja3.Cells(L, 4) 'Razon social del proveedor
Exit For
End If
Next
For j = 1 To final   'la varible guardó última fila de hoja salidas
If Hoja3.Cells(j, 4) = VALOR Then
CONTAR = CONTAR + 1
'funciona salidas
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 2) = Hoja3.Cells(j, 1) 'Codigo Item
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 3) = Hoja3.Cells(j, 2) 'Descricpion Item
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 4) = Hoja3.Cells(j, 3) 'Cantidad
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 5) = Hoja3.Cells(j, 5) 'Fecha Salida
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 6) = Hoja3.Cells(j, 6) 'No de documento
Application.Workbooks(ORIGEN).Worksheets(1).Cells(CONTAR, 7) = Hoja3.Cells(j, 7) 'Valor
End If
Next
'Formatos
 Range("A1:G1").Select
    With Selection
        .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    With Selection
        .HorizontalAlignment = xlLeft
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = True
    End With
    With Selection.Interior
        .ColorIndex = 10
        .Pattern = xlSolid
    End With
    Selection.Font.ColorIndex = 2
    Selection.Font.Bold = True
    Range("A4").Select
    ActiveCell.FormulaR1C1 = "CLIENTE / DESTINO"
    Range("B5:F5").Select
    With Selection
        .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    With Selection
        .HorizontalAlignment = xlLeft
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = True
    End With
    Range("B4:F4").Select
    With Selection
        .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    With Selection
        .HorizontalAlignment = xlLeft
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = True
    End With
    Range("B3:F3").Select
    With Selection
        .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    With Selection
        .HorizontalAlignment = xlLeft
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = True
    End With
    Range("B6:F6").Select
    With Selection
        .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    With Selection
        .HorizontalAlignment = xlLeft
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = True
    End With
'Encabezados del informe
    Range("B8").Select
    ActiveCell.FormulaR1C1 = ""
    Range("B9").Select
    ActiveCell.FormulaR1C1 = "CODIGO ITEM"
    Range("C9").Select
    ActiveCell.FormulaR1C1 = "DESCRIPCION"
    Range("D9").Select
    ActiveCell.FormulaR1C1 = "CANTIDAD"
    Range("E9").Select
    ActiveCell.FormulaR1C1 = "F.SALIDA"
    Range("F9").Select
    ActiveCell.FormulaR1C1 = "No. DCMTO"
    Range("G9").Select
    ActiveCell.FormulaR1C1 = "VALOR"
    Range("B8:G8").Select
    With Selection
         .HorizontalAlignment = xlCenter
        .VerticalAlignment = xlBottom
        .WrapText = False
        .Orientation = 0
        .AddIndent = False
        .IndentLevel = 0
        .ShrinkToFit = False
        .ReadingOrder = xlContext
        .MergeCells = False
    End With
    Selection.Merge
    Selection.Borders(xlDiagonalDown).LineStyle = xlNone
    Selection.Borders(xlDiagonalUp).LineStyle = xlNone
    With Selection.Borders(xlEdgeLeft)
        .LineStyle = xlContinuous
        .Weight = xlMedium
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeTop)
        .LineStyle = xlContinuous
        .Weight = xlMedium
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeBottom)
        .LineStyle = xlContinuous
        .Weight = xlMedium
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeRight)
        .LineStyle = xlContinuous
        .Weight = xlMedium
        .ColorIndex = xlAutomatic
    End With
    Selection.Borders(xlInsideVertical).LineStyle = xlNone
    ActiveCell.FormulaR1C1 = "SALIDAS PARA EL CLIENTE/DESTINO SELECCIONADO"
    Range("B8:G8").Select
    With Selection.Interior
        .ColorIndex = 3
        .Pattern = xlSolid
    End With
    Selection.Font.ColorIndex = 2
    Selection.Font.Bold = True
    Range("B9:G9").Select
    Selection.Font.Bold = True
    Range("B9:G9").Select
    Selection.Borders(xlDiagonalDown).LineStyle = xlNone
    Selection.Borders(xlDiagonalUp).LineStyle = xlNone
    With Selection.Borders(xlEdgeLeft)
        .LineStyle = xlContinuous
        .Weight = xlThin
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeTop)
        .LineStyle = xlContinuous
        .Weight = xlThin
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeBottom)
        .LineStyle = xlContinuous
        .Weight = xlThin
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlEdgeRight)
        .LineStyle = xlContinuous
        .Weight = xlThin
        .ColorIndex = xlAutomatic
    End With
    With Selection.Borders(xlInsideVertical)
        .LineStyle = xlContinuous
        .Weight = xlThin
        .ColorIndex = xlAutomatic
    End With
    Cells.Select
    Cells.EntireColumn.AutoFit
    Range("A2").Select
'Formato de fecha
    Range("E11").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.NumberFormat = "m/d/yyyy"
    Range("B11").Select
    Informe_por_Cliente.ComboBox1 = ""
'Formato de numero
    Range("G11").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.NumberFormat = "#,##0.00"
    Range("E10").Select
Unload Me
Unload Informes
Unload Menu_Principal
'Hoja2.Visible = xlSheetVeryHidden
'Hoja3.Visible = xlSheetVeryHidden
'Hoja5.Visible = xlSheetVeryHidden
'Hoja6.Visible = xlSheetVeryHidden
 Hoja1.Cells(3, 1) = ""
End Sub

.

'S aludos. Dante Amor. Recuerda valorar la respuesta. G racias

.

Avísame cualquier duda

.

Añade tu respuesta

Haz clic para o

Más respuestas relacionadas

Ejecutar macro si en la celda B1 hay datos

Esta es la macro que utilizo siempre, pero tengo la dificultad que a veces me olvido de colocar el numero de curso (Ejp. 1era_1era o 2da_2da ...)en la celda B1, todo este proceso se ejecuta en la hoja Activa "Curso", lo que busco que la macro se...
2 respuestas

¿Qué hay de mal en este macro?

Quiero hacer un macro para hacer un filtro de una base de datos poniendo como condición un periodo de tiempo, teniendo una celda como fecha de inicio y otra celda como fecha de terminación. Te mando imagen de mi libro y código. Muchas gracias Sub...
1 respuesta

Dividir un valor en excel y que el resultado sea igual al de una calculadora manual

Sres. T.E La Consulta es la siguiente: Como dividir 4.888,00 / 30 y que el resultado sea igual tanto en excel como una calculadora manual o la calculado de win xp. Ejemplos: En excel 2003 si divido 4.888,00 / 30 = 162,93 En una Calculadora 4.888,00 /...
2 respuestas

No puedo registrar más de 30k renglones, sale error de desbordamiento (Error 6)

Tengo el siguiente código, que de un formulario pasa los datos a una tabla. El problema es que después de aprox. 35,000 renglones registrados, al querer seguir registrando me aparece el error de desbordamiento (Error 6). Comparto código, si de favor...
3 respuestas

Optimizar macro para combinar filas

Tengo una macro que acumula valores de filas determinados según columnas similares, funciona bien con una tabla de datos intermedia en cuanto a su tiempo de ejecución, lo que quiero es optimizar esta macro o mejorarla ya que tengo tablas de hasta...
3 respuestas

Ordenar listado en excel por numero de tienda

Tengo un listado de productos en excel y en la misma celda viene el numero de la tienda y los totales de cada tienda, pero si tengo más de 100 tiendas quiero agregar una columna donde me diga que tienda es y así poder ordenar mi listado por el número...
1 respuesta

Hacer un nuevo archivo copiando la hoja2 y hoja3 de una lista de nombres que están en la hoja1.

En la hoja1 tengo una lista de nombres y quiero copiar la hoja=evaluación y hoja=metas a un nuevo libro con el nombre de la lista de la hoja1 las dos hojas en el mismo nuevo libro.
1 respuesta

Tengo una macro , que necesita que de una lista de alumnos

Imprime un informe de un alumno en particular, necesito que el usuario digite un numero y luego siga con los pasos siguientes, es decir que imprima el usuario seleccionado . ¿Cómo puedo pausar una macro?
2 respuestas

Convertir Texto a Número con Decimales

Tengo mis TextBox donde guardo información pero esta la guarda como texto y deseo convertirlos a Números con Decimales ya utilice Val pero me los convierte en Enteros es al momento de guardar el dato y al momento de modificar el dato. Abocándome a...
4 respuestas

Fórmula Excel para sumar columnas con fechas

Estoy tratando de buscar alguna fórmula que me pueda ayudar a sumar diferentes columnas con fechas; indicándole el rango de fechas y el criterio a considerar. Es decir que tome en cuenta lo siguiente: Fecha de Inicio, Fecha de Termino, Producto....
3 respuestas