Нужно ли все время инстанцировать классы в Backbone.js?

Question

[RaulDuke]

Здравствуйте,

в приведенном ниже коде при определенном состоянии роутера инстанцируется представление без помещения в переменную. В этой вьюхе происходит довольно банальная история: она аппендит в тело документа div, в котором показывает текст новости, после закрытия удаляет себя (this.$el.remove).

Pbl.classes.Router = Backbone.Router.extend({
	routes: {
		'' : 'index',
		'news/add' : 'newsAdd',
		'news/:id' : 'getNews',
	},

getNews: function(id) {
		new Pbl.classes.Views.NewsItem({
			model: new Pbl.classes.Models.NewsItem(id)
		});
	}
});

Подскажите пожалуйста:

1. Не приведет ли такой подход к утечке памяти?
2. Какие существует best practice на случай если если надо просто показать модель во вьюхе, без ее повторного использования.
3. Правильно ли выборку по id осуществлять в моделе (new Pbl.classes.Models.NewsItem(id)) или это должно происходить в роутере/представлении?

Заранее благодарен.

Answer 1

[Константин Китманов]

Нет смысла постоянно создавать и удалять вьюхи, можно просто скрывать стилями. Это, кстати, позволит анимировать показ/скрытие.

Какие существует best practice на случай если если надо просто показать модель во вьюхе, без ее повторного использования.

Я не думаю, что есть принципиальная разница, сколько раз показывать вьюху, 1 раз или 10. Сегодня так, а завтра проект развился, требования поменялись, и стало эдак.

Правильно ли выборку по id осуществлять в моделе

Хм, обычно выбирают из коллекции... А откуда вызывать эту выборку -- вопрос организации. У меня всегда роутеры тупые и просто стреляют событиями, а весь связущий код во вьюхах (которые вообще-то контроллеры, особенно если юзать двухстороннее связывание). Но, наверное, можно и как-то по другому, главное, помнить принцип единственной ответственности.

Comments

[RaulDuke]

Спасибо за Ваши ответы, по последнему пункту уточните пожалуйста. Если продолжать пример с выборкой новостей: есть две вьюхи NewsItem (показывает конкретную новость по id из роутера) и NewsList (показывает 3 последние новости на главной в списке). Теперь представим, что у нас ацки-популярный портал и новостей там за год появилось десятки тысяч и, следовательно, при инициализации страницы было бы слишком расточительно все эти многие тысячи записей хранить в коллекции? И при таких исходных лучше делать точечную выборку для модели NewsItem по id, верно рассуждаю?

[Константин Китманов]

RaulDuke: вы правы в том, что хранить 10k моделей в памяти не надо. Но в том use case, который вы описали, нужна пагинация, а не выборка по ID.

[RaulDuke]

Константин Китманов: кхм не очень понимаю чем тут поможет пагинация? Вы про коллекцию для NewsItem?

[Константин Китманов]

RaulDuke: да

[RaulDuke]

Константин Китманов: гмм, так и не понял что вы имели ввиду. Есть ссылка, например на двугодичную новость, example.com#news/239091, как мне может помочь пагинация?

[Константин Китманов]

RaulDuke: понял вас. Да, в таком случае придется выбирать по ID:)

Однако не обязательно заранее держать его целиком в памяти. Можно создать новый инстанс модели, пустой, с одним только ID и сделать ему fetch -- он подтянет свои данные с сервра.