public statics

Developer
Mar 8, 2009 at 9:08 PM
Edited Mar 8, 2009 at 9:08 PM

 

Hi, just wandering if there was a reason for keeping these as public statics, and not private's in the CacheFunctions class.


public
static string Register(object o, string key)
public static string Register(object o)
public static object Lookup(string key)

 

 

 

 

Coordinator
Mar 9, 2009 at 6:01 AM
My thinking was that people using this library from their own code may want to add objects to the cache/inspect what is in there already from the backend.



On Sun, Mar 8, 2009 at 11:08 PM, incred <notifications@codeplex.com> wrote:

From: incred

 

Hi, just wandering if there was a reason for keeping these as public statics, and not private's


public
static string Register(object o, string key)
public static string Register(object o)
public static object Lookup(string key)

 

Read the full discussion online.

To add a post to this discussion, reply to this email (ExcelDnaContrib@discussions.codeplex.com)

To start a new discussion for this project, email ExcelDnaContrib@discussions.codeplex.com

You are receiving this email because you subscribed to this discussion on CodePlex. You can unsubscribe or change your settings on codePlex.com.

Please note: Images and attachments will be removed from emails. Any posts to this discussion will also be available online at codeplex.com




--
Regards,
Kevin Trethewey

Driven Software

tel:     +27 (0) 82 579 9154
fax:    +27 (0) 82 131 579 9154
email: kevint@drivensoftware.net
web:   http://www.drivensoftware.net
blog:   http://kevint.drivensoftware.net
Developer
Mar 9, 2009 at 11:33 AM
My only issue was really that by my making them public you expose them to the excel environment.

I don't have a suggestion for an alternative at the moment
Coordinator
Mar 9, 2009 at 5:02 PM
I switched them to internal for now, we can address the issue when there is an actual usage case for it.

Thanks.


On Mon, Mar 9, 2009 at 1:33 PM, incred <notifications@codeplex.com> wrote:

From: incred

My only issue was really that by my making them public you expose them to the excel environment.

I don't have a suggestion for an alternative at the moment

Read the full discussion online.

To add a post to this discussion, reply to this email (ExcelDnaContrib@discussions.codeplex.com)

To start a new discussion for this project, email ExcelDnaContrib@discussions.codeplex.com

You are receiving this email because you subscribed to this discussion on CodePlex. You can unsubscribe or change your settings on codePlex.com.

Please note: Images and attachments will be removed from emails. Any posts to this discussion will also be available online at codeplex.com




--
Regards,
Kevin Trethewey

Driven Software

tel:     +27 (0) 82 579 9154
fax:    +27 (0) 82 131 579 9154
email: kevint@drivensoftware.net
web:   http://www.drivensoftware.net
blog:   http://kevint.drivensoftware.net