Как зарефакторить код?

Question

[mmind]

// renameFields items->children
    const renameFields = (obj)=> {
        let newObj = {}
        newObj.name = obj.title
        newObj.url = obj.url
        newObj.children = obj.items
        return newObj
    }

    // transform Data
    const transformData = (obj) => {
        if(obj.items === null) {
            obj = renameFields(obj)
            obj.children = []
        } else if(obj.items) {
            obj.items = obj.items.map(item => transformData(item))
            obj = renameFields(obj)
        } else {
            obj = renameFields(obj)
        }
        return obj
    }

Такое ощущение, что можно так-то лучше написать

Answer 1

[WbICHA]

/** renameFields items->children **/
const renameFields = (obj) => ({
  name: obj.title,
  url: obj.url,
  children: obj.items,
});

/** transform Data **/
const transformData = (data) => {
  const renamedData = renameFields(data);
  
  if(data.items) {
    renamedData.children = renamedData.children.map((child) => renamedData(child));
    return renamedData;
  }
  
  if(data.items === null) {
    renamedData.children = [];
    return renamedData;
  }
  
  return renamedData;
}

Comments

[WbICHA]

Единственное я не понял, почему только если чилдрен нулл делать его массивом, а если андефайнд?

[mmind]

WbICHA, на практике бывает только null

[WbICHA]

mmind, тогда так правильнее:

/** transform Data **/
const transformData = (data) => {
  const renamedData = renameFields(data);
  
  if(renamedData.children) {
    renamedData.children = renamedData.children.map((child) => renamedData(child));
    return renamedData;
  }
  
  renamedData.children = [];
  return renamedData;
}

[WbICHA]

mmind, или вообще можно так:

/** transform Data **/
const transformData = (data) => {
  const renamedData = renameFields(data);
  
  renamedData.children = renamedData.children?.map((child) => renamedData(child)) || [];
  
  return renamedData;
}

Но это по последней спецификации, альтернатива:

/** transform Data **/
const transformData = (data) => {
  const renamedData = renameFields(data);
  
  renamedData.children = renamedData.children ? 
    renamedData.children.map((child) => renamedData(child)) : 
    [];
  
  return renamedData;
}