put results on worksheets in another workbook

helpexcel

Well-known Member
Joined
Oct 21, 2009
Messages
656
I'm using this code to count items on each worksheet in workbook 1. How do alter the code so that this part (ws.Range("C1").Value = cnt) is put into a different workbook? both workbooks have the same sheet names, if that helps.

Thanks!


Code:
Dim ws As Worksheet

With Sheet1
    Dim x As Long
   x = .Range("A" & Rows.Count).End(xlUp).Row
End With


For Each ws In Worksheets
  Dim cnt As Integer
  cnt = 0
  For i = 1 To x
    If ws.Cells(i, 1).Value = ws.Name And ws.Cells(i, 2).Value = "Football" Then
      cnt = cnt + 1
    End If
  Next i




ws.Range("C1").Value = cnt
Next
 

Excel Facts

Round to nearest half hour?
Use =MROUND(A2,"0:30") to round to nearest half hour. Use =CEILING(A2,"0:30") to round to next half hour.
I'm using this code to count items on each worksheet in workbook 1. How do alter the code so that this part (ws.Range("C1").Value = cnt) is put into a different workbook? both workbooks have the same sheet names, if that helps.

Thanks!

Is
Code:
ws.Range("C1").Value = cnt
meant to be inside or outside the loop. As it is, you will just be overwriting the same cell each time the loop moves to another worksheet. Additionaly, is Sheet1 the actual name of the worksheet or are you referring to its codename.

Please read the comments
Code:
Dim ws As Worksheet, cnt As Long, x As Long, TempA As Variant

x = ThisWorkbook.Worksheets("Sheet1").Range("A" & Rows.Count).End(xlUp).Row
'assumes "Sheet1" is the name of the worksheet
For Each ws In ThisWorkbook.Worksheets
  
  With ws.UsedRange.Columns(1)
    TempA = .Resize(.Rows.Count, 2).Value2
  End With
  
  For I = 1 To x
    If TempA(I, 1) = ws.Name And TempA(I, 2) = "Football" Then
      cnt = cnt + 1
    End If
  Next I

Next
'assumes worksheet name in 2nd workbook is also "Sheet1" and that the cell is meant to be
'updated after exiting all loops

Workbooks("Name_of_Workbook_Here_with_extension").Worksheets("Sheet1").Range("C1").Value = cnt
 
Last edited:
Upvote 0
Thanks. Sheet1 is the Codename. I did want that section in the loop, because there are multiple worksheets I want counted. For example i have sheets named NY, Philly, Dallas, Chicago, LA in both workbooks. I want to loop through each sheet in workbook1 and count, then i want it to loop through each corresponding sheet in workbook2 and put the result in C1.
 
Upvote 0
Thanks. Sheet1 is the Codename. I did want that section in the loop, because there are multiple worksheets I want counted. For example i have sheets named NY, Philly, Dallas, Chicago, LA in both workbooks. I want to loop through each sheet in workbook1 and count, then i want it to loop through each corresponding sheet in workbook2 and put the result in C1.

Try this
Code:
Dim ws As Worksheet, cnt As Long, x As Long, TempA As Variant, Workbook_2 As Workbook


x = Sheet1.Range("A" & Rows.Count).End(xlUp).Row


Set Workbook_2 = Workbooks("Name_of_Workbook_Here_with_extension") 'workbook must be open


For Each ws In ThisWorkbook.Worksheets
  
    With ws.UsedRange.Columns(1)
        TempA = .Resize(.Rows.Count, 2).Value2
    End With
  
    With ws
  
        For i = 1 To x
            If TempA(i, 1) = .Name And TempA(i, 2) = "Football" Then
                cnt = cnt + 1
            End If
        Next i
  
        Workbook_2.Worksheets(.Name).Range("C1").Value = cnt


    End With
    
cnt = 0 'reset value of cnt


Next
 
Upvote 0
I couldn't get the TempA code to run without getting an out of range error, so changed it back to .Cells and it ran perfectly. THANK YOU!!!!
 
Upvote 0
I couldn't get the TempA code to run without getting an out of range error, so changed it back to .Cells and it ran perfectly. THANK YOU!!!!
If you want to use an array rather than checking each cell then change
Code:
[COLOR=#333333]TempA = .Resize(.Rows.Count, 2).Value2[/COLOR]

to
Code:
[COLOR=#333333]TempA = .Resize(x, 2).Value2[/COLOR]
 
Upvote 0

Forum statistics

Threads
1,224,823
Messages
6,181,177
Members
453,021
Latest member
Justyna P

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top