How to have promise for the multiple functions in angularjs

Question

Code explanation:

If internet is available it will sync the unsynced data from the different tables based on different methods.But, I would like to know how to add promise for these different functions.The logic which I used in making all the functions to start simultaneously but, I would like to start function2 based on the success of function1 and repeat the same process can anyone tell me how to do it.

function syncApp () {
  $log.log('offlineOnlineSync got called');

  Contact.syncApp().then(function (resp) {
    $log.log('Contact sync got called', resp);

    WorkerGroup.syncApp().then(function (resp) {
      $log.log('WorkerGroup sync got called', resp);

      Category.syncApp().then(function (resp) {
        $log.log('Category sync got called', resp);

        Vehicle.syncApp().then(function (resp) {
          $log.log('Vehicle sync got called', resp);

          Daybook.syncApp().then(function (resp) {
            $log.log('Daybook sync got called', resp);
          }, CommonService.errorHandler);

        }, CommonService.errorHandler);

      }, CommonService.errorHandler);

    }, CommonService.errorHandler);

  }, CommonService.errorHandler);
}

In the first method the process which takes place is this but before completing the above process the second method gets called.

    prom = DB.update('contacts', servResp, 'id', key)
      .then(function () {
        if (servResp.type === 'Worker') {
            WorkerGroup.checkGroupForTempIds(key, servResp.id)
             .then(function (resp) {
                  $log.log('Response in contact service', resp);
            }, function (err) {
                  $log.log('err: ', err);
            });
        } // fix me // not needed
    }, function (err) {
          $log.log('err: ', err);
    });

    $log.log('serverresponseid', servResp.id);
    $log.log('key', key);
    var daybook_updatequery = 'UPDATE daybook SET contact_id = ? WHERE contact_id = ?';

    $cordovaSQLite.execute(Database.db, daybook_updatequery, [servResp.id, key])
      .then(function (resp) {
        $log.log('response', resp);
        defer.resolve('success');
    }, function (err) {
        q.reject(err);
        $log.log(err);
    });

    proms.push(prom);
});
$q.all(proms).then(function () {
    defer.resolve('success');
});

What is the problem with your example code besides being ugly? Does it work? — Tamas Hegedus
– Tamas Hegedus, Commented Aug 2, 2017 at 13:13
It is difficult to tell what is wrong with the getAllProms function because parts of it are missing. On first observation, it fails to use return statements in the .then handler functions and it uses broken $q.defer anti-patterns. — georgeawg
– georgeawg, Commented Aug 2, 2017 at 15:30

Tamas Hegedus · Accepted Answer · 2017-08-03 06:50:48Z

2

You can chain the promises, so you only need one error handler:

function syncApp () {
  $log.log('offlineOnlineSync got called');
  return Contact.syncApp().then(function (resp) {
    $log.log('Contact sync got called', resp);
    return WorkerGroup.syncApp();
  }).then(function (resp) {
    $log.log('WorkerGroup sync got called', resp);
    return Category.syncApp();
  }).then(function (resp) {
    $log.log('Category sync got called', resp);
    return Vehicle.syncApp();
  }).then(function (resp) {
    $log.log('Vehicle sync got called', resp);
    return Daybook.syncApp();
  }).then(function (resp) {
    $log.log('Daybook sync got called', resp);
  }).catch(function(err) {
    CommonService.errorHandler(err);
  });
}

edited Aug 3, 2017 at 6:50

answered Aug 2, 2017 at 13:16

Tamas Hegedus

30.1k12 gold badges67 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Nidhin Kumar Over a year ago

i have tried with the above code but before completing the task of first method the second one gets started

georgeawg · Accepted Answer · 2017-08-02 16:46:06Z

To have parent promises wait for completion of chained promises, it is important to return the chained promise to the parent .then handler function:

function getProm(servResp, key) {
    ͟r͟e͟t͟u͟r͟n͟ DB.update('contacts', servResp, 'id', key)
      .then(function () {
        if (servResp.type === 'Worker') {
            ͟r͟e͟t͟u͟r͟n͟ WorkerGroup.checkGroupForTempIds(key, servResp.id)
             .then(function (resp) {
                  $log.log('Response in contact service', resp);
                  ͟r͟e͟t͟u͟r͟n͟  resp;
            }, function (err) {
                  $log.log('err: ', err);
                  //IMPORTANT
                  throw err;
            });
        } else {
            ͟r͟e͟t͟u͟r͟n͟ "something";
        };
    }, function (err) {
          $log.log('err: ', err);
          //IMPORTANT
          throw err;
    });
}

Also to avoid converting rejected promises to fulfilled promises, it is important to use a throw statement in the rejection handler function.

When a .then method handler function omits a return statement, the method returns a new promise that resolves undefined and that promise does not wait for any asychronous operations started in that handler function.

For more information, see You're Missing the Point of Promises

Collectives™ on Stack Overflow

How to have promise for the multiple functions in angularjs

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related