|
|
|
|
|
|
|
|
|
|
|
|
|
|
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 --
Learn Notes and Domino 8 at your place and pace!
Learn Notes and Domino in your office and/or home! TLCC's highly acclaimed distance learning courses for users, developers, and admins will enhance your career and your resume.
The many included activities and demos will make you a pro! Expert instructor help is a click away.
Click here to try a FREE demo course!! |
-- Advertisement --
Mark your calendar for in-depth Lotus training, May 12-14, Boston
Join experts and peers May 12-14 in Boston for educational and networking events that deliver real-world Lotus training so you can increase productivity and efficiency in your company, advance your skills, and squeeze the most from your current environment. One registration gets you into THE VIEW's Admin2010 and Lotus Developer2010.
Register by April 10 to save $200. |
|
|
|
|
|
|
|
|
|
|
