オブジェクトのソート

Question

オブジェクトを、➀grpIdをまず降順にすると、grpIdの２の３つが上に行き、１の２つが下にいく。
その後、➁grpId２の中でjobNameの昇順に、grpId１の中でもjobNameの昇順にという実装をしたいです。
参考サイトなどを見て、同じコードを書いたのに、➀ができません。
ご教授お願いいたします。

Answer 1

ソートをするのがdate ではなくdate内のjobsだからでしょう

let date = [{
  "jobs": [
    { "jobId" : 5,
      "jobName" : "Dentist",
      "grpId" : 2},
    { "jobId" : 2,
      "jobName" : "Firefighter",
      "grpId" : 1},
    { "jobId" : 3,
      "jobName" : "Doctor",
      "grpId" : 2},
    { "jobId" : 1,
      "jobName" : "Police",
      "grpId" : 1},
    { "jobId" : 4,
      "jobName" : "Nurse",
      "grpId" : 2},
    ]
}];

let result=[{jobs:null}];
result[0].jobs = date[0].jobs.sort(function(a,b){
  return a.grpId === b.grpId ?a.jobId-b.jobId:b.grpId-a.grpId;
});
console.log(result);

grpId降順→jobName照準

let result=[{jobs:null}];
result[0].jobs = date[0].jobs.sort(function(a,b){
  return a.grpId === b.grpId ?(a.jobName>b.jobName?1:-1):b.grpId-a.grpId;
});
console.log(result);

考え方

sort処理は2つの値を比較してプラスなら前にマイナスなら後ろに入れ替わります
2次ソートは1次ソートが同じ値だった場合に処理をします冗長にかくとこうなります。

const a = [
  [0,2,2],
  [1,3,1],
  [2,3,2],
  [3,1,3],
  [4,2,3],
  [5,2,1],
  [6,3,3],
  [7,1,1],
  [8,1,2],
];
const b=a.sort((x,y)=>{
  let ret=0;
  if(x[1]==y[1]){
    if(x[2]>y[2]){
      ret=1;
    }else{
      ret=-1;
    }
  }else if(x[1]>y[1]){
    ret=1;
  }else{
    ret=-1;
  }
  return ret;
})
console.log(b);

ただし数値の場合
もしx[1]>y[1]なら1そうでないなら-1という処理をプラスマイナスだけの判断でいいので
x[1]-y[1]でできてしまいます

const b=a.sort((x,y)=>x[1]==y[1]?x[2]-y[2]:x[1]-y[1]);

Comments

[退会済みユーザー]

すみません、この実装でできたのですが、理解が乏しく、コードの説明を頂けたら嬉しいです！
そのコードの場合、➁をするときはどのようにすればいいのでしょうか？
教えていただけると幸いです！

[yambejp]

(2)も対応していますよね？なにか挙動がおかしいですか？

[退会済みユーザー]

grpIdは２の３つが上、下に１が２つきているのですが、
jobNameは、
上から、"Doctor"、"Nurse"、"Dentist"、"Police"、"Firefighter"の順番になっています。

[yambejp]

失礼しましたjobIdじゃなくてjobName順ですね
数値の比較ではない場合は比較演算子をつかって三項演算子でプラスマイナスを明示的に返します
追記分を確認してみてください

[退会済みユーザー]

いえいえ、こちらこそすみません汗
お手数おかけいたします汗

[退会済みユーザー]

すみません、ありがとうございます！
差し出がましいのですが、理解を深めたいので、ざっくりでもいいので、コードの説明を頂けたら嬉しいです！

[yambejp]

ソートはコールバック関数の感覚がわかりづらいですよね
プラスを返すかマイナスを返すか意識するとなんとなくわかるかもしれません

[退会済みユーザー]

何から何までありがとうございます！
let result=[{jobs:null}];でjobｓをなかったことにして、result[0]は配列が１つだから[0]というのはわかるのですが、その後がいまいちわかりませんでした汗
もう少し考えてみます！

[退会済みユーザー]

なるほど、すごくわかりやすいです！！
ご丁寧に例題までつけてくださってありがとうございます！！
無事に実装もできましたので、ベストアンサーにさせて頂きます！
本当にありがとうございました！

Answer 2

42であなたの問題は解決します。googleで検索したので間違いないです。
参考：The Answer to the Ultimate Question of Life, the Universe, and Everything is 42

Answer 3

dateをソートしようとしていますが、dateの要素は{jobs: ...}の1つだけです。
おそらく、date[0].jobsをソートしたいのではないでしょうか

Comments

[退会済みユーザー]

私もjobsの扱い方がダメなのかなと思ってはいたのですが、
どう扱えばいいのかわからず、上記のようなやり方にするやり方があるのですね！
参考にさせていただきます！

Answer 4

「参考サイト」とやらが間違っているのかもしれませんね。
sort() に指定する関数は、負の数値・0・正の数値の3種類の値を返す必要があります。真偽値ではありません。
➀に関しては、return b.grpId - a.grpId とすればよいです。

a.grpId と b.grpId が等しいときは jobName の比較をすれば➁もできます。

Comments

[退会済みユーザー]

そうなのですね！
すみません、
let result = date.sort(function(a,b){
return b.grpId - a.grpId;
});
console.log(result);
でやって見たのですが、順番が変わりませんでした汗

[int32_t]

あとは他の回答で指摘されているとおり、ソート対象が違うことですかね。
date[0].jobs.sort(function(a, b) { ... })

[退会済みユーザー]

ですよね、、汗
yambejpさんのコードで解決はしたのですが、コードの意味がちゃんと理解できてなくて、
まだまだだなと痛感しました汗
迅速にご回答いただきありがとうございました！