table_with_headers = $(this.dom.table).find("thead tr th").map(-> $(this).text() ).get().join("\t") Can one write it nicer? I especially don't like the inner function in map syntax enforced by CoffeeScript and jQuery.
\$\begingroup\$ \$\endgroup\$
Add a comment |
2 Answers
\$\begingroup\$ \$\endgroup\$
CoffeeScript is not my forte, however I quite like this approach:
elements = $(this.dom.table).find("thead tr th") table_with_headers2 = ($(element).text() for element in elements ).join('\t') This way you call out which elements you are going to process, and then use a list comprehension. In a style that is closer to JS than true CoffeeScript.
\$\begingroup\$ \$\endgroup\$
4 You can use explicit find like so
$ 'thead tr th', @dom.table and put some indentations instead of braces
table_with_headers = $ "thead tr th", @dom.table .map -> @.text() .join '\t' Also try to use list comprehesion
table_with_headers = [ $(th).text() for th in $ "thead tr th", @dom.table ].join '\t' 
- \$\begingroup\$ sorry, but proposed solution does not work. \$\endgroup\$andilabs– andilabs2014-10-01 12:07:26 +00:00Commented Oct 1, 2014 at 12:07
- \$\begingroup\$ @andi check updates \$\endgroup\$outoftime– outoftime2014-10-01 12:21:50 +00:00Commented Oct 1, 2014 at 12:21
- \$\begingroup\$ May I suggest renaming
elemtoth? \$\endgroup\$200_success– 200_success2014-10-01 17:54:17 +00:00Commented Oct 1, 2014 at 17:54 - \$\begingroup\$ @200_success, done. \$\endgroup\$outoftime– outoftime2014-10-03 13:39:47 +00:00Commented Oct 3, 2014 at 13:39