Suppose we were attempting to write parseLogInteractively by just returning errors (as in idiomatic Go).
If parseLog just returned an error upon failure we would have to same problem as exceptions do, as explained in section 1. So we have to make parseLog return two lists: one of entries, and one of errors. We would then have to search through the error list fixing and reparsing each error. In Python this would be something like,
def parseLogInteractively(file):
return parseLinesInteractively(file.readLines())
def parseLinesInteractively(lines):
entries, errors = parseLines(lines)
if errors:
entries += parseLinesInteractively(askToFixEntry(e.text) for e in errors)
return entries
Again we lose the ability to abstract the function 'parseLog'. And I don't know about you, but this looks far worse to me than "resume FixEntry" in terms of exposing implementation details.
In python, I usually deal with this with generators. It's not as elegant as conditions, but it works well enough. I've never tried to permit custom restart-like behavior, but now that generators are coroutines, it should be doable.
Common lisp could really use some better coroutine support (and some way to specify generic sequences). Sure, you can do it with macros, but it gets horrible fast. Have you looked at SERIES, for example?
> I've never tried to permit custom restart-like behavior, but now that generators are coroutines, it should be doable.
It would be extremely awkward, you'd need a big dispatch table handling the generator's result and send()ing restarts up to the generator, and of course you lose condition's ability to work through stacks of blissfully unaware code, and to have default behaviors (in Smalltalk, the default behavior — in an interactive image — is to pop up a dialog asking the user if he wants to unwind the stack [as in an exception], to open a debugger at condition point or to try resuming the condition [under the assumption that the user fixed some incorrect method, or checked his network access])
Thanks for the pointer; I've been learning some Smalltalk lately, but I haven't yet looked into the condition system.
And, yes, I'm not saying that manually implementing pseudo-restarts is something I'd ever want to do, but some vague skeleton of the possibility is there.
If you want the restart, you need to take advantage of the PEP-342 two-way yield(), as in:
...
except SomethingBad:
yield None
restartRequest = yield None
if restartRequest:
do whatever is necessary to go on
The problem with the above is that the double yield() ugly. You cannot do this with single yield() because the caller would have to know in advance that next() will return None.
Generic sequences are coming to common lisp. I'm working on a simple implementation inspired by clojure which I'll be posting to /r/lisp in the coming week.
Do not separate errors and entries. Instead use something like Haskell's Maybe, but instead of an empty Nothing-object you return something which contains error information:
def parseLogInteractively(file):
entries = list()
for line in file:
entry = parseEntry(line)
if entry.failed:
entry = askToFixEntry(entry)
entries.append(entry)
return entries
A design question arises: Do you need to fix parse errors before you can proceed with next entry? If not, the error handling can be done by whomever calls parseLog.
def parseLog(file):
list = []
for line in file:
list.append(parseEntry(line))
return list
Or better (more pythonic imho):
def parseLog(file):
for line in file:
yield parseEntry(line)
The second version is lazy (if file reading is lazy), so you can even abort parsing or fix the state on an error. Here are your different versions all reusing parseLog from above and equivalent line count to condition handling:
def parseLogLoudly(file):
for entry in parseLog(file):
if entry.failed:
print "Warning:", entry.message
else
yield entry
def parseLogSilently(file):
for entry in parseLog(file):
if not entry.failed:
yield entry
def parseLogInteractively(file):
for entry in parseLog(file):
if entry.failed:
yield askToFixEntry(entry)
else
yield entry
(Haskell's Either has "Left a" or "Right b". I want something like "Just value" or "Error msg", but you are right Either is nearer than Maybe)
Haskell's Either type is conventionally used exactly like that--pretend that Right val is Just val and Left err is Error err.
The reason it doesn't have those names is because it is more general; you can also use Either to model things like early termination. The generic names just make it clearer that it isn't exclusively for error handling unlike exceptions. In other languages using exceptions for more generic control flow is considered poor style, but in Haskell using Either for non-error cases is completely reasonable.
Because that only works for very flat stacks, otherwise your callback will start infecting more and more method less and less related to what's actually causing the error, and then you'll have to move to a mapping of callbacks as different parts of the stack will want their own error callback.
Do you really want to handle error conditions in a resumey way far further up the stack? I'd expect most cases are either handle the error close to the cause, or a single application-wide handler (which could be provided with dependency injection).
The condition handling approach just seems too magic to me. It seems to mean the "inner" code calls another method defined somewhere else - but that "somewhere else" is defined by the call stack, something that's not really very visible in the code.
> Do you really want to handle error conditions in a resumey way far further up the stack?
"Way far" is relative and depends on a number of factor. And I don't consider half a dozen stack frames "way far", it's very easy to attain when using a coding style emphasizing small short methods which call one another.
You also have no bloody idea how the library works internally.
> I'd expect most cases are either handle the error close to the cause
Again, "close" and "far" is relative. I want to handle the error at the closest point to its triggering, but I'm also limited by the data available to know how to handle it. One pushes the handler down the stack, the other up.
> The condition handling approach just seems too magic to me.
There's nothing magic to it.
> It seems to mean the "inner" code calls another method defined somewhere else - but that "somewhere else" is defined by the call stack
Welcome to dynamic scoping, there are cases where it's useful.
> something that's not really very visible in the code.
How is "I'm signaling a condition" not very visible in the code? It says right there that it signals a condition.
If parseLog just returned an error upon failure we would have to same problem as exceptions do, as explained in section 1. So we have to make parseLog return two lists: one of entries, and one of errors. We would then have to search through the error list fixing and reparsing each error. In Python this would be something like,
Again we lose the ability to abstract the function 'parseLog'. And I don't know about you, but this looks far worse to me than "resume FixEntry" in terms of exposing implementation details.