Comments on: Identifying Sheets

By: Egon

Egon — Thu, 30 Oct 2008 19:31:00 +0000

Sounds like this CodeName instability adds a bunch of annoyance, no matter which approach one takes.

Hope this bug will be fixed before Excel 2007 is deployed here (wishful thinking...).

By: Doug Glancy

Doug Glancy — Thu, 16 Oct 2008 17:00:00 +0000

AlexJ, I do that to. Something that I learned a while back from one of the newsgroup experts I think. It makes it easier to test code too. I do it in addins as well, for instance, AddinInstall or WorkBookOpen.

By: AlexJ

AlexJ — Thu, 16 Oct 2008 14:31:00 +0000

I've been trying to put all my ThisWorkbook and Worksheet code in regular modules, with event driven calls from ThisWorkbook or Worksheet. This way I can easily swap out an old module for a new one without copying class module code to the sheet or ThisWorkbook.

Obviously not using an addin.

By: Jon Peltier

Jon Peltier — Wed, 15 Oct 2008 23:12:00 +0000

On occasion I’ve put code into my templates, including code that really should live in a regular module. But it’s easier to distribute a few templates with code, especially code that works specifically on the template’s worksheet, than to distribute an add-in widely.

By: Dick Kusleika

Dick Kusleika — Tue, 14 Oct 2008 14:55:00 +0000

I never put any code in my templates, but if it’s all in the sheet’s class module I may be persuaded to make an exception.

By: Simon Murphy

Simon Murphy — Tue, 14 Oct 2008 14:50:00 +0000

Doug, yeah, they're a hard lot to please maintainers. Me, I like to try and keep logic blocks either code side or grid side.

sh needs to be an object so the code will compile, as a normal worksheet doesn't have an IsAnInvoice method. Just those with the added VBA method (or it could be a property).

Actually I think I would use a value in a cell where possible because then the logic is clear for everyone. That would depend on how meddlesome the users are though.

By: Doug Glancy

Doug Glancy — Mon, 13 Oct 2008 23:14:00 +0000

Simon,that's cool, I never thought of doing something like that. Although I'm not sure it's less work for the maintainer - it might just make my head spin an extra orbit!

Can you explain why your function requires sh to be declared as object, not worksheet?

By: Simon Murphy

Simon Murphy — Mon, 13 Oct 2008 21:50:00 +0000

I'd add a IsAnInvoice boolean function to the invoice sheets that returns true then change your call to

Public Function IsInvoice(sh As object) As Boolean
On Error Resume Next
IsInvoice = sh.IsAnInvoice

End Function

That way its all upfront in the VBA for the maintainers that follow

By: Stephen Bullen

Stephen Bullen — Mon, 13 Oct 2008 14:20:00 +0000

When I last looked at custom sheet properties, they weren’t copied across when you copied a sheet – defeating the point of Dick’s requirement. I use hidden sheet-level defined names to identify worksheets and workbook document properties for books.

By: jkpieterse

jkpieterse — Mon, 13 Oct 2008 11:14:00 +0000

Never knew about custom sheet props, very useful!
Dick: And of course you use Name Manager to hide the range name, not?