JavaScript数组或对象检查-代码改进

Question

我接收数据=>这些数据可以是对象数组，也可以只是一个对象。我写了一些代码，但也许有一种方法可以让这个代码更性感、更清晰或更短，而且没有任何错误

这是代码：

export const CalculateIt = (props) => {
const conversionValues = []
if (props) {
if (props.length > 0) {
for (let i = 0; i < props.length; i++) {
const clicks = props[i]?.insights?.[0].inline_link_clicks
const actionsNumber = props[i]?.insights?.[0]?.actions?.length || 0
let result = 0
if (clicks && actionsNumber) {
result = devideNumbers(clicks, actionsNumber, 8)
}
conversionValues.push(result)
}
return conversionValues
}
const clicks = props?.insights?.[0].inline_link_clicks
const actionsNumber = props?.insights?.[0]?.actions?.length || 0
let result = 0
if (clicks && actionsNumber) {
result = devideNumbers(clicks, actionsNumber)
}
return conversionValues.push(result)
}
}

正如你在那里看到的，你可以找到代码的一些相似部分，比如：

const clicks = props[i]?.insights?.[0].inline_link_clicks
and 
const clicks = props?.insights?.[0].inline_link_clicks

有可能把它写得更聪明吗？

最佳

Answer 1

可能会移动函数中的公共代码：

function getResult(data) {
const clicks = data?.insights?.[0].inline_link_clicks
const actionsNumber = data?.insights?.[0]?.actions?.length || 0
let result = 0
if (clicks && actionsNumber) {
result = devideNumbers(clicks, actionsNumber, 8)
}
return result;
}

并使用原始函数中的辅助函数：

export const CalculateIt = (props) => {
const conversionValues = []
if (props) {
if (props.constructor === Array) {
props.forEach((item) => conversionValues.push(getResult(item)))
} else {
conversionValues.push(getResult(props));
}
return conversionValues;
}
}

Answer 2

事实上，您可以强制所有data进入一维数组，并且只能安全地使用对象数组。

这个代码够性感吗？

const obj = {id: 1, val: 1};
const arr = [{id: 1, val: 1},{id: 2, val: 2},{id: 3, val: 3}];
const normalize = (data) => [data].flat();
console.log(normalize(obj)[0]);
console.log(normalize(arr)[0]);
// -------------------
// and you can use this approach in code:
const getResult = (obj) => obj.val * 10;
const CalculateIt = (props) => [props].flat().map(getResult);
console.log(CalculateIt(obj));
console.log(CalculateIt(arr));

.as-console-wrapper{min-height: 100%!important; top: 0}