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

1

Я новичок в 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 (Immediately invoked function expression), чтобы выполнить код мгновенно, но когда я это сделал, я не смог получить объект обратного вызова appointments и моя логика тоже сломана.

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

  • 0
    Смотрите этот ответ для получения дополнительной информации о обратных вызовах и циклах
  • 0
    На самом деле я рассматривал это раньше, но это не относится к моему делу. Не могли бы вы быть более конкретным?
Показать ещё 3 комментария
Теги:
schedule
iife

2 ответа

0
Лучший ответ

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

Поделиться
0

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

При сохранении назначения неправильно, запрашивающий не знает об этом, потому что 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");
    }
  )
});
Поделиться
  • 1
    После вашего комментария я много изучил об обещаниях и реорганизовал свой код для их использования. Теперь у меня намного больше контроля над решением. Спасибо!

Ещё вопросы

Сообщество Overcoder
Наверх
Меню