代码之家 › 专栏 › 技术社区 › RRTW

多中频保理建议

refactoring

RRTW · 技术社区 · 7 年前

我有一些像这样难看的老密码

if(!isLogin)
{
  if(confirm("Login first ?"))
  {
    doLogin();
    return;
  }
  else
  {
    doStuff1();
    doStuff2();
    doStuff3();
  }
}
else
{
  doStuff1();
  doStuff2();
  doStuff3();
}

对于重构,我是这样做的

if(!isLogin && confirm("Login first ?"))
{
  doLogin();
}
else
{
  doStuff1();
  doStuff2();
  doStuff3();
}

我不确定…新的代码逻辑是否等于旧的部分,并有可能缩短它?

1 回复 | 直到 7 年前

Jonathan Leffler 7 年前

你需要添加 return; 之后 doLogin(); 保持不变。你可以放下 else { 和 } 也一样,不让这三个 doStuff?() 功能:

if (!isLogin && confirm("Login first ?"))
{
    doLogin();
    return;
}
doStuff1();
doStuff2();
doStuff3();

推荐文章

Glory Raj · 有什么方法可以使这两个块具有共同的功能吗

3 年前

Justin · Python自属性重命名

8 年前

g.pickardou · 如何重构元组<..>到具有描述性名称的类型化类?

8 年前

Enoy · Javascript:重构现有代码以分离关注点,我们如何使用类/原型并从它自己的对象方法调用?

8 年前

Steve Harrison iDhaval · 为什么某些调用者没有显示在显示相关项目列表(Xcode)中?

8 年前

Sabutobi · 长dict属性行的重构

8 年前

Falco Alexander · 简单LINQ重构/样式

8 年前

05bs001 · Euler 12需要优化

8 年前

slartidan · 如何将lambda表达式重构为方法引用?

8 年前

Alex · 在Java类中频繁使用与映射键相同的字符串[已关闭]

8 年前