|
|
|
|
|
|
|
|
|
|
|
|
|
|
CODING BASICS
A simple way of commenting if/else follow-up
By Joshua B. Jore
| Erwin van Hunen's article "A simple way of commenting if/else for better understanding" in the August issue of DominoPower Magazine has generated quite a bit of feedback. This time around, we have a piece sent in by Joshua B. Jore, a Domino developer for Imation Corporation, regarding guard statements. Take it away Joshua . . .
|
Mr. van Hunen's recent article dealing with deeply nested conditional statements at http://www.dominopower.com/issues/issue200408/00001323001.html appeared to be lacking the mention of guard statements.
When you have a block of code you wish to run conditional on a number of criteria, perhaps with some set-up work along the way, guard statements allow for readable code instead keeping the deep nesting and trusting to yet more text to deobfuscate it.
The key here is to use Goto in a way that doesn't violate the Dijkstra's original frustrations. More on this can be found at http://c2.com/cgi/wiki?GotoConsideredHarmful, but specifically Knuth's expansion on Goto at http://pplab.snu.ac.kr/courses/PL2001/papers/p261-knuth.pdf.
Goto has a bad reputation, deservedly so, since it enables some horrible, horrible flow control. It has the ability to clarify the important parts of a program -- the code being executed for its effects over the control flow.
The key thing here is that all the uses of Goto are forward and local. There is a distinct sense of a block being skipped over, that the destination is always further down, and when code is skipped, there isn't going to be any tricky-reentering code.
I normally use this sort of construction when writing loops where there is a significant number of conditionals (more than two), or where there is a significant block (more than two lines).
The idea is to find a balance that makes the code visually pleasing, so the most important parts (the effect generating code, normally) can stand out more. The indentation effects of multiple conditionals serve to make the most important code less visible by pushing it farther away and also making it less clear.
If docCurrent Is Nothing Then
' The docCurrent UIDocument is not available.
If Not ViewCurrent Is Nothing Goto SkipFormChange
' The viewCurrent object is not set.
Set viewCurrent = dbCurrent.getView( "vlu-Names" )
If viewCurrent Is Nothing Goto SkipFormChange
Set docCurrent = viewCurrent.GetFirstDocument()
If docCurrent Is Nothing Goto SkipFormChange
' Do something with the document.
Call docCurrent.ReplaceItemValue( "Form", "f-Test" )
End If
SkipFormChange:
|
The counter argument to this is your exact same code with the addition of some code inside the nested ifs but after one of the inner end-ifs. Normally such a beast would be poor coding as-is but it also limits the number of nested ifs that can be stripped down to a SKIP-OVER block.
If ... Then
If ... Then
' Guard statements may be put here
If ... Then
If ... Then
End If
End If
End If
SkipTheAbove: ' This should have a descriptive label
Call SomeFunction()
End If
|
I hope this sheds a little more light on what can sometimes be a complicated and confusing coding scenario.
Joshua B. Jore is a Domino Developer for Imation Corporation and can be reached at jjore@imation.com.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
-- Advertisement --
Sophisticated Meets Simple For Document Management
Share. Control. Manage.
Documents, emails, and content in the context of how work is done.
Native to Lotus Domino. The User Experience unseen for Lotus Domino.
Do more with less. Really.
See the possibilities Docova unleashes for Lotus Domino. |
-- Advertisement --
Teamstudio Edition 25 has shipped
It's finally here! Now that Teamstudio Edition 25 has shipped, listen to our latest Tool Time audio program to find out what's changed. Updates to all your favorite Teamstudio tools will be discussed.
Plus, you'll get an introduction to Teamstudio Undo (formerly known as Teamstudio Snapper).
Tap here to get started! |
|
|
|
|
|
|
|
|
|
|
