Question

我在Ramda中编写了一个映射器，该映射器简化了对象并删除了具有空数组的属性。这是一件非常简单的事情，但是当我研究该函数时，不得不组合在一起的Ramda辅助方法会使我在数周的时间内挠头。

我想知道是否有一种更干净的方法来达到相同的结果。

代码如下：

const filters = {
  Brand: [],
  BusinessUnit: [
    {
      FilterName: 'BusinessUnit',
      KeyItem: 'Beauty'
    },
    {
      FilterName: 'BusinessUnit',
      KeyItem: 'Sports'
    }
  ],
  Category: [],
  SKU: [
    {
      FilterName: 'SKU',
      KeyItem: '9023'
    }
  ]
}

const expectedFilters = {
  BusinessUnit: ['Beauty', 'Sports'],
  SKU: ['9023']
};

///////////////////////////////////////////////
// is there any way to write it cleaner

const result = R.compose(
  R.pickBy(R.pipe(R.isEmpty, R.not)),
  R.map(R.map(R.prop('KeyItem')))
)(filters);

///////////////////////////////////////////////

console.log('(expectedFilters:: ', expectedFilters);
console.log('result:: ', result);
console.log('is equal? ', R.equals(expectedFilters, result)); // true

这是此代码在Stackblitz上运行的链接。

Answer 1

更新

@codepic指出，使用reject代替组合filter和complement还是更好。这显然是一种优越的解决方案：

const transform = pipe (
  reject (isEmpty), 
  map (pluck ('KeyItem') )
)

const filters = {Brand: [], BusinessUnit: [{FilterName: 'BusinessUnit', KeyItem: 'Beauty'}, {FilterName: 'BusinessUnit', KeyItem: 'Sports'}], Category: [], SKU: [{FilterName: 'SKU', KeyItem: '9023'}]}

console .log (
  transform (filters)
)

<script src="//cdnjs.cloudflare.com/ajax/libs/ramda/0.26.1/ramda.js"></script>
<script> const {pipe, reject, isEmpty, map, pluck} = R   </script>

原始答案

肯定有一些清理工作。我认为complement (isEmpty)比pipe (isEmpty, not)更干净。 map (prop ('foo') )具有内置功能pluck ('foo')。尽管pickBy可以正常工作，但由于filter可以覆盖对象和数组，因此它在一段时间前变得多余了。

所以我的（现在已过时）版本如下所示：

const transform = pipe (
  filter (complement (isEmpty) )
  map (pluck ('KeyItem') )
)

const filters = {Brand: [], BusinessUnit: [{FilterName: 'BusinessUnit', KeyItem: 'Beauty'}, {FilterName: 'BusinessUnit', KeyItem: 'Sports'}], Category: [], SKU: [{FilterName: 'SKU', KeyItem: '9023'}]}

console .log (
  transform (filters)
)

<script src="//cdnjs.cloudflare.com/ajax/libs/ramda/0.26.1/ramda.js"></script>
<script> const {pipe, filter, complement, isEmpty, map, pluck} = R   </script>

当然，要为将complement (isEmpty)提取到notEmpty并可能将filter (notEmpty)提取成类似removeEmpties的方法进行讨论。显然，这些都是容易做到的。

Answer 2

我认为您的解决方案没有任何错误或复杂之处。

我唯一要进行的调整是：

将R.pickBy(R.pipe(R.isEmpty, R.not))替换为R.pickBy(R.complement(R.isEmpty))。
在映射前先选择。（当前，您正在映射空数组）

在Ramda.js中编写特定对象映射器的更简洁方法

2 个答案:

更新

原始答案