uniapp：code review 商品分类、商品品牌

YunaiV
1 parent a233ab13
Showing 4 changed files with 12 additions and 2 deletions
yudao-module-mall/yudao-module-product-biz/src/main/java/cn/iocoder/yudao/module/product/controller/admin/brand/BrandController.java
yudao-module-mall/yudao-module-product-biz/src/main/java/cn/iocoder/yudao/module/product/controller/admin/category/CategoryController.java
yudao-module-mall/yudao-module-product-biz/src/main/java/cn/iocoder/yudao/module/product/dal/dataobject/category/CategoryDO.java
yudao-module-mall/yudao-module-product-biz/src/main/java/cn/iocoder/yudao/module/product/service/category/CategoryServiceImpl.java
@@ -68,6 +68,7 @@ public class BrandController {
         return success(BrandConvert.INSTANCE.convert(brand));
     }
+    // TODO @JeromeSoar：这个接口，是不是没用哈
     @GetMapping("/list")
     @ApiOperation("获得品牌列表")
     @ApiImplicitParam(name = "ids", value = "编号列表", required = true, example = "1024,2048", dataTypeClass = List.class)
@@ -68,6 +68,7 @@ public class CategoryController {
         return success(CategoryConvert.INSTANCE.convert(category));
     }
+    // TODO @JeromeSoar：这个接口，是不是没用哈
     @GetMapping("/listByIds")
     @ApiOperation("获得商品分类列表")
     @PreAuthorize("@ss.hasPermission('product:category:query')")
@@ -76,6 +77,7 @@ public class CategoryController {
         return success(CategoryConvert.INSTANCE.convertList(list));
     }
+    // TODO @JeromeSoar：这应该是个 app 的接口，提供商品分类的树结构。这个调整下，后端只返回列表，前端构建 tree。注意，不需要返回创建时间、是否开启等无关字段。
     @GetMapping("/listByQuery")
     @ApiOperation("获得商品分类列表")
     @PreAuthorize("@ss.hasPermission('product:category:query')")
@@ -28,6 +28,7 @@ public class CategoryDO extends BaseDO {
     /**
      * 父分类编号
      */
+    // TODO @JeromeSoar：改成 parentId 字段
     private Long pid;
     /**
      * 分类名称
@@ -38,7 +39,9 @@ public class CategoryDO extends BaseDO {
      */
     private String icon;
     /**
-     * 分类图片
+     * 分类 Banner 图片
+     *
+     * 第一层的商品分类，会有该字段，用于用户 App 展示
      */
     private String bannerUrl;
     /**
@@ -51,7 +54,7 @@ public class CategoryDO extends BaseDO {
     private String description;
     /**
      * 开启状态
-     * <p>
+     *
      * 枚举 {@link CommonStatusEnum}
      */
     private Integer status;
@@ -29,6 +29,7 @@ public class CategoryServiceImpl implements CategoryService {
     @Override
     public Long createCategory(CategoryCreateReqVO createReqVO) {
+        // TODO JeromeSoar：校验父分类
         // 插入
         CategoryDO category = CategoryConvert.INSTANCE.convert(createReqVO);
         categoryMapper.insert(category);
@@ -38,6 +39,7 @@ public class CategoryServiceImpl implements CategoryService {
     @Override
     public void updateCategory(CategoryUpdateReqVO updateReqVO) {
+        // TODO JeromeSoar：校验父分类
         // 校验存在
         this.validateCategoryExists(updateReqVO.getId());
         // 更新
@@ -47,6 +49,8 @@ public class CategoryServiceImpl implements CategoryService {
     @Override
     public void deleteCategory(Long id) {
+        // TODO JeromeSoar：校验删除的商品分类是否存在
+        // TODO 芋艿 补充只有不存在商品才可以删除
         // 校验存在
         this.validateCategoryExists(id);
         // 删除