"(PeteCresswell)" <x@y.Invalid> wrote in
news:44lbj6prjdrhdd1nqgv0mfsf88845ibhe4 (AT) 4ax (DOT) com:

Is the database referenced there really a database, or is it a temp
file? I'm just guessing here, but it looks like a compile problem.
Have you decompiled and recompiled?

I don't see anything in particular, but I really think I'd change
the date argument of your IsBizDate() function to be ByVal instead
of ByRef, since when you pass a control to a ByRef, you're operating
on the control, and not on the value in the control. I assume you've
defined it as variant because you want it to work with a Null, but
you don't have a guard clause in your IsBizDate() function that
handles Nulls. It may be that it will fall over to returning False
on Null, but I'd still want to handle it explicitly.

Also, you're not using your explanation variable, which makes your
code more confusing than it needs to be.

Indeed, the whole things looks quite convoluted to me -- it looks
like vastly overengineered code to me, and when that's the case it
often becomes very hard to troubleshoot. I'd get back to basics and
see if I can generate the same error with much simpler code.

--
David W. Fenton http://www.dfenton.com/
contact via website only http://www.dfenton.com/DFA/

"(PeteCresswell)" <x@y.Invalid> wrote in
news:0llbj614o6h3pjohkdl38qv40ukth14aeq (AT) 4ax (DOT) com:

Do you decompile regularly? If not, why not? It really is something
that is necessary to maintain good code health (as well as having
COMPILE ON DEMAND turned off, OPTION EXPLICIT in all modules, and
compiling every few minutes when coding).

--
David W. Fenton http://www.dfenton.com/
contact via website only http://www.dfenton.com/DFA/

Per David-W-Fenton:
It's like a religion with me: multiple times per day.

I suffix my development versions with a letter and every time I
take a break or something, I zip up the current version, move the
..ZIP to an archive location, /Decompile, Compile, and then
Compact/Repair into the next-lettered version.

--
PeteCresswell

On 19/01/2011 14:55:14, "(PeteCresswell)" wrote:
Intersting, because I have seen numerous postings on this NG that the
automatic compact & repair on closing a Db is a bad idea. Are you suggesting
it is a good idea if you have decompiled first?

Phil

Phil wrote:


Automatic CR on close is a bad idea, but that's not what
Pete's doing. He first makes the critically important
backup BEFORE doing the other things. Auto CR skips this
steo so if anything goes wrong with the CR, you're dead.

I would consider moving the Compile step to after the CR
step, but I think his procedure is absolutely safe and
sound.

--
Marsh

On 19/01/2011 16:24:05, Marshall Barton wrote:
Yup, that sounds very sensible.
Thanks
Phil

Per Phil:
I don't know from automatic.

I close the thing, open it up again with /Decompile, compile it,
close it, and then compact/repair.
--
PeteCresswell

Per (PeteCresswell):
But before I do any of that, I zip up the archive copy.
--
PeteCresswell

Marshall Barton <marshbarton (AT) wowway (DOT) com> wrote in
news:gd3ej6p7hpi22meamsnlj2gqmvjp1u0q34 (AT) 4ax (DOT) com:

I always do a decompile cycle with:

1. backup

2. compact

3. decompile

4. compact

5. compile

6. compact

My theory is that there is less chance of any discarded data pages
being retained with a compact before and after every step.

--
David W. Fenton http://www.dfenton.com/
contact via website only http://www.dfenton.com/DFA/