Hi Bobby,



Try the attached update. The main issues were that the code was:
• starting from row 3 on the 'Welder Totals' sheet instead of row 2; and
• testing column B on each worksheet to get the count of used rows. On some sheets, that column was either empty or incomplete. It now tests column A.

Attached Files

Weld ProgressionPOSTONLINE2.xlsm (307.2 KB, 15 views)

__________________
Cheers,
Paul Edstein
[Fmr MS MVP - Word]

Thank You very much under quick investigation it appears as though this will do everything i can throw @ it, I pasted the macro in my small example database i sent yesterday and i added some stencils here and
everywhere and it appears to work great. I will import this into my big workbook tommorow cross my fingers and I will report back as soon as possible. So Macropod do you do this as a living for MS? or do you just dabble on the side and moonlight as a Macro god! If the latter i would love to donate you have helped me tremendiously.

Hi Bobby,

When you apply it to your workbook at work, just be sure to make the worksheet names exactly match the column headings on the 'Welder Totals' sheet. The work version of the 'Welder Totals' sheet will also need its layout changed to match what I've posted.
No, I'm retired and have never worked for MS. Whilst I dabble in Excel a bit, my MVP status with MS was awarded on the bais of my proficiency in Word and the help I give people with that application.

Despite superficial appearances to the contrary, I'm not a programmer - my vba coding skills are self-taught, mostly as part of helping others solve their problems. Most of the on-line stuff I've helped people with over the years concern uses of Word & Excel that I never had any real use for in my own career.

__________________
Cheers,
Paul Edstein
[Fmr MS MVP - Word]

Macropod

I got a question about the part of the macro that looks in column AG for a date, the ACC column. could there be an (or) this column statement.
Basically could the macro look in the AG column or the AL column only one of these two will have the date we are looking for the tally both columns will not have info only one or the other? Because one column shows welds that xray good on the first try the other column shows welds that xray good after being repaired. Any help whenever you get the chance. Much Thanks.

Hi Bobby,

Change:
'Check column AG for X-Ray dates
If .Cells(j, 33).Value Like "*#/*#/####" Then iXRay = iXRay + 1
to:
'Check column AG & AL for X-Ray dates
If .Range("AG" & j).Value Like "*#/*#/####" Or _
.Range("AL" & j).Value Like "*#/*#/####" Then iXRay = iXRay + 1

There's nothing significant about using .Range instead of .Cells here except that it makes it easier to recognise which column the code is working with.

__________________
Cheers,
Paul Edstein
[Fmr MS MVP - Word]

works like a charm. I am most definately gonna get on this tomorrow. it seem to work great on preliminary try now I wont dread being on a 5000 plus weld Job. because its alot of long hand the old way. boy oh boy I cant thank you enough.

Hi Bobby,

I take it the code's working well? FWIW, if at any time you decide to add/delete a weld sheet, the code will accomodate the change with no modifications.

__________________
Cheers,
Paul Edstein
[Fmr MS MVP - Word]

I was running through old emails and ran across the ones from MSofficeForums and I just wanted to tell you again how grateful I am, and the Macro is still working Really well. Thanks again for all the work you put into making this thing the killer macro I knew it could be. BTW i now can edit and adapt it to all my temp. jobs and Thanks for the lessons learned.