From d1298206056f60285154b6c8ef3eacd9acbca1aa Mon Sep 17 00:00:00 2001
From: YunaiV <zhijiantianya@gmail.com>
Date: Wed, 27 Dec 2023 23:13:26 +0800
Subject: [PATCH] =?UTF-8?q?=F0=9F=93=96=20code=20review=EF=BC=9A=E5=88=86?=
 =?UTF-8?q?=E4=BA=AB=E6=B5=B7=E6=8A=A5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../s-share-modal/canvas-poster/index.vue     |  12 +-
 .../s-share-modal/canvas-poster/useCanvas.js  |   2 -
 .../s-share-modal/s-share-modal.vue           | 318 +++++++++---------
 3 files changed, 166 insertions(+), 166 deletions(-)

diff --git a/sheep/components/s-share-modal/canvas-poster/index.vue b/sheep/components/s-share-modal/canvas-poster/index.vue
index 5c453663..b5212569 100644
--- a/sheep/components/s-share-modal/canvas-poster/index.vue
+++ b/sheep/components/s-share-modal/canvas-poster/index.vue
@@ -1,4 +1,4 @@
-<!-- 页面 -->
+<!-- 海报弹窗 -->
 <template>
   <su-popup :show="show" round="10" @close="onClosePoster" type="center" class="popup-box">
     <view class="ss-flex-col ss-col-center ss-row-center">
@@ -20,8 +20,8 @@
           height: poster.height + 'px',
           width: poster.width + 'px',
         }"
-		:show-menu-by-longpress="true"
-      ></image>
+        :show-menu-by-longpress="true"
+      />
       <canvas
         class="hideCanvas"
         :canvas-id="poster.canvasId"
@@ -33,7 +33,7 @@
       />
       <view
         class="poster-btn-box ss-m-t-20 ss-flex ss-row-between ss-col-center"
-        v-if="poster.src != ''"
+        v-if="poster.src !== ''"
       >
         <button class="cancel-btn ss-reset-button" @tap="onClosePoster">取消</button>
         <button class="save-btn ss-reset-button ui-BG-Main" @tap="onSavePoster">
@@ -73,9 +73,12 @@
 
   const emits = defineEmits(['success', 'close']);
   const vm = getCurrentInstance();
+
   const onClosePoster = () => {
     emits('close');
   };
+
+  // 保存海报图片
   const onSavePoster = () => {
     if (['WechatOfficialAccount', 'H5'].includes(sheep.$platform.name)) {
       sheep.$helper.toast('请长按图片保存');
@@ -95,6 +98,7 @@
     });
   };
 
+  // 使用 canvas 生成海报
   async function getPoster(params) {
     poster.src = '';
 
diff --git a/sheep/components/s-share-modal/canvas-poster/useCanvas.js b/sheep/components/s-share-modal/canvas-poster/useCanvas.js
index cd779472..8097de0e 100644
--- a/sheep/components/s-share-modal/canvas-poster/useCanvas.js
+++ b/sheep/components/s-share-modal/canvas-poster/useCanvas.js
@@ -5,8 +5,6 @@
  * @param {Object} options - 海报参数
  * @param {Object} vm - 自定义组件实例
  */
-
-import sheep from '@/sheep';
 import QSCanvas from 'qs-canvas';
 import { getPosterData } from './poster';
 
diff --git a/sheep/components/s-share-modal/s-share-modal.vue b/sheep/components/s-share-modal/s-share-modal.vue
index 87dc96f4..0f0b92db 100644
--- a/sheep/components/s-share-modal/s-share-modal.vue
+++ b/sheep/components/s-share-modal/s-share-modal.vue
@@ -1,197 +1,195 @@
+<!-- 全局分享弹框 -->
 <template>
-	<!-- 分享弹框 -->
-	<view>
-		<su-popup :show="state.showShareGuide" :showClose="false" @close="onCloseGuide"></su-popup>
-		<view v-if="state.showShareGuide" class="guide-wrap">
-			<image class="guide-image" :src="sheep.$url.static('/static/img/shop/share/share_guide.png')"></image>
-		</view>
+  <view>
+    <su-popup :show="state.showShareGuide" :showClose="false" @close="onCloseGuide" />
+    <view v-if="state.showShareGuide" class="guide-wrap">
+      <image class="guide-image" :src="sheep.$url.static('/static/img/shop/share/share_guide.png')" />
+    </view>
 
-		<su-popup :show="show" round="10" :showClose="false" @close="closeShareModal">
-			<!-- 分享tools -->
-			<view class="share-box">
-				<view class="share-list-box ss-flex">
-					<!--          <button
+    <su-popup :show="show" round="10" :showClose="false" @close="closeShareModal">
+      <!-- 分享 tools -->
+      <view class="share-box">
+        <view class="share-list-box ss-flex">
+          <!-- 操作 ①：发送给微信好友 -->
+          <button
             v-if="shareConfig.methods.includes('forward')"
             class="share-item share-btn ss-flex-col ss-col-center"
             open-type="share"
             @tap="onShareByForward"
+          >
+            <image class="share-img" :src="sheep.$url.static('/static/img/shop/share/share_wx.png')" mode="" />
+            <text class="share-title">微信好友</text>
+          </button>
+
+          <!-- 操作 ②：生成海报图片 -->
+          <button
+            v-if="shareConfig.methods.includes('poster')"
+            class="share-item share-btn ss-flex-col ss-col-center"
+            @tap="onShareByPoster"
           >
             <image
               class="share-img"
-              :src="sheep.$url.static('/static/img/shop/share/share_wx.png')"
+              :src="sheep.$url.static('/static/img/shop/share/share_poster.png')"
               mode=""
-            ></image>
-            <text class="share-title">微信好友</text>
-          </button> -->
-					<button v-if="shareConfig.methods.includes('poster')"
-						class="share-item share-btn ss-flex-col ss-col-center" @tap="onShareByPoster">
-						<image class="share-img" :src="sheep.$url.static('/static/img/shop/share/share_poster.png')"
-							mode=""></image>
-						<text class="share-title">生成海报</text>
-					</button>
-					<!-- 
+            />
+            <text class="share-title">生成海报</text>
+          </button>
+
+          <!-- 操作 ③：生成链接 -->
           <button
             v-if="shareConfig.methods.includes('link')"
             class="share-item share-btn ss-flex-col ss-col-center"
             @tap="onShareByCopyLink"
           >
-            <image
-              class="share-img"
-              :src="sheep.$url.static('/static/img/shop/share/share_link.png')"
-              mode=""
-            ></image>
+            <image class="share-img" :src="sheep.$url.static('/static/img/shop/share/share_link.png')" mode="" />
             <text class="share-title">复制链接</text>
-          </button> -->
-				</view>
-				<view class="share-foot ss-flex ss-row-center ss-col-center" @tap="closeShareModal">
-					取消
-				</view>
-			</view>
-		</su-popup>
-		<!-- 分享海报 -->
-		<canvas-poster ref="SharePosterRef" :show="state.showPosterModal" :shareInfo="shareInfo"
-			@close="state.showPosterModal = false" />
-	</view>
+          </button>
+        </view>
+        <view class="share-foot ss-flex ss-row-center ss-col-center" @tap="closeShareModal">
+          取消
+        </view>
+      </view>
+    </su-popup>
+
+    <!-- 分享海报，对应操作 ② -->
+    <canvas-poster
+      ref="SharePosterRef"
+      :show="state.showPosterModal"
+      :shareInfo="shareInfo"
+      @close="state.showPosterModal = false"
+    />
+  </view>
 </template>
 <script setup>
-	/**
-	 * 分享弹窗
-	 */
-	import {
-		ref,
-		unref,
-		reactive,
-		computed
-	} from 'vue';
-	import sheep from '@/sheep';
-	import canvasPoster from './canvas-poster/index.vue';
-	import {
-		showShareModal,
-		closeShareModal,
-		showAuthModal
-	} from '@/sheep/hooks/useModal';
+  /**
+   * 分享弹窗
+   */
+  import { ref, unref, reactive, computed } from 'vue';
+  import sheep from '@/sheep';
+  import canvasPoster from './canvas-poster/index.vue';
+  import { closeShareModal, showAuthModal } from '@/sheep/hooks/useModal';
 
-	const show = computed(() => sheep.$store('modal').share);
-	const shareConfig = computed(() => sheep.$store('app').platform.share);
-	const SharePosterRef = ref('');
+  const show = computed(() => sheep.$store('modal').share);
+  const shareConfig = computed(() => sheep.$store('app').platform.share);
+  const SharePosterRef = ref('');
 
-	const props = defineProps({
-		shareInfo: {
-			type: Object,
-			default () {},
-		},
-	});
+  const props = defineProps({
+    shareInfo: {
+      type: Object,
+      default() {},
+    },
+  });
 
-	const state = reactive({
-		showShareGuide: false, //H5的指引。
-		showPosterModal: false, //海报弹窗
-	});
+  const state = reactive({
+    showShareGuide: false, // H5 的指引
+    showPosterModal: false, // 海报弹窗
+  });
 
-	// 生成海报分享
-	const onShareByPoster = () => {
-		closeShareModal();
-		if (!sheep.$store('user').isLogin) {
-			showAuthModal();
-			return;
-		}
-		unref(SharePosterRef).getPoster();
-		state.showPosterModal = true;
-	};
+  // 操作 ②：生成海报分享
+  const onShareByPoster = () => {
+    closeShareModal();
+    if (!sheep.$store('user').isLogin) {
+      showAuthModal();
+      return;
+    }
+    unref(SharePosterRef).getPoster();
+    state.showPosterModal = true;
+  };
 
-	// 直接转发分享
-	const onShareByForward = () => {
-		closeShareModal();
+  // 操作 ①：直接转发分享
+  const onShareByForward = () => {
+    closeShareModal();
 
-		// #ifdef H5
-		if (['WechatOfficialAccount', 'H5'].includes(sheep.$platform.name)) {
-			state.showShareGuide = true;
-			return;
-		}
-		// #endif
+    // #ifdef H5
+    if (['WechatOfficialAccount', 'H5'].includes(sheep.$platform.name)) {
+      state.showShareGuide = true;
+      return;
+    }
+    // #endif
 
-		// #ifdef APP-PLUS
-		uni.share({
-			provider: 'weixin',
-			scene: 'WXSceneSession',
-			type: 0,
-			href: props.shareInfo.link,
-			title: props.shareInfo.title,
-			summary: props.shareInfo.desc,
-			imageUrl: props.shareInfo.image,
-			success: (res) => {
-				console.log('success:' + JSON.stringify(res));
-			},
-			fail: (err) => {
-				console.log('fail:' + JSON.stringify(err));
-			},
-		});
-		// #endif
-	};
+    // #ifdef APP-PLUS
+    uni.share({
+      provider: 'weixin',
+      scene: 'WXSceneSession',
+      type: 0,
+      href: props.shareInfo.link,
+      title: props.shareInfo.title,
+      summary: props.shareInfo.desc,
+      imageUrl: props.shareInfo.image,
+      success: (res) => {
+        console.log('success:' + JSON.stringify(res));
+      },
+      fail: (err) => {
+        console.log('fail:' + JSON.stringify(err));
+      },
+    });
+    // #endif
+  };
 
-	// 复制链接分享
-	const onShareByCopyLink = () => {
-		sheep.$helper.copyText(props.shareInfo.link);
-		closeShareModal();
-	};
+  // 操作 ③：复制链接分享
+  const onShareByCopyLink = () => {
+    sheep.$helper.copyText(props.shareInfo.link);
+    closeShareModal();
+  };
 
-	function onCloseGuide() {
-		state.showShareGuide = false;
-	}
+  function onCloseGuide() {
+    state.showShareGuide = false;
+  }
 </script>
 
 <style lang="scss" scoped>
-	.guide-image {
-		right: 30rpx;
-		top: 0;
-		position: fixed;
-		width: 580rpx;
-		height: 430rpx;
-		z-index: 10080;
-	}
+  .guide-image {
+    right: 30rpx;
+    top: 0;
+    position: fixed;
+    width: 580rpx;
+    height: 430rpx;
+    z-index: 10080;
+  }
 
-	// 分享tool
-	.share-box {
-		background: $white;
-		width: 750rpx;
-		border-radius: 30rpx 30rpx 0 0;
-		padding-top: 30rpx;
+  // 分享tool
+  .share-box {
+    background: $white;
+    width: 750rpx;
+    border-radius: 30rpx 30rpx 0 0;
+    padding-top: 30rpx;
 
-		.share-foot {
-			font-size: 24rpx;
-			color: $gray-b;
-			height: 80rpx;
-			border-top: 1rpx solid $gray-e;
-		}
+    .share-foot {
+      font-size: 24rpx;
+      color: $gray-b;
+      height: 80rpx;
+      border-top: 1rpx solid $gray-e;
+    }
 
-		.share-list-box {
-			.share-btn {
-				background: none;
-				border: none;
-				line-height: 1;
-				padding: 0;
+    .share-list-box {
+      .share-btn {
+        background: none;
+        border: none;
+        line-height: 1;
+        padding: 0;
 
-				&::after {
-					border: none;
-				}
-			}
+        &::after {
+          border: none;
+        }
+      }
 
-			.share-item {
-				flex: 1;
-				padding-bottom: 20rpx;
+      .share-item {
+        flex: 1;
+        padding-bottom: 20rpx;
 
-				.share-img {
-					width: 70rpx;
-					height: 70rpx;
-					background: $gray-f;
-					border-radius: 50%;
-					margin-bottom: 20rpx;
-				}
+        .share-img {
+          width: 70rpx;
+          height: 70rpx;
+          background: $gray-f;
+          border-radius: 50%;
+          margin-bottom: 20rpx;
+        }
 
-				.share-title {
-					font-size: 24rpx;
-					color: $dark-6;
-				}
-			}
-		}
-	}
-</style>
\ No newline at end of file
+        .share-title {
+          font-size: 24rpx;
+          color: $dark-6;
+        }
+      }
+    }
+  }
+</style>