Solved

Best practice for CFQuery: using DBName as attribute or in SQL?

Posted on 2008-10-28
7
683 Views
Last Modified: 2012-06-27
When using CFQuery, is it best to have the database name as an attribute (example code 1) or as part of the SQL (example code 2)?  

Does this make a difference for ColdFusion query caching (cachedWithin)?

(currently using ColdFusion 8)
example 1:
<cfquery name="qCountry" datasource="LOCALDSN" dbname="#curr_db_name#">
SELECT * FROM tblCountry WHERE country_id = #cid#
</cfquery
 
example 2
<cfquery name="qCountry" datasource="LOCALDSN">
SELECT * FROM #curr_db_name#.dbo.tblCountry WHERE country_id = #cid#
</cfquery

Open in new window

0
Comment
Question by:paid_tech
  • 3
  • 3
7 Comments
 
LVL 19

Expert Comment

by:erikTsomik
ID: 22822790
it does not really matter the datasource get setup in CF administrator which will point to the DB file. So what you doing is absolutely identicall and has no effect on the perforance
0
 
LVL 36

Accepted Solution

by:
SidFishes earned 125 total points
ID: 22822896
actually dbname is deprecated and should not be used

"Deprecated the connectString, dbName, dbServer, provider, providerDSN, and sql attributes, and all values of the dbtype attribute except query. They do not work, and might cause an error, in releases later than ColdFusion 5. "

from livedocs
0
 

Author Comment

by:paid_tech
ID: 22823575
thank you for pointing out the deprecation of dbname
(http://www.cfquickdocs.com/cf8/?getDoc=cfquery#cfquery)
just to clarify I have about 40 databases, one for each site, and they are all setup through the same ColdFusion Datasource/DSN.

1) if its possible to switch to a DSN for each database, should we change to that?
2) So should I use the format of example 2, with the dname in front of each table name in the SQL? (example below)

<cfquery name="qCountry" datasource="LOCALDSN">
SELECT * 
FROM #curr_db_name#.dbo.tblCountry  AS c
LEFT OUTER JOIN #curr_db_name#.dbo.tblCountryStatus AS cs ON c.country_id = cs.country_id
WHERE country_id = #cid#
</cfquery

Open in new window

0
Master Your Team's Linux and Cloud Stack!

The average business loses $13.5M per year to ineffective training (per 1,000 employees). Keep ahead of the competition and combine in-person quality with online cost and flexibility by training with Linux Academy.

 
LVL 36

Assisted Solution

by:SidFishes
SidFishes earned 125 total points
ID: 22823745
yes example 2 would be the best approach...i believe it's actually a bit better than multiple dsn's as it use the db's power rather than cf's and that's always a good thing

you could try putting the variable in application scope if each site has it's own codebase

FROM #application.curr_db_name#.dbo.tblCountry  AS c

0
 

Author Comment

by:paid_tech
ID: 22824209
thank you SidFish

also does this approach (with dbname in SQL) affect the ColdFusion query caching?

would ex 3 & 4 be considered different queries by ColdFusion, and hence be cached as different queries?

ex 3:
<cfquery name="qCountry" datasource="LOCALDSN">
SELECT * 
FROM funSite.dbo.tblCountry  AS c
LEFT OUTER JOIN funSite.dbo.tblCountryStatus AS cs ON c.country_id = cs.country_id
WHERE country_id = 5
</cfquery>
 
ex 4:
<cfquery name="qCountry" datasource="LOCALDSN">
SELECT * 
FROM testSite.dbo.tblCountry  AS c
LEFT OUTER JOIN testSite.dbo.tblCountryStatus AS cs ON c.country_id = cs.country_id
WHERE country_id = 5
</cfquery>

Open in new window

0
 
LVL 36

Assisted Solution

by:SidFishes
SidFishes earned 125 total points
ID: 22824626
"To pull from the cache, more than just the name of the query must match. Here's the list:

    * Same query "Name"
    * Exact same SQL statement - "where username='bubbaLouie'" and "where username = 'samIam'" are 2 different statements, ergo 2 different queries in the cache - even if they are both "named" NightOnTown.
    * Same Datasource - for those of you who fail to assume and stumbled onto that thought.
    * Same Username and password - This is interesting to note. If you have a site with a shared datasource but multiple db usernames you may not get the benefit from caching that you think you should.
    * Same DBTYPE"

http://mkruger.cfwebtools.com/index.cfm?mode=entry&entry=EAA0D1CA-01F6-F3EC-5520AAD6EEC68061


that being said, if you're not using (which your examples don't)

cachedwithin="#crateTimespan(0,010,0)#" in your cfquery tag you're not using caching anyways... (and caching has to be enabled in cfadmin)

for the benefit of future readers of this q, as noted in the article, versions prior to CF8 could not use cached queries -and- cfqueryparam. This is a major problem as imho, there is no circumstance where you should eliminate the use of cfqueryparam as protection agaisnt sql injection even if it means giving up server performance.




0
 

Author Closing Comment

by:paid_tech
ID: 31510776
Thank you very much, your answers were detailed and easy to understand
0

Featured Post

Master Your Team's Linux and Cloud Stack!

The average business loses $13.5M per year to ineffective training (per 1,000 employees). Keep ahead of the competition and combine in-person quality with online cost and flexibility by training with Linux Academy.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

The technique is by far very Simple! How we can export the ColdFusion query results to DOC file?  Well before writing this I researched a lot in Internet but did not found a good Answer anyways!  So i thought now i should share my small snippet w…
PROBLEM: How to add your own buttons to the bottom toolbar with paging info ( result count ). While creating a cfgrid, I ran into an issue where I wanted to embed my own custom buttons where the default ones ( insert / delete / etc… ) are for aes…
Microsoft Active Directory, the widely used IT infrastructure, is known for its high risk of credential theft. The best way to test your Active Directory’s vulnerabilities to pass-the-ticket, pass-the-hash, privilege escalation, and malware attacks …
Finds all prime numbers in a range requested and places them in a public primes() array. I've demostrated a template size of 30 (2 * 3 * 5) but larger templates can be built such 210  (2 * 3 * 5 * 7) or 2310  (2 * 3 * 5 * 7 * 11). The larger templa…

839 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question