From 84c5eb9942f7823980096aca80e556c4696e853c Mon Sep 17 00:00:00 2001
From: YunaiV <zhijiantianya@gmail.com>
Date: Sat, 26 Aug 2023 11:19:29 +0800
Subject: [PATCH] =?UTF-8?q?code=20review=EF=BC=9A=E5=95=86=E5=93=81?=
 =?UTF-8?q?=E8=AF=84=E8=AE=BA?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/views/mall/product/comment/CommentForm.vue               | 1 +
 src/views/mall/product/comment/components/SkuTableSelect.vue | 1 +
 src/views/mall/product/comment/components/SpuTableSelect.vue | 1 +
 src/views/mall/product/comment/index.vue                     | 4 +++-
 src/views/mall/product/spu/addForm.vue                       | 2 ++
 5 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/views/mall/product/comment/CommentForm.vue b/src/views/mall/product/comment/CommentForm.vue
index 85458c3d..f4180f45 100644
--- a/src/views/mall/product/comment/CommentForm.vue
+++ b/src/views/mall/product/comment/CommentForm.vue
@@ -36,6 +36,7 @@
       <el-form-item label="评论内容" prop="content">
         <el-input type="textarea" v-model="formData.content" />
       </el-form-item>
+      <!-- TODO @疯狂：formData.scores 是通过后端计算的哈，不要前端传递 -->
       <el-form-item label="评分星级" prop="scores">
         <el-rate v-model="formData.scores" />
       </el-form-item>
diff --git a/src/views/mall/product/comment/components/SkuTableSelect.vue b/src/views/mall/product/comment/components/SkuTableSelect.vue
index 8bbc50cf..f72870e4 100644
--- a/src/views/mall/product/comment/components/SkuTableSelect.vue
+++ b/src/views/mall/product/comment/components/SkuTableSelect.vue
@@ -37,6 +37,7 @@ import { ElTable } from 'element-plus'
 import * as ProductSpuApi from '@/api/mall/product/spu'
 import { propTypes } from '@/utils/propTypes'
 
+// TODO @疯狂：是不是挪到 spu 的 components 下哈
 defineOptions({ name: 'SkuTableSelect' })
 
 const props = defineProps({
diff --git a/src/views/mall/product/comment/components/SpuTableSelect.vue b/src/views/mall/product/comment/components/SpuTableSelect.vue
index f8560aa3..2a62a2fa 100644
--- a/src/views/mall/product/comment/components/SpuTableSelect.vue
+++ b/src/views/mall/product/comment/components/SpuTableSelect.vue
@@ -86,6 +86,7 @@ import { defaultProps, handleTree } from '@/utils/tree'
 import * as ProductCategoryApi from '@/api/mall/product/category'
 import * as ProductSpuApi from '@/api/mall/product/spu'
 
+// TODO @疯狂：是不是挪到 spu 的 components 下哈
 defineOptions({ name: 'SpuTableSelect' })
 
 const message = useMessage() // 消息弹窗
diff --git a/src/views/mall/product/comment/index.vue b/src/views/mall/product/comment/index.vue
index ecb519c9..7647aff1 100644
--- a/src/views/mall/product/comment/index.vue
+++ b/src/views/mall/product/comment/index.vue
@@ -60,6 +60,7 @@
   <ContentWrap>
     <el-table v-loading="loading" :data="list" :stripe="true" :show-overflow-tooltip="false">
       <el-table-column label="评论编号" align="center" prop="id" min-width="60" />
+      <!-- TODO @疯狂：后端貌似没读取？ -->
       <el-table-column label="用户名称" align="center" prop="userNickname" width="80" />
       <el-table-column label="商品信息" align="center" min-width="210">
         <template #default="scope">
@@ -210,7 +211,7 @@ const getList = async () => {
   loading.value = true
   try {
     const data = await CommentApi.getCommentPage(queryParams)
-    // visible 如果为 null，会导致刷新的时候触发e-switch的change事件
+    // visible 如果为 null，会导致刷新的时候触发 e-switch 的 change 事件
     data.list.forEach((item) => {
       if (!item.visible) {
         item.visible = false
@@ -241,6 +242,7 @@ const openForm = (type: string, id?: number) => {
   formRef.value.open(type, id)
 }
 
+// TODO @疯狂：要不回复，也搞一个组件出去？
 /** 回复 **/
 const replyFormRef = ref()
 const replyDialog = reactive({
diff --git a/src/views/mall/product/spu/addForm.vue b/src/views/mall/product/spu/addForm.vue
index b577ece9..a536ba35 100644
--- a/src/views/mall/product/spu/addForm.vue
+++ b/src/views/mall/product/spu/addForm.vue
@@ -44,6 +44,8 @@ import { BasicInfoForm, DescriptionForm, OtherSettingsForm } from './components'
 import * as ProductSpuApi from '@/api/mall/product/spu'
 import { convertToInteger, formatToFraction } from '@/utils'
 
+// TODO @芋艿：后续稍微调整下；
+
 defineOptions({ name: 'ProductSpuForm' })
 
 const { t } = useI18n() // 国际化