From be3f2cbce9b543e5abd475c9cad2b4e03f35af53 Mon Sep 17 00:00:00 2001
From: chenjiahan <chenjiahan@youzan.com>
Date: Mon, 14 Sep 2020 19:22:09 +0800
Subject: [PATCH] chore(CouponList): refactor with composition api

---
 package.json             |   2 +-
 src/coupon-list/index.js | 284 +++++++++++++++++++--------------------
 yarn.lock                |   8 +-
 3 files changed, 145 insertions(+), 149 deletions(-)

diff --git a/package.json b/package.json
index 8a43ddfe0..27236e84c 100644
--- a/package.json
+++ b/package.json
@@ -55,7 +55,7 @@
   "dependencies": {
     "@babel/runtime": "7.x",
     "@vant/icons": "1.3.0",
-    "@vant/use": "^0.0.1",
+    "@vant/use": "^0.0.2",
     "vue-lazyload": "1.2.3"
   },
   "peerDependencies": {
diff --git a/src/coupon-list/index.js b/src/coupon-list/index.js
index c909246b8..5a6d09207 100644
--- a/src/coupon-list/index.js
+++ b/src/coupon-list/index.js
@@ -1,6 +1,12 @@
+import { watch, computed, nextTick, onMounted, reactive } from 'vue';
+
 // Utils
 import { createNamespace } from '../utils';
 
+// Composition
+import { useWindowSize } from '@vant/use';
+import { useRefs } from '../composition/use-refs';
+
 // Components
 import Tab from '../tab';
 import Tabs from '../tabs';
@@ -65,178 +71,168 @@ export default createComponent({
 
   emits: ['change', 'exchange', 'update:code'],
 
-  data() {
-    return {
+  setup(props, { emit }) {
+    const [couponRefs, setCouponRefs] = useRefs();
+
+    const state = reactive({
       tab: 0,
-      winHeight: window.innerHeight,
-      currentCode: this.code || '',
-    };
-  },
+      code: props.code || '',
+    });
 
-  computed: {
-    buttonDisabled() {
-      return (
-        !this.exchangeButtonLoading &&
-        (this.exchangeButtonDisabled ||
-          !this.currentCode ||
-          this.currentCode.length < this.exchangeMinLength)
-      );
-    },
+    const { height: windowHeight } = useWindowSize();
 
-    listStyle() {
-      return {
-        height: this.winHeight - (this.showExchangeBar ? 140 : 94) + 'px',
-      };
-    },
-  },
+    const buttonDisabled = computed(
+      () =>
+        !props.exchangeButtonLoading &&
+        (props.exchangeButtonDisabled ||
+          !state.code ||
+          state.code.length < props.exchangeMinLength)
+    );
 
-  watch: {
-    code(code) {
-      this.currentCode = code;
-    },
+    const listStyle = computed(() => ({
+      height: windowHeight.value - (props.showExchangeBar ? 140 : 94) + 'px',
+    }));
 
-    currentCode(code) {
-      this.$emit('update:code', code);
-    },
-
-    displayedCouponIndex: 'scrollToShowCoupon',
-  },
-
-  mounted() {
-    this.scrollToShowCoupon(this.displayedCouponIndex);
-  },
-
-  methods: {
-    onClickExchangeButton() {
-      this.$emit('exchange', this.currentCode);
+    const onExchange = () => {
+      emit('exchange', state.code);
 
       // auto clear currentCode when not use vModel
-      if (!this.code) {
-        this.currentCode = '';
+      if (!props.code) {
+        state.code = '';
       }
-    },
+    };
 
-    // scroll to show specific coupon
-    scrollToShowCoupon(index) {
-      if (index === -1) {
-        return;
-      }
-
-      this.$nextTick(() => {
-        const { card, list } = this.$refs;
-
-        /* istanbul ignore next */
-        if (list && card && card[index]) {
-          list.scrollTop = card[index].$el.offsetTop - 100;
+    const scrollToCoupon = (index) => {
+      nextTick(() => {
+        if (couponRefs.value[index]) {
+          couponRefs.value[index].scrollIntoView();
         }
       });
-    },
+    };
 
-    genEmpty() {
-      return (
-        <div class={bem('empty')}>
-          <img src={this.emptyImage} />
-          <p>{t('empty')}</p>
-        </div>
-      );
-    },
-
-    genExchangeButton() {
-      return (
-        <Button
-          plain
-          type="danger"
-          class={bem('exchange')}
-          text={this.exchangeButtonText || t('exchange')}
-          loading={this.exchangeButtonLoading}
-          disabled={this.buttonDisabled}
-          onClick={this.onClickExchangeButton}
-        />
-      );
-    },
-  },
-
-  render() {
-    const { coupons, disabledCoupons } = this;
-
-    const count = this.showCount ? ` (${coupons.length})` : '';
-    const title = (this.enabledTitle || t('enable')) + count;
-
-    const disabledCount = this.showCount ? ` (${disabledCoupons.length})` : '';
-    const disabledTitle = (this.disabledTitle || t('disabled')) + disabledCount;
-
-    const ExchangeBar = this.showExchangeBar && (
-      <div class={bem('exchange-bar')}>
-        <Field
-          vModel={this.currentCode}
-          clearable
-          border={false}
-          class={bem('field')}
-          placeholder={this.inputPlaceholder || t('placeholder')}
-          maxlength="20"
-        />
-        {this.genExchangeButton()}
+    const renderEmpty = () => (
+      <div class={bem('empty')}>
+        <img src={props.emptyImage} />
+        <p>{t('empty')}</p>
       </div>
     );
 
-    const onChange = (index) => () => this.$emit('change', index);
-
-    const CouponTab = (
-      <Tab title={title}>
-        <div
-          class={bem('list', { 'with-bottom': this.showCloseButton })}
-          style={this.listStyle}
-        >
-          {coupons.map((coupon, index) => (
-            <Coupon
-              ref="card"
-              key={coupon.id}
-              coupon={coupon}
-              chosen={index === this.chosenCoupon}
-              currency={this.currency}
-              onClick={onChange(index)}
+    const renderExchangeBar = () => {
+      if (props.showExchangeBar) {
+        return (
+          <div class={bem('exchange-bar')}>
+            <Field
+              vModel={state.code}
+              clearable
+              border={false}
+              class={bem('field')}
+              placeholder={props.inputPlaceholder || t('placeholder')}
+              maxlength="20"
             />
-          ))}
-          {!coupons.length && this.genEmpty()}
-        </div>
-      </Tab>
+            <Button
+              plain
+              type="danger"
+              class={bem('exchange')}
+              text={props.exchangeButtonText || t('exchange')}
+              loading={props.exchangeButtonLoading}
+              disabled={buttonDisabled.value}
+              onClick={onExchange}
+            />
+          </div>
+        );
+      }
+    };
+
+    const renderCouponTab = () => {
+      const { coupons } = props;
+      const count = props.showCount ? ` (${coupons.length})` : '';
+      const title = (props.enabledTitle || t('enable')) + count;
+
+      return (
+        <Tab title={title}>
+          <div
+            class={bem('list', { 'with-bottom': props.showCloseButton })}
+            style={listStyle.value}
+          >
+            {coupons.map((coupon, index) => (
+              <Coupon
+                key={coupon.id}
+                ref={setCouponRefs(index)}
+                coupon={coupon}
+                chosen={index === props.chosenCoupon}
+                currency={props.currency}
+                onClick={() => emit('change', index)}
+              />
+            ))}
+            {!coupons.length && renderEmpty()}
+          </div>
+        </Tab>
+      );
+    };
+
+    const renderDisabledTab = () => {
+      const { disabledCoupons } = props;
+      const count = props.showCount ? ` (${disabledCoupons.length})` : '';
+      const title = (props.disabledTitle || t('disabled')) + count;
+
+      return (
+        <Tab title={title}>
+          <div
+            class={bem('list', { 'with-bottom': props.showCloseButton })}
+            style={listStyle.value}
+          >
+            {disabledCoupons.map((coupon) => (
+              <Coupon
+                disabled
+                key={coupon.id}
+                coupon={coupon}
+                currency={props.currency}
+              />
+            ))}
+            {!disabledCoupons.length && renderEmpty()}
+          </div>
+        </Tab>
+      );
+    };
+
+    watch(
+      () => props.code,
+      (value) => {
+        state.code = value;
+      }
     );
 
-    const DisabledCouponTab = (
-      <Tab title={disabledTitle}>
-        <div
-          class={bem('list', { 'with-bottom': this.showCloseButton })}
-          style={this.listStyle}
-        >
-          {disabledCoupons.map((coupon) => (
-            <Coupon
-              disabled
-              key={coupon.id}
-              coupon={coupon}
-              currency={this.currency}
-            />
-          ))}
-          {!disabledCoupons.length && this.genEmpty()}
-        </div>
-      </Tab>
+    watch(
+      () => state.code,
+      (value) => {
+        emit('update:code', value);
+      }
     );
 
-    return (
+    watch(() => props.displayedCouponIndex, scrollToCoupon);
+
+    onMounted(() => {
+      scrollToCoupon(props.displayedCouponIndex);
+    });
+
+    return () => (
       <div class={bem()}>
-        {ExchangeBar}
-        <Tabs vModel={this.tab} class={bem('tab')} border={false}>
-          {CouponTab}
-          {DisabledCouponTab}
+        {renderExchangeBar()}
+        <Tabs vModel={state.tab} class={bem('tab')} border={false}>
+          {renderCouponTab()}
+          {renderDisabledTab()}
         </Tabs>
         <div class={bem('bottom')}>
           <Button
-            vShow={this.showCloseButton}
+            vShow={props.showCloseButton}
             round
             block
             type="danger"
             class={bem('close')}
-            text={this.closeButtonText || t('close')}
-            onClick={onChange(-1)}
+            text={props.closeButtonText || t('close')}
+            onClick={() => {
+              emit('change', -1);
+            }}
           />
         </div>
       </div>
diff --git a/yarn.lock b/yarn.lock
index 955f996de..2f80426f0 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -2156,10 +2156,10 @@
   resolved "https://registry.yarnpkg.com/@vant/touch-emulator/-/touch-emulator-1.2.0.tgz#486300b23e57db9ce9231a04e0a0c621c68692d8"
   integrity sha512-sJ97zU85zOq51qoi7+CpBEcOyH3CitjP1KC7/GQwqaurUJni+EP7/F9n0HMnAh8GXMjgtgDBNJ5z48x+coNKYQ==
 
-"@vant/use@^0.0.1":
-  version "0.0.1"
-  resolved "https://registry.npmjs.org/@vant/use/-/use-0.0.1.tgz#66ed5dd186e54a6ec1b48604bfd8c0f466d9d568"
-  integrity sha512-AQrKFoiC4sZqRI6mx4a4iHekcpcvNYcZDo/0/WTMejtjKygaKMlImmOR36JWYinhvQwjO6GnAgQDvuO/hNwCPA==
+"@vant/use@^0.0.2":
+  version "0.0.2"
+  resolved "https://registry.npmjs.org/@vant/use/-/use-0.0.2.tgz#9ea72642e0d6ef664f19e80b73fdb59b9f403d21"
+  integrity sha512-kPgqwBMKaeKPQ/LZjjVDHKxo2Thi4w5M186+Yg4jATW7qCcfd0CujDmqlETfFjBCbw40C1HVu/kmhXE0cQPPUg==
 
 "@vue/babel-helper-vue-transform-on@^1.0.0-rc.2":
   version "1.0.0-rc.2"