[javascript]swith構文の関数を何度も使用したいのですが、できません。

前提・実現したいこと

swith構文の関数を1つの.jsファイルの中で何度も使用したいですが、うまくいきません。

発生している問題・エラーメッセージ

taskBの内容が「taskA = 3」の内容になってしまいます。

該当のソースコード

var YearMonth = '20201';
var taskA;
function gettask(){
switch (YearMonth){
case '20201':
taskA = 3;
break;
case '20202':
taskA = 4;
break;}
return taskA;}

var TaskB = gettask('20202');

試したこと

YearMonth = '20202';
var TaskB = gettask();

と書いたら、今度はTaskAの内容が「4」になってしまいました。

補足情報（FW/ツールのバージョンなど）

miyabi_pudding

2020/02/19 06:55

最終的に、どの変数がどういう状態になるのを想定していますでしょうか？

x_x

2020/02/19 06:56

大文字小文字を区別するので気を付けて書いてください。TaskA ですか？

kmyhskzh

2020/02/19 07:25

x_x様ご返信頂きまして、誠にありがとうございましたm(_ _)m タイプミスをしました。。申し訳ございません。

行動規範の内容に同意します

回答2件

ベストアンサー

まず、スコープに関して勉強しましょう。
ブロック - JavaScript | MDN

スコープに関して学ぶと、自ずとなぜ今回のようなことが起こっているか理解できると思いますよ。

さて、回答です。

taskAがグローバルスコープなのに、
gettask関数の中で、taskAの再代入を行っているからです。
つまり、TaskBの実行時に、taskAの中身を入れ替えちゃってるってわけですね。

ではどうすればいいかというと、関数の中のみのローカルスコープ変数を返すようにすればいいわけです。
（というか、switch文の中でreturnさせればいい）

javascript
1var YearMonth = '20201';
2// というかここで宣言する意味がないのでコメントアウト
3// var taskA;
4function gettask(yearInner){
5  // 文字列を、引数で受け取るようにしたほうがいいかと。
6  switch (yearInner){
7    case '20201':
8      // 下で改めて返す必要ないので、ここで返しちゃう
9      return 3;
10      // returnの後にbreakはいらなくなる
11      // break;
12    case '20202':
13      return 4;
14      // break;
15    default:
16      // 上のいずれのパターンにも属さない時にここを通る
17      return 4;
18  }
19  // return taskA;
20}
21// 一応、上の変数を引数に入れてます。（場合によっては必要ありません）
22var taskA = gettask(YearMonth);
23var TaskB = gettask('20202');

以上でどうでしょうか。

投稿2020/02/19 07:07

編集2020/02/19 07:08

miyabi_pudding

総合スコア9555

kmyhskzh

2020/02/19 07:24

miyabi_takatsuk様この度は誠にありがとうございましたm(_ _)m miyabi_takatsuk様にご回答頂き、解決に至りました。「taskA = 」や「taskB = 」という形でなく、「return」にすれば良いのですね。参考文献についてもご紹介頂きまして、ありがとうございます。初心者ゆえに問いの立て方もたどたどしく、タイプミスもありながら、意図を汲み取って頂きましてありがとうございました。