blog: 更新文章

docs/_posts/随笔/代码评审/代码评审优化建议-210802.md

@@ -24,26 +24,12 @@ sidebar: false
 
 
 
-3. `196`和`197`行，获取文件名和文件后缀的方法可以使用数组解构来简写：
 
-   ```js
-   const [fileName='', ext=''] = '我最好看.jpg'.split('.');
-   ```
+3. `210`行，这一行代码似乎没起什么作用
 
 
 
-
-4. `210`行，这一行代码似乎没起什么作用
-
-
-
-5. `212`行，鉴于`209`行的match方法在匹配不到时会返回null, 因此`matchRes`的值有可能是`null`，直接把它当成数组去取值有可能会导致报错。
-
-   - `209`行是否可以使用同上面第3点的方法取值？
-
-
-
-6. `319`/`371`行 等多处判断参数是否为函数类型是没必要的，因为props已经限制了参数必须是函数类型。
+4. `319`/`371`行 等多处判断参数是否为函数类型是没必要的，因为props已经限制了参数必须是函数类型。
 
 
 
@@ -60,7 +46,7 @@ sidebar: false
 
    - 你已经是一个成熟的组件了，要学会适应不同的api
 
-   <img src="https://cdn.jsdelivr.net/gh/xugaoyi/image_store@master/blog/xh.4k9nyou0jfu0.png" alt="xh" style="zoom:50%;" />
+   
 
 
 
@@ -75,6 +61,6 @@ sidebar: false
 
 1. `139`行，添加了`maxSize`配置没有在文档里写明。
 
-2. 看了代码逻辑，只要`isMultipart`为true，就会强制分片上传。所以：
+2. 看了代码逻辑，只要`isMultipart`为true，就会强制分片上传。文档应该更清楚的写明：
    -  `partSize` 配置说明应改为：**自动开启分片上传的临界点(单位M)。仅在`isMultipart`为false时生效。**
    - `isMultipart` 配置说明应改为：**是否强制分片上传**