Комментарии (10)

1. 

   kahi4

   21.09.2017 09:30

   #10429206

   -1

   Идея несколько необычна, но вполне интересна — диспетчер задач для вашего кода в браузере. Даже с учётом современной ситуации js-пей полных аналогов не найдено.

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

   
   

   1. 

      Electrohedgehog Автор

      21.09.2017 13:39

      #10429860

      +2

      Вы, простите, статью читали? Специально кидаете ссылки на модули Node.js, утверждая что это одно и то же? Статью обязательно следует прочесть перед комментированием.
      Из ваших ссылок сколько-нибудь адекватной может считаться только первая, но это 1) библиотека поверх библиотеки 2) не имеющая системы управления заданиями 3)без логирования 4)без графического интерфейса.
      Я писал инструмент для прозрачного выполнения кода с возможностью управления им. Вы даёте ссылки на планировщики заданий. Разница весьма существенна.

      
      

      1. 

         mayorovp

         21.09.2017 13:45

         #10429868

         Почему вы считаете третью ссылку модулем node.js, который нельзя запустить в браузере?

         
         

         1. 

            Electrohedgehog Автор

            21.09.2017 13:55

            #10429890

            +1

            Возможно я ошибся, но беглый просмотр не дал никаких оснований считать, что он запустится в браузере. Есть причины думать иначе?
            

            
            

            1. 

               mayorovp

               21.09.2017 13:57

               #10429898

               Конечно же такие основания есть. Он не использует ни одного nodejs-специфичного модуля.

               
               

               1. 

                  Electrohedgehog Автор

                  21.09.2017 14:05

                  #10429922

                  +1

                  Да, почитал код, был неправ. Тем не менее, это опять же, другой инструмент.

      
      

      1. 

         kahi4

         21.09.2017 18:09

         #10430596

         +3

         В каком месте у вас в библиотеке управление задачами? У вас такой же ровно крон, позволяющий исключать из расписания задачи, выставлять обратно, запускать мгновенно. По факту, что я кинул, позволяют плюс-минус тоже самое, только с разным синтаксисом.
         Беглый осмотр говорит, что это позволяет делать тоже самое, что и ваше

         
         

         Нет логирования — косяк библиотеки, хотя скорее всего он есть.
         Графический интерфейс вообще странное преимущество. Зачем он мне в проде? В лучшем случае, это должно быть подключаемым модулем.

         
         

         До тех пор пока вы не сможете останавливать выполняемые задачи прямо сейчас и переключать контексты — у вас будет такой же планировщик как и любой другой, только с гуишкой, логами и другим интерфейсом, поэтому я, даже три раза прочитав статью, не понимаю разницу.

         
         

         И я еще могу понять зачем нужен планировщик задач на js, но если вы хотите сделать удобную обертку над setTimeout и setInterval, которая позволяет вместо кода

         
         

         const timer = setTimeout(() => {}, 1000);
         
         clearTimeout();

         
         

         писать

         
         

         const task = new Task(() => {}, 1000);
         task.stop(1000)

         
         

         • я не понимаю, зачем это делать синглтоном, кроме как порождать лишние утечки памяти, забыв где-то что-то закрыть. Один черт вам дальше в коде нужно таскать PID или название таски, чтобы знать что останавливать. И опять же раз, два, много

         
         

         Хорошо, допустим, вы хотите написать некий дашборд, в котором каждый виджет живет своей жизнью и является приложением самим в себе, и вы хотите централизованно иметь возможность останавливать в них "жизнь" и восстанавливать и все такое, тогда согласен — ваша реализация может выступать в роли "уникальной", хотя конечно же не очень

         
         

         Ах да, в ней нет GUI, беру свои слова обратно.

         
         

         Немного конструктивного по коду

           //Watch keyup and waiting for Ctr Ctrl Ctrl to open control panel
           window.addEventListener("keyup", function(e){
             if (e.key == "Control")
               self.controlCount++;
             else if (e.key == "Escape")
               self.closeControlPanel();
             if (self.controlCount>=3){
               self.controlCount = 0;
               if(!self.cpOpened)
                 self.openControlPanel(true);
               else
                 self.closeControlPanel(true);
             }
           });
         
           this.verboseLogs = verboseLogs;
           this.verboseTime = verboseTime;
           this.writeLogs = writeLogs;
           this.runTask( () => {
             console.log("Example function executed");
           },
           42000000,
           false,
           "example",
           "This is a sample task. It will do nothing.");

         
         

         Че это оно безусловно шлет что-то в консоль, да еще и само навешивается на keypress? Дайте хотя бы метод registerGUI или типа того.

         
         

             while (this.tasks[name])
               name = "task_"+Math.random().toString(32).substr(2);

         
         

         Зачем? Введите статическое свойство у chronoshift как name, увеличивайте на 1 каждый раз при генерировании имени и не будет конфликта.
         И опять же, при попытке создать таску с уже имеющимся именем (если пользователь задал его явно) лучше кидать исключение, скорее всего это ошибка, чем автоматом присваивать случайное.

         
         

         setInterval зло. Следует всегда избегать его по-возможности хотя бы в пользу setTimeout. Да и неплохо было бы добавить возможность "приостанавливать" таску в автоматическом режиме при переходе в другую вкладку (делается через requestAnimationFrame).

         
         

         Ну и по мелочи: airbnb codestyle (хотя ваше право, но он вроде как близок к победе). Вы используете let и const, но не используете class. В итоге ни туда, ни сюда.

         
         

           this.redrawReqest = function(){
             let e = new Event("csredrawrequired"); // зачем это? А почему просто вызвать redrawReqest нельзя?
             // через свой же диспетчер задач, создав первую задачу как system.
             window.dispatchEvent(e);
           }
         
           window.addEventListener("csredrawrequired",function(){
             if (!self.cpOpened)
               return;
             let vl = this.verboseLogs,
               vt = this.verboseTime,
               wl = this.writeLogs;
             self.setLogging(0, 0, 0);
             self.closeControlPanel();
             self.openControlPanel();
             self.setLogging(vl, vt, wl);
           }, false);

         
         

         Ну это жесть конечно. И то, что перерисовывается много ненужного, и то, что стили инлайново внутри функции, а не вынесены в переменную до кода, и куча куча куча всего.

         
         

         И опять же, umd, он не просто так существует.
         Да и научите функцию принимать конкретный элемент вместо body, чтобы я сам решал где оно будет отрисовываться.

         
         

         1. 

            Electrohedgehog Автор

            22.09.2017 06:15

            #10431292

            Большое спасибо вам за конструктивную критику.
            По первому пункту — просто опечатка, не доглядел. Должен вызываться this.log.
            По второму — мне не понравилась куча однотипных имён, глазу проще зацепиться за разницу в несколько символов чем в один.
            Остальное буду улучшать.
            Я правильно понимаю, что основная ваша претензия заключается в том, что аналогичные библиотеки уже существуют а я просто написал велосипед с квадратными колёсами?
            

            
            

            1. 

               kahi4

               22.09.2017 10:15

               #10431546

               Я правильно понимаю, что основная ваша претензия заключается в том, что аналогичные библиотеки уже существуют а я просто написал велосипед с квадратными колёсами?

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

               
               

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

               
               

               P.S. Вот вам еще один feature-request: Вообще было бы неплохо добавить еще два метода: suspend и resume, которые бы запоминали через сколько должна выполнится функция и перевыставляли таймер с этим значением.

               
               

               И, если хочется развивать эту библиотеку в реально полезную для широкого круга задач, разбейте ее на модули. Дайте возможность создавать задачи без контейнера, управлять ими через дескриптор самой задачи, а не через контейнер, контейнер сделайте модулем, заодно это упростит реализацию более сложных задач. Вынесите рендер GUI в отдельный модуль, в идеале в отдельный репозиторий (и npm пакет соответственно). Ах да, это то тоже неплохо было бы поместить в npm, уже давно никто напрямую файлики не качает.

               
               

               Ну либо двигайтесь в обратную сторону от обертки над timeout в сторону более реального планировщика ОС, заставляйте передавать не просто функцию, а объект с методами run, exit, suspend, не знаю. Дайте возможность устанавливать приоритеты "потокам", в общем — тут тоже есть где разыграться. Если сильно упороться, можно вот что сделать: каждый "процесс" должен дергать функцию типа dispatcher.processTasks, который позволит в это время переключить на другую функцию, если эта выполнялась уже слишком долго, и все такое. Это будет даже здорово и интересно, но опять же придется решать проблему с теми "потоками", которые выполняются слишком долго, при этом не дергая эту функцию. (Подсказка: webWorkers можно убивать принудительно, но сами webWorkers сделают всю вашу разработку бессмысленной, покуда их будет контроллировать "взрослый" диспетчер задач. Пока же можно просто кидать предупреждение, что "вот такой поток жрет слишком много ресурсов".)

               
               

               Хм, кажется, я только что придумал тестовое задание :)

1. 

   Alex_1985

   21.09.2017 13:22

   #10429804

   Ну не знаю, но лично я ссылочку сохранил, вдруг пригодится…