combining arrays

I have two arrays that are created with the following code and then attempted to be concatenated with the last for loop. However it gives a "subscript out of range" error after only one iteration!? help!?

VB Code:

intmon = Format(strx, "mm")
intyr = Format(strx, "yy")
 
Open "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm" For Input As #1
    rtb1.LoadFile "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm"
Close #1
    arrmon = Split(rtb1, "</a></div>")
i = UBound(arrmon, 1)
 
intmon = Format(Now, "mm")
intyr = Format(Now, "yy")
 
Open "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm" For Input As #1
    rtb1.LoadFile "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm"
Close #1
    arrmon2 = Split(rtb1, "</a></div>")
    
For x = 0 To UBound(arrmon2, 1)
     arrmon(i) = arrmon2(x)
     i = i + 1
Next x

VB Code:

redim arrmon(Ubound(arrmon2))
For i = Lbound(arrmon2) To UBound(arrmon2)
     arrmon(i) = arrmon2(i)
Next i

actually, you can probably just assign the array in one line.

arrmon = arrmon2

well since you start off the index at the upper bound of the array, you'd pretty much expect to get an index error after one iteration, huh?

you need to redim the array before attempting to concatenate the 2nd one onto the end of it. Otherwise your code is fine.

cafeenman seems to have mistaken you question for how to reassign one array to another, not concatenate as you asked.

ok, I thought there was something wrong with that.

here's what I've done:

VB Code:

ReDim arrmon(UBound(arrmon) + UBound(arrmon2))
For x = 0 To UBound(arrmon2, 1)
     arrmon(i) = arrmon2(x)
     i = i + 1
Next x

However, when I do that, it makes all of the initial array nothing. I've tried using "Preserve", but that appears to simply put the second array in the new array twice!?! :(

scratch that... my

VB Code:

strx = Now -1
intmon = Format(strx, "mm")

is giving me the wrong date (this month).... !?

hm ... your code looks good to me, but I'm don't use redim. there must be something we're missing, obviously.

you could always just make a large 3rd array and move the 1st then the 2nd into it. that's inelegant, but memory's cheap

Code analysis:

Set the value of var "strx" to TODAY - 1 DAY
Get the month

So unless if it's the first of the month (which is not the case), it would return this month. Try this instead:

VB Code:

strx = DateAdd("n", -1, Now)
intmon = Format(strx, "mm")

actually i just did:

VB Code:

strx = Now - 30
intmon = Format(strx, "mm")

That works fine now. I'm still having problems with the arrays tho.

The first array[portion] /edit in the concatenated array comes up as empty elements, and there's a few extra elements on the end of the concatenated array(which I can deal with).

anyone?

I've just noticed something. Your code will replace the last variable in the first array. To prevent that, use:

VB Code:

intmon = Format(strx, "mm")
intyr = Format(strx, "yy")
 
Open "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm" For Input As #1
    rtb1.LoadFile "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm"
Close #1
arrmon = Split(rtb1, "</a></div>")
i = UBound(arrmon, 1) + 1
 
intmon = Format(Now, "mm")
intyr = Format(Now, "yy")
 
Open "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm" For Input As #1
    rtb1.LoadFile "O:\ER2002\Inetlog\" & intyr & "\" & intmon & "\" & intmon & "e.htm"
Close #1
    arrmon2 = Split(rtb1, "</a></div>")
    
ReDim Preserve arrmon(UBound(arrmon) + UBound(arrmon2) + 1)
For x = 0 To UBound(arrmon2, 1)
     arrmon(i) = arrmon2(x)
     i = i + 1
Next x

Quote:

---

Originally posted by ober5861
actually i just did:

VB Code:

strx = Now - 30
intmon = Format(strx, "mm")

---

:eek: NO! That is NOT RIGHT! Set your computer date to July 31, 2002 and see if that code works. It will warp back to July 1, which is still July!!!

Also, if you set your calender to March 1 and use that code, it will return January, NOT February!!!

i just saw that myself ;)

and I knew about the problem with the 30 days, I was going to fix that later. I just wanted to get the array catonation to work.

It works perfectly now. Thank you all. :)

Quote:

---

PMbyober5861
hey, thanks for your help, but a piece of code you gave me doesn't work like you said:

strx = DateAdd("n", -1, Now)

That still returns the current month!?!

---

You PMed me concerning my code. OK, here's the modified code:

VB Code:

strx = DateAdd("[b]m[/b]", -1, Now)
intmon = Format(strx, "mm")

Iactuallychangedtheminutesinsteadofthemonth...Iamstupid...:(