Можно ли уменьшить код не заварачивая в функцию ?

[pokk]

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

if(IsBookErrorNotExist()){
                    GetJson('GetBoorkError.CGI',HandlerState,function(){
                        GetJson('GetPageVar.CGI',HandlerState,function(){
                            //-----------------------------------------------
                            //-----------------------------------------------
                                setTimeout(function(){
                                    GetJson('GetSerNumber.CGI',HandlerState);
                                }, 20);
                            //-----------------------------------------------
                                clearTimeout(timerId||0);
                                timerId = setTimeout(function tick() {
                                  GetJson('GetPageVar.CGI',HandlerState);
                                  timerId = setTimeout(tick, 1000);
                                }, 100);                    
                            //-----------------------------------------------
                        });                 
                    }
                    );
                }else{
                        GetJson('GetPageVar.CGI',HandlerState,function(){
                            //-----------------------------------------------
                            //-----------------------------------------------
                                setTimeout(function(){
                                    GetJson('GetSerNumber.CGI',HandlerState);
                                }, 20);
                            //-----------------------------------------------
                                clearTimeout(timerId||0);
                                timerId = setTimeout(function tick() {
                                  GetJson('GetPageVar.CGI',HandlerState);
                                  timerId = setTimeout(tick, 1000);
                                }, 100);                    
                            //-----------------------------------------------
                        });                     
                }

[Aetae]

pokk, проблема старая, известная как callback hell. Прочитайте этот раздел учебника: https://learn.javascript.ru/async , тут всё подробно расписано по этому вопросу. Скипать частями - не рекомендую, потому что одно из другого там вытекает и могут остаться проблемы с пониманием.

[pokk]

О благодарю много нового узнал, правильный подход это использование
asynс функций с await вместо цепочки промиксов?

[SuperZen]

О благодарю много нового узнал, правильный подход это использование asynс функций с await вместо цепочки промиксов?

нет, это делает код проще

clearTimeout(timerId || 0);
timerId = setTimeout(function tick() {
  GetJson('GetPageVar.CGI', HandlerState);
  timerId = setTimeout(tick, 1000);
}, 100);

тут написано что-то странное...

[ksa]

тут написано что-то странное...

Что именно там странно?

var timerId;
clearTimeout(timerId || 0);
timerId = setTimeout(function tick() {
	alert('Ok');
	timerId = setTimeout(tick, 1000);
}, 100);

[SuperZen]

|| 0, с таким подходом можно удалить не тот таймаут, который "в уме" подразумевается...

[ksa]

с таким подходом можно удалить не тот таймаут, который "в уме" подразумевается...

Это вряд ли...
В браузерах это положительное число больше 0.
Т.е. таймера с 0 не будет.

Просто чел решил сэкономить на

if (timerId) {};

Ниндзястайл и не более того...

[SuperZen]

ksa, действительно, больше нуля ... но, все равно, там должен быть setInterval вместо setTimeout...

[Aetae]

О благодарю много нового узнал, правильный подход это использование
asynс функций с await вместо цепочки промиксов?

В принципе то да, но asynс/await пока не везде доступны, а потому если вы ещё не используете транспайлер вам придётся изучить ещё и данный вопрос. Если же обойтись без транспайлера, то цепочки промисов сами по себе достаточно наглядны и удобны.

if (timerId) {};

А в итоге дописал лишних 4 символа.) clearTimeout вызывается с любой чушью(приводимой к number без exception) в качестве аргумента(или вовсе без оного), по историческим причинам.

... но, все равно, там должен быть setInterval вместо setTimeout...

Неа, setInterval - он только для очень узких случаев.
И хотя данный конкретный код технически можно было бы заменить на setInterval, но только потому что сам код ошибочен.
Тут, как и в случае с setInterval: если GetJson займёт более 100мс, то одновременные вызовы будут накапливаться пока не станет совсем грустно. Правильный подход вызывать следующий setTimeout только после того как GetJson гарантировано отработал(в данном случае в коллбэке).

[SuperZen]

Aetae, +