SQL Clone
SQLServerCentral is supported by Redgate
Log in  ::  Register  ::  Not logged in

Guest editorial: SQL Code Metrics

By Phil Factor,

SQL has a wonderful way of teaching us humility. Just when we think we have it sussed, we realise that there are better ways of doing it. I've never found a routine that can't be improved in some way; particularly in my own work. Writing good SQL code is a fascinating activity, a constant fight against hasty assumptions, outdated rules, and learned procedural programming practices. SQL is very different from any other type of programming.

This truth of this was brought home to me sharply the other day whilst reading about 'code smells' and quality metrics in C#. How wonderful it would be, I thought, if we could use the same techniques in order to detect 'smelly' SQL code in a database system. All we'd need to do is to check on the complexity of the code, the number of lines of code in a procedure, the number of SQL Statements, and possibly the complexity of the execution plan.

The only tool I've found that makes some of this available is SQLTAC. However, a lot more is attainable. For example, we could check for an over-reliance on cursors, the use of the * to insert into a table, the presence of SQL Statements that consistently take a long time to execute, code without comments or documentation, over-use of table variables, tables without a primary key, over-denormalized tables, and so on. It would be deeply satisfying to be able to produce a graphical representation of TSQL code quality, to wave in front of some poor programmer's face when you're checking his work!

Just as quickly as the thought had occurred to me, however, I dismissed it. The trouble is that when I supervise a programmer who is learning SQL, I sometimes learn new ideas that come from a fresh mind being applied to old problems. The code may, in general, stink, but the new ideas are priceless. If we apply 'quality metrics' and 'best practices' without applying our own judgement and intuition, we run the risk of ossifying rules that are merely intended to be guides. Code metrics are a good servant but a very poor master.

Phil Factor.

Total article views: 376 | Views in the last 30 days: 2
Related Articles

Actionable Metrics

Metrics should help you learn, or otherwise change the behavior of the system.


Real World: SQL Server Quick Quality Check

Here is a script that can give the DBA a SQL Instance quick quality check.


Useful Links for learning & using in Data Quality Projects in SQL Server

Data Quality Services in SQL Server is a component that is based around the concept of knowledge-dri...


Data Quality

Today we have an editorial reprinted from Dec 12, 2005 as Steve is on vacation. Steve talks about th...


Human Data Quality

Disclaimer: I read and speak one language, having failed pretty well at learning Latin, Spanish, and...