連想配列の並べ変えについて教えてください。

Question

連想配列の要素を小さい順に並べ変えたいと思いますが、
以下のコードで何が問題でしょうか？
（超初心者です）

'use strict';

{ 
  let ar =[
    {id:1,name:'morita'},
    {id:2,name:'kenji'},
    {id:4,name:'uro'},
    {id:3,name:'ken'}
    ];

    ar.sort(function(a,b) {
      if (a.id > b.id ) {
        return 1}
      else {return -1};

      console.log(ar);
    })
}

コンソールには何も表示されません

以上　よろしくお願いいたします。

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

また、そもそもこのコードはどのように実行されているのでしょうか。

[m.ts10806]

>（超初心者です）
「初心者アイコン」を質問につけられるので、そちらでご対応ください。質問本文やタイトルにわざわざ書く必要はありません。

[m.ts10806]

初心者アイコン・・・

Answer 1

console.log(ar) が絶対に実行されない場所にあるからかと思います。

ar.sort(function(a,b) {
  if (a.id > b.id ) {
    // ここで関数を抜けるか
    return 1
  } else {
    // ここで関数を抜ける
    return -1
  };
  // 絶対に上記二つのうちのいずれかの return が実行されているためここには到達しない 
  console.log(ar);
})

Comments

[kinsncn]

ありがとうございました。 consoleの場所を移動したらすぐ直りました。

[think49]

To: ＠kinsncnさん
修正依頼の@m.ts10806さんの指摘が無視されていますが、正当な指摘だと思います。

[think49]

To: ＠kinsncnさん
念のため、追記しますが、私がいっているのは「回答」ではなくて「修正依頼」の方です。

[kinsncn]

ただいま、修正したつもりですが、確認お願いします。

[think49]

> ただいま、修正したつもりですが、確認お願いします。

markdownは良いですが、初心者アイコンが付いていないようですね。
指摘した@m.ts10806さんに無言で返すのは、礼を欠いていると私は思います。

Answer 2

console.log(ar) の位置がおかしいです。

"use strict";

{
    let ar = [
        { id: 1, name: "morita" },
        { id: 2, name: "kenji" },
        { id: 4, name: "uro" },
        { id: 3, name: "ken" },
    ];

    ar.sort(function (a, b) {
        if (a.id > b.id) {
            return 1;
        } else {
            return -1;
        }

        console.log(ar); //これar.sortの外側に記述するのでは？
    });
}

Comments

[kinsncn]

ありがとうございました。　ご指摘通りです。

Answer 3

console.log(ar);
がar.sort()の中の通らない場所で実行されてるからでは。
コードをきちんとインデント付ければ明らかです。

'use strict';

{
	let ar = [{
		id: 1,
		name: 'morita'
	}, {
		id: 2,
		name: 'kenji'
	}, {
		id: 4,
		name: 'uro'
	}, {
		id: 3,
		name: 'ken'
	}];

	ar.sort(function(a, b) {
		if (a.id > b.id) {
			return 1
		} else {
			return -1
		};
//必ずreturnされるので通らない
		console.log(ar);
	})
}

「実行後」じゃないときちんとソートされたものにならないので、外に出してください。

Comments

[kinsncn]

ありがとうございました。ご指摘どおりでした。