Comment directives let you ignore code review problems. When you wish to ignore problems in a certain section of code, you write a comment directive in your code. Comment directives are specially formatted comments. Example: '$PROBHIDE ALL hides all problems on a single line.
With comment directives you can:
For a start, let's see what a typical directive looks like:
That's it! This will hide a "Goto statement found" problem for a line.
Now let's look at the syntax. Comment directives always start with '$. The '$ prefix marks the start of a comment directive line.
'$ directive [ : directive [ : directive ...]]
Note that Rem $ is not acceptable as a comment directive. You can put multiple directives on a line by separating them with a colon ( : ). Usually you would just put a single directive on a line, though.
The available directives are: PROBHIDE to hide problems, PROBSHOW to show them again, END to finish a BEGIN..END block and USED, which is now obsolete.
The PROBHIDE directive hides problems of one or more types. Use it to disable problem detection at locations you don't want to change. Syntax:
PROBHIDE <problem type> [<problem type> ...] [EXCEPT <problem type> [<problem type> ...] ] [BEGIN]
|<problem type>||Problem type(s) to hide. See the Code review rules for the name of each problem type. You can also use combination types, see below. You can list any number of problem types separated by space.|
|EXCEPT||Optional. Problems types listed after this parameter are not affected by the rule.|
|BEGIN||Optional. Starts a BEGIN...END block. The directive affects all code until a matching '$ END directive or end-of-file. If BEGIN is omitted, the directive affects just one line of code. See Scope.
|DEAD||Dead code related problems|
|FUNC, FUNCTIONALITY||Functionality related problems|
|METRICS||Metrics related problems|
|OPT, OPTIMIZATION||Optimization related problems (also includes DEAD)|
|SCOPE||Combination of EXCESS_SCOPE, NO_SCOPE, MEMBER_SCOPE and GLOBAL|
|STYLE||Style related problems (also includes METRICS)|
|VB.NET||VB.NET Compatibility Check problems|
|INFO, INFORMATION||Informational problems|
Each code review rule has its own directive keyword. They are given in UPPER_CASE after each problem description in Code review rules. Certain keywords may cover two or more closely related problems.
A procedure does not require an error handler. Hide the "Error handler missing" problem for that procedure but show it everywhere else.
'$ PROBHIDE NO_ERROR_HANDLER
' This procedure needs no error handling
A procedure is dead at the moment but you don't want to remove it. You want to flag the procedure live without hiding any other dead code related problems. Notice how the syntax affects just the next line (the line with the dead procedure problem). Any other lines are not affected and the dead variable problem will thus be shown. Notice that hiding a dead procedure problem propagates down along the call tree.
'$ PROBHIDE DEAD
' This procedure should be kept for later use
Dim DeadVariable As Integer ' But this variable should be removed
' This procedure shows up as live because it's called by DeadProcedure which was flagged live
Hide all problems classified as "warning" or "information" until end-of-file.
'$ PROBHIDE WARNING INFORMATION BEGIN
code with problems
that I want to hide
Hide all problems in a range of code between BEGIN and END. Keep showing dead code related problems at all locations.
'$ PROBHIDE ALL EXCEPT DEAD BEGIN
code with problems
show all problems after this point
A Goto statement is acceptable at a certain location. Hide the problem on that specific line. There are two alternative ways to write the same directive: write the comment either on the line with the problem or add it on a new line immediately above it.
Goto 1999 '$ PROBHIDE GOTO
'$ PROBHIDE GOTO
Enables (shows) problems that would otherwise be hidden by PROBHIDE. Syntax:
PROBSHOW <problem type> [<problem type> ...] [EXCEPT <problem type> [<problem type> ...] ] [BEGIN]
The syntax is similar to that of PROBHIDE. You should use PROBSHOW only inside a PROBHIDE BEGIN ... END block.
Hide all problems in a class, but enable dead code problems after class declarations to remove any dead procedures and local variables/constants.
'$ PROBHIDE ALL BEGIN
Dim x, y
Const PI = 3.14
Declare Function ChooseFont Lib "comdlg32.dll" Alias "ChooseFontA" (pChoosefont As CHOOSEFONT) As Long
'$ PROBSHOW DEAD BEGIN
Sub MyProc ( x )
Ends a BEGIN...END directive block. See Scope below for more information.
This obsolete directive marks things as being in use. It is provided for only backward compatibility. Older Project Analyzer versions used to support it until 1999. You should not use this directive any more since future versions may not support it. Project Analyzer replaces it internally as follows:
USED => PROBHIDE DEAD
USED PROC => PROBHIDE DEAD_PROC
USED PARAM => PROBHIDE DEAD_PARAM
How do you work out where to put the directives and what code they will then affect? First of all, look at the problem icons in the hypertext view of Project Analyzer. Your comment directives should be placed to affect the lines with the icons. If your directives don't seem to work, pay attention to the exact line and the scope you're using. There are several alternative ways to define the scope of a comment directive.
'$ PROBHIDE DEAD
Dim MyVariable As Integer
A directive on an otherwise empty line affects the immediately following line. In this case, MyVariable is not reported dead.
Dim MyVariable As Integer '$ PROBHIDE DEAD
A directive after the code on a given line affects that line. In this case, MyVariable is not reported dead.
'$ PROBHIDE DEAD BEGIN
Dim MyVariable As Integer
Dim Text As String
The directive affects all code between the BEGIN and the END directive. In this case, MyVariable and Text are not reported dead.
You can nest multiple BEGIN...END blocks. You may also omit the END directive. In that case, the block ends at end-of-file.
'$ PROBHIDE DEAD WHERE expression
To hide problems on specific lines use the WHERE scope. The directive affects all lines containing expression. Examples:
The WHERE expression is case insensitive. expression can be any string, but a colon : cannot be included. If expression contains multiple words, spaces between the words are significant. Spaces before and after expression don't have an effect. The WHERE directive can exist in any file. It affects all files in the analysis (similar to ANYWHERE).
'$ PROBHIDE DEAD IN_THIS_FILE
To hide problems in an entire file, regardless of which line they affect, use the IN_THIS_FILE scope. You can put this keyword anywhere in the file to make a directive apply to all lines in the file, also lines before the directive. IN_THIS_FILE is useful when you need to hide problems at the start of a file, but that area is reserved for comment block and a directive would not look good there.
'$ PROBHIDE DEAD IN_THIS_PROJECT
To hide problems in the whole project, regardless of which file they appear in, use the IN_THIS_PROJECT scope. You can put this keyword anywhere to make a directive apply to all files of the project. This directive may be overridden in each individual file by scopes A-D. On the other hand, it will override ANYWHERE directives.
IN_THIS_PROJECT is useful when your regular coding standards don't apply to a certain project. As an example, '$ PROBHIDE STYLE IN_THIS_PROJECT ignores style rules in the project while they are still enabled for other projects.
'$ PROBHIDE DEAD ANYWHERE
To hide problems in all files, regardless of which project the file may be in, use the ANYWHERE scope. You can put this keyword wherever you like to make a directive apply to all files of the analysis. Note that this directive may be overridden in each individual file by scopes A-D and each individual project by scope E.
Use ANYWHERE instead of IN_THIS_PROJECT when you want to control an entire analysis instead of an individual project. It will cover all files when analyzing one or more solutions, project groups or a directory tree.
Scope ANYWHERE has the lowest precendence, IN_THIS_PROJECT is in the middle. The other scopes are on the highest precedence level.
To avoid any conflicts put all ANYWHERE directives in a single file. The precedence of conflicting ANYWHERE directives mixed in different files is undefined. The same goes for conflicting IN_THIS_PROJECT directives. You may want to put them in a global declarations file to make them easier to manage.
Directives coming later override directives on preceding lines. Thus, you can always override a previous directive by writing a new directive after it. As an example, you can have a '$ PROBHIDE ALL early in the file and then override it with a '$ PROBSHOW DEAD IN_THIS_FILE later. In this case, the later PROBSHOW has higher precedence and will override the PROBHIDE for dead code problems.
If you don't want a directive to override others, use the EXCEPT keyword. As an example, '$ PROBSHOW ALL EXCEPT STYLE IN_THIS_FILE would show all other problems but it wouldn't have any effect on style related problems. Style problems would be entirely controlled by other directives, should any exist.
'$ PROBHIDE STYLE IN_THIS_PROJECT causes style related problems to be ignored in a project where style doesn't matter.
'$ PROBSHOW ALL IN_THIS_FILE enables all problems for important.bas regardless of any setting in general.bas.
'$ PROBHIDE TYPECHAR WHERE &
This example disables the rules "Function/constant/variable with type character" on lines containing a & to allow
Dim Number& instead of
Dim Number As Long.
'$ PROBHIDE GOTO SINGLELINEIF WHERE Goto Quit
This example allows the use of the Goto keyword to jump to Quit. In addition, it also allows a single-line If..Then to exist on the same line. Code that would be allowed:
If somecondition Then Goto Quit
'$ PROBHIDE SINGLELINEIF ANYWHERE
'$ PROBSHOW SINGLELINEIF WHERE Else
This example allows a single line If..Then but not a single-line If..Then..Else.
You can flag a procedure as live code in several ways. The recommended way is by '$ PROBHIDE DEAD or '$ PROBHIDE DEAD_PROC.
Hiding a dead procedure problem flags the affected procedure as live code. What is more, also the procedures it calls are flagged live, down along the entire call tree. Consider a call tree such as A -> B -> C. If you flag A as a live procedure, also B and C will show up as live. Thus, using ' $PROBHIDE to hide one dead procedure also propagates downwards in the call tree. You can mark just the root procedure as live and Project Analyzer will take care of the rest. The call tree propagation only affects procedures. Variables, classes or any other code is not affected.
There's a command line option that tells Project Analyzer to ignore any comment directives. Start Project Analyzer like this: project.exe /NOCDIRECTIVES. Use this option to detect all problems even if $PROBHIDE has been used to hide some of them. You can also use it if your code happens to have existing comment lines that start with $PROBHIDE, $PROBSHOW or $USED.
PROJECT_ANALYZER is a built-in "compiler constant" that allows you to tweak the code analysis.You can add missing definitions or exclude sections from being analyzed. During code analysis, PROJECT_ANALYZER=True. To Visual Basic, PROJECT_ANALYZER=False.
PROJECT_ANALYZER is not a comment directive. It's a constant for #If..#ElseIf..#Else..#End If.
The following trick lets you run your code with late binding, but analyze it as if it were early-bound.
#If PROJECT_ANALYZER Then Dim o As MyClass ' Analyze early-bound calls only #Else Dim o As Object ' Run with late binding #End If
The next trick lets you write code that won't be analyzed at all. You can also add code for the purpose of code analysis only.
#If PROJECT_ANALYZER Then FakeCall ' Analyze what would happen with this call #Else SecretCall ' This runs but won't be analyzed #End If
Versions. Suppose your code compiles two versions of the same program. To compile version 1, you set VERSION=1 in your project settings. To compile the other version, you set VERSION=2. Now, you exclude and include sections of code with #If to compile the 2 versions. This causes problems in the analysis, because Project Analyzer should read your code in its entirety and not just one of the versions.
The following trick lets you compile several versions of your program while keeping Project Analyzer analyzing the full code.
#If VERSION=1 Or PROJECT_ANALYZER Then Version 1 code goes here #End If #If VERSION=2 Or PROJECT_ANALYZER Then Version 2 code goes here #End If
Tweak metrics. Suppose you have a large section of generated code that shouldn't be counted in the logical lines of code (LLOC) metric, because it exaggerates the size of your program. By adding the generated code in a False block you exclude it from LLOC, even when it still compiles and runs just fine.
#If Not PROJECT_ANALYZER Then excluded code goes here #End If
©Aivosto Oy - Project Analyzer Help Contents