On Tue, Jan 28, 2014 at 8:39 PM, Alex Kocharin <[email protected]> wrote:
> > It's the same deal as with hungarian notation. Terrible idea. > *applause* Rick > > Your function name should already describe what this variable contains. > There is no need to duplicate this information with underscores. Name it > "on_save" or "save_cb" instead. Even "save" will do fine in small functions. > > This code will be read by humans, not computers. So there is no need to > invent a "notation" just for the sake of it. Computers don't need it, and > humans should get this info from the name/context anyway. > > > 26.01.2014, 18:24, "Filipe Deschamps" <[email protected]>: > > Guys, what do you think to prefix all callback functions with double > underscores? (or any other pattern that makes more sense to identify a > callback) > > For example: > > model.save(data, *__save*); > > function *__save*(err, return) { > ... > } > > I'm node/javascript newbie, but using a prefix for callback names is > making a huge different to scan the code (including using find features). > > Are you guys using some name convention approach for callbacks? Or is this > just a newbie problem? > > Best regards, > > Filipe Deschamps > > > -- > -- > 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 > > --- > You received this message because you are subscribed to the Google Groups > "nodejs" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/groups/opt_out. > > -- > -- > 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 > > --- > You received this message because you are subscribed to the Google Groups > "nodejs" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/groups/opt_out. > -- -- 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 --- You received this message because you are subscribed to the Google Groups "nodejs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
