CODEWITHSUNDEEP
vbaexcel
SDROB
SDROB

Reputation: 125

Excel VBA row copy and pasting error

The following code is copy and pasting rows from sheet1 to sheet2 depending on keyword criteria being met in column E in the first sheet.

Between code lines 5 to 10 I am trying to copy the whole of the tenth row to sheet2 to use as column headers for this sheet, where a pivot will be created.

I am getting "Applicated-defined or Object-defined error" whenever I try to run any operation to copy and paste this row.

Can anyone help?

Sub mileStoneDateChanger()
    Dim r As Long, pasteRowIndex As Long, v() As Long, i As Long
    Dim lastRow As Long
    Dim lCol As Long
    Sheets("Sheet1").Select
    Rows("10:10").Select
    Selection.Copy
    Sheets("Sheet2").Select
    Range("A1").Select
    Selection.Paste
    lastRow = Cells.Find(What:="*",
                    After:=Range("A1"),
                    LookAt:=xlPart,
                    LookIn:=xlFormulas,
                    SearchOrder:=xlByRows,
                    SearchDirection:=xlPrevious,
                    MatchCase:=False).Row
    MsgBox "Last Row: " & lastRow
    pasteRowIndex = 1

    With Sheets("Sheet1")
        For r = 1 To lastRow
            If .Cells(r, "E").Value Like "Milestone*" Then
                If UBound(Split(.Cells(r, "E"), ",")) > 0 Then
                    i = i + 1
                    ReDim v(1 To i)
                    v(i) = pasteRowIndex
                End If
                Sheets("Sheet1").Rows(r).Copy Sheets("Sheet2").Rows(pasteRowIndex)
            pasteRowIndex = pasteRowIndex + 1
            End If
        Next r
    End With

    With Sheets("Sheet2")
        newLastRow = pasteRowIndex
        If IsArray(v) Then
            .Columns(6).Insert shift:=xlToRight
         For i = 1 To newLastRow
                If InStr(1, .Cells(i, "E"), ",") Then
                    .Cells(i, "F").Value = Split(.Cells(i, "E"), ",")(1)
                End If
            Next i
        End If
    End With
End Sub

Upvotes: 1

Views: 301

Answers (3)

Vityata
Vityata

Reputation: 43565

Range("A1") cannot be pasted with one whole row. Thus, use the following:

Sub TestME()

    Sheets(1).Select
    Rows("10:10").Select
    Selection.Copy
    Sheets(2).Select
    Range("A1").Select
    ActiveSheet.Paste
    Application.CutCopyMode = False

End Sub

Disclaimer - you really do not want to use Select and Active in VBA. How to avoid using Select in Excel VBA

Best case scenario is probably to avoid Copy like this:

Sub TestME()
    WorkSheets(2).Rows(1) = WorkSheets(1).Rows(10)
End Sub

Upvotes: 1

mooseman
mooseman

Reputation: 2017

Here is the copy for row 10 in Sheet1 to A1 in Sheet2

Sheets("Sheet1").Rows("10").Copy Sheets("Sheet2").Range("A1")

Upvotes: 4

GSD
GSD

Reputation: 1252

Replace Selection.Paste with Selection.PasteSpecial

Upvotes: 1

Related Questions