Question

If I have a static method like this

    private static bool TicArticleExists(string supplierIdent)
    {
        using (TicDatabaseEntities db = new TicDatabaseEntities())
        {
            if((from a in db.Articles where a.SupplierArticleID.Equals(supplierIdent) select a).Count() > 0)
                return true;                
        }
        return false;
    }

and use this method in various places in foreach loops or just plain calling it numerous times, does it create and open new connection every time?

1. If so, how can I tackle this? Should I cache the results somewhere, like in this case, I would cache the entire Classifications table in Memory Cache? And then do queries vs this cached object?
2. Or should I make TicDatabaseEntities variable static and initialize it at class level?
3. Should my class be static if it contains only static methods? Because right now it is not..
4. Also I’ve noticed that if I return result.First() instead of FirstOrDefault() and the query does not find a match, it will issue an exception (with FirstOrDefault() there is no exception, it returns null).

Thank you for clarification.

Answer 1

• new connections are non-expensive thanks to connection caching. Basically, it grabs an already open connection (I htink they are kept open for 2 minutes for reuse).

• Still, caching may be better. I do really not like the “firstordefault”. Thinks of whether you can acutally pull in more in ONE statement, then work from that.

For the rest, I can not say anything – too much depends on what you actually do there logically. What IS TicDatabaseEntities? CAN it be cached? How long? Same with (3) – we do not know because we do not know what else is in there.

If this is something like getting just some lookup strings for later use, I would say….

• Build a key out of classI, class II, class III
• load all classifications in (I assume there are only a couple of hundred)
• Put them into a static / cached dictionary, assuming they normally do not change (and I htink I have that idea here – is this a financial tickstream database?)

Without business knowledge this can not be answered.

4: yes, that is as documented. First gives first or an exception, FirstOrDefault defaults to default (empty struct initialized with 0, null for classes).