Cargar Matriz en Excel con VBA

Anónimas
2013-07-03T14:07:20+00:00

Buenas como estan,

mire tengo u gran problema en Excel con VBA tengo este codigo, y deseo simplificarlo y corregir el error de que no me carga bien la matriz en la hoja llamada General, que podre hacer, en negrita es donde esta la simplificacion y error:

Public NuevoNombre As String

Sub CopiarHojaDeArchivosEnDirectorio()

  Dim Ruta As String, Archivo As String

    Application.ScreenUpdating = False

    With ThisWorkbook

      Ruta = .Path & ""

      Archivo = Dir(Ruta & "*.xls*")

      Do While Archivo <> ""

        If Archivo = .Name Then GoTo Siguiente

        Application.StatusBar = "Procesando..." & Archivo

        Workbooks.Open Ruta & Archivo

        Worksheets(3).Copy After:=.Worksheets(.Worksheets.Count)

        CambiaNombre (Archivo)

        .Worksheets(.Worksheets.Count).Name = NuevoNombre

        For C = 3 To 49

            For A = 5 To 40

                .Worksheets(.Worksheets.Count).Activate

        '                Asigna valor de fila G + valor(A) en Dato

                Dato = Worksheets(.Worksheets.Count).Cells(A, 7).Value

                Porciento = Worksheets(.Worksheets.Count).Cells(1, 8).Value

                Programa1 = Worksheets(.Worksheets.Count).Cells(5, 2).Value                Programa2 = Worksheets(.Worksheets.Count).Cells(8, 2).Value                Programa3 = Worksheets(.Worksheets.Count).Cells(13, 2).Value                Programa4 = Worksheets(.Worksheets.Count).Cells(18, 2).Value                Programa5 = Worksheets(.Worksheets.Count).Cells(21, 2).Value                Programa6 = Worksheets(.Worksheets.Count).Cells(25, 2).Value                Programa7 = Worksheets(.Worksheets.Count).Cells(32, 2).Value                Programa8 = Worksheets(.Worksheets.Count).Cells(37, 2).Value                Programa9 = Worksheets(.Worksheets.Count).Cells(39, 2).Value                Windows("Consolida.xlsm").Activate                Worksheets(2).Cells(.Worksheets.Count, 4).Value = Programa1                Worksheets(2).Cells(.Worksheets.Count, 8).Value = Programa2                Worksheets(2).Cells(.Worksheets.Count, 14).Value = Programa3                Worksheets(2).Cells(.Worksheets.Count, 20).Value = Programa4                Worksheets(2).Cells(.Worksheets.Count, 24).Value = Programa5                Worksheets(2).Cells(.Worksheets.Count, 29).Value = Programa6                Worksheets(2).Cells(.Worksheets.Count, 38).Value = Programa7                Worksheets(2).Cells(.Worksheets.Count, 44).Value = Programa8                Worksheets(2).Cells(.Worksheets.Count, 47).Value = Programa9

B = A - 1                If Worksheets(2).Cells(C, B).Value = "" Then                    If A < 41 Then        '                    Asigna Dato en fla D + valor(C)                        Worksheets(2).Cells(C, B).Value = Dato                    Else                        B = A        '                    Asigna Dato en fla D + valor(C)                        Worksheets(2).Cells(C, B).Value = Dato                    End If                End If                Next A            A = 5        Next C

        Windows("Consolida.xlsm").Activate

        Worksheets(2).Cells(.Worksheets.Count, 2).Value = NuevoNombre

        Worksheets(2).Cells(.Worksheets.Count, 3).Value = Porciento

        Workbooks(Archivo).Close False

Siguiente:

            Archivo = Dir()

          Loop

        End With

        Application.StatusBar = "Listo"

End Sub

Public Function CambiaNombre(cadena As String) As String

Dim Posicion As Integer, Caracter As Variant

'definimos una matriz sin especificar elementos

Dim Matr() As Variant

'la redefinimos y le asignamos un número de elementos

'igual al número de caracteres que componen la cadena a evaluar

ReDim Preserve Matr(Len(cadena))

'recorremos cada uno de dichos caracteres

For Posicion = 1 To Len(cadena)

    Caracter = Mid$(cadena, Posicion, 1)

    If Caracter <> " " Then

        Matr(Posicion) = Caracter

        NuevoNombre = Join(Matr, "")

    Else

        Exit For

    End If

    Next Posicion

End Function

Microsoft 365 y Office | Excel | Para la casa | Windows

Pregunta bloqueada. Esta pregunta se migró desde la Comunidad de Soporte técnico de Microsoft. Puede votar si es útil, pero no puede agregar comentarios o respuestas ni seguir la pregunta.

0 comentarios No hay comentarios
Respuesta aceptada por el autor de la pregunta
Héctor Miguel 71,595 Puntos de reputación
2013-07-03T20:03:16+00:00

sube tu archivo a una carpeta pública de TU SkyDrive, obtén una ruta para su descarga y compártela en tu siguiente mensaje

de todas formas, hay algunos "conceptos" que no has analizado completamente (p.e.)

  1. lo que hace la función, ya lo "leí", le pasas como parámetro el nombre del libro que se abre y lo "recortas" en caso de encontrarse algún espacio " ", para asignarlo como nombre a la hoja recién agregada, la pregunta es: porque recortas al espacio ?

=> (no pusiste dos o tres ejemplos)

  1. Windows(... es claro que se refiere a un libro abierto, PERO... "ThisWorkbook" NO ES... ninguna hoja, es EL LIBRO donde esta pegado el código que expones, por lo que NO CAMBIA de nombre (según el bucle) a "XXXX"

nueva (RE)pregunta:

hay algún rango "especifico" para obtener los datos de la hoja(3) copiada de cada libro y agregada al libro (ThisWorkbook) ?

saludos,

hector.

p.d. agrego: cómo se llama el libro donde está el código que expones ?

¿Le ha resultado útil esta respuesta?

0 comentarios No hay comentarios

41 respuestas adicionales

Ordenar por: Muy útil
  1. Anónimas
    2013-07-05T13:59:30+00:00

    Aquí mi solución. SIN MACROS. Espero tus críticas.

    Ya sé que no es lo que pide el interesado, pero es como debe hacerse: los datos se escriben en la hoja Datos, con una tabla dinámica obtienes los resúmenes deseados y, si necesitas rellenar el formulario, lo haces con macro o fórmulas. 

    https://skydrive.live.com/#!/edit.aspx?cid=C1A49B2EC89DEB86&resid=C1A49B2EC89DEB86%21306&app=Excel

    ¿Le ha resultado útil esta respuesta?

    0 comentarios No hay comentarios
  2. Anónimas
    2013-07-05T00:06:38+00:00

    Hector se que esta conclusion ya la sabes sos un genio en la materia, muchas gracias, espero llegar algun dia a ser como vos.

    te felicito

    Muchas Gracias.

    ¿Le ha resultado útil esta respuesta?

    0 comentarios No hay comentarios
  3. Héctor Miguel 71,595 Puntos de reputación
    2013-07-04T22:08:57+00:00

    esto ES lo que necesitas hacer (en la hoja "general" del libro "consolida")...

    1. reenumerar las celdas:

    [AY35] => 6.6 (o 6,6 según tu separador)

    [Ay36] => 6.7 (o 6,7 según tu separador)

    1. ELIMNAR la columna AK (ya que NO existe un 6.8 -de acuerdo ?-)
    2. ELIMINA el modulo de código donde tienes la macro que llamas con el "CommandButton" de la hoja2
    3. copia/pega (o escribe) el siguiente procedimiento EN el modulo de código de la hoja2 (en el evento '_click' del "CommandButton2)

    Private Sub CommandButton1_Click()

      Dim Ruta As String, Archivo As String, n As Integer

      Application.ScreenUpdating = False

      Ruta = ThisWorkbook.Path & ""

      n = Worksheets(2).Cells(Rows.Count, 3).End(xlUp).Row ' o... => Worksheets("general")

      If n > 2 Then _

        If MsgBox("Ya existen registros en la BD" & vbCr & "Deseas reemplazar los datos ?", _

          vbOKCancel, "ALERTA !!!") = vbCancel Then Exit Sub Else [c3].CurrentRegion.Offset(2).ClearContents

      Archivo = Dir(Ruta & "*.xls*")

      n = 2

      Do While Archivo <> ""

        If Archivo = ThisWorkbook.Name Then GoTo Siguiente Else n = n + 1

        Workbooks.Open Ruta & Archivo

        Archivo = StrConv(Left(Archivo, InStr(Archivo, " ") - 1), vbProperCase)

        Application.StatusBar = "Procesando: " & Archivo & " ..."

        With Worksheets(3)

          ThisWorkbook.Worksheets(2).Range("b" & n).Resize(, 47) = Array(Archivo, .[h1], _

            .[b5], .[g5], .[g6], .[g7], _

            .[b8], .[g8], .[g9], .[g10], .[g11], Null, _

            .[b13], .[g13], .[g14], .[g15], .[g16], .[g17], _

            .[b18], .[g18], .[g19], .[g20], _

            .[b21], .[g21], .[g22], .[g23], .[g24], _

            .[b25], .[g25], .[g26], .[g27], .[g28], .[g29], .[g30], .[g31], _

            .[b32], .[g32], .[g33], .[g34], .[g35], .[g36], _

            .[b37], .[g37], .[g38], _

            .[b39], .[g39], .[g40])

        End With

        ActiveWorkbook.Close False

    Siguiente:

        Archivo = Dir()

      Loop

      Worksheets(2).Activate

      Application.StatusBar = False

    End Sub

    saludos,

    hector.

    ¿Le ha resultado útil esta respuesta?

    0 comentarios No hay comentarios
  4. Anónimas
    2013-07-04T19:00:46+00:00

    Esmérate. ;-)))

    ¿Le ha resultado útil esta respuesta?

    0 comentarios No hay comentarios