原因是判斷用的 obj 的每個屬性都被計算了一次，可以加條件阻塞改進：

var convert = function(obj) {
  return obj.map(o => ({
    'number': o.type === 'number ' && 1,
    'string': o.type === 'string ' &&  's',
    'array': o.type === 'array ' &&  convert(o.children)
  }[o.type]))
}

當要判斷的條件少的時候可以用多個三目條件判斷，太多這樣的判斷，這種寫法要美觀一點，接受不了的可能只能寫 if else 了。

因為你的遞迴沒有終止條件

報錯是第一個的時候沒有children

var convert = function(obj) {
  return obj.map(o => ({
    'number': 1,
    'string': 's',
    'array': o.children?convert(o.children):""//假设没有的时候返回空咯
  }[o.type]))
}