JavaScript 如果已经使用了 promise，我该如何克服 JavaScript 的异步性？

Question

这不是我第一次遇到这个问题，但我总能找到解决方法，我想这很好，但现在我想了解如何改进。因此，在我的示例中，我有一个类(class)列表，我需要计算每个类(class)的平均评分:

var vm = this;

course.averageRating = vm.getAverageRating(course.id);

现在，什么不起作用:

vm.getAverageRating = function (courseId) {
       let sum = 0, courses = [];
       courseContext.getUserCourseFeedback(courseId).then(function(results){
            courses = results;
        });

       courses.forEach(function (course) {
            if(course && course.feedback_rating) {
                sum += course.feedback_rating;
            }
       });
       return courses.length > 0 ? sum / courses.length : null;
};

显然，courses.forEach 在应用程序实际从服务器下载 userCourses 列表之前触发，并且该函数始终返回 null。

我的解决方法如下:

vm.averageRatings = {};

vm.getAverageRating = function (courseId) {
  let sum = 0, courses = [];
  courseContext.getUserCourseFeedback(courseId).then(function(results){
       courses = results;

       courses.forEach(function (course) {
            if(course && course.feedback_rating) {
                sum += course.feedback_rating;
            }
       });

       vm.averageRatings[courseId] = courses.length > 0 ? sum/courses.length : null;
  });
};

我认为这不是解决此问题的最佳方法。在我的第一个示例中，我不能在 then 中使用 return 语句，我也不能在 promise 上使用 promise。如果你处在我的位置，你会如何解决这个问题？

Answer 1

使用 promise 没问题，但你应该

• 返回值的 promise ，而不是写入一些预先确定的对象
• 简化您的代码，特别是变量声明

function getAverageRating(courseId) {
  return courseContext.getUserCourseFeedback(courseId).then(function(courses) {
//^^^^^^                                                ^^^^^^^
    let sum = 0;
    for (const course of courses)
      sum += course && course.feedback_rating || 0;
    return courses.length > 0 ? sum/courses.length : null;
  });
};

getAverageRating(course.id).then(avg => {
  course.averageRating = avg;
});