0

バックボーンとマリオネットを使用して、サイドバーに部門のリストを表示し、メイン コンテナーに従業員のリストを表示する小さなアプリを開発しています。

ユーザーは部門を選択し、部門ごとに従業員リストをフィルタリングできます。ここにコントローラーとルーターのコードがありますが、コントローラーでアプリを初期化する方法に満足していません。

コントローラー:

Users = require('../collections/users.coffee')
Departments = require('../collections/departments.coffee')
UserListView = require('../views/user-list.coffee')
DepartmentListView = require('../views/department-list.coffee')


module.exports = class Controller extends Marionette.Controller


    ###
    initialize
    ----------
    ###
    initialize: ->
        @users = new Users()
        @userListView = new UserListView(collection: @users)
        @departments = new Departments()
        @departmentListView = new DepartmentListView(collection: @departments)

        @app = new Backbone.Marionette.Application()
        @app.addRegions(
            users: '.js-users-container'
            departments: '.js-departments-container'
        )
        @app.users.show(@userListView)
        @app.departments.show(@departmentListView)

        @app.on("initialize:after", ->
            Backbone.history.start()
        )

        # App will start when the department list is rendered
        @listenTo(@departmentListView, 'composite:collection:rendered', =>
            @app.start()
        )
        # catch filter events
        @listenTo(App.events, 'filter-users', (filters) =>
            @users.fetch(data: filters)
        )
        # load the departments collection, will render department list view
        @departments.fetch(data: active: 1)



    ###
    index
    -----
    ###
    index: ->
        App.events.trigger('filter-users', null)


    ###
    getByDepartment
    ---------------
    ###
    getByDepartment: (id) ->
        App.events.trigger('filter-users', department: id)

ルーター:

Controller = require('./controllers/main.coffee')


module.exports = class Router extends Backbone.Marionette.AppRouter
    appRoutes:
        "": "index"
        "department/:id": "getByDepartment"

    initialize: ->
        @controller = new Controller()

アプリの初期化:

router = new Router()

このコードを改善するにはどうすればよいですか?

よろしくお願いします。

4

1 に答える 1

2

コントローラー内で新しいアプリケーションを作成するべきではありません。単一のアプリケーションを作成し、必要な機能のさまざまな領域を含むモジュールをそれに追加する必要があります。以下に基本的な例を示します。

var app = new Backbone.Marionette.Application();

app.addRegions({
  users: '.js-users-container',
  departments: '.js-departments-container'
});

// more app initialisation from the function in your controller

app.module('i-need-a-good-name', function (Module, App, Backbone, Marionette, $, _) {
  // controller etc in here
});

app.start();
于 2013-07-30T03:29:18.600 に答える