How can I improve my function?

Question

JavaScript

Hello! I have different drop-down blocks in my layout when I click on a button.
So I hang the block opening event

$('button').on('click', function(){
  $(this).toggleClass('act');
  $('div').toggleClass('act');
});

And so, I hide the block, when clicking outside it:

$(document).on('click', function(event){
    var target = event.target;

    var closeByDocument = function(docTarget, butTrigger, triggerElem, removeClass, butTriggerBullClass){
      // docTarget - Элемент, при клике по документу
      // butTrigger - Кнопка открытия элемента
      // triggerElem - Открытие блока, при нажатии на кнопку
      // removeClass - Имя удаляемого класса с блока и кнопки
      // butTriggerBullClass - Удалять ли класс у кнопки 

      if(docTarget !== $(triggerElem)[0] && !$(triggerElem).has(docTarget).length && docTarget !== $(butTrigger)[0] && !$(butTrigger).has(docTarget).length){
        $(triggerElem).removeClass(removeClass);

        if(butTriggerBullClass){
          $(butTrigger).removeClass(removeClass);
        }
      }
    };

    // Закроем блок
    closeByDocument(target, 'button', 'div', 'act', true);
  });

Everything works as it should. But there was a desire to somehow rewrite or improve the code, because after looking after, I realized that somehow it looks like shit code! Help with advice or, if possible, show your ideas directly on my example. I would be very grateful

Answer 1

do caching and do not do unnecessary operations! why look for elements of the same type on each click, or why redefine a function that does not change the logic on each click? it can be done once and then called.

var divs = $('div');
$('button').on('click', function(){
  $(this).toggleClass('act');
   divs.toggleClass('act');
});

var closeByDocument = function(docTarget, butTrigger, triggerElem, removeClass, butTriggerBullClass){
      // docTarget - Элемент, при клике по документу
      // butTrigger - Кнопка открытия элемента
      // triggerElem - Открытие блока, при нажатии на кнопку
      // removeClass - Имя удаляемого класса с блока и кнопки
      // butTriggerBullClass - Удалять ли класс у кнопки 

      if(docTarget !== $(triggerElem)[0] && !$(triggerElem).has(docTarget).length && docTarget !== $(butTrigger)[0] && !$(butTrigger).has(docTarget).length){
        $(triggerElem).removeClass(removeClass);

        if(butTriggerBullClass){
          $(butTrigger).removeClass(removeClass);
        }
      }
    };
$(document).on('click', function(event){
    var target = event.target;
    // Закроем блок
    closeByDocument(target, 'button', 'div', 'act', true);
  });

Answer 2

The simplest example of a modal window: jsfiddle.net/ph8zd19j/1
Instead of creating 100500 different popup blocks, you can copy them inside one #modal
I can sketch an example.