diff --git a/src/main/java/cn/chjyj/szwh/service/impl/GoodsServiceImpl.java b/src/main/java/cn/chjyj/szwh/service/impl/GoodsServiceImpl.java
index d660f51..ffe8924 100644
--- a/src/main/java/cn/chjyj/szwh/service/impl/GoodsServiceImpl.java
+++ b/src/main/java/cn/chjyj/szwh/service/impl/GoodsServiceImpl.java
@@ -46,11 +46,6 @@ public class GoodsServiceImpl implements GoodsService {
     // 节假日mapper
     @Autowired
     private FestivalsMapper festivalsMapper;
-    /**
-     * springboot 自带事务
-     */
-    @Autowired
-    private PlatformTransactionManager platformTransactionManager;
 
     @Override
     public List<Goods> getGoodsList(int page) {
@@ -457,13 +452,15 @@ public class GoodsServiceImpl implements GoodsService {
      * @param type
      * @return
      */
+
     @Override
     public int ranksort(String goodsIsli, int type) {
         //去除排序列表
-        String where="is_deleted=0 and is_recommend=1";
-        List<Goods> tmpRanklist = goodsMapper.getGoodsByCondition(where, 8, "recommend_sort", "DESC");
+        //String where="is_deleted=0 and is_recommend=1";
+        //List<Goods> tmpRanklist = goodsMapper.getGoodsByCondition(where, 8, "recommend_sort", "DESC");
+        List<Goods> tmpRanklist = queryRankList();
         int ret = 0;
-        for(int i=0;i<tmpRanklist.size()-1;i++){
+        for(int i=0;i<tmpRanklist.size();i++){
             if(goodsIsli.equals(tmpRanklist.get(i).getGoodsIslicode()) ){
                 // 当前项目sort
                 int curSort = tmpRanklist.get(i).getRecommendSort();
@@ -475,6 +472,7 @@ public class GoodsServiceImpl implements GoodsService {
                         //前序sort
                         int frontSort = tmpRanklist.get(i-1).getRecommendSort();
                         int afterSort= (frontSort==curSort)?curSort+1:frontSort;
+                        log.info("更新后的sort："+afterSort);
                         // 前序变更recommend sort
                         goodsMapper.updateRecommend(tmpRanklist.get(i-1).getGoodsIslicode(),curSort);
                         ret = goodsMapper.updateRecommend(tmpRanklist.get(i).getGoodsIslicode(),afterSort);
@@ -511,6 +509,16 @@ public class GoodsServiceImpl implements GoodsService {
         return opGoodsInfo(jsonObject);
     }
 
+    /**
+     * query rank list
+     * @return
+     */
+    private List<Goods> queryRankList(){
+        String where="is_deleted=0 and is_recommend=1";
+        List<Goods> tmpRanklist = goodsMapper.getGoodsByCondition(where, 8, "recommend_sort", "DESC");
+        return tmpRanklist;
+    }
+
     /**
      * 操作商品信息
      *
diff --git a/src/main/resources/mapper/szwh/GoodsMapper.xml b/src/main/resources/mapper/szwh/GoodsMapper.xml
index 4bb7648..94c8f08 100644
--- a/src/main/resources/mapper/szwh/GoodsMapper.xml
+++ b/src/main/resources/mapper/szwh/GoodsMapper.xml
@@ -257,7 +257,7 @@
     <update id="updateRecommend">
         update
         <include refid="tbName"/>
-        set is_recommend=#{vale}
+        set recommend_sort=#{vale}
         where goods_islicode=#{goodsisli}
     </update>
 
diff --git a/src/test/java/cn/chjyj/szwh/service/impl/GoodsServiceImplTest.java b/src/test/java/cn/chjyj/szwh/service/impl/GoodsServiceImplTest.java
index f32faf2..2c03d93 100644
--- a/src/test/java/cn/chjyj/szwh/service/impl/GoodsServiceImplTest.java
+++ b/src/test/java/cn/chjyj/szwh/service/impl/GoodsServiceImplTest.java
@@ -78,37 +78,42 @@ public class GoodsServiceImplTest {
     /**
      * 测试排序操作
      */
-    @Rollback(true)
+    @Rollback(value = false)
     @Test
     public void ranksort() {
         //商品编号
-        String goods_isliCode="010007-00000000187499999999-1";
+        String goods_isliCode="010007-00000000278199999999-1";
         // 1:add 2: reduce
-        int sortType=1;
+        int type=1;
 
         String where="is_deleted=0 and is_recommend=1";
         List<Goods> tmpRanklist = goodsMapper.getGoodsByCondition(where,8, "recommend_sort", "DESC");
         //tmpRanklist.get(0);
-        for(int i=0;i<tmpRanklist.size()-1;i++){
+        int ret = 0;
+        for(int i=0;i<tmpRanklist.size();i++){
             if(goods_isliCode.equals(tmpRanklist.get(i).getGoodsIslicode()) ){
-                // for add
-                if(sortType==1) {
+                // 当前项目sort
+                int curSort = tmpRanklist.get(i).getRecommendSort();
+                // 上移
+                if(type==1) {
                     if(i==0){
                         break;
                     }else {
-                        goodsMapper.updateRecommend(tmpRanklist.get(i-1).getGoodsIslicode(),
-                                tmpRanklist.get(i).getRecommendSort());
-                        goodsMapper.updateRecommend(tmpRanklist.get(i).getGoodsIslicode(),
-                                tmpRanklist.get(i-1).getRecommendSort());
+                        //前序sort
+                        int frontSort = tmpRanklist.get(i-1).getRecommendSort();
+                        int afterSort= (frontSort==curSort)?curSort+1:frontSort;
+                        // 前序变更recommend sort
+                        ret = goodsMapper.updateRecommend(goods_isliCode,afterSort);
                     }
                 }else{
                     if(i==tmpRanklist.size()-1){
                         break;
                     }else{
-                        goodsMapper.updateRecommend(tmpRanklist.get(i+1).getGoodsIslicode(),
-                                tmpRanklist.get(i).getRecommendSort());
-                        goodsMapper.updateRecommend(tmpRanklist.get(i).getGoodsIslicode(),
-                                tmpRanklist.get(i+1).getRecommendSort());
+                        //前序sort
+                        int frontSort = tmpRanklist.get(i+1).getRecommendSort();
+                        int afterSort= (frontSort==curSort)?curSort-1:frontSort;
+                        // 前序变更recommend sort
+                        ret = goodsMapper.updateRecommend(goods_isliCode, afterSort);
                     }
                 }
             }