Merge branch 'feat/group-buy' into 'master'

fix: code review

See merge request !9

src/views/goodsDetail/index.vue

@@ -427,27 +427,24 @@ export default {
         this.detailInfo = res;
         this.setStartStatus(res);
         this.showPage = true;
-        try {
-          this.detailInfo.contentDetailUrl = this.detailInfo.contentDetailUrl.replace(
-            'http://',
-            'https://'
-          );
-          const detailImages = await goods.getDetailPic(this.detailInfo.contentDetailUrl);
-          // const imgReg = new RegExp('(?<=src=").[^"]*', 'g');
-          // this.detailImgList = (detailImages || '').match(imgReg);
-          let imgReg = /<img.*?(?:>|\/>)/gi;
-          // eslint-disable-next-line no-useless-escape
-          let srcReg = /src=[\'"]?([^\'"]*)[\'"]?/i;
-          let arr = (detailImages || '').match(imgReg);
-          let srcArr = [];
-          for (let i = 0; i < arr.length; i++) {
-            let src = arr[i].match(srcReg)[1].replace('http://', 'https://');
-            srcArr.push(src);
-          }
-          this.detailImgList = srcArr;
-        } catch (error) {
-          console.error(error);
+
+        this.detailInfo.contentDetailUrl = this.detailInfo.contentDetailUrl.replace(
+          'http://',
+          'https://'
+        );
+        const detailImages = await goods.getDetailPic(this.detailInfo.contentDetailUrl);
+        // const imgReg = new RegExp('(?<=src=").[^"]*', 'g');
+        // this.detailImgList = (detailImages || '').match(imgReg);
+        let imgReg = /<img.*?(?:>|\/>)/gi;
+        // eslint-disable-next-line no-useless-escape
+        let srcReg = /src=[\'"]?([^\'"]*)[\'"]?/i;
+        let arr = (detailImages || '').match(imgReg);
+        let srcArr = [];
+        for (let i = 0; i < arr.length; i++) {
+          let src = arr[i].match(srcReg)[1].replace('http://', 'https://');
+          srcArr.push(src);
         }
+        this.detailImgList = srcArr;
 
         this.selectedGoods.skuId = this.detailInfo.skuNo;
         if (res.activitySkuTotalCount === 0) {

src/views/goodsDetail/smallPic.vue

@@ -239,28 +239,24 @@ export default {
         this.imgList = res.imageUrl || [];
         this.detailInfo = res;
         this.setStartStatus(res);
-        try {
-          this.detailInfo.contentDetailUrl = this.detailInfo.contentDetailUrl.replace(
-            'http://',
-            'https://'
-          );
-          const detailImages = await goods.getDetailPic(this.detailInfo.contentDetailUrl);
-          console.log(detailImages);
-          // const imgReg = new RegExp('(?<=src=").[^"]*', 'g');
-          // this.detailImgList = (detailImages || '').match(imgReg);
-          let imgReg = /<img.*?(?:>|\/>)/gi;
-          // eslint-disable-next-line no-useless-escape
-          let srcReg = /src=[\'"]?([^\'"]*)[\'"]?/i;
-          let arr = (detailImages || '').match(imgReg);
-          let srcArr = [];
-          for (let i = 0; i < arr.length; i++) {
-            let src = arr[i].match(srcReg)[1].replace('http://', 'https://');
-            srcArr.push(src);
-          }
-          this.detailImgList = srcArr;
-        } catch (error) {
-          console.error(error);
+        this.detailInfo.contentDetailUrl = this.detailInfo.contentDetailUrl.replace(
+          'http://',
+          'https://'
+        );
+        const detailImages = await goods.getDetailPic(this.detailInfo.contentDetailUrl);
+        console.log(detailImages);
+        // const imgReg = new RegExp('(?<=src=").[^"]*', 'g');
+        // this.detailImgList = (detailImages || '').match(imgReg);
+        let imgReg = /<img.*?(?:>|\/>)/gi;
+        // eslint-disable-next-line no-useless-escape
+        let srcReg = /src=[\'"]?([^\'"]*)[\'"]?/i;
+        let arr = (detailImages || '').match(imgReg);
+        let srcArr = [];
+        for (let i = 0; i < arr.length; i++) {
+          let src = arr[i].match(srcReg)[1].replace('http://', 'https://');
+          srcArr.push(src);
         }
+        this.detailImgList = srcArr;
       } catch (e) {
         console.log(e);
       }