質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.54%

javascriptのsort ダブルreturnについての疑問

受付中

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,085

garchomp

score 124

codeprepの配列の練習問題の回答例に関する質問です。

現在配列の文字列のソートをやっていますが、皆さんはどのようにコードを書くでしょうか?

また、回答例では、以下のようになっていました。

<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<script src="https://code.jquery.com/jquery-3.1.1.min.js"></script>
<script src="index.js"></script>
<script>
function array2str(array) {
  return "[" + array.join(", ") + "]";
}
$(document).ready(function() {
  $("#a1").text(array2str(sort(["ccc", "dddd", "a", "bb", "eeeee"])));
  $("#a2").text(array2str(sort(["apple", "banana", "chicken", "doughnut", "egg"])));
});
</script>
</head>
<body>
<h1>ソート</h1>
<p>配列 ["ccc", "dddd", "a", "bb", "eeeee"]のソート結果は</p>
<p id="a1"/><p>配列 ["apple", "banana", "chicken", "doughnut", "egg"]のソート結果は</p>
<p id="a2"/>
</body>
</html>
function sort(array){
  return array.sort((a,b)=>{ return a.length > b.length ? -1 : 1} );
}

ここで見る限り、function sortの中に実際の処理を書いていくわけですが、最初のreturnの中にaray.sort((a,b)=>{return...
といった感じでなんとreturnが連続して続いているのですが、これは関数の中に関数が入っているという解釈でよろしいのでしょうか?

また、これはいわゆるクローシャというものと関係しているのでしょうか?

追記:問題文追加し忘れて板の失礼いたしました。

こちらです。

配列を文字列の長さの長い順でソートする関数sortを完成させてください。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • think49

    2017/05/01 15:46

    そもそも、問題文が分からないのですが…。模範解答から問題文を読み取れ、という事でしょうか。

    キャンセル

  • garchomp

    2017/05/01 16:15

    失礼いたしました。問題文追加しておきました!

    キャンセル

回答 3

+3

問題文が不明ですが、「回答例」から読み取る限りでは「String 値の要素からなる配列を文字列の長い順にソートせよ」というところでしょうか。
私なら次のように書きます。

function lengthSort (a, b) {
  return String(b).length - String(a).length;
}

console.log(JSON.stringify(["ccc", "dddd", "a", "bb", "eeeee"].sort(lengthSort)));               // ["eeeee","dddd","ccc","bb","a"]
console.log(JSON.stringify(["apple", "banana", "chicken", "doughnut", "egg"].sort(lengthSort))); // ["doughnut","chicken","banana","apple","egg"]

length は String 型の保証とならない(配列も存在しうる)為、String 型に変換し、JSON.stringify でシリアライズします。

 アロー関数

アロー関数/関数宣言/関数式が混在していて、コード全体として統一感のないコードになっている印象を受けます。
sort のコードだけ別の人が書いたコードなのでしょうか。
アロー関数を使うのなら、$(document).ready(function() {}); もアロー関数で書いた方がすっきりするのではないかと思います。

 クロージャ

関数の中に関数があればクロージャといえますが、「array.sort() 内のアロー関数」は単純なコールバック関数なので、クロージャならではの使い方ではないように思います。
function sort (){} 内の return は Array.prototype.sort の返り値を返しているだけです。

function sort (array) {
  return array.sort();
}

上記コードにコールバック関数が付与されたところで、コードが表す意味は変わりません。

 array2str() のシリアライズ

本題から逸れるのでしょうが、array2str() の設計がどうにも気になります。
配列を「配列初期化子」となる文字列に変換する事が期待されますが、配列の要素に , が存在するだけで出力値が不正になります。

function array2str(array) {
  return "[" + array.join(", ") + "]";
}

var array = ['a, bb, ccc, dddd, eeeee', 'ffffff'];
console.log(array2str(array));       // [a, bb, ccc, dddd, eeeee, ffffff]
console.log(JSON.stringify(array));  // ["a, bb, ccc, dddd, eeeee","ffffff"]

Re: garchomp さん

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/01 16:14

    なるほど。変換するというのも大事なんですね・・・ありがとうございます!

    キャンセル

+1

皆さんはどのようにコードを書くでしょうか?

function sort(array){
  return array.sort( (a,b)=> b.length - a.length );
}

関数の中に関数が入っているという解釈でよろしいのでしょうか?

だいたいそんな理解であってますが、「高階関数」という言葉を覚えておくといいかと思います。

高階関数(こうかいかんすう、英: higher-order function)とは、関数(手続き)を引数にしたり、あるいは関数(手続き)を戻り値とするような関数のことである。
高階関数 - Wikipedia

引数 

compareFunction 
ソート順を定義する関数を指定します。省略された場合、配列は各要素の文字列比較に基づき辞書順にソートされます。
Array.prototype.sort() - JavaScript | MDN

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/01 16:15

    高階関数ですね。覚えておきます!

    キャンセル

0

関数の中に関数が入っているという解釈

はい。そのとおりです。

クローシャというものと関係しているのでしょうか?

あります。Closure クロージャですね。
Javascript の無名関数はクロージャそのものです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/01 16:14

    そうなんですね!ありがとうございます.

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.54%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる