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

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

ただいまの
回答率

90.99%

  • JavaScript

    13868questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    1559questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • 関数型プログラミング

    25questions

    関数型プログラミングとは、関数を用いて演算子を構築し、算出し、コンピュータプログラムを構成する枠組みです。

どっちのコードが好ましいですか?その理由は何故ですか?

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 720
退会済みユーザー

退会済みユーザー

2パターンのコードを書いてみました。

1つ目は関数内部で外のスコープの関数を直接使っているパターンです。
2つ目は関数の引数に外のスコープの関数を値として渡して、引数で渡された関数を実行しているパターンです。

どっちが良いかというと、おそらく、2つ目の方なのではないかというのは感覚的に分かります!
だって2つ目のはDependency Injection(依存性の注入)みたいなものですもんね!!
(あまりDIもよく理解できていないので困ったものです)

ただ、仮に2つ目のほうが好ましいとした場合、なぜ、好ましいのか、どういったメリットがあるのか、1つ目だと何がまずいのか、どういうときに困るのかってのがよくイメージ出来ません。

どなたか解説いただけませんでしょうか?

 A

// 実際は他ファイルからimportしていると仮定して…
// import importedFunction from './somewhere'

const importedFunction = (arg) => {
    return arg + '!!!'
}

const returnSomething = () => {

    return importedFunction('something');
}

returnSomething()

 B

// 実際は他ファイルからimportしていると仮定して…
// import importedFunction from './somewhere'

const importedFunction = (arg) => {
    return arg + '!!!'
}

const returnSomething = (func) => {

    return func('something');
}

returnSomething(importedFunction)
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • mts10806

    2017/10/10 18:55

    ご自身ではどのようにお考えでしょうか?

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2017/10/10 18:59

    質問文に書いたとおり、ざっくりとしたところでは後者のほうが依存性が外から注入されているので、粗結合でテストもしやすく、再利用性も高まるのではないかと思います。という、それらしいことは言えるのですが、自分で言っててよくわかってません(苦笑)

    キャンセル

回答 1

checkベストアンサー

+4

AとBは目的に応じて使い分けるものであって、
どちらのほうが好ましいですか?という類のものじゃないね。

まずはAを選択すべき例を挙げてから、Bの使い方を見ていこうか。
例として台形の面積を求める関数を、三角形の面積のロジックを流用して作る事をやってみる。

// 引数と戻り値はマスキング
// 三角形の面積を求める関数
const triangle => (xxxxx) {  
  return yyyyy;
}
// 台形の面積を求める関数
const trapezoid => (xxxxx) {
  return yyyyy;
}

さて、これを使うコードはどちらが好ましいだろうか?

trapezoid(3, 4, 5);
trapezoid(3, 4, 5, triangle);

台形の面積は(上底 + 下底)* 高さ / 2と決まりきってるわけで、
必要な引数は「上底、下底、高さ」の3つ。
つまり台形の面積を出すのに、三角形の面積を導く公式なんて不要なんだよ。

従って前者と書くのが自然であり、後者はノイズが混じっている。
要するに簡単な事を難しくやってしまっている悪い例になってしまうわけだね。
結局、これが自然。

const triangle = (width, height) => {
  return width * height / 2;
}
const trapezoid = (upperBase, lowerBase, height) => {
  return triangle(upperBase, height) + triangle(lowerBase, height);
}

trapezoid(3, 4, 5);

Bのやり方を行う場合、ロボットの右手に剣をもたせるか、弓をもたせるか、銃をもたせるかを選ぶように、
ベースになる関数が沢山あってその中から選択する仕組みでなければならない。
カリー化や部分適用を利用して、関数の中身で同じように叩いても動作が変わるとかね

この辺は使い方も事情もDIと全く一緒。
オブジェクトになってコンストラクタメソッドの引数になるか、
カリー化でもされて第一引数の関数として挿入されるかの違いでしかない。


サンプルとしてカリー化や部分適用を組み合わせて色々やってみた。
まぁ、JSで凝った事がしたければRamda.jsは必須かな。

const R = require('ramda')
const triangle = R.curry((width, height) => width * height / 2)
const trapezoid = R.curry((fn, upperBase, lowerBase, height) =>
  R.pipe(
    R.append(upperBase),
    R.append(lowerBase),
    R.map(triangle(R.__, height)),
    R.sum
  )([])
)(triangle)

const trapezoid_at_height5 = trapezoid(R.__, R.__, 5)
trapezoid_at_height5(3, 4); // 17.5

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/11 09:09

    なるほど!ありがとうございます。用途が具体的な場合は関数を直接中で使えば良いし、汎用性をもたせたものであれば、外部から注入させて柔軟にするといった感じですかね!

    キャンセル

  • 2017/10/11 10:54

    前にご紹介頂いた緑表紙のJS関数型プログラミングの本、翻訳本ということで買ってから読むのを少し躊躇してましたが、読みやすくて分かりやすくて良いですね!!一日1章くらいのペースで読んでみます!

    キャンセル

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

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

関連した質問

  • 解決済

    外部jsの中で更に外部jsを読み込む方法

    簡単なテストとしてfunction test()をapp.jsの中で呼び出すのには、下記のような構成&scriptでは呼び出せないのですが 何処が間違っているのかがわかりません。ご

  • 解決済

    cakephp require_onceすると処理がされなくなる

    app/modules/test.php(自作フォルダ/ファイル) $nowdate=date("Y/m/d g:i:s"); WriteFinFile($fin_file,"S

  • 解決済

    UITableViewControllerとUITableViewDataSourceの関係について

    Swift初心者です。 現在UITableViewControllerを使ってテーブルビューを実装中です(正確には、UITableViewControllerを継承したクラス内に

  • 解決済

    apply()やcall()を使う機会を知りたい

    apply()、call()メソッドについて この2つのメソッドについては、使用している実際のコードを目にすることもあり、これまでにもネットで調べて自分でカスタマイズしつつ検証し

  • 解決済

    Reduxのaction周りと,ES6の記法がわかりません

    このページとこのページを参考にしてReduxを勉強をしています。 このページの著者のgithubはここにあります。 このサイトの中にこんなコードが有りました。 // s

  • 解決済

    入力された文字列の行数を返すJavascriptの関数

    実現したい関数 入力された文字列に対して、その行数を返す関数を作りたいのですが、 その際最小値と最大値を設定する必要があります。 function(text) {

  • 解決済

    Goのif文について

    まだGo初心者なので教えていただきたいです。 以下のメソッドでbool値を返したいのですが func main(){ result := result(messag

  • 解決済

    Listを引数とするメソッドでのエラー

    前提・実現したいこと javaでListを引数とし、Listの中身があるか無いかをチェックするメソッドを作りたかった のですが、メインクラスで実装しようとすると エラーが発生し躓い

同じタグがついた質問を見る

  • JavaScript

    13868questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    1559questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • 関数型プログラミング

    25questions

    関数型プログラミングとは、関数を用いて演算子を構築し、算出し、コンピュータプログラムを構成する枠組みです。