Итерация с обратными вызовами / анонимными функциями - PullRequest
Новая
       7

Итерация с обратными вызовами / анонимными функциями

0 голосов
/ 29 апреля 2018

Я новичок в Node.JS и продвинутом Javascript в целом, но я пытаюсь создать приложение менеджера расписаний самостоятельно и столкнулся с проблемой (я опишу ее позже) при попытке выполнить следующий код : 

router.get('/', function (req, res) {
var day = new Date(req.query.day);
Location.getLocations(function (err, locations) {
    if (locations.length > 0) {
        var i;
        for (i = 0; i < locations.length; i++) {
            var location = locations[i];
            Appointment.getAppointments(day, location, function (err, appointments) {
                if (err) throw err;
                if (appointments.length == 0) {
                    // CREATE APPOINTMENTS
                    for (var j = location.available_time_start; j <= location.available_time_end; j += location.appointment_duration) {
                        var newAppointment = new Appointment();

                        newAppointment.start_date = new Date(day.getFullYear(), day.getMonth() + 1, day.getDate(), j);
                        newAppointment.appointment_duration = location.appointment_duration;
                        newAppointment.location = location.id;
                        newAppointment.booked = false;
                        newAppointment.locked = false;

                        Appointment.createAppointment(newAppointment, function (err, appointment) {
                            if (err) throw err;
                            console.log(appointment.location + ' - ' + appointment.start_date);
                        });
                    }
                }
            });
        }
    } else {
        // THERE ARE NO LOCATIONS
    }

    res.render('appointments', { locations: locations });
});

Проблема:
Когда я пытаюсь выполнить итерацию объекта locations, а затем выполнить функцию getAppointments, в этот момент код не выполняется. Позже, когда он выполняется, объект location всегда одинаков (итерация не работает), что приводит к неожиданному результату (все встречи с одинаковым / последним местоположением).

Я пытался использовать IIFE (выражение для немедленного вызова функции), чтобы выполнить код мгновенно, но когда я сделал это, я не смог получить объект обратного вызова appointments, и моя логика тоже сломалась.

Заранее спасибо!

Ответы [ 2 ]

0 голосов
/ 29 апреля 2018

Ваш код, кажется, сохраняет встречи, но ничего не делает с сохраненными встречами (вы меняете местоположения?).

Когда сохранение встречи идет не так, запросчик об этом не знает, потому что createAppointment является асинхронным и к тому времени обратный вызов res.render('appointments', { locations: locations }); уже выполнен.

Вы можете попробовать преобразовать свои функции обратного вызова в обещания: 

const asPromise = (fn,...args) =>
  new Promise(
    (resolve,reject)=>
      fn.apply(undefined,
        args.concat(//assuming one value to resole
          (err,result)=>(err)?reject(err):resolve(result)
        )
      )
  );
const savedAppointmentsForLocation = (day,location,appointments) => {
  const savedAppointments = [];
  if (appointments.length == 0) {
    // CREATE APPOINTMENTS
    for (var j = location.available_time_start; j <= location.available_time_end; j += location.appointment_duration) {
      var newAppointment = new Appointment();
      newAppointment.start_date = new Date(day.getFullYear(), day.getMonth() + 1, day.getDate(), j);
      newAppointment.appointment_duration = location.appointment_duration;
      newAppointment.location = location.id;
      newAppointment.booked = false;
      newAppointment.locked = false;
      savedAppointments.push(
        asPromise(
          Appointment.createAppointment.bind(Appointment),
          newAppointment
        )
      );
    }
  }
  //you are not doing anything with the result of the saved appointment
  //  I'll save it as promise to see if something went wrong to indicate
  //  to the requestor of the api that something went wrong
  return Promise.all(savedAppointments);
}

router.get('/', function (req, res) {
  var day = new Date(req.query.day);
  asPromise(Location.getLocations.bind(Location))
  .then(
    locations=>
      promise.all(
        locations.map(
          location=>
            asPromise(Appointment.getAppointments.bind(Appointment),[day,location])
            .then(appointments=>[location,appointments])
        )
      )
  )
  .then(
    results=>//results should be [ [location,[appointment,appointment]],...]
      Promise.all(
        results.map(
          ([location,appointments])=>
            savedAppointmentsForLocation(day,location,appointments)
            .then(ignoredSavedAppointment=>location)
        )
      )
  )
  .then(locations=>res.render('appointments', { locations: locations }))
  .catch(
    error=>{
      console.log("something went wrong:",error);
      res.status(500).send("Error in code");
    }
  )
});
0 голосов
/ 29 апреля 2018

Проблема была решена с помощью let вместо var, как предложено @ JaromandaX.

...