0

好吧，这是我的第一个堆栈溢出帖子！

大家好，最近学习了很多函数式编程。我在funfunfunction上发现了一些旧视频，它们激起了我的胃口，我一直在玩它们。

我的问题是……有没有更好/不同/首选的方式来重构这段代码：

if ( bodyClasses.contains("first-class") || bodyClasses.contains("second-class") || bodyClasses.contains("third-class") ) {
        pages.filter((page) => searchAndReplace( page.postType, ".selector" ));
}

我尝试使用reduce()但很快了解到我需要在第一次迭代返回时过早地中断 reduce true。我觉得 if 语句在这里很有意义，但我只是好奇。

我知道这是一小段没有上下文的代码，但问题是关于概念而不是功能。

对于任何回应的人，我提前感谢您！

4

2 回答 2

1

归根结底，||可能需要在某个地方，但也许我们可以把它们藏起来。我喜欢在我的 if 子句中使用谓词来增加可读性并创建可重用的函数。

const hasSupport = classes =>
  classes.some(cls => cls === "class1" || cls === "class2" || cls === "class3");

if (hasSupport(bodyClasses)) { ... }

最后要考虑的部分是在您的if陈述中。该过滤器将返回一个新列表，但您不会将数据存储在任何地方。因此，如果您确实想要一个新列表，请将其存储在一个变量中：

const supportedPages = hasSupport(bodyClasses) 
  ?  pages.filter(...) 
  : []

如果您想要这些副作用，请使用 forEach 循环：

if (hasSupport(bodyClasses) { pages.forEach(...) }
于 2019-03-05T20:18:15.703 回答
1

要摆脱 or，匹配.some数组值：

if(["first-class", "second-class", "third-class"].some(cls => bodyClasses.contains(cls)))
 /*...*/;

.filter如果你没有对返回的值做任何事情，我不会使用，那么只需使用.forEach.

于 2019-03-05T20:33:42.127 回答