Assigning aliases to selected fields in a query (AssignAliasFieldsInQuery)¶
Type | Scope | Severity | Activated by default |
Minutes to fix |
Tags |
---|---|---|---|---|---|
Code smell |
BSL |
Major |
Yes |
1 |
standard sql badpractice |
Description¶
It is recommended to specify optional query constructs, first, to explicitly assign aliases to fields in order to increase the clarity of the query text and the "steadiness" of the code that uses it.
For example, if the algorithm uses a query with a field declared as
CashBox.Currency
CashBox.Currency As Currency
You should be especially careful about automatically assigned aliases for fields - attributes of other fields, such as "... CashBox.Currency.Name...". In the above example, the field will be automatically getting aliased CurrencyName, but not Name.
Be sure to include the AS keyword before the alias of the source field.
The aliases of tables and fields from secondary queries from "UNION" are not checked by the diagnostics.
Examples¶
Query = New Query;
Query.Text =
"SELECT
| Currencies.Ref, // Incorrectly
| Currencies.Ref AS AliasFieldsRef, // Correctly
| Currencies.Code Code // Incorrectly
|FROM
| Catalog.Currencies AS Currencies // Ignored
|
|UNION ALL
|
|SELECT
| Currencies.Ref, // Ignored
| Currencies.Ref, // Ignored
| Currencies.Code // Ignored
|FROM
| Catalog.Currencies AS Currencies // Ignored
|;
|
|////////////////////////////////////////////////////////////////////////////////
|SELECT
| Currencies.Ref, // Incorrectly
| Currencies.Ref AS AliasFieldsRef, // Correctly
| Currencies.Code Code // Incorrectly
|FROM
| Catalog.Currencies AS Currencies // Ignored
|
|UNION ALL
|
|SELECT
| Currencies.Ref, // Ignored
| Currencies.Ref, // Ignored
| Currencies.Code // Ignored
|FROM
| Catalog.Currencies AS Currencies"; // Ignored
Query1 = New Query;
Query1.Text =
"SELECT
| NestedRequest.Ref AS Ref // Correctly
|FROM
| (SELECT
| Currencies.Ref // Incorrectly
| FROM
| Catalog.Currencies AS Currencies) AS NestedRequest"; // Ignored
Sources¶
Source: Making query text
Snippets¶
Diagnostic ignorance in code¶
// BSLLS:AssignAliasFieldsInQuery-off
// BSLLS:AssignAliasFieldsInQuery-on
Parameter for config¶
"AssignAliasFieldsInQuery": false