Good Morning,
I currently have two separte macros but I'm trying to combine them but it's
going to require some different commands. What I need it to do is something
like this: start off we are in Summaryworkbook now if there is INV in column
D then do a straight search for 123 (which is in column E) in workbooks 1 - 7.
if there is a CM in column D then do a search for CM123 in workbooks 1-7 and
just go down the list from there. There are over 60000 of them to go through.
..Any ideas??

--
Message posted via http://www.officekb.com

post the 2 codes that you have already working.........
susan

On Apr 17, 9:35 am, "Joe via OfficeKB.com" <u27679@uwe wrote:
Good Morning,
I currently have two separte macros but I'm trying to combine them but it's
going to require some different commands. What I need it to do is something
like this: start off we are in Summaryworkbook now if there is INV in column
D then do a straight search for 123 (which is in column E) in workbooks 1 - 7.
if there is a CM in column D then do a search for CM123 in workbooks 1-7 and
just go down the list from there. There are over 60000 of them to go through.
.Any ideas??

--
Message posted viahttp://www.officekb.com

Sub Zurnprt2()
Const SummaryWorkbook = "ZurnOpenItemsspreadsheet.xls"
Const MainInvoiceCol = 5
Const MainPasteCol = 38
Const WbkInvoiceCol = 5
Const WbkStartCol = 1
Const WbkEndCol = 14

Dim i As Integer


Set wsh1 = Workbooks(SummaryWorkbook).Worksheets(1)

wsh1.Activate
Lastrow = wsh1.Cells(Rows.Count, MainInvoiceCol).End(xlUp).Row
Set InvoiceRange = wsh1. _
Range(Cells(1, MainInvoiceCol), Cells(Lastrow, MainInvoiceCol))

For Each cell1 In InvoiceRange
InvoiceNumber = cell1.Value

i = Len(InvoiceNumber) 'count the characters in the string
i = i - 2 'Remove 1 from the count to accommodate YM, WM or XM
InvoiceNumber = Right(InvoiceNumber, i) 'counting back from the right
'select only the numbers


For Each wbk1 In Application.Workbooks

If StrComp(wbk1.Name, SummaryWorkbook) < 0 Then
With wbk1.Worksheets(1)
.Activate

Lastrow = .Cells(Rows.Count, WbkInvoiceCol).End(xlUp).Row
Set InvoiceRange2 = .Range(Cells(1, WbkInvoiceCol), Cells(Lastrow,
WbkInvoiceCol))

For Each cell2 In InvoiceRange2

If (cell2.Value = "" & InvoiceNumber) Then 'add the prefix and
the number here

.Range(Cells(cell2.Row, WbkStartCol), _
Cells(cell2.Row, WbkEndCol)).Copy _
Destination:=wsh1.Cells(cell1.Row, MainPasteCol)
End If
Next cell2
End With
End If
Next wbk1
Next cell1
End Sub

Both are similiar the other one just doesn't take away two characters...



Susan wrote:
post the 2 codes that you have already working.........
susan

Good Morning,
I currently have two separte macros but I'm trying to combine them but it's
[quoted text clipped - 7 lines]
--
Message posted viahttp://www.officekb.com

--
Message posted via http://www.officekb.com

ok, joe
so the only thing that's different between the two macros is this area
of code?

i = Len(InvoiceNumber) 'count the characters in the string
i = i - 2 'Remove 1 from the count to accommodate YM, WM or XM
InvoiceNumber = Right(InvoiceNumber, i) 'counting back from the right
'select only the numbers

if your answer is yes, then you just need to devise some sort of if-
then test as to whether or not this section is performed.......
this is the only area where i could see you were deleting any
characters.
maybe this helps......
susan




On Apr 17, 12:21 pm, "Joe via OfficeKB.com" <u27679@uwe wrote:
Sub Zurnprt2()
Const SummaryWorkbook = "ZurnOpenItemsspreadsheet.xls"
Const MainInvoiceCol = 5
Const MainPasteCol = 38
Const WbkInvoiceCol = 5
Const WbkStartCol = 1
Const WbkEndCol = 14

Dim i As Integer

Set wsh1 = Workbooks(SummaryWorkbook).Worksheets(1)

wsh1.Activate
Lastrow = wsh1.Cells(Rows.Count, MainInvoiceCol).End(xlUp).Row
Set InvoiceRange = wsh1. _
Range(Cells(1, MainInvoiceCol), Cells(Lastrow, MainInvoiceCol))

For Each cell1 In InvoiceRange
InvoiceNumber = cell1.Value

i = Len(InvoiceNumber) 'count the characters in the string
i = i - 2 'Remove 1 from the count to accommodate YM, WM or XM
InvoiceNumber = Right(InvoiceNumber, i) 'counting back from the right
'select only the numbers

For Each wbk1 In Application.Workbooks

If StrComp(wbk1.Name, SummaryWorkbook) < 0 Then
With wbk1.Worksheets(1)
.Activate

Lastrow = .Cells(Rows.Count, WbkInvoiceCol).End(xlUp).Row
Set InvoiceRange2 = .Range(Cells(1, WbkInvoiceCol), Cells(Lastrow,
WbkInvoiceCol))

For Each cell2 In InvoiceRange2

If (cell2.Value = "" & InvoiceNumber) Then 'add the prefix and
the number here

.Range(Cells(cell2.Row, WbkStartCol), _
Cells(cell2.Row, WbkEndCol)).Copy _
Destination:=wsh1.Cells(cell1.Row, MainPasteCol)
End If
Next cell2
End With
End If
Next wbk1
Next cell1
End Sub

Both are similiar the other one just doesn't take away two characters...

Susan wrote:
post the 2 codes that you have already working.........
susan

Good Morning,
I currently have two separte macros but I'm trying to combine them but it's
[quoted text clipped - 7 lines]
--
Message posted viahttp://www.officekb.com

--
Message posted viahttp://www.officekb.com

Ok the first macro looks for nothing but 123 which is great so it works. But
I wasn't sure on how to write the second one because it's not working the way
I planned, when the second macro runs it find's 123 then does a search for
CM3 (because it replaces the -2 charcters with the CM) and the crazy thing
it's finding matches!! which is incorrect. The whole reason I wanted the -2
is because there are other numbers like this XY456 and in our system it's
labled at CM456 so I figure just take two away with out thinking of the other
numbers without the letters in front. So I came up with this idea...I'm not
sure on how to write it though...

Susan wrote:
ok, joe
so the only thing that's different between the two macros is this area
of code?

i = Len(InvoiceNumber) 'count the characters in the string
i = i - 2 'Remove 1 from the count to accommodate YM, WM or XM
InvoiceNumber = Right(InvoiceNumber, i) 'counting back from the right
'select only the numbers

if your answer is yes, then you just need to devise some sort of if-
then test as to whether or not this section is performed.......
this is the only area where i could see you were deleting any
characters.
maybe this helps......
susan

Sub Zurnprt2()
Const SummaryWorkbook = "ZurnOpenItemsspreadsheet.xls"
[quoted text clipped - 60 lines]
--
Message posted viahttp://www.officekb.com

--
Message posted via OfficeKB.com
http://www.officekb.com/Uwe/Forums.a...mming/200704/1

Both are similiar the other one just doesn't take away two characters

so, not really, because one is searching for things & the other one
isn't......

why don't you post BOTH macro codes.
label one "THIS ONE RUNS FIRST"
and the other "THIS ONE RUNS SECOND"

then i (or anybody else) can see exactly what you're trying to do, in
the order you're trying to do it in.

finding incorrect matches, it only finds what you tell it to. so
there must be some kind of error in telling it what you want it to
find.
:)
susan



On Apr 17, 1:12 pm, "Joe via OfficeKB.com" <u27679@uwe wrote:
Ok the first macro looks for nothing but 123 which is great so it works. But
I wasn't sure on how to write the second one because it's not working the way
I planned, when the second macro runs it find's 123 then does a search for
CM3 (because it replaces the -2 charcters with the CM) and the crazy thing
it's finding matches!! which is incorrect. The whole reason I wanted the -2
is because there are other numbers like this XY456 and in our system it's
labled at CM456 so I figure just take two away with out thinking of the other
numbers without the letters in front. So I came up with this idea...I'm not
sure on how to write it though...





Susan wrote:
ok, joe
so the only thing that's different between the two macros is this area
of code?

i = Len(InvoiceNumber) 'count the characters in the string
i = i - 2 'Remove 1 from the count to accommodate YM, WM or XM
InvoiceNumber = Right(InvoiceNumber, i) 'counting back from the right
'select only the numbers

if your answer is yes, then you just need to devise some sort of if-
then test as to whether or not this section is performed.......
this is the only area where i could see you were deleting any
characters.
maybe this helps......
susan

Sub Zurnprt2()
Const SummaryWorkbook = "ZurnOpenItemsspreadsheet.xls"
[quoted text clipped - 60 lines]
--
Message posted viahttp://www.officekb.com

--
Message posted via OfficeKB.comhttp://www.officekb.com/Uwe/Forums.aspx/excel-programming/200704/1- Hide quoted text -

- Show quoted text -