Adding up hours from separate sheets using VBA

TAPS_MikeDion

Well-known Member
Joined
Aug 14, 2009
Messages
622
Office Version
  1. 2011
Platform
  1. MacOS
Hello everybody,
I'm hoping someone will be able to figure this out for me...please.

Here's what I need to have VBA do:
  1. Scan through column D (ID numbers) in my Division Member Info (DMI) sheet
  2. Scan through column G (ID numbers) in my monthly sheets (D-JAN, D-FEB, D-MAR, etc to D-DEC) and find the ID number matching the ID number in the DMI sheet column D
  3. Individually add up the hours found for each ID number from column N in the monthly (D-JAN, D-FEB, etc) sheets
  4. Put the final total of hours (per ID number) in column AG (Detail Hours) of the DMI sheet
I have a sample TestBook Excel file made to make things easier, but I'm not sure how to get it to anyone willing to help.

Thank you in advance!
-Mike
 

Excel Facts

Create a chart in one keystroke
Select the data and press Alt+F1 to insert a default chart. You can change the default chart to any chart type
Hi @TAPS_MikeDion. Thanks for posting on the board.

You could upload a copy of your file to a free site such www.dropbox.com or google drive. Once you do that, mark it for 'Sharing' and you will be given a link to the file that you can post here. If the workbook contains confidential information, you could replace it with generic data.

--------------
I hope to hear from you soon.
Respectfully
Dante Amor
--------------​
 
Upvote 0
Hi Mike:

Try this macro:
VBA Code:
Sub sum_hours()
  Dim sh As Worksheet
  Dim c As Range
  Dim tot As Double
  
  With Sheets("Division Member Info")
    For Each c In .Range("D3", .Range("D" & Rows.Count).End(3))
      tot = 0
      For Each sh In Sheets
        If UCase(Left(sh.Name, 2)) = "D-" Then
          tot = tot + WorksheetFunction.SumIf(sh.Range("G:G"), c.Value, sh.Range("N:N"))
        End If
      Next
      .Range("AG" & c.Row).Value = tot
    Next
  End With
End Sub

Note: The format in the AG column must be [h]:mm to show the total hours:
1684969691119.png


--------------
Let me know the result and I'll get back to you as soon as I can.
Cordially
Dante Amor
--------------​
 
Upvote 0
Hi Dante,
I'm amazed that you were able to get that in such a short macro, awesome!!! You're the best!!!

I was informed that it will only be full hours, so I formatted the cells using just [h] and it works perfectly.

Here's the only issue I'm having now...
As you can see by the screenshot, the hours aren't adding correctly, which I'm assuming may have to do with formatting???

The TOTAL HOURS (column AI) should read:
Smith - 50
Jones - 53
Doe - 45

The REMAINING HOURS (column AJ) cells have the formula "=MAX(0,50-AI3)" and should read:
Smith - 0
Jones - 0
Doe - 5

Do you know what I would need to do to these add up correctly?

Thank you so much again!

Screen Shot 2023-05-25 at 7.30.13 AM.png
 
Upvote 0
Hi Mike:

As you can see by the screenshot, the hours aren't adding correctly, which I'm assuming may have to do with formatting???

The TOTAL HOURS (column AI) should read:
Smith - 50
Jones - 53
Doe - 45

The REMAINING HOURS (column AJ) cells have the formula "=MAX(0,50-AI3)" and should read:
Smith - 0
Jones - 0
Doe - 5

That is the ultimate goal, you should mention it at the beginning of your request, that way I will work on a complete answer.
I explain:
3:00 hours, correspond to part of a day (24 hours), that is, 0.125 of a day. So to get the integer value of those hours, you must multiply by 24.

So this you must do in your sheet.
1)
Change the format of the AG, AI and AJ columns to general format.
2) Use this macro, which already converts hours to integers.

Rich (BB code):
Sub sum_hours()
  Dim sh As Worksheet
  Dim c As Range
  Dim tot As Double
  
  With Sheets("Division Member Info")
    For Each c In .Range("D3", .Range("D" & Rows.Count).End(3))
      tot = 0
      For Each sh In Sheets
        If UCase(Left(sh.Name, 2)) = "D-" Then
          tot = tot + WorksheetFunction.SumIf(sh.Range("G:G"), c.Value, sh.Range("N:N"))
        End If
      Next
      .Range("AG" & c.Row).Value = tot * 24
    Next
  End With
End Sub


After running the macro:
1685026132729.png




---------------------------------
Supplementary Note:
You need to update your file as you have too many formats and you use up to a million cells but they are empty.
I suggest the following, create a new workbook, in that workbook create new sheets and give them the same name as you have in the current workbook, copy only the cells with data from each sheet in the current workbook and paste it into the new worksheet inside the new book.
Save your new file.
Your file that you shared is 5 MB, after performing the above procedure the book was 27 KB for me.

--------------
Let me know the result and I'll get back to you as soon as I can.
Cordially
Dante Amor
--------------​
 
Upvote 0
Solution
Hi Dante,
I'll try what you suggested and let you know. Unfortunately, I didn't realize this was going to be an issue or I would have mentioned it at the start of my post.

I'll also try your suggestion and see if I can drop the file size.

Thanks again for everything!
 
Upvote 0
Thanks again for everything Dante!

Also, your suggestion worked wonders for my file. After doing what you recommended I went from a 19 MB file to a 128 KB file!!!
 
Upvote 1

Forum statistics

Threads
1,214,918
Messages
6,122,246
Members
449,075
Latest member
staticfluids

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