Javascript if else 文を短縮し処理を軽くしたい

Question

現在、javascriptの学習で　スプレットシートからタイムを取得し、条件付きで出力をする処理を行っております。

条件として、(例)A列に〇〇〇〇年〇〇月〇〇日を格納し　
B列に各日付のベストタイムを格納しかつ配列にし年、月、日が合致したら、そのタイムの各月で最もタイムが早いものを各月のベストラップとして配列から取り出し、スプレットシートのセルに書き込むようにしております。

そこで上記に記載したような処理をif else文で作成をしたのですが、コードが膨大になってしまい
処理が重くなってしまってしまいました。

下記が今回のコードになります。

 if (2021 == y && 1 == m) {
                inputSheet.getRange(2, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 2 == m) {
                inputSheet.getRange(3, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 3 == m) {
                inputSheet.getRange(4, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 4 == m) {
                inputSheet.getRange(5, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 5 == m) {
                inputSheet.getRange(6, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 6 == m) {
                inputSheet.getRange(7, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 7 == m) {
                inputSheet.getRange(8, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 8 == m) {
                inputSheet.getRange(9, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 9 == m) {
                inputSheet.getRange(10, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 10 == m) {
                inputSheet.getRange(11, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 11 == m) {
                inputSheet.getRange(12, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2021 == y && 12 == m) {
                inputSheet.getRange(13, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 1 == m) {
                inputSheet.getRange(18, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 2 == m) {
                inputSheet.getRange(19, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 3 == m) {
                inputSheet.getRange(20, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 4 == m) {
                inputSheet.getRange(21, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 5 == m) {
                inputSheet.getRange(22, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 6 == m) {
                inputSheet.getRange(23, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 7 == m) {
                inputSheet.getRange(24, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 8 == m) {
                inputSheet.getRange(25, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 9 == m) {
                inputSheet.getRange(26, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 10 == m) {
                inputSheet.getRange(27, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 11 == m) {
                inputSheet.getRange(28, 8).setValue(Math.min.apply(null, maleTimeArray));
            } else if (2022 == y && 12 == m) {
                inputSheet.getRange(29, 8).setValue(Math.min.apply(null, maleTimeArray));
            }

※ yは年　mは月です。
if (2021 == y && 1 == m)は2021年がy(年と同じ)かつ1月がm(月と同じ)だったら

私はまだjavascript初学者でどのようにしたらもっと処理を軽くし管理がしやすくなるか
お時間のある方がいらっしゃいましたら、ご助言を頂ければ幸いです。

何卒宜しくお願い致します。

Comments

[m.ts10806]

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

[rate_o]

承知いたしました。
ただいま修正させていただきました。
申し訳ございませんでした。

[gentaro]

言葉に気をつけたほうが良いと思うが…「処理が重い」ってのは計測した結果で判断してる？
正直体感できるほど「重い」と感じる事はないと思う。
単にコードが冗長、複雑になりすぎた、というだけの話じゃない？

[rate_o]

まさにその通りだと思います。
言葉選びが適切でなかったこと誠に申し訳ございません。

Answer 1

計算で求まるんだったら y_waiwaiさんのようにするのがいいと思いますが
もしそうでない場合はこんな方法はどうでしょうか?

function getRangeVal(y, m) {
    const data = {
        '202101':2,  '202102':3,  '202103':4,  '202104':5,
        '202105':6,  '202106':7,  '202107':8,  '202108':9,
        '202109':10, '202110':11, '202111':12, '202112':13,
        '202201':18, '202202':19, '202203':20, '202104':21,
        '202205':22, '202206':23, '202207':24, '202108':25,
        '202209':26, '202210':27, '202211':28, '202112':29};

    return data[y*100 + m];
}

let year = 2022;
let month = 1;

console.log(getRangeVal(year, month));

※ getRange() の第一パラメタ以外は同じようなので、それを求める方法って意味です。

Comments

[rate_o]

引数を使用して共通の箇所を一つにし、処理を軽くする考え方でしょうか？
わざわざコードまで記述くださりありがとうございます！

Answer 2

yが２０２２の場合はmに１２足せば、1から２４で判断できますな。
ついでに、その数値からRangeの数値出すように工夫すれば、数行で収まります

2021のときはmに１を足す、2022のときはmに１７足す、だけでいけますな。
＃３行でいけるやん

Comments

[rate_o]

コメントが遅くなってしまい申し訳ございません。
ご助言ありがとうございます！
やってみます！

[rate_o]

申し訳ございません、私の学習量が足りなく、[yが２０２２の場合はmに１２足せば、1から２４で判断] [2021のときはmに１を足す、2022のときはmに１７足す]の意味が理解ができないのですが、月をfor文で
回して足していくことでコードを少なくしていく感じなのでしょうか？
その場合、月ごとの条件をどのように検証はどのようにするのでしょうか？

お忙しい中、コメントを頂くことができたのにも関わらず、私の知識が少なく
大変申し訳ございません。

[y_waiwai]

inputSheet.getRange(m+(y==2021)?1:17, 8).setValue(Math.min.apply(null, maleTimeArray));

この１行で。

[think49]

2023年以降も考慮するなら

inputSheet.getRange(1+(y-2021)*16+m, 8).setValue(Math.min.apply(null, maleTimeArray));

[rate_o]

御忙しい中、ご回答いただき誠にありがとうございます！
また質問になってしまい、申し訳ないのですが、
(1+(y-2021)*16+m, 8)について具体的に教えていただけないでしょうか？

[loppta]

では分かりやすく書き直したコードを。

year=2022,mouth=1

mouthCalc = mouth-1+2 // 月の計算 1月は+2開始
yearCalc = (year-2021)*16 // 年の計算 2022では16+2になるように計算
Calc = yearCalc+mouthCalc
console.log(Calc)

出力は 18

Answer 3

規則性

@y_waiwaiさんのコメント欄より引用します。

(1+(y-2021)*16+m, 8)について具体的に教えていただけないでしょうか？

冗長なコードを短くまとめるコツは規則性を見出すことです。
洗練されたコードは、一定の規則を持ってコードが書かれています。

異なるコードの洗い出し

各コードを順番に確認していき、「共通するコード」と「異なるコード」に分けます
「異なるコード」は次の3つです。

• 変数 y の値
• 変数 m の値
• getRange メソッドの第一引数値(対象セルの行数)

データの洗い出し

先の3つをスプレッドシートに入力します。

y m linage(行数)
2021 1 2
2021 2 3
2021 3 4
2021 4 5
2021 5 6
2021 6 7
2021 7 8
2021 8 9
2021 9 10
2021 10 11
2021 11 12
2021 12 13
2022 1 18
2022 2 19
2022 3 20
2022 4 21
2022 5 22
2022 6 23
2022 7 24
2022 8 25
2022 9 26
2022 10 27
2022 11 28
2022 12 29

数値を分解

linageをスプレッドシート内の行数において、意味のあるグループ別に分解します。
数字から逆算する必要がある我々回答者と違い、@rate_o さんは目の前に実物があるのでこの作業は難しくないはずです。

• 1行目を飛ばして、2行目に最初のデータ(2021/01)がある事をあなたは知っています
• 年の変わり目(2021/12～2022/01)に「4行の間隔」がある事をあなたは知っています

y m linage(行数) 分解した式
2021 1 2 1+1
2021 2 3 1+2
2021 3 4 1+3
...
2021 12 13 1+12
2022 1 18 1+12+4+1
2022 2 19 1+12+4+2
2022 3 20 1+12+4+3
...
2022 12 29 1+12+4+12
2023 1 34 1+12+4+12+4+1
2023 2 35 1+12+4+12+4+2
2023 3 36 1+12+4+12+4+3
...

変数を当てはめる

「分解した式」のどこに y, m を代入できるか考えます。
2021年が次の式であることは明らかです。

linage === 1 + m;

• 2021/01と2022/01を比較すれば、12+4 の差があることが分かります。
• 2021/01と2023/01を比較すれば、(12+4)*2 の差があることが分かります。
• 2021/01と2022/01の比較は (12+4)*1 に置き換えることが可能であり、「1と2の違い」が2021年からの経過年数であることも分かります。

正当性検査

スプレッドシート内で数式を組み、比較演算子 = による確認が全て TRUE ならその数式は正解です。

y m linage 行数を求める数式 正誤判定
2021 1 2 =1+(A2-2021)*16+B2 =C2=D2

コードに落とし込む

「共通するコード」はそのまま利用し、「異なるコード」を算術式に置き換えます。
linage(getRange メソッドの第一引数)は変数 y, m から算術式で演算できる事がここまでの確認で判明している為、1行のコードになります。

inputSheet.getRange(1+(y-2021)*16+m, 8).setValue(Math.min.apply(null, maleTimeArray));

Re: rate_o さん

Comments

[rate_o]

休日にも関わらず、ご教示いただき誠にありがとうございます！
上記の説明をもとに今一度、設計しなおして参ります。

誠に有り難うございました。