ＪＳの冗長な処理を簡易にしたい

Question

###実現したいこと
対象の配列から
【Ａ】idの配列
【Ｂ】ソートされたオブジェクトの配列
を取得したいです。

いちおう処理はできたのですが、いかんせん素人レベルのコピペだらけで、余計な処理があるように思います。

それが最後にあるソースコードですが、もっとスッキリさせた書き方にできないかと思っています。
もしアドバイスいただけましたら幸いです。
###対象の配列
まずこれが対象の配列です。ここから【Ａ】【Ｂ】を取得したいわけです。

const iniArray = [
  {id: '1', x_num: '100', y_num: null, z_num: null },
  {id: '2', x_num: null, y_num: '400', z_num: null },
  {id: '3', x_num: null, y_num: null, z_num: '300' },
  {id: '4', x_num: null, y_num: '200', z_num: null }
];

###【Ａ】
対象の配列から extKeysの指定値 が null でないオブジェクトの id の配列を取得するのが【Ａ】です。

たとえば extKeys = [ 'x_num', 'y_num' ] を指定した場合ならば、これらがnullである3を除外して、次の配列がほしいということです。

[ '1', '2', '4' ] 

###【Ｂ】
対象の配列から extKeys が null でないオブジェクトを、「●_num」でソートして取得するのが【Ｂ】です。

先の【Ａ】で取得したidを持つオブジェクトを「●_num」でソートし、次のオブジェクトの配列がほしいということです。

[
  {id: '1', x_num: '100', y_num: null, z_num: null },
  {id: '4', x_num: null, y_num: '200', z_num: null }, 
  {id: '2', x_num: null, y_num: '400', z_num: null }
];

###素人ながら試したこと
以下があれこれ試行錯誤してできたコードになります。

1、2、3、4という処理ですが、ほしいのは【Ａ】【Ｂ】の情報（下記なら1、3の情報）だけなので、もっとスッキリさせた書き方にできないかと思っています。

ご指導ご鞭撻のほど、よろしくお願い申し上げます。

/*
対象の配列
*/
const iniArray = [
  {id: '1', x_num: '100', y_num: null, z_num: null },
  {id: '2', x_num: null, y_num: '400', z_num: null },
  {id: '3', x_num: null, y_num: null, z_num: '300' },
  {id: '4', x_num: null, y_num: '200', z_num: null }
];

/*
1. extKeysがnullじゃなければidを検索
*/
function getExtIds( iniArray, extKeys ){
  let array = [];
  for (let i=0; i < iniArray.length; i++) {
    for (let j=0; j < extKeys.length; j++) {
      if( iniArray[i][extKeys[j]] !== null ){
        const id = iniArray[i]["id"];
        array.push(id);
      }
    }
  }
  // extKeysが [ 'x_num', 'y_num' ] なので、[ '1', '2', '4' ] が取得される
  console.log('---検索されたidの配列---');
  console.log(array);
  return array;
}

/*
2. 検索したidを持つオブジェクトを抽出する
*/
function getExtObj( iniArray, extIds ){
  let array = [];
  for (let i=0; i < iniArray.length; i++) {
    for (let j=0; j < extIds.length; j++) {
      if( iniArray[i]["id"] == extIds[j] ){
        const extObj = iniArray.find( e=>e.id && e.id === extIds[j] );
        array.push(extObj);
      }
    }
  }
  console.log('---抽出されたオブジェクト---');
  console.log(array);
  return array;
}

/*
3. 抽出されたオブジェクトを「●_num」で並び替える
*/
function getSortObj( extObj ) {
	return extObj.sort((a,b) => (a.x_num || a.y_num || a.z_num) - (b.x_num || b.y_num || b.z_num));
}

/*
4. 処理を実行
*/
const extKeys = [ 'x_num', 'y_num' ];
const extIds  = getExtIds( iniArray, extKeys );
const extObj  = getExtObj( iniArray, extIds );
const result  = getSortObj( extObj );
console.log('---「●_num」でソートされたオブジェクト---');
console.log( result );

Answer 1

まずA

const iniArray = [
  {id: '1', x_num: '100', y_num: null, z_num: null },
  {id: '2', x_num: null, y_num: '400', z_num: null },
  {id: '3', x_num: null, y_num: null, z_num: '300' },
  {id: '4', x_num: null, y_num: '200', z_num: null }
];
const extKeys = [ 'x_num', 'y_num' ]
const newArray=iniArray.filter(x=>!extKeys.map(y=>x[y]).every(z=>z==null)).map(x=>x.id);
console.log(newArray);

つづいてB

const newArray2=iniArray
.filter(x=>!extKeys.map(y=>x[y]).every(z=>z==null))
.map(x=>{
  x.sort=parseInt(x[Object.keys(x).filter(y=>y.match(/._num$/) && x[y]!=null)[0]]);
  return x;
}).sort((x,y)=>x.sort-y.sort).map(x=>{
  delete(x.sort);
  return x;
});
console.log(newArray2);

Comments

[kuroean]

引き続きどうもありがとうございます。目玉が飛び出るくらいスッキリしましたね…！自分のコードと比べて思わず笑ってしまいましたｗ

Answer 2

「extKeys が null でないオブジェクトだけピックアップした配列を作る」という関数を作っておけば、両方で使い回せます。

/*
対象の配列
*/
const iniArray = [
  {id: '1', x_num: '100', y_num: null, z_num: null },
  {id: '2', x_num: null, y_num: '400', z_num: null },
  {id: '3', x_num: null, y_num: null, z_num: '300' },
  {id: '4', x_num: null, y_num: '200', z_num: null }
];

// extkeysでのフィルタリング
function filtered(arr, extKeys) {
  return arr.filter(function(item){
    return extKeys.some(function(key){
      return item[key] !== null;
    });
  });
}

// idだけ抽出
function pickIds(arr, extKeys) {
  return filtered(arr, extKeys).map(function(item){
    return item.id;
  });
}



//並べ替えた配列
function sorted(arr, extKeys) {
  // 補助関数
  function pickValue(item) {
    for(var i = 0; i < extKeys.length; ++i) {
      var val = item[extKeys[i]];
      if(val) return val;
    }
  }

  // filteredで別な配列が生成するので、そのままsortに投げて問題なし
  return filtered(arr, extKeys).sort(function(a, b) {
    const aValue = pickValue(a);
    const bValue = pickValue(b);
    if(aValue < bValue) return -1;
    if(aValue > bValue) return 1;
    return 0;
  });
}

Comments

[kuroean]

そのご発想ステキです。かなり使えそうです。どうもありがとうございます。わたしは質問のコードを書くのに２日かけたのですが(笑)、みなさん１０分やそこらで出来ちゃうんですね…。

[maisumakun]

Array.prototypeには色々メソッドがありますので、これらをどれだけ使いこなせるかが配列処理の肝になってきます。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/prototype#Methods

[kuroean]

とても便利なものがたくさんありますね。これは知っておいてよかったです。情報感謝申し上げます。