switch文の結果を定数に代入したい

Question

【やりたいこと】
ドロップダウン選択後、クリックイベントで、選択したoptionの値を取得。
optionの値ごとにswitch文で文字列を定数に格納したい。

【確認したことエラー】
クリックイベントで、optionの値取得後、定数部分の即時関数内に値を渡すところまでは確認できてます。
switch 文の引数に直接、文字列でoptionの値を入れると理想通り、定数に代入されます。

しかし、switch 文の引数にoptionValを入れるとエラーになってしまいます。
即時関数内に値は渡せているはずだと思っているのですが、どうすれば、直接文字列を入れたときのように
うまくswitch文を機能させ、constに定数を代入できるか教えていただきたいです。

$("button").click(function(){

        let pageOption =  $("#pageList").val();　//選択したoptionの値を代入

        const HOGE = (function(optionVal) {
                switch (optionVal) {
                        case 'optionvalue1': return 'hoge1'; break;
                        case 'optionvalue2': return 'hoge2'; break;
                        case 'optionvalue3': return 'hoge3'; break;
                        default: alert('エラー'); break;
                } 
        })(pageOption); 
            
        console.log(HOGE); //ここがundefinedになります。。
            

});//f_button-click END

Comments

[shiracamus]

どのcaseにも該当しなかった可能性は？

[paccuman]

ありがとうございます。
検証してみました。defaultの判定でした。
それに伴い質問文を調整させていただきましたのでご確認いただけますと幸いです m( )m

Answer 1

optionValが'optionvalue1' でも 'optionvalue2' でも 'optionvalue3' でもないなら、エラーが表示されるのは当然では？

> const f = function(optionVal) {
...         switch (optionVal) {
.....                 case 'optionvalue1': return 'hoge1'; break;
.....                 case 'optionvalue2': return 'hoge2'; break;
.....                 case 'optionvalue3': return 'hoge3'; break;
.....                 default: console.log('エラー'); break;
.....         }
... };
undefined
> f('optionvalue3');
'hoge3'
> f('hoge');
エラー
undefined

ちなみに、この内容でしたら辞書変換の方が効率いい気がします。

> const convert = {
...   optionvalue1: 'hoge1',
...   optionvalue2: 'hoge2',
...   optionvalue3: 'hoge3',
... };
undefined
> convert["optionvalue3"]
'hoge3'
> convert["hoge"]
undefined

Comments

[paccuman]

ありがとうございます。辞書変換…試してみます！！
optionValの値は、選択したプルダウンによりますが、文字列でoptionvalue1(3)のいずれかは入っていたんですよね。。
switch文ではなく、console.logやreturn optionValだけで試してみました????

とりあえず、明日もう一度確認してみます。。！

Answer 2

そのcaseのどれにも合わなかったときにどうなるか考えてみよう

Comments

[paccuman]

ありがとうございます。
検証してみました。defaultの判定でした。
同じ回答文で恐縮ですが、それに伴い質問文を調整させていただきましたのでご確認いただけますと幸いです m( )m

[y_waiwai]

optionValのナカミは何なのか調べてみては

[paccuman]

ありがとうございます。optionValの中身はちゃんとプルダウンで選択した値が入っていたんですよね…。
そこまで確認して、いざswitch文にしてみたときに、エラーになってしまい
わからなくなってしまいました。。

Answer 3

皆様ありがとうございました。無事解決できました。
結論、jsの記述は正常でした。

原因はvalueのダブルクォーテーションの記述ミスでした。
誤：value= ”name_often”
正：value= "name_often"

js内では処理が進んで、見た目も似ていて気付けなかったです。
js以外が原因ですみませんでした。。

皆様のソースコードでわからない箇所を調べながら気づくことができました。
見ていただいた皆様、ありがとうございました！

Answer 4

switchの各レベルで処理が一つしか無いなら配列に押し込んだほうが楽ですね
そもそもswitchでreturnするのもフロー的にはあまり好ましくないでしょう
コールバックで取っているのもさほど効率的とは言えません

ざっくりまとめるとこう

<script>
$(function(){
  $("button").on('click',function(){
    const pageOption =  $("#pageList").val();
    const HOGES = {
      'optionvalue1':'hoge1',
      'optionvalue2':'hoge2',
      'optionvalue3':'hoge3',
    } ;
    if (Object.keys(HOGES).indexOf(pageOption)<0){
      alert("エラー");
    }else{
      console.log(HOGES[pageOption]);
    }
  });
});
</script>
<select id="pageList">
<option value="">--</option>
<option value="optionvalue1">1</option>
<option value="optionvalue2">2</option>
<option value="optionvalue3">3</option>
</select>
<button>button</button>

ちょっと手を抜くとこう

<script>
$(function(){
  $("button").on('click',function(){
    const pageOption =  $("#pageList").val();
    const HOGES = {
      'optionvalue1':'hoge1',
      'optionvalue2':'hoge2',
      'optionvalue3':'hoge3',
    } ;
    const msg=HOGES[pageOption];
    (msg || alert("エラー")) && console.log(msg);
  });
});
</script>
<select id="pageList">
<option value="">--</option>
<option value="optionvalue1">1</option>
<option value="optionvalue2">2</option>
<option value="optionvalue3">3</option>
</select>
<button>button</button>