»
Home
Products
Support
Developer Community
Company
Buy
Demo
Documentation
Price
More Info
ColdFusion Code Review Rules
accessibility
Missing ALT attributes
You need to use ALT attributes if your going to be section 508 compatable
Missing inner content in OBJECT tag
Enter a HTML description or replacement of the object inside the object tag.
Missing TITLE attribute in OBJECT tag
You should use the title tag to explain your object.
bugprone
Nested CFLOCK tags
Nesting CFLOCK tags may cause a deadlock, which will crash your server.
Variable in CFERROR template
If there is a variable in the CFERROR template it will throw an error.
Relative URL in CFSCHEDULE
You must use an absolute url in the URL attribute of CFSCHEDULE
Nested CFTRANSACTION tags
CFTRANSACTION tags cannot be nested if the inner transaction action is BEGIN
OnRequestEnd.cfm must be in same directory as Application.cfm
The OnRequestEnd.cfm file must be in the same directory as the Application.cfm file
CFINCLUDE Template does not exists
The template in the CFINCLUDE or CFERROR tag does not exist.
Query String in CFINCLUDE
Query strings will not be processed within a cfinclude
Missing maxlength attribute
You should use the maxlength attribute in your form elements
Unnecessary CFOUTPUT tag
Some tags implicitly define cfoutput for you such as cfmail, and cfquery
Missing Column List in Insert Into Statement
Although it is syntatically correct to omit the column list, it leads to difficult maintenance
deprecated
Use of ParameterExists
The ParameterExists function has been deprecated since CF4, use IsDefined("var") instead
documentation
Missing HINT attribute
Use the HINT attribute to document a CFC or Function
Missing DISPLAYNAME attribute
Use the DISPLAYNAME attribute to document a CFC or Function
performance
Use of IIF
The IIF function is slower, and less readable than using conditionals
Use of SELECT *
Using SELECT * often returns more columns than necessary.
Use of SetVariable in CFSET
Use the equals operator in the CFSET tag instead for a speed and readibiliity increase.
StructFind Usage
StructFind is slower and less readable than dot or bracket notation.
Boolean Usage
Variables that resolve to true/false,yes/no, or 1/0 can be treated as booleans. Use cfif var, or cfif NOT var instead of cfif var IS true.
Use of IncrementValue function
IncrementValue is very slow use the addition operator inside of cfset instead.
Missing allow overwrite argument in StructInsert
Set the allow overwrite argument to true for a speed increase
Zero length String comparison
Use the Len function instead of IS ""
portability
File name case must be Application.cfm for UNIX
UNIX have case sensitive file systems you must use Application.cfm with a capital A
File name case must be OnRequestEnd.cfm for UNIX
UNIX have case sensitive file systems you must use OnRequestEnd.cfm
CFINCLUDE Unix Compatibility
The template path in CFINCLUDE is case sensitive on unix.
security
Missing CFQUERYPARAM
Using CFQUERYPARAM both increases performance, and prevents common security vulnerabilities.
Missing accept attribute in CFFILE
Without the accept attribute it is possible for users to upload files of any type.
style
Pound Sign # Usage
Pound Signs only need to be used in CFSET or CFIF Tags when they are contained within qoutes. They also have slight performance implications.
Missing End Tags
Closing tags are recommended on tags such as li, and option.
Quoted Numerals
Quotes are not needed around numbers in expressions.
Sign up for our newsletter:
| Subscribe with RSS:
RSS
© ActivSoftware 1999 to 2005 |
Privacy Statement