diff --git a/src/main/java/com/sqx/modules/course/service/impl/CourseCollectServiceImpl.java b/src/main/java/com/sqx/modules/course/service/impl/CourseCollectServiceImpl.java
index 19162c12..0bb9fa58 100644
--- a/src/main/java/com/sqx/modules/course/service/impl/CourseCollectServiceImpl.java
+++ b/src/main/java/com/sqx/modules/course/service/impl/CourseCollectServiceImpl.java
@@ -56,7 +56,7 @@ public class CourseCollectServiceImpl extends ServiceImpl<CourseCollectDao, Cour
         try {
             if (courseCollect.getClassify() == 3) {
                 if (courseCollect.getCourseId() == null || courseCollect.getCourseDetailsId() == null) {
-                    return Result.success("操作成功！");
+                    return Result.success("操作失败！");
                 }
                 QueryWrapper<CourseCollect> queryWrapper = new QueryWrapper<>();
                 queryWrapper.eq("user_id", courseCollect.getUserId())
@@ -67,9 +67,9 @@ public class CourseCollectServiceImpl extends ServiceImpl<CourseCollectDao, Cour
                 if (collect != null) {
                     baseMapper.update(null, new LambdaUpdateWrapper<CourseCollect>()
                             .eq(CourseCollect::getCourseCollectId, collect.getCourseCollectId())
-                            .eq(CourseCollect::getCourseId, collect.getCourseId())
+                            .eq(CourseCollect::getUserId, collect.getUserId())
                             .set(CourseCollect::getUpdateTime, DateUtil.now())
-                            .set(CourseCollect::getCourseDetailsId, collect.getCourseDetailsId()));
+                            .set(CourseCollect::getCourseDetailsId, courseCollect.getCourseDetailsId()));
                     return Result.success("操作成功！");
                 }
                 collect = new CourseCollect()