javascript 関数の副作用について

Question

クリックしたら、今日の日付を付けたファイルを作成したいとします。
その時に、イベントハンドラ(関数)の内部にmoment()で日付を生成するのはよくないでしょうか？
以下はreact×typescriptのコードです(ファイル作成のコードではなく単に日付を表示する内容)

function Timer: FC = () => {

  const handleClick = () => {
    alert(`${moment().format("YYYYMMDD")}`)
  }
  
  return (
    <button onClick={handleClick}>
      Click
    </button>
  );
}

function App {
  ReactDOM.render(
    <Timer />,
    document.getElementById('root')
  );
}

そんなに複雑でない(テストするまでもない)関数なら内部で値が変わるようなものを持たせて良いでしょうか？

Comments

[maisumakun]

どういう観点で「よくない」かもしれないと考えているのでしょうか？

[yochun02]

一般に関数内部で値が変わってしまうようなプログラムは関数型プログラミングにおいては忌み嫌われているものと理解しております。
つまりある一定の値(hoge)を入力したときに必ず意図したものが返ってくる(piyo)
function(hoge) => piyo
の作法を守ることで予期せぬバグを避けられるからだと。
その意味においてこのような書き方はよくないのかなと思うのですが、今回の場合はクリックしたときのハンドル関数ですし、一々テストするようなものでもないので
「クリックしたら今日付けの日時も一緒に合わせて何かする」程度ならこの実装でも良いのかな？と思ったのです。

Answer 1

一般に関数内部で値が変わってしまうようなプログラムは関数型プログラミングにおいては忌み嫌われているものと理解しております。

イベントハンドラは、本質的に副作用のために実行される関数ですので、「関数型プログラミング」の考え方を適用するのが適切ではない、と考えます。

Comments

[maisumakun]

Purescriptのように「外部の環境も引数・返り値の1つとして考える」ような道具立てをしたのなら話は別ですが、そうでないならイベントハンドラから外部に変化を起こすには副作用を使うより他に手段がありません。

[yochun02]

> イベントハンドラは、本質的に副作用のために実行される関数

なるほど、そうなんですね。
解答ありがとうございました。

Answer 2

質問者様

moment()で日付を生成するのはよくないでしょうか？

「moment.jsを使った処理「moment()」を使って生成するのはよくないのでしょうか？」
という意味としてお答えします。

長所

• 日時の変換等共通化されていて簡単に行える

短所

• 拡張機能にバグやバージョンアップ、別の拡張機能と重なったときにきちんと管理できるか？

ですね日時関連は「ロケール」問題がかならずあります。
他の国の方が見られることも想定するなら「moment.js」もありかと思います。
また日時を様々な箇所で使うのであれば問題ないかとも思います。

ただ1箇所だけ日付を取得する為だけに導入するのは「長所 < 短所」となるように
個人的には思います。

Comments

[yochun02]

momentの扱いについて教えていただきありがとうございます。