On 11Feb2019 08:17, Irv Kalb <i...@furrypants.com> wrote:
On Feb 11, 2019, at 7:25 AM, Neal Becker <ndbeck...@gmail.com> wrote:
I have code with structure:
```
if cond1:
[some code]
if cond2: #where cond2 depends on the above [some code]
[ more code]
else:
[ do xxyy ]
else:
[ do the same xxyy as above ]
```
So what's the best style to handle this? As coded, it violates DRY.
Try/except could be used with a custom exception, but that seems a bit heavy
handed. Suggestions?
I like the additional function approach others have mentioned. But if you want
a different approach, you could do this:
runExtraCode = True # set some Boolean
if cond1:
[some code]
if cond2:
[more code]
runExtraCode = False # both conditions met, no need to run the extra
code
if runExtraCode:
[do xxyy]
I want to second this. As others have said, there's a trivialness
threshold where you wouldn't bother, and that is subjective. But when
you do, the above is often useful. If "runExtraCode" bothers you,
consider "need_task", with a suitable "task" name.
I've got several things in the above form. The most recent I fiddled
with was last night, in a monitor script which updates some information.
It takes the form:
updated = False
while True:
time.sleep(INTERSCAN_DELAY)
for rpath, dirnames, filenames in os.walk(...):
...
for filename in filenames:
if new file found:
scan file ...
updated = true
if updated:
update top level record ...
updated = False
...
which updates a state file on a per subdirectory frequency if something
new is found. This falls into exactly the pattern Irv describes.
You'll notice my example starts with the flag variable being False, and
makes it True if the extra code should run. I personally prefer that
pattern (I like things to generally start False, or zero, or empty), but
what values you use will depend on what you're doing.
Cheers,
Cameron Simpson <c...@cskk.id.au>
--
https://mail.python.org/mailman/listinfo/python-list
