来请教大家，这种算冗余代码吗？

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

This topic created in 2177 days ago, the information mentioned may be changed or developed.

<template>
  <div>
    <button @click="tryAgain">点击重试</button>
  </div>
</template>

<script>
export default {
  data() {
    return {};
  },
  mounted() {
    this.unsubscribe();
  },
  methods: {
    unsubscribe() {
      // do something
    },
    tryAgain() {
      this.unsubscribe();
    },
  },
};
</script>

类似这么一段 Vue 代码，给同事 review 代码时候我指出tryAgain可以删掉，但他认为很直观没问题。

我觉得只是单纯再包一层函数直观但没啥意义，更倾向删掉。

39 replies • 2020-06-24 08:59:41 +08:00

12tall

Jun 23, 2020

个人倾向于删掉，后面有需求可以再加上(doge

chaos1019

Jun 23, 2020

我觉得不算

syscall

Jun 23, 2020

很直观没问题。
“很直观”就是个意义。

dumbass

Jun 23, 2020

如果按钮还要做`unsubscribe()`之外的操作，或许就不冗余了

mikoshu

Jun 23, 2020

看习惯吧我个人是会删的...减少代码量哈哈哈

axeprpr1

Jun 23, 2020

xiaoming1992

Jun 23, 2020 via Android

这样子语义更明晰吧

wxsm

Jun 23, 2020

这叫直观？惊了。

<button @click="unsubscribe">点击重试</button>
这才叫直观好吧！

bQ3u9RAQt4125PUK

Jun 23, 2020

直观是啥意思？
看不懂 HTML 里面的那四个“点击重试”四个大字非得用一个 function 名来把它翻译一下么？
你们团队里有外国人？

pinkSlime

Jun 23, 2020

我感觉吧这种有别人强烈自我风格表达意愿的代码
如果不是违反你们公司的规范的话还是不要提的好
回字的四种写法没有绝对的是非对错
而在你的队友看来你是在否定他的审美
就好比: 你怎么穿这双鞋奇丑无比
没有实质收益还挺容易得罪人何苦来着

xingyue

Jun 23, 2020

@zackwan95 #9 本来还挺纠结的，被老哥一语点醒，抱拳.jpg

ps: 如果真要表达“直观”这一层，似乎要这样写：
<button @click="handleBtnClick('点击重试')">点击重试</button>
handleBtnClick(type,...args){
switch(type){
case '点击重试':
this.unsubscribe();
break;
}
}

tryAgain 和 unsubscribe 的语义在同一层，都告诉你具体干什么；而 handleBtnClick 就语义很“直观”，我负责处理点击事件，具体干什么请去函数内找，与前两者不在同一层~

zjsxwc

Jun 23, 2020

我倾向于套个娃

ChanKc

Jun 23, 2020 via Android

倾向于不删

zhuweiyou

Jun 23, 2020

看情况，如果 tryAgain 还要做别的事情，那就有意义。

bnm965321

Jun 23, 2020

unsubscribe 之后不应该重新 subscribe 吗

mmlovely

Jun 23, 2020

见仁见智，个人认为没什么问题

yiyi11

Jun 23, 2020 via Android

但问题是，冗余≠多余，冗余设计为了维护扩展，那是可行的。
如果真要较真，应该举证，根据业务需求的复杂度，认为绝大部分情况不需要在 tryAgain 中写额外逻辑，即 undescribe 的逻辑足够简单且能表达整个操作的意思，那就不需要该冗余。

yiyi11

Jun 23, 2020 via Android

上面写错了，是 unsubscribe

lcs1998

Jun 23, 2020

这个还得看你们之后是不是要在这上面拓展功能了

Mutoo

Jun 23, 2020

这不只是冗余，还是脱裤子放屁。

UIXX

Jun 23, 2020

这是代码风格的问题，只要该程序员写的代码风格一致即可。

no1xsyzy

Jun 23, 2020

@xingyue #11 你这也不对，“我符合处理点击事件” 是 @click 表达的语义。
我觉得这里的问题是这个组件只用于 unsubscribe ？
难道不应该外层把 unsubscribe 函数和 “点击重试” 字符串作为 props 传进去吗？

sonice

Jun 23, 2020

点击重试和 unsubscribe 有逻辑关系吗？
如果根据上下文工作流程已经很确定了，重试就一定是 unsubscribe，那去掉没毛病。
还有其他人有提到的业务扩展的问题。

review 代码不都是走过场吗？只要功能保证，没有明显的逻辑错误就行，你这样搞是会没“朋友”的

swirling

Jun 23, 2020

Verbosity is not a bad thing.

atwoodSoInterest

Jun 23, 2020

不冗余
按钮的显示是“点击重试”，但是实际执行的方法是 unsubscribe，意义其实是不对等的。
这个调用的意义就在于说明，重试的内容是取消订阅。

yinjunjian0

Jun 23, 2020

删
按钮的方法名是 tryAgain，看方法名实质不知道重试什么
换成 unsubscribe 语义更明确

asLw0P981N0M0TCC

Jun 23, 2020

万一 tryagin 里后期还要加别的操作呢不只是 unsubscribe 呢

Lanayaaa

Jun 23, 2020

如果明天 PM 就给你提个需求让在 tryagain 里多一个操作, 就有意义了。
简单的说就是赌博。老梗了。

cassyfar

Jun 23, 2020

不冗余。unsub 只是 try again 点下去后发生的一步。

tutou

Jun 23, 2020

不想删，因为不知道什么时候产品加奇怪的需求

djs

Jun 23, 2020

没问题吧，加入你这个 tryagain 又要加一堆奇怪的东西，就有用了，逻辑上也分离了

djs

Jun 23, 2020

听听你同事咋说，如果只是直观，我是觉得删了吧

GzhiYi

Jun 23, 2020

是我没法留下来，心里不爽。所以我选删。

ipwx

Jun 23, 2020

tryAgain => handleBtnTryAgain

我打赌这么重命名楼上的绝对会一致赞同。

ipwx

Jun 23, 2020

或者 tryAgain => onBtnTryAgainClick

本质上楼主的同事可能觉得 tryAgain 指的就是 tryagain 这个按钮，和 unsubscribe 不在同一层。但是楼主认为 tryAgain 语义上和 unsubscribe 在同一层。这么一种解读差别，所以带来了不同的判断。