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 强制放入一个 one-dimensional 数组中，并且只使用对象数组来安全工作。

这段代码够性感吗？

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}

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

JavaScript array or object checking - code improvements

javascript

arrays

if-statement

reactjs