Copier coller à la suite VBA

Fermé
Stevou - 20 oct. 2011 à 12:57
pijaku Messages postés 12263 Date d'inscription jeudi 15 mai 2008 Statut Modérateur Dernière intervention 4 janvier 2024 - 20 oct. 2011 à 13:44
Bonjour,

je suis un novice sur VBA, donc j'ai quelques souci pour un code.
Mon code est ts simple:
En cliquant sur un bouton de commande, la macro doit copier coller une plage sur une feuille, doit aller sur une autre feuille. A partir de la, elle doit chercher la premiere cellule non vide (en partant du bas), et coller la selection.
Voici mon code:

Sub bouton()
Dim WB_Principal As Workbook
Set WB_Principal = ActiveWorkbook
Sheets("Macro1").Range("J20:K24").Copy
Workbooks("Machin.xlsx").Activate
Sheets("col").Activate
Do Until Not IsEmpty((ActiveCell))
Range("A65536").Offset(1, 0).End(xlUp).Select
Selection.Paste
Loop

WB_Principal.Activate
End Sub

A partir de cette ligne: Range("A65536").Offset(1, 0).End(xlUp).Select
j'ai l'erreur 1004 "erreur définie par l'application ou par l'objet"
Donc j'aimerai bien savoir pq j'ai cette erreur et par la suite avoir si possible une solution.

Merci de votre aide

Stevou


A voir également:

1 réponse

Patrice33740 Messages postés 8556 Date d'inscription dimanche 13 juin 2010 Statut Membre Dernière intervention 2 mars 2023 1 776
Modifié par Patrice33740 le 20/10/2011 à 13:25
Range("A65536").Offset(1, 0).End(xlUp).Select
C'est comme
Range("A65537").End(xlUp).Select
c'est une cause d'erreur sur un fichier .xls (mais pas sur un xlsx), il faudrait écrire :
Range("A65536").End(xlUp).Offset(1, 0).Select
et pour un xlsx :
Range("A1048576").End(xlUp).Offset(1, 0).Select

plus universel :
Cells(Rows.Count, 1).End(xlUp).Offset(1, 0).Select

Je préfère utiliser un objet Range :
Dim rng As Range
Set rng = ActiveSheet.Columns("A").Find("*", , , , , xlPrevious)
If Not rng Is Nothing Then
Set rng = rng.Offset(1, 0)
' ....
End If


Cordialement
Patrice
0
pijaku Messages postés 12263 Date d'inscription jeudi 15 mai 2008 Statut Modérateur Dernière intervention 4 janvier 2024 2 744
20 oct. 2011 à 13:28
Salut Patrice.
la boucle Do Until bla bla semble également inutile...
A supprimer!
0
Patrice33740 Messages postés 8556 Date d'inscription dimanche 13 juin 2010 Statut Membre Dernière intervention 2 mars 2023 1 776
20 oct. 2011 à 13:44
Bonjour pijaku,
Je suis entièrement d'accord avec toi.
Je dirais même qu' il faudrait tout ré-écrire en utilisant des objets pour pour désigner l'origine et la cible sans ambigüité et éviter les Select et Activate inutiles.
0
pijaku Messages postés 12263 Date d'inscription jeudi 15 mai 2008 Statut Modérateur Dernière intervention 4 janvier 2024 2 744
20 oct. 2011 à 13:44
tout est dit!
0