Return to Answer

typo in variable name

edit approved Jan 19, 2012 at 11:40

I'd prefer writing it like this:

dataSearcher = (dataElementelement, identifier) -> 
 return element if element.identifier is identifier
 for child in element.children
 found = dataSearcher child, identifier
 return found if found

Changes:

guard style instead of else (one less level of indentation)
postfix if (probably a question of style)
is instead of ==
shorter variable names

I'd prefer writing it like this:

dataSearcher = (dataElement, identifier) -> 
 return element if element.identifier is identifier
 for child in element.children
 found = dataSearcher child, identifier
 return found if found

Changes:

guard style instead of else (one less level of indentation)
postfix if (probably a question of style)
is instead of ==
shorter variable names

I'd prefer writing it like this:

dataSearcher = (element, identifier) -> 
 return element if element.identifier is identifier
 for child in element.children
 found = dataSearcher child, identifier
 return found if found

Changes:

guard style instead of else (one less level of indentation)
postfix if (probably a question of style)
is instead of ==
shorter variable names

Post Migrated Here from stackoverflow.com (revisions)

occurred Jan 9, 2012 at 2:07

Source Link

answered Jan 6, 2012 at 19:18

thejh

answered Jan 6, 2012 at 19:18

thejh

I'd prefer writing it like this:

dataSearcher = (dataElement, identifier) -> 
 return element if element.identifier is identifier
 for child in element.children
 found = dataSearcher child, identifier
 return found if found

Changes:

guard style instead of else (one less level of indentation)
postfix if (probably a question of style)
is instead of ==
shorter variable names

lang-js