Как определить событие вызова функции?

Question

[Анатолий Сидоров]

Добрый день.
Есть довольно большой класс на js. И требуется возможность при переходе в определенный режим отрубить 80% функционала, а при попытке обращения к функционалу ругаться. Как можно определить при вызове функции что функция вызвана и сделать какие-либо проверки и прерывание функции, при этом не дописывая в начало каждой функции условие с выходом?
Просто при наличии ~100 методов класса редактировать каждый муторно, а если появится еще один режим работы - это вообще повеситься:(

UPD. Немного перефразирую. Есть примерно вот что:

vat test = function() {
	this.block1 = $('block1');
	this.block2 = $('block2');
	
	//переключатель
	//при true все вызовы должны быть перенаправлены на метод с кастомной ошибкой
	this.enable = false; 
	
	//обработчики событий
	this.events = {
		block1: {
			event1: function(e){},
			event2: function(e){}
		},
		block2 ...
	}
	
	this.bindEvents = function() {
		this.block1.on('click', this.events.block1.event1)
		this.block2.on('click', this.events.block1.event2)
		...
		//еще куча привязок
	}
	
	//общие методы для обработки данных, 
	//которые могу вызываться как из разных внутренних событий
	//так и извне
	this.someMethod1 = function() {}
	this.someMethod2 = function() {}
	...
	
	this.bindEvents();
}

Как можно видоизменить структуру, чтобы (new test()).enable = true требуемые события и методы(не все) блокировались , при этом не дописывать условия выхода в каждом методе.
Честно говоря не представляю как это сделать абстрагированием:(

Answer 1

[Денис Инешин]

Попахивает неправильным проектированием.

Нужно сделать ровно наоборот. Базовый класс должен обладать минимумом методов, которые используют все, а уж дальше, в зависимости от сценария, наследники расширяют его функционал своими методами.

Answer 2

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

Вам нужно AOP. А еще в ES7 есть декораторы.

Comments

[Иван Соломенников]

ES6

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

capable: ES6 уже утвержден и в нем нет декораторов.

[Иван Соломенников]

Константин Китманов: >>>ES6 уже утвержден
Ну как бэ, это давно уже многим известно.
>>>в нем нет декораторов
Правду говоришь.

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

capable: К чему был этот ваш комментарий? Или так просто, модное словечко ввернуть?

[Иван Соломенников]

Константин Китманов: >>>К чему был этот ваш комментарий?
К тому что браузерами многое из спецификации ES6 не реализовано.
>>>Или так просто, модное словечко ввернуть?
OMG.

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

capable: понятно. Все-таки модное словечко ввернуть.

[Иван Соломенников]

Константин Китманов: П0циент вы с какой больницы вещаете? Танки видели?

Answer 3

[M-ka]

А еще почитайте это и это.
Но в любом случаи переделывать нужно

Answer 4

[Dmitriy Mozgovoy]

Месье знает толк...
Тут явно что то не то, но все же можно , но не нужно, что то наподобие этого:

var test=function(){

           this.enable = false;

           this.protectedFunc1=function(){
                console.warn('Protected func1 called!');
           }

           this.protectMethod=function(fn){
               var self=this,
                   fn;

               if(typeof fn!=='function'){
                   throw TypeError('Given thing is not a function!');
               }


               return function(){

                   console.log(self.enable);

                   if(!self.enable){
                       throw Error('Access to protected method not allowed');
                   }

                   return fn.apply(self,arguments);
               };
           }

           this.protectedFunc1=this.protectMethod(this.protectedFunc1);

           this.protectedFunc2=this.protectMethod(function(args){
               console.warn('Protected func1 called!');
           })


       }


       var obj=new test;

       obj.enable=true;
       obj.protectedFunc1(); //Protected func1 called!

       obj.enable=false;
       obj.protectedFunc1();// Error: Access to protected method not allowed

Comments

[Анатолий Сидоров]

Ну если это называется извращением, как тогда должна выглядеть архитектура приложения на js рассчитанного на работу в нескольких режимах?

[Dmitriy Mozgovoy]

Тут надо еще понять что такое у Вас "в нескольких режимах"=)

[Анатолий Сидоров]

Допустим есть таблица с данными в ней и кнопки управления данными(например сортировки столбцов).
И есть кнопка "редактировать" включающая режим редактирования,в котором у большинства остальных кнопок события должны перенаправляться на заглушку аля перекючитесь в режим просмотра данных.
При этом нужно учитывать, что в модуле >100 таких событий.
Как должна выглядеть такая архитектура?
Костылить не хочется, если бы был нормальный пример архитектуры то лучше бы переписал полностью.

[Dmitriy Mozgovoy]

Анатолий Сидоров: А зачем события от ненужных кнопок "перенаправлять" да еще и на заглушку? Если речь идет чтоб пользователь не мог активировать недоступное действие, то "кнопки" должны при малом количестве делаться disabled и соответственно не генерировать событий от пользователя, либо убираться, что лучше вообще, с экрана если их достаточно много.

Если уж вообще огромный функционал у того класса, то надо его разбить на меньшие. Поступить как MS Office - функционал разнести по панелям визуально, в коде разнести по разных классах. Т.е как класс "таблицы" скажем GridView назовем его, в зависимости от своего режима создает и показывает класс какойнить GridViewEditor. Поменялся режим- ненужные панели вместе с классом уничтожить, нужные создать и/или показать. Но это излишнее усложняет реализацию, при небольшом и не расширяемом функционале.