There are only two hard things in Computer Science: cache invalidation and naming things -- Phil Karlton
Naming convention is a set of rules for choosing the character sequence to be used for identifiers which denote variables, types, functions, and other entities in source code and documentation.
Reasons for using a naming convention (as opposed to allowing programmers to choose any character sequence) include the following:
- To reduce the effort needed to read and understand source code.
- To enable code reviews to focus on more important issues than arguing over syntax and naming standards.
- To enable code quality review tools to focus their reporting mainly on significant issues other than syntax and style preferences.
- SQL Server Object Name Convention
- SQL Server Data Types Recommendation
- T-SQL Programming T-SQL Style
- Reference and useful links
Object | Code | Notation | Length | Plural | Prefix | Suffix | Abbreviation | Char Mask | Example |
---|---|---|---|---|---|---|---|---|---|
Database | UPPERCASE | 30 | No | No | No | Yes | [A-z] | MYDATABASE |
|
Schema | lowercase | 30 | No | No | No | Yes | [a-z][0-9] | myschema |
|
Global Temporary Table | PascalCase | 117 | No | No | No | Yes | ##[A-z][0-9] | ##MyTable |
|
Local Temporary Table | PascalCase | 116 | No | No | No | Yes | #[A-z][0-9] | #MyTable |
|
File Table | PascalCase | 128 | No | FT_ |
No | Yes | [A-z][0-9] | FT_MyTable |
|
Memory-optimized SCHEMA_AND_DATA Table | PascalCase | 128 | No | MT_ |
_SD |
Yes | [A-z][0-9] | MT_MyTable_SD |
|
Memory-optimized SCHEMA_ONLY Table | PascalCase | 128 | No | MT_ |
_SO |
Yes | [A-z][0-9] | MT_MyTable_SO |
|
Temporal Table | PascalCase | 128 | No | No | _TT |
Yes | [A-z][0-9] | MyTable_TT |
|
Disk-Based Table | U | PascalCase | 128 | No | No | No | Yes | [A-z][0-9] | MyTable |
Disk-Based Wide Table - SPARSE Column | U | PascalCase | 128 | No | No | _SPR |
Yes | [A-z][0-9] | MyTable_SPR |
Table Column | PascalCase | 128 | No | No | No | Yes | [A-z][0-9] | MyColumn |
|
Table Column SPARSE | PascalCase | 128 | No | No | _SPR |
Yes | [A-z][0-9] | MyColumn_SPR |
|
Columns Check Constraint | C | PascalCase | 128 | No | CTK_ |
No | Yes | [A-z][0-9] | CTK_MyTable_MyColumn_AnotherColumn |
Column Check Constraint | C | PascalCase | 128 | No | CK_ |
No | Yes | [A-z][0-9] | CK_MyTable_MyColumn |
Column Default Values | D | PascalCase | 128 | No | DF_ |
No | Yes | [A-z][0-9] | DF_MyTable_MyColumn |
Table Primary Key | PK | PascalCase | 128 | No | PK_ |
No | Yes | [A-z][0-9] | PK_MyTableID |
Table Unique (Alternative) Key | UQ | PascalCase | 128 | No | AK_ |
No | Yes | [A-z][0-9] | AK_MyTable_MyColumn_AnotherColumn |
Table Foreign Key | F | PascalCase | 128 | No | FK_ |
No | Yes | [A-z][0-9] | FK_MyTable_ForeignTableID |
Table Clustered Index | PascalCase | 128 | No | IXC |
No | Yes | [A-z][0-9] | IXC_MyTable_MyColumn_AnotherColumn |
|
Table Non Clustered Index | PascalCase | 128 | No | IX_ |
No | Yes | [A-z][0-9] | IX_MyTable_MyColumn_AnotherColumn |
|
DDL Trigger | TR | PascalCase | 128 | No | TR_ |
_DDL |
Yes | [A-z][0-9] | TR_LogicalName_DDL |
DML Trigger | TR | PascalCase | 128 | No | TR_ |
_DML |
Yes | [A-z][0-9] | TR_MyTable_LogicalName_DML |
Logon Trigger | TR | PascalCase | 128 | No | TR_ |
_LOG |
Yes | [A-z][0-9] | TR_LogicalName_LOG |
View | V | PascalCase | 128 | No | VI_ |
No | No | [A-z][0-9] | VI_LogicalName |
Indexed View | V | PascalCase | 128 | No | VIX_ |
No | No | [A-z][0-9] | VIx_LogicalName |
Stored Procedure | P | PascalCase | 128 | No | usp_ |
No | No | [A-z][0-9] | usp_LogicalName |
Scalar User-Defined Function | FN | PascalCase | 128 | No | udf_ |
No | No | [A-z][0-9] | udf_FunctionLogicalName |
Table-Valued Function | FN | PascalCase | 128 | No | tvf_ |
No | No | [A-z][0-9] | tvf_FunctionLogicalName |
Synonym | SN | camelCase | 128 | No | sy_ |
No | No | [A-z][0-9] | sy_logicalName |
Sequence | SO | PascalCase | 128 | No | sq_ |
No | No | [A-z][0-9] | sq_TableName |
CLR Assembly | PascalCase | 128 | No | CA |
No | Yes | [A-z][0-9] | CALogicalName |
|
CLR Stored Procedures | PC | PascalCase | 128 | No | pc_ |
No | Yes | [A-z][0-9] | pc_CAName_LogicalName |
CLR Scalar User-Defined Function | PascalCase | 128 | No | cudf_ |
No | No | [A-z][0-9] | cudf_CAName_LogicalName |
|
CLR Table-Valued Function | PascalCase | 128 | No | ctvf_ |
No | No | [A-z][0-9] | ctvf_CAName_LogicalName |
|
CLR User-Defined Aggregates | PascalCase | 128 | No | ca_ |
No | No | [A-z][0-9] | ca_CAName_LogicalName |
|
CLR User-Defined Types | PascalCase | 128 | No | ct_ |
No | No | [A-z][0-9] | ct_CAName_LogicalName |
|
CLR Triggers | PascalCase | 128 | No | ctr_ |
No | No | [A-z][0-9] | ctr_CAName_LogicalName |
More details about SQL Server data types and mapping it with another databases and program languages you can find here
General Type | Type | ANSI | Recommended | What use instead | Why use or not |
---|---|---|---|---|---|
Exact Numerics | bit | No | Maybe | tinyint | bit convert any number (except 0) to 1, 0 converted to 0 |
Exact Numerics | tinyint | No | Maybe | int | for saving 3 bytes compare to int data type or for replacing bit data type |
Exact Numerics | smallint | Yes | Maybe | int | for saving 2 bytes compare to int data type |
Exact Numerics | int | Yes | Yes | - | |
Exact Numerics | bigint | No | Yes | int | if you work more than |
Exact Numerics | decimal | Yes | Yes | - | |
Exact Numerics | smallmoney | No | Maybe | decimal | possibility to loss precision due to rounding errors |
Exact Numerics | money | No | Maybe | decimal | possibility to loss precision due to rounding errors |
Approximate Numerics | real | Yes | Yes | - | |
Approximate Numerics | float(1-24) | Yes | No | real | SQL Server automatically converts float(1-24) to real data type |
Approximate Numerics | float(24-53) | Yes | Yes | - | |
Date and Time | date | Yes | Yes | - | |
Date and Time | smalldatetime | No | Maybe | date | |
Date and Time | time | Yes | Yes | - | |
Date and Time | datetime2 | No | Yes | - | |
Date and Time | datetime | Yes | Maybe | datetime2 | On the Advantages of DateTime2(n) over DateTime |
Date and time | datetimeoffset | No | Yes | - | |
Character Strings | char | Yes | Maybe | varchar | Save 1 byte from varchar , but be ready for trailing spaces |
Character Strings | varchar | Yes | Yes | - | |
Character Strings | varchar(max) | Yes | Yes | - | |
Character Strings | nchar | Yes | Maybe | nvarchar | |
Character Strings | nvarchar | Yes | Yes | - | |
Character Strings | nvarchar(max) | Yes | Yes | - | |
Character Strings | ntext | No | Deprecated | nvarchar(max) | NVARCHAR(MAX) VS NTEXT in SQL Server |
Character Strings | text | No | Deprecated | varchar(max) | Differences Between Sql Server TEXT and VARCHAR(MAX) Data Type |
Binary Strings | image | No | Deprecated | varbinary(max) | VARBINARY(MAX) Tames the BLOB |
Binary Strings | binary | Yes | Deprecated | varbinary | Conversions between any data type and the binary data types are not guaranteed |
Binary Strings | varbinary | Yes | Yes | - | |
Binary Strings | varbinary(max) | Yes | Yes | - | |
Other Data Types | cursor | No | Yes | - | |
Other Data Types | sql_variant | No | Yes | - | |
Other Data Types | hierarchyid | No | Yes | - | |
Other Data Types | rowversion | No | Maybe | - | |
Other Data Types | timestamp | No | Deprecated | rowversion | it is just synonym to rowversion data type and must be removed |
Other Data Types | uniqueidentifier | No | Yes | - | |
Other Data Types | xml | Yes | Yes | - | |
Other Data Types | table | No | Maybe | - | |
Spatial Data Types | geometry | No | Yes | - | |
Spatial Data Types | geography | No | Yes | - |
SQL Server T-SQL Coding Conventions, Best Practices, and Programming Guidelines.
-
For database objects names in code use only schema plus object name, do not hardcode server and database names in your code:
dbo.MyTable
is good and badPRODSERVER.PRODDB.dbo.MyTable
. More details here, here and here. -
Delimiters: spaces (not tabs)
-
Avoid using asterisk in select statements
SELECT *
, use explicit column names. More details here. -
No square brackets
[]
and reserved words in object names and alias, use only Latin symbols[A-z]
and numeric[0-9]
. -
Prefer ANSI syntax and functions (
CAST
insteadCONVERT
,COALESE
insteadISNULL
, etc.). -
All finished expressions should have semicolon
;
at the end. This is ANSI standard and Microsoft announced with the SQL Server 2008 release that semicolon statement terminators will become mandatory in a future version so statement terminators other than semicolons (whitespace) are currently deprecated. This deprecation announcement means that you should always use semicolon terminators in new development. More details here. -
All script files should end with
GO
and line break. -
Keywords should be in UPPERCASE:
SELECT
,FROM
,GROUP BY
etc. -
Data types declaration should be in lowercase:
varchar(30)
,int
,real
,nvarchar(max)
etc. More details here. -
All system database and tables must be in lowercase for properly working for Case Sensitive instance:
master, sys.tables …
. -
Avoid non-standard column aliases, use, if required, double-quotes for special characters and always
AS
keyword before alias:SELECT p.LastName AS "Last Name" FROM dbo.Person AS p;
More details here. All possible ways using aliases in SQL Server:
/* Recommended due to ANSI */ SELECT SCHEMA_NAME(schema_id) + '.' + "name" AS "Tables" FROM sys.tables; /* Not recommended but possible */ SELECT SCHEMA_NAME(schema_id) + '.' + [name] AS "Tables" FROM sys.tables; SELECT Tables = SCHEMA_NAME(schema_id) + '.' + [name] FROM sys.tables; SELECT "Tables" = SCHEMA_NAME(schema_id) + '.' + [name] FROM sys.tables; SELECT [Tables] = SCHEMA_NAME(schema_id) + '.' + [name] FROM sys.tables; SELECT 'Tables' = SCHEMA_NAME(schema_id) + '.' + [name] FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] [Tables] FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] 'Tables' FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] "Tables" FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] Tables FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] AS [Tables] FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] AS 'Tables' FROM sys.tables; SELECT SCHEMA_NAME(schema_id) + '.' + [name] AS Tables FROM sys.tables;
-
The first argument in
SELECT
expression should be on the next line:SELECT FirstName
-
Arguments are divided by line breaks, commas should be placed before an argument:
SELECT FirstName , LastName
-
For SQL Server >= 2012 use
FETCH-OFFSET
insteadTOP
. More details here. But if you useTOP
avoid useTOP
in aSELECT
statement without anORDER BY
. More details here. -
If you using
TOP
(instead recommendedFETCH-OFFSET
) function with round brackets becauseTOP
has supports use of an expression, such as(@Rows*2)
, or a sub query:SELECT TOP(100) LastName …
. More details here. AlsoTOP
without brackets does not work withUPDATE
andDELETE
statements./* Not working without brackets () */ DECLARE @n int = 1; SELECT TOP@n name FROM sys.objects;
-
For demo queries use
TOP(100)
or lower value because SQL Server uses one sorting method forTOP
1-100 rows, and a different one for 101+ rows. More details here. -
Avoid specifying integers in the
ORDER BY
clause as positional representations of the columns in the select list. The statement with integers is not as easily understood by others compared with specifying the actual column name. In addition, changes to the select list, such as changing the column order or adding new columns, requires modifying theORDER BY
clause in order to avoid unexpected results. More details here./* bad */ SELECT ProductID, Name FROM Production.Production ORDER BY 2; /* good */ SELECT ProductID, Name FROM Production.Production ORDER BY Name;
-
Avoid using
ISNUMERIC
function. Use for SQL Server >= 2012TRY_CONVERT
function and for SQL Server < 2012LIKE
expression:CASE WHEN STUFF(LTRIM(TapAngle),1,1,'') NOT LIKE '%[^-+.ED0123456789]%' /* is it a float? */ AND LEFT(LTRIM(TapAngle),1) LIKE '[-.+0123456789]' AND TapAngle LIKE '%[0123456789][ED][-+0123456789]%' AND RIGHT(TapAngle ,1) LIKE N'[0123456789]' THEN 'float' WHEN STUFF(LTRIM(TapAngle),1,1,'') NOT LIKE '%[^.0123456789]%' /* is it numeric? */ AND LEFT(LTRIM(TapAngle),1) LIKE '[-.+0123456789]' AND TapAngle LIKE '%.%' AND TapAngle NOT LIKE '%.%.%' AND TapAngle LIKE '%[0123456789]%' THEN 'float' ELSE NULL END
More details here.
-
Avoid using
INSERT INTO
a permanent table withORDER BY
. More details here. -
Avoid using shorthand (
wk, yyyy, d
etc.) with date/time operations, use full names:month, day, year
. More details here. -
Avoid ambiguous formats for date-only literals, use
CAST('yyyymmdd' AS DATE)
format. -
Avoid treating dates like strings and avoid calculations on the left-hand side of the
WHERE
clause. More details here. -
Avoid using hints except
RECOMPILE
if needed andNOEXPAND
(see next tip). More details here. -
Use
NOEXPAND
hint for indexed views on non enterprise editions and Prior to SQL Server 2016 (13.x) SP1 to let the query optimizer know that we have indexes. More details here. -
Avoid use of
SELECT…INTO
for production code, use insteadCREATE TABLE
+INSERT INTO …
approach. More details here. -
Use only ISO standard JOINS syntaxes. The old style Microsoft/Sybase
JOIN
style for SQL, which uses the=*
and*=
syntax, has been deprecated and is no longer used. Queries that use this syntax will fail when the database engine level is 10 (SQL Server 2008) or later (compatibility level 100). The ANSI-89 table citation list (FROM tableA, tableB
) is still ISO standard forINNER JOINs
only. Neither of these styles are worth using. It is always better to specify the type of join you requireINNER
,LEFT OUTER
,RIGHT OUTER
,FULL OUTER
andCROSS
, which has been standard since ANSI SQL-92 was published. While you can choose any supportedJOIN
style, without affecting the query plan used by SQL Server, using the ANSI-standard syntax will make your code easier to understand, more consistent, and portable to other relational database systems. More details here. -
Do not use a scalar user-defined function (UDF) in a
JOIN
condition,WHERE
search condition, or in aSELECT
list, unless the function is schema-bound. More details here. -
Use
EXISTS
orNOT EXISTS
if referencing a subquery, andIN
orNOT IN
when have a list of literal values. More details here. -
For concatenate unicode strings:
- always using the upper-case
N
; - always store into a variable of type
nvarchar(max)
; - avoid truncation of string literals, simply ensure that one piece is converted to
nvarchar(max)
. Example:
DECLARE @nvcmaxVariable nvarchar(max); SET @nvcmaxVariable = CAST(N'ಠ russian anomaly ЯЁЪ ಠ ' AS nvarchar(max)) + N'something else' + N'another'; SELECT @nvcmaxVariable;
More details here.
- always using the upper-case
-
Always specify a length to any text-based data type such as
varchar
,nvarchar
,char
,nchar
:/* bad */ DECLARE @myBadVarcharVariable varchar; DECLARE @myBadNVarcharVariable nvarchar; DECLARE @myBadCharVariable char; DECLARE @myBadNCharVariable nchar; /* good */ DECLARE @myGoodVarchareVariable varchar(50); DECLARE @myGoodNVarchareVariable nvarchar(90); DECLARE @myGoodCharVariable char(7); DECLARE @myGoodNCharVariable nchar(10);
More details here.
-
Use only
ORIGINAL_LOGIN()
function because is the only function that consistently returns the actual login name that we started with regardless of impersonation. More details here. -
Always use
IF
statement withBEGIN-END
block to prevent errors with multi line statements:DECLARE @x int = 0; DECLARE @y int = 1; /* bad */ IF @y > @x SET @x = @x + 1; SET @y = @y - 1; ELSE PRINT(1); /* Msg 156, Level 15, State 1, Line 8 Incorrect syntax near the keyword 'ELSE'. */ /* good */ IF @y > @x BEGIN SET @x = @x + 1; SET @y = @y - 1; END; ELSE BEGIN PRINT(1); END;
-
FROM, WHERE, INTO, JOIN, GROUP BY, ORDER BY
expressions should be aligned so, that all their arguments are placed under each other (see Example below)
TSQL Example with formating:
WITH CTE_MyCTE AS (
SELECT
t1.Value1 AS Val1
, t1.Value2 AS Val2
, t2.Value3 AS Val3
INNER JOIN dbo.Table3 AS t2
ON t1.Value1 = t2.Value1
WHERE t1.Value1 > 1
AND t2.Value2 >= 101
)
SELECT
t1.Value1 AS Val1
, t1.Value2 AS Val2
, t2.Value3 AS Val3
INTO #Table3
FROM CTE_MyCTE AS t1
ORDER BY t2.Value2;
- All stored procedures and functions should use
ALTER
statement and start with the object presence check (see example below) ALTER
statement should be preceded by 2 line breaks- Parameters name should be in camelCase
- Parameters should be placed under procedure name divided by line breaks
- After the
ALTER
statement and beforeAS
keyword should be placed a comment with execution example - The procedure or function should begin with parameters checks (see example below)
- Create
sp_
procedures only inmaster
database - SQL Server will always scan through the system catalog first - Always use
BEGIN TRY
andBEGIN CATCH
for error handling - Always use multi-line comment
/* */
instead in-line comment--
- Use
SET NOCOUNT ON;
for stops the message that shows the count of the number of rows affected by a Transact-SQL statement and decreasing network traffic. More details here. - Do not use
SET NOCOUNT OFF;
because it is default behavior - Use
RAISERROR
insteadPRINT
if you want to give feedback about the state of the currently executing SQL batch without lags. More details here and here. - All code should be self documenting
- T-SQL code, triggers, stored procedures, functions, scripts, should have a standard comment-documentation banner:
<documentation>
<summary>Get all databases meta data using dynamic T-SQL</summary>
<returns>1 data set: temp table #DatabaseInfo.</returns>
<issues>No</issues>
<author>Konstantin Taranov</author>
<created>2018-03-01</created>
<modified>2019-11-14 by Konstantin Taranov</modified>
<version>1.2</version>
<sourceLink>https://github.com/ktaranov/sqlserver-kit/blob/master/Scripts/Databases_Report.sql</sourceLink>
</documentation>
Stored Procedure Example:
IF OBJECT_ID('dbo.usp_StoredProcedure', 'P') IS NULL
EXECUTE('CREATE PROCEDURE dbo.usp_StoredProcedure as SELECT 1');
GO
ALTER PROCEDURE dbo.usp_StoredProcedure(
@parameterValue1 smallint
, @parameterValue2 nvarchar(300)
, @debug bit = 0
)
/*
<documentation>
<summary>Simple example of tsql procedure</summary>
<returns>nothing</returns>
<issues>No</issues>
<author>Konstantin Taranov</author>
<created>2019-01-01</created>
<modified>2019-11-25 by Konstantin Taranov</modified>
<version>1.2</version>
<sourceLink>-</sourceLink>
<example1>EXECUTE dbo.usp_StoredProcedure
@parameterValue1 = 0
, @parameterValue2 = N'BULK'</example1>
</documentation>
*/
AS
SET NOCOUNT ON;
BEGIN TRY
IF (@parameterValue1 < 0 OR @parameterValue2 NOT IN ('SIMPLE', 'BULK', 'FULL'))
RAISERROR('Not valid data parameter!', 16, 1);
IF (@debug) PRINT @parameterValue2;
END TRY
BEGIN CATCH
/* Print error information. */
PRINT 'Error: ' + CAST(ERROR_NUMBER()) AS varchar(50)) +
', Severity: ' + CAST(ERROR_SEVERITY(), varchar(5)) +
', State: ' + CAST(ERROR_STATE(), varchar(5) ) +
', Procedure: ' + COALESCE(ERROR_PROCEDURE(), '-') +
', Line: ' + CAST(ERROR_LINE(), varchar(5)) +
', User name: ' + CAST(ORIGINAL_LOGIN(), sysname);
PRINT ERROR_MESSAGE();
END CATCH;
GO
Highly recommended to read awesome detailed article about dynamic T-SQL by Erland Sommarskog: The Curse and Blessings of Dynamic SQL
Dynamic SQL is a programming technique that allows you to construct SQL statements dynamically at runtime. It allows you to create more general purpose and flexible SQL statement because the full text of the SQL statements may be unknown at compilation. For example, you can use the dynamic SQL to create a stored procedure that queries data against a table whose name is not known until runtime.
More details here.
- Do not use nvarchar(max) for your object’s name parameter, use sysname instead (synonym for nvarchar(128) except that, by default, sysname is NOT NULL).
/* Bad */ DECLARE @tableName nvarchar(max) = N'MyTableName'; /* Good */ DECLARE @tableName sysname = N'MyTableName';
- Do quote the names of your objects properly.
/* Bad */ DECLARE @tsql nvarchar(max); DECLARE @tableName sysname = N'My badly named table!'; SET @tsql = N'SELECT object_id FROM ' + @tableName; /* Good */ DECLARE @tsql nvarchar(max); DECLARE @tableName sysname = N'My badly named table 111!'; SET @tsql = N'SELECT object_id FROM ' + QUOTENAME(@tableName);
- Always use
sp_executesql
insteadEXEC
to prevent sql injection. Alsosp_executesql
can parameterizing your dynamic statement that means plans can be reused as well (when the value of the dynamic object is the same). Alsosp_executesql
can even be used to output values as well (see example below)./* Bad EXEC example with sql injection*/ DECLARE @tsql nvarchar(max); DECLARE @tableName sysname = N'master.sys.tables; SELECT * FROM master.sys.server_principals;'; SET @tsql = N'SELECT "name" FROM ' + @tableName + N';'; EXEC (@tsql); /* Good sp_executesql example*/ DECLARE @tsql nvarchar(max); DECLARE @tableName sysname = N'master.sys.tables'; DECLARE @id int = 2107154552; SET @tsql = N'SELECT name FROM ' + @tableName + N' WHERE object_id = ' + CONVERT(nvarchar(max), @id); EXEC sp_executesql @tsql, N'@ID int', @ID = @id; /* Good sp_executesql example with OUTPUT */ DECLARE @tsql nvarchar(max); DECLARE @tableName sysname = N'master.sys.tables'; DECLARE @count bigint; SET @tsql = N'SELECT @countOUT = COUNT(*) FROM ' + @tableName + N';'; EXEC sp_executesql @tsql, N'@countOUT bigint OUTPUT', @countOUT = @count OUTPUT; PRINT('@count = ' + CASE WHEN @count IS NULL THEN 'NULL' ELSE CAST(@count AS varchar(30)) END);
- Do not use dynamic T-SQL if your statement is not dynamic.
/* Bad */ DECLARE @tsql nvarchar(max); DECLARE @id int = 2107154552; SET @tsql = N'SELECT object_id, "name" FROM master.sys.tables WHERE object_id = ' + CAST(@id AS nvarchar(max)); EXEC sp_executesql @tsql; /* Good */ DECLARE @id int = 2107154552; SELECT object_id, "name" FROM master.sys.tables WHERE object_id = @id;
- Do not debug the code that creates the dynamic T-SQL first, debug the generated T-SQL statement instead.
Use
@debug
variable to print (or aSELECT
statement if your dynamic T-SQL is over 4000 characters) dynamic statement instead executing it. See example below. - Do take the time to format your dynamic T-SQL.
/* Bad @tsql formating */ DECLARE @tsql nvarchar(max); DECLARE @sep nvarchar(30) = ' UNION ALL '; DECLARE @debug bit = 1; SELECT @tsql = COALESCE(@tsql, N'') + N'SELECT N' + QUOTENAME(name,'''') + N' AS DBName, (SELECT COUNT(*) FROM ' + QUOTENAME(name) + N'.sys.tables) AS TableCount' + @sep FROM sys.databases ORDER BY name; SET @tsql = LEFT(@tsql, LEN(@tsql) - LEN(@sep)); IF @debug = 1 SELECT @tsql AS "tsql" ELSE EXEC sp_executesql @tsql; /* Good @tsql formating */ DECLARE @tsql nvarchar(max); DECLARE @sep nvarchar(30) = ' UNION ALL '; DECLARE @debug bit = 1; DECLARE @crlf nvarchar(10) = NCHAR(13) + NCHAR(10); SELECT @tsql = COALESCE(@tsql, N'') + @crlf + N'SELECT N' + QUOTENAME(name,'''') + N' AS DBName' + @crlf + N' , (SELECT COUNT(*) FROM ' + QUOTENAME(name) + N'.sys.tables) AS TableCount' + @crlf + @sep FROM sys.databases ORDER BY name; SET @tsql = LEFT(@tsql, LEN(@tsql) - LEN(@sep)) + N';'; IF @debug = 1 SELECT @tsql AS "tsql" ELSE EXEC sp_executesql @tsql;
- Transact-SQL Formatting Standards (by Robert Sheldon)
- Subjectivity: Naming Standards (by Aaron Bertrand)
- General Database Conventions (by Thomas Kejser)
- Writing Readable SQL (by Red Gate)
- SQL Style Guide (by Simon Holywell)
- SQL Code Layout and Beautification (by William Brewer)
- TSQL Coding Style (by Gregory Larsen)
- User-Defined Functions
- Synonyms (Database Engine)
- Primary and Foreign Key Constraints
- sys.objects
- SQL Server Constraints
- CHECK Constraint TECHNET
- SQL Server CLR Integration
- Deploying CLR Database Objects
- CLR Stored Procedures
- User-defined Functions
- SET NOCOUNT ON (Transact-SQL)
- T-SQL Coding Guidelines Presentation (by Chris Adkin)
- Sql Coding Style
- SQL Server Code Review Checklist for Developers (by Samir Behara)
- SQL Formatting standards – Capitalization, Indentation, Comments, Parenthesis (by ApexSQL)
- In The Cloud: The Importance of Being Organized
- Naming Conventions in Azure
- The Basics of Good T-SQL Coding Style – Part 3: Querying and Manipulating Data
- SQL naming conventions (by Phi Factor)
- SQL Server Compact Object Limitations
- Dos and Don'ts of Dynamic SQL (by Thom Andrews)