Как проверить на javascript существование свойств у динамического объекта и создать их в случае отсутствия?

Question

[Mike]

Друзья, пытаюсь на ноде сделать парсинг данных и построить дерево. В dataChapter прилетают строки вида Сайт1\Рубрика1\Заголовок1, надо из этих строк построить дерево вида:

root = {
  name: 'Сайт1',
  children: [{
    name: 'Рубрика1',
    children: [{
      name: 'Заголовок1'
    }]
  }]
}

Получается мне надо на каждой итерации проверять существует ли name на текущем уровне, если не существует, то добавлять его и создавать children для следующей итерации. Привожу кусок кода, на который ругается консоль:

...
var chapters = dataChapter.split('\\');
var root = {}
var tree_pointer = 'root';
for(var j=0, jlen = chapters.length; j < jlen; j++){
  if(!tree_pointer.hasOwnProperty("name")) {
    tree_pointer['name'] = chapters[j];
  }
  tree_pointer += '.children';
}
...

Подскажите как решить эту проблему? Чувствую что скорее всего не хватает eval()...

Answer 1

[alex]

function makeTree(data) {
	return data.reduceRight((acc, cur) => {
		return {
			name: cur,
			children:[acc]
		};
	}, { name: data.pop() });
}

spoiler

let dataChapter = 'a/bb/ccc'.split('/');
let root = makeTree(dataChapter);
console.log(JSON.stringify(root, null, 2));
// {
//   "name": "a",
//   "children": [
//     {
//       "name": "bb",
//       "children": [
//         {
//           "name": "ccc"
//         }
//       ]
//     }
//   ]
// }

Comments

[Mike]

Alex спасибо, для локальной задачи работает, но вот попробовал встроить в проект и столнулся с проблемой, там получается что на каждой итерации содержимое затирается, вот привожу полный код:

var regexp_chapter = /<meta chapter="(.*?)" order="([0-9]+)">/i
var root = {};
fs.readdir(dataFolder, (err, fileNames) => {
  for (var i = 0, len = fileNames.length; i < len; i++) {
    if(path.extname(fileNames[i]) === '.html'){
      fs.readFile(dataFolder+fileNames[i], (err, data) => {
        var dataChapter = regexp_chapter.exec(data)[1];
        //ToDo add sort by order
        if(dataChapter.indexOf('\\') !== -1){
          let chapters = dataChapter.split('\\');
          let root = makeTree(chapters);
        }
      })
    }
  }
})

[alex]

dillix: root и будет затираться, у вас же цикл. сделайте массив например:

var root = [];
...
if(dataChapter.indexOf('\\') !== -1){
  var chapters = dataChapter.split('\\');
  root.push( makeTree(chapters) );
}

spoiler

просто вы же сами просили в вопросе сделать из этого:
Сайт1\Рубрика1\Заголовок1
это:

root = {
  name: 'Сайт1',
  children: [{
    name: 'Рубрика1',
    children: [{
      name: 'Заголовок1'
    }]
  }]
}

makeTree и делает эту конвертацию

Answer 2

[Николай]

var tree_pointer = 'root'; - tree_pointer - это строка.
После этого вы от строки пытаетесь сделать hasOwnProperty. Очевидно, что любая консоль вам скажет о том, что такого метода нет.
Чего вы пытаетесь добиться конкатенацией строк - остается загадкой.

Comments

[Mike]

Николай, вот что пытаюсь сделать:

мне надо на каждой итерации проверять существует ли name на текущем уровне, если не существует, то добавлять его и создавать children для следующей итерации.

Answer 3

[Евгений Терещенко]

var chapters = dataChapter.split('\\');
var root = {}
var tree_pointer = {};  // зачем тут была строка?
for(var j=0, jlen = chapters.length; j < jlen; j++){
  if(!tree_pointer.hasOwnProperty("name")) {
    tree_pointer['name'] = chapters[j];
  }
  tree_pointer['children'] = {};  // зачем тут была строка?
}

Comments

[Mike]

Jman, у меня данный код лежит внутри другого цикла, кроме строки var root = {}. В вашем коде не хватает проверки связанной с root и глубина на каждой итерации цикла должна увеличиваться для этого использовал строки, но как их конвертировать в вызов объекта и его дочерних свойств я не разобрался...