Adding copy-paste into loop VBA formula

[dangorka]

Hi everyone

I have a quite simple Excel sheet with two tabs in it (Sheet1 and Sheet2)

Sheet 2 contains numbers in row Q and hyperlinks in column T
Basically any time when there is a number "1" in row Q, macro should copy paste hyperlink from reflective row to A4 in Sheet1, call few macros and then move back to Sheet2 and proceeds to another row with number "1" in row Q

So far I came up with this but I am not sure how can I add copy-pasting from one sheet to another into loop

Sub ScrapingAllTransfermarkt()

    Const START_ROW As Integer = 1
    
    For I = START_ROW To Cells(Rows.Count, 1).End(xlUp).Row
    
    If Cells(I, 17) = 1 Then
    
    Call transfermarkthome
    Call storeDataTransfermarktHome
    Call TransfermarktHomeClean


    End If
    Sheets("Sheet2").Select
    Next I
    
    End If
    Next I
    
End Sub

Will be grateful for any help!

 

[mumps]

Doing what you want should not be a problem. Just one question for clarification: Your current macro runs 3 other macros for every row in column Q. So if you have data in column Q down to row 100 and 50 of those rows have value of 1, the 3 macros will be run 50 times. Is this what you want to do. If not, can you post the other 3 macros and explain in detail what you are trying to do.

 

[dangorka]

Hi mumps!

Thanks for the response. Yes, that's exactly what I want, to run those 3 macros 50 times if 50 rows have value of 1! (can take some time to run everything but for the purpose of this I will be using virtual machine anyway)

 

[mumps]

Cross-posted at: https://www.excelforum.com/excel-pr...-with-looping-formula-needed.html#post5127795

 

[Fluff]

@dangorka
While we do not prohibit Cross-Posting on this site, we do ask that you please mention you are doing so and provide links in each of the threads pointing to the other thread (see rule 13 here along with the explanation: Forum Rules).
This way, other members can see what has already been done in regards to a question, and do not waste time working on a question that may already be answered.

 

[dangorka]

Thank you guys, will remember in the future about providing links to other forum!!

 

[mumps]

Try:

Sub ScrapingAllTransfermarkt()
    Application.ScreenUpdating = False
    Dim LastRow As Long, rng As Range, desWS As Worksheet, x As Long: x = 4
    Set desWS = Sheets("Sheet1")
    LastRow = Cells.Find("*", SearchOrder:=xlByRows, SearchDirection:=xlPrevious).Row
    For Each rng In Range("Q1:Q" & LastRow)
        If rng = 1 Then
            desWS.Range("A" & x) = rng.Offset(0, 3)
            x = x + 1
            Call transfermarkthome
            Call storeDataTransfermarktHome
            Call TransfermarktHomeClean
        End If
    Next rng
    Application.ScreenUpdating = True
End Sub