standData是福还是祸？

[wjgogogo]

从注释来看：引入标准数据为了转换提高效率，然而🤔 ：

目前标准数据流导致的问题：

• 以前是按照values遍历，保证了生成的数据都是以values为准，而现在的指标顺序和数据的内部顺序强关联，甚至出现如果没有一个对应字段的数据，根本不会在最终结果显示该 value 的单元格，和values配置不匹配。
• data数据处理和下钻处理数据方法不匹配：
  

当然可行的解决方法也有：

1. 可以再写一个工具方法来处理原始数据不存在某些字段的情况，但是我们的初衷是提高转换速度，这样只会加大难度💀。
2. 或者在生成rowPivot或者colPivot做补全处理，又得多if/else几层判断
3. 最简单的方法就是用回原来的转换流程，考虑到现在没几个用户在用，成本也低

相关 issues: #224, #292

各位怎么看这个问题？

[xingwanying]

我站提供工具函数外部转换。
理由如下：

1、我们最终的标准就是一个格子对应一个数据，dirllDown 只能说是去除 standData 没有挪干净。
2、增加一个 standData 的概念增加了用户的理解成本。
3、 我们不能预判用户的数据是怎样的，因为很有可能用户的默认数据就是 standData, 或者是其他的各种格式，即使用回原来也没办法保证用户不需要进行一次格式转换，我们没办法涵盖所有场景，之前纯粹是按照我们自己的场景来适配的。
4、把这层逻辑放在 core 层反而是增加了性能负担。

[wjgogogo]

是的，关键在于我们提供的工具函数也得说明其针对的场景，毕竟这个逻辑不管是放在外部还是内部，都没法处理所有情况。
首要是将数据的问题导致 values 没有完全显示的问题解了， 现在不是按照values的配置排序的

[xingwanying]

不是按照 values 的配置排序的? 具体指的是什么？可以描述详细一些嘛？

[wjgogogo]

举个🌰 :

1. values配置是 ['price', 'cost']
2. data为：

[
    {
        "province": "浙江",
        "city": "杭州",
        "type": "笔",
        "cost": "1"
    },
    {
        "province": "浙江",
        "city": "杭州",
        "type": "笔",
        "price": "2"
    }
]

3. 根据 standard 上面截图的处理来看，会生成：

[
    {
        "province": "浙江",
        "city": "杭州",
        "type": "笔",
        "cost": "1",
        "extra-field": "cost",
        "field-value": "1",
    },
    {
        "province": "浙江",
        "city": "杭州",
        "type": "笔",
        "price": "2",
        "extra-field": "price",
        "field-value": "2",

    }
]

4. 最终会按data顺序生成sortedDimensionValues时，导致 cost在 price之前 ：
   

   S2/packages/s2-core/src/utils/dataset/pivot-data-set.ts

   Lines 119 to 134 in cfc77b7

   	export function transformDimensionsValues(
   	record: DataType,
   	dimensions: string[],
   	sortedDimensionValues: Map<string, Set<string>>,
   	): string[] {
   	return map(dimensions, (dimension) => {
   	const dimensionValue = record[dimension];
   	if (!sortedDimensionValues.has(dimension)) {
   	sortedDimensionValues.set(dimension, new Set());
   	}
   	const values = sortedDimensionValues.get(dimension);
   	values.add(record[dimension]);
   	return dimensionValue;
   	});
   	}

[xingwanying]

ref: #528