Saltar al contenido

Macro par importar datos de un libro a otro de excel


Recommended Posts

publicado

Buenos días

Podríais ayudarme a refinar esta macro que realice con la grabadora.

Gracias

 

Sub MetodoAbrirLibro()

Workbooks.Open "C:\Users\jctorres\Desktop\pruebas partes automaticos\C2020-0138_Carga_Horas (1)2.xls"


    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("A8:f78").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("Personal").Range("A8:f78").PasteSpecial xlPasteValues
        
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("F2").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("Personal").Range("G3").PasteSpecial xlPasteValues
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("I2").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("Personal").Range("H3").PasteSpecial xlPasteValues
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("H2").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("h2").PasteSpecial xlPasteValues
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("H3:m3").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("H3:m3").PasteSpecial xlPasteValues
 
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("l2").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("l2").PasteSpecial xlPasteValues
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("G8:H78").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("G8:H78").PasteSpecial xlPasteValues

    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("J8:K78").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("J8:K78").PasteSpecial xlPasteValues
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Sheets(13).Range("M8:M78").Copy
    Windows("C2020-0138_Carga_Horas (1)2.xls").Activate
    Sheets("OT1").Range("M8:M78").PasteSpecial xlPasteValues
   
    
    Windows("PARTE DE TRABAJOS EE-II.xlsm").Activate
    Application.CutCopyMode = False


End Sub

 

publicado

@Juan Carlos torres ruiz , te dejo una solución un poco a vuelapluma... Comprueba los rangos a copiar en cada hoja, creo que los he seguido bien, pero compruebalo. Como no has subido tus archivos para ver si se puede optimizar algo más, prueba el siguiente código:
 

Sub MetodoAbrirLibro()
Dim rngCopy As Range, rngOT1 As Range, cel As Range, adres$
Dim wbOr As Workbook, wbDes As Workbook

Application.ScreenUpdating = False

Set wbOr = ThisWorkbook
Set wbDes = Workbooks.Open("C:\Users\jctorres\Desktop\pruebas partes automaticos\C2020-0138_Carga_Horas (1)2.xls")

With wbOr.Sheets(13)
    .Range("A8:F78").Copy wbDes.Sheets("Personal").Range("A8:F78")
    .Range("F2").Copy wbDes.Sheets("Personal").Range("G3")
    .Range("I2").Copy wbDes.Sheets("Personal").Range("H3")

    Set rngOT1 = .Range("A:M")
    Set rngCopy = Intersect(rngOT1, Union(.Range("H2"), .Range("H3:M3"), .Range("12"), .Range("G8:H78"), .Range("J8:K78"), .Range("M8:M78")))

    For Each cel In rngCopy
        adres = cel.Address
        cel.Copy wbDes.Sheets("OT1").Range(adres)
    Next cel
End With

Application.CutCopyMode = False

End Sub

 

publicado
Hace 2 horas, Juan Carlos torres ruiz dijo:

Buenas , muchas gracias, me da un error con las celdas combinadas el resto lo hace perfecto.

Esto pasa por no subir los archivos... :(

Cambia el código del siguiente bucle a lo siguiente y prueba:
 

For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT1").Range(adres).FormulaR1C1 = cel.Text
Next cel

 

publicado
Hace 21 horas, Juan Carlos torres ruiz dijo:

Ahora que he pasado la macro al archivo original , me aparece este error, y por mas que le doy vueltas no consigo ver el fallo

Ese error es bastante inespecífico, aunque suele indicar que algo no encuentra. En tu caso creo que es por todos los nombres con rango que tienes. La manera de poder pasar los datos es separar el copy y el paste. Usa el siguiente código:
 

With wbOr.Sheets("EPYC")
    .Range("A8:F78").Copy
    wbDes.Sheets("Personal").Range("A8:F78").PasteSpecial xlPasteValues
    .Range("F2").Copy
    wbDes.Sheets("Personal").Range("G3").PasteSpecial xlPasteValues
    .Range("I2").Copy
    wbDes.Sheets("Personal").Range("H3").PasteSpecial xlPasteValues

    Set rngOT1 = .Range("A:M")
    Set rngCopy = Intersect(rngOT1, Union(.Range("H2"), .Range("H3"), .Range("l2"), .Range("G8:H78"), .Range("J8:K78"), .Range("M8:M78")))

    For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT1").Range(adres).FormulaR1C1 = cel.Text
    Next cel
End With

 

publicado

Muchas gracias , ya no da ese error.

Estoy duplicando la macro para que rellene mas hojas del archivo y me encuentro que que los datos de la celda h3 en la hoja ot2 se copia lo mismo que en la hoja ot1 .

Sub MetodoAbrirLibro()
Dim rngCopy As Range, rngOT1 As Range, cel As Range, adres$
Dim wbOr As Workbook, wbDes As Workbook

Application.ScreenUpdating = False

Set wbOr = ThisWorkbook
Set wbDes = Workbooks.Open("C:\Users\jctorres\Desktop\pruebas partes automaticos\C2020-0138_Carga_Horas (1)2.xls")

With wbOr.Sheets("EPYC")
    .Range("A8:F78").Copy
    wbDes.Sheets("Personal").Range("A8:F108").PasteSpecial xlPasteValues
    .Range("F2").Copy
    wbDes.Sheets("Personal").Range("G3").PasteSpecial xlPasteValues
    .Range("I2").Copy
    wbDes.Sheets("Personal").Range("H3").PasteSpecial xlPasteValues

    Set rngOT1 = .Range("A:M")
    Set rngCopy = Intersect(rngOT1, Union(.Range("H2"), .Range("H3"), .Range("l2"), .Range("G8:H108"), .Range("J8:K108"), .Range("M8:M108")))

    For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT1").Range(adres).FormulaR1C1 = cel.Text
    Next cel
    
    Set rngOT2 = .Range("A:u")
    Set rngCopy = Intersect(rngOT2, Union(.Range("p2"), .Range("p3"), .Range("t2"), .Range("o8:p108"), .Range("r8:s108"), .Range("u8:u108")))

    For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT2").Range(adres).FormulaR1C1 = cel.Text
    Next cel
    
     Set rngOT3 = .Range("W:AC")
    Set rngCopy = Intersect(rngOT3, Union(.Range("H2"), .Range("H3"), .Range("l2"), .Range("G8:H108"), .Range("J8:K108"), .Range("M8:M108")))

    For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT3").Range(adres).FormulaR1C1 = cel.Text
    Next cel
    
      Set rngOT4 = .Range("AE:AK")
    Set rngCopy = Intersect(rngOT4, Union(.Range("H2"), .Range("H3"), .Range("l2"), .Range("G8:H108"), .Range("J8:K108"), .Range("M8:M108")))

    For Each cel In rngCopy
        adres = cel.Address
        wbDes.Sheets("OT4").Range(adres).FormulaR1C1 = cel.Text
    Next cel
End With

Application.CutCopyMode = False

End Sub

 

C2020-0138_Carga_Horas (1)2.xls

publicado
En 6/1/2021 at 19:44 , Juan Carlos torres ruiz dijo:

Estoy duplicando la macro para que rellene mas hojas del archivo

En el archivo que has subido no está la macro ?

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
      187
    • Comentarios
      97
    • Revisiones
      28

    Más información sobre "Un juego del Rabino en Excel"
    Última descarga
    Por pegones1

    1    1

  • Crear macros Excel

  • Mensajes

    • Perdona @Abraham Valencia pero he estado liado estos días. En realidad todo el problema se reduce a estos dos problemas: Problema 1: El script que "fabrica" SAP y que luego "pego" en la macro, no es capaz de  guardar archivos en SharePoint. He estado buscando, y en realidad muchas personas tienen ese problema (no poder guardar un Excel en SAP a través de VBA). Eso muy probablemente sean problemas de permisos, que no puedo cambiar (no soy administrador de nada). Como no puedo solucionarlo así, paso al plan B, que es guardar en Excel que me genera SAP en el ordenador de cada usuario que ejecute la plantilla (y que sí está guardada en SharePoint), para después con PowerQuery llamar a ese Excel (el export). Para ello, pretendo guardar el export, en la ruta relativa "C:\..\..\..\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" donde los \..\..\..\ saltan las rutas personales de cada usuario (tipo C:\users\IDusuario\). Eso lo hace bien, y el archivo se guarda en la ruta de cada usuario que lo usa, pero surge el problema 2 Problema 2: PowerQuery no trabaja con rutas relativas del tipo  "C:\..\..\..\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" necesita que sea del tipo fija "C:\users\IDusuario\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" pero claro, IDusuario es diferente para cada usuario.   Pero escribiendo todo esto, creo que he dado con una posible solución, no grabar el export en una ruta de usuario, sino en una en la raiz de C:, que siempre será igual para todos los usuarios, del tipo C:\Sap\export.xlsx que seria igual en todos los ordenadores. Asi sí podría decirle a PowerQuery que vaya siempre a la ruta C:\Sap\ que existirá en todos los ordenadores. Mañana intentaré hacer pruebas, aunque mi solución ideal seria que se pudiera guardar en el SharePoint. Saludos.
    • Hola La opción brindada por @torquemada es correcta, funciona, pero hay algunos inconvenientes que (desde mi punto de vista) no la convierten en mi primera elección. Los inconvenientes son: Tendrías que ir columna por columna haciendo los reemplazos, claro que no se harían a mano sino que utilizarías la opción reemplazar o la opción texto en columnas, aun asi demorará un poquito y será trabajoso. Cada vez que descargues otro listado, tendrás que volver a realizar los reemplazos. Me parece una mejor propuesta lo siguiente: Descarga los movimientos a un archivo de Excel Desde tu control de pagos (otro archivo) cargas los movimientos del archivo descargado mediante Power Query Power Query hará los reemplazos y reconocerá todo correctamente (sin que tengas que hacer nada especial) Cuando descargues los movimientos un día posterior, solamente tendrás que hacer clic en "Actualizar" y todo funcionará en automático
    • Hola a todos, Efectivamente, me temo que tal como trabajan las funciones =HOY() y/o =AHORA() (volátiles), sólo con macros puedes obtener soluciones. Un recurso pedestre podría ser, cada vez que quieras que se fije un dato, te sitúes en esa celda y pulses F2, F9 e INTRO.  Pero claro, puede ser un inconveniente si hay que hacerlo repetitivamente en muchas ocasiones,.............. en fin, lo comento sólo como posibilidad. Saludos,
    • Hola nuevamente, mi duda sigue siendo la ruta, o rutas, finales que quedan, esas que llamas "relativas"; igual por si acaso pon 3 o 4 de esas, tal cual son y/o se ven en el explorador de cada PC y, de ser posible, en cualquier otro "lado" en que las veas.
    • Hola, tal cual se plantea, solamente con macros (VBA); en todo caso, hacerlo a mano o con "CTRL + ;". Saludos.
  • 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.