D'oh- sent without finishing the subject line.
Hey guys, new(ish) to Node, first time posting. I made an npm account
shortly after I started for programming, but I haven't written anything
that really called for a reusable npm package until now.
So, the code I'm writing doesn't use any flow-control/promise-chain
libraries like seq or step or q or first or async or queue-async
(although I have used the last one a little). However, it still has a
few callbacks, and it rarely (if ever) cares about what to do if there's
an error (other than panic, of course).
Anyway, to avoid endless "if(err){console.error(err)} else {" typing, I
found myself copy-pasting something like this at the top of all my
scripts to wrap all my callbacks in:
function errorHandlingCb(errCb){
return function onSuccess(successCb){
return function(err) {
if (err) {
errCb(err);
} else {
successCb.apply(this,Array.prototype.slice.call(arguments,1));
}
}
}
(or some variant on the onSuccess function with console.error or "throw
err")
Eventually I started writing something with its functionality spread
across files, got tired of having the same 7-9 lines in multiple files
in one project (let alone across multiple projects, and decided I'd turn
it into a module:
https://github.com/stuartpb/catcher/blob/master/index.js
I reset my npm password (which I'd unsurprisingly forgotten) and
published it:
https://npmjs.org/package/catcher
What do you think? I don't know if I'm replicating some existing thing,
or if I've overlooked something, or made some assumption I shouldn't.
Here are some ideas I have for 0.2:
- "errorize" any non-Error strings / objects (to amerliorate the
problems detailed in
http://www.devthought.com/2011/12/22/a-string-is-not-an-error/) by
replacing line 5 with this:
if (err instanceof Error) ecb(err);
else ecb(new Error(err));
- drop throwNewErr (in light of the above change)
- New functions oriented toward different combinations of wrapping the
method and error handler for calling the method with the success
callback, rather than just wrapping the error handler, then the
success callback, then using that in the method
- Maybe not wrap the provided common error handlers if I'm including
multiple ways of using them, ie. just having something like:
exports.thrower = function(err){throw err}
--
Job Board: http://jobs.nodejs.org/
Posting guidelines:
https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines
You received this message because you are subscribed to the Google
Groups "nodejs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/nodejs?hl=en?hl=en
