Classical JS pitfall

base_view.js.coffee

class Amplifr.BaseView extends Backbone.View

  constructor: ->
    @createSelectors()
    super

  els: {}

  createSelectors: ->
    for name, selector of @els
      cachedMatches = selector.match(/^cached (.+)/)
      @['$' + name] = =>
        if cachedMatches
          @['$_' + name] ||= @$(cachedMatches[1])
        else
          @$(selector)

Code contain critical bug. What kind is bug?

Coffeescript.

@private-face nice try, but wrong.

Ставлю на то, что при вызове @"$_'" + name этот самый name внутри функции будет уже не тот, что надо. Кому-то нужно еще одно замыкание и я солидарен с @private-face, использование coffeescript само по себе критичный баг ;)

А что хранится в @els? Можно примерчик?

@itrelease, просто javascript вообще не нужен. В топку его!

@whitequark программирование говно!

@itrelease ага, cachedMatches будет из последнего вызова.

  createSelectors: ->
    for name, selector of @els
      do (name, selector) =>
        cachedMatches = selector.match(/^cached (.+)/)
        @['$' + name] = =>
          if cachedMatches
            @['$_' + name] ||= @$(cachedMatches[1])
          else
            @$(selector)

@milushov примерно вот так:

  els:
    new:   '@new'
    link:  '@link'
    tabs:  '@tab'
    input: 'cached @input'

@new, @link это алиас на [role='new'].

@whitequark @itrelease и компьютеры отстой.

for is probably the worst part of Coffee

@inossidabile maybe worst part of JS?