関数型JavaScriptのFizzBazz問題の最適化

Question

判定するdecision関数を最適化したいです。
いい方法はないですか？

const min = 1
const max = 100

//計算式
let remainder = function(array,f) {
  return array.map(f)
}

//判定
let decision = function(s){

  if (s % 15 == 0) {
    return "FizzBuzz"
  } else if (s % 3 == 0) {
    return "Fizz"
  } else if (s % 5 == 0) {
    return "Buzz"
  } else {
    return s
  }
}

//数値の生成
let renge = function(min,max) {
  return [...Array(max - min + 1).keys()].map(x => x + min)
  
}

//物質化
let answer = remainder(renge(min,max),decision)

//出力
console.log(answer)

Answer 1

  if (s % 15 === 0) return "FizzBuzz";
  if (s % 3 === 0) return "Fizz";
  if (s % 5 === 0) return "Buzz";
  return s

Answer 2

なにをもって最適化なのですか？
速度？可読性？コードの量？

もし速度なら質問文のコードより速くする手段はありません。
精々JITコンパイラに働きかける小細工を施すくらいです。

可読性ならte2jiさんのコードかswitchの2択ですが、
さほど質問文のコードから変わったように思えません。

最後のコード量を減らす案ですが、FizzとBuzzの文字列を別々に作成するテクニックがあります。
手慰みに作った私の記事があるのでそちらを共有します。
FizzBuzz関数の選択肢 - ES2015でFizzBuzz

Comments

[退会済みユーザー]

オレの回答主旨は `===` だよん。

[miyabi-sun]

他の場面で変なバグ出さない為にNull比較以外は全部`===`にする癖を付けた方が良いというのは同意見です。
今回のケース限定で言えば`s % 15`が先に入るので確実にNumber型にキャストされます。
「遊戯の奴…そこまで考えて…」いたのであれば`==`でも別に良いんじゃないかな？と思って別に指摘しませんでした。

Answer 3

何をもって「最適化」というのかにもよりますが、こんな方法もあります。

const remainderTable = [
  'FizzBuzz', null, null, 'Fizz', null, 'Buzz', 'Fizz', null, null,
  'Fizz', 'Buzz', null, 'Fizz' // 13・14はundefinedでいいので略
];

const decision = s => remainderTable[s % 15] || s;

Comments

[HiiragYukina]

返答ありがとうございます。
最適化したいのは処理速度と判定( if )の数を減らせないかです。

Answer 4

もうクローズしてるけど、楽しそうなので。

let decision = function(x) {
  const m3 = !(x % 3);
  const m5 = !(x % 5);
  return !(m3 + m5) * x || 'FizzBuzz'.slice(!m3 * 4, m5 * 4 + 4);
}

Comments

[Lhankor_Mhy]

miyabi-sun さんのリンクをあとから発見してしょんぼりした。
でもほら、三項演算子使ってないし？　短絡評価使って文字列生成を減らしてるし？

Answer 5

たとえばこう

var min = 1
var max = 100

var answer=Array(max-min+1).fill(null).map(function(x,y){
  var s=y+min;
  return s%15==0?"FizzBuzz":s%3==0?"Fizz":s%5==0?"Buzz":s;
  //return s%15==0?"FizzBuzz":(s%3==0?"Fizz":(s%5==0?"Buzz":s));
});
console.log(answer)

「三項演算子はネストするな！」の原則からいうとコメントアウトした
論理節でわける方がよいかもしれません