質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.36%

  • JavaScript

    18713questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Javascript: 冗長なif文をスッキリした書き方にしたい。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 87

rails_ruby

score 29

前提

コードを見て「何したいの」と思うかもしれませんが、そこはグッと抑えて頂いてご教示いただけたら幸いです。
下記のコードでは、セレクトボックスで選択した曜日の1週間前の日付に遡るプログラムになります。
もっとスマートな書き方はあると思いますが、私の今持っている知識で実装すると下記のようなコードになります。
一応、デベロッパーツールを使って検証したらので1週間前の日付には遡ることが出来ています。

実現したいこと

コードを見ていただくとわかると思いますが、if文を多用した書き方をしており可読性が悪いと私自身でも感じおります。しかしながら私の今の知識ではこの書き方しか思いつかず、他に可読性が高い書き方があればと思いご意見、ご教示をいただくたく質問させて頂きました。
何か、スッキリしたコードの書き方があればご教示いただけると幸いです。
よろしくお願いします。

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

該当のソースコード

<select id="select"></select>
<input type="button" value="go" id="btn_date">
//セレクトボックス
var week = [ "日曜日", "月曜日", "火曜日", "水曜日", "木曜日", "金曜日", "土曜日" ];

for(var i=0;i<week.length;i++){
let op = document.createElement("option");
op.text = week[i];
op.value = i;
document.getElementById("select").appendChild(op);
}

window.addEventListener('DOMContentLoaded', function(){
  document.querySelector('#btn_date').addEventListener('click',function(){

  var dObj = new Date();
  dObj.setDate(dObj.getDate() + 2);//検証で日付をいじっている
  var wDay = dObj.getDay();
  //console.log(week[wDay]);
  console.log('今日は'+ dObj);

  const str = document.getElementById("select").value;
  //console.log(str);
  console.log(wDay);
  if(wDay == 0){
    if (str == 0){
      console.log(str);
      console.log('日曜日です');
      dObj.setDate(dObj.getDate() - 7);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 1){
      console.log(str);
      console.log('月曜日です');
      dObj.setDate(dObj.getDate() - 6);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 2){
      console.log(str);
      console.log('火曜日です');
      dObj.setDate(dObj.getDate() - 5);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 3){
      console.log(str);
      console.log('水曜日です');
      dObj.setDate(dObj.getDate() - 4);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 4){
      console.log(str);
      console.log('木曜日です');
      dObj.setDate(dObj.getDate() - 3);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 5){
      console.log(str);
      console.log('金曜日です');
      dObj.setDate(dObj.getDate() - 2);
      console.log('1週間前は'+ dObj + 'です');
    }else if(wDay == 6){
      console.log(str);
      console.log('土曜日です');
      dObj.setDate(dObj.getDate() - 1);
      console.log('1週間前は'+ dObj + 'です');
    }
  }else if(wDay == 1){
    if (str == 0){
      console.log(str);
      console.log('日曜日です');
      dObj.setDate(dObj.getDate() - 8);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 1){
      console.log(str);
      console.log('月曜日です');
      dObj.setDate(dObj.getDate() - 7);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 2){
      console.log(str);
      console.log('火曜日です');
      dObj.setDate(dObj.getDate() - 6);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 3){
      console.log(str);
      console.log('水曜日です');
      dObj.setDate(dObj.getDate() - 5);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 4){
      console.log(str);
      console.log('木曜日です');
      dObj.setDate(dObj.getDate() - 4);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 5){
      console.log(str);
      console.log('金曜日です');
      dObj.setDate(dObj.getDate() - 3);
      console.log('1週間前は'+ dObj + 'です');
    }else if(wDay == 6){
      console.log(str);
      console.log('土曜日です');
      dObj.setDate(dObj.getDate() - 2);
      console.log('1週間前は'+ dObj + 'です');
    }
  }else if(wDay == 2){
    if (str == 0){
      console.log(str);
      console.log('日曜日です');
      dObj.setDate(dObj.getDate() - 9);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 1){
      console.log(str);
      console.log('月曜日です');
      dObj.setDate(dObj.getDate() - 8);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 2){
      console.log(str);
      console.log('火曜日です');
      dObj.setDate(dObj.getDate() - 7);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 3){
      console.log(str);
      console.log('水曜日です');
      dObj.setDate(dObj.getDate() - 6);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 4){
      console.log(str);
      console.log('木曜日です');
      dObj.setDate(dObj.getDate() - 5);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 5){
      console.log(str);
      console.log('金曜日です');
      dObj.setDate(dObj.getDate() - 4);
      console.log('1週間前は'+ dObj + 'です');
    }else if(wDay == 6){
      console.log(str);
      console.log('土曜日です');
      dObj.setDate(dObj.getDate() - 3);
      console.log('1週間前は'+ dObj + 'です');
    }
  }else if(wDay == 3){
    if (str == 0){
      console.log(str);
      console.log('日曜日です');
      dObj.setDate(dObj.getDate() - 10);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 1){
      console.log(str);
      console.log('月曜日です');
      dObj.setDate(dObj.getDate() - 9);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 2){
      console.log(str);
      console.log('火曜日です');
      dObj.setDate(dObj.getDate() - 8);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 3){
      console.log(str);
      console.log('水曜日です');
      dObj.setDate(dObj.getDate() - 7);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 4){
      console.log(str);
      console.log('木曜日です');
      dObj.setDate(dObj.getDate() - 6);
      console.log('1週間前は'+ dObj + 'です');
    }else if(str == 5){
      console.log(str);
      console.log('金曜日です');
      dObj.setDate(dObj.getDate() - 5);
      console.log('1週間前は'+ dObj + 'です');
    }else if(wDay == 6){
      console.log(str);
      console.log('土曜日です');
      dObj.setDate(dObj.getDate() - 4);
      console.log('1週間前は'+ dObj + 'です');
    }
  }

  });
});

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

js

追記

@kei344さんから

console.log(str);
      console.log( week[ str ] + 'です');
      dObj.setDate(dObj.getDate() - 7 - wDay + str);
      dObj.setDate(dObj.getDate() + 1);
      console.log('1週間前は'+ dObj  + 'です');
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+2

if(wDay == 0)以降を丸々下記に書き換えでできませんか?

      console.log(str);
      console.log( week[ str ] + 'です');
      dObj.setDate(dObj.getDate() - 7 - wDay + str );
      console.log('1週間前は'+ dObj + 'です');

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/11 18:38

    なんども回答して頂きありがとうございます。
    もしかしたら私のプログラムの原因かもしれませんが、ご教示して頂いたコードを試したところ、1日多く戻りすぎていたので僭越ながら一文だけ追記させて頂き、意図したプログラムになりました。

    キャンセル

+2

こんな感じかな(dayjs利用)

<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<script src="https://cdnjs.cloudflare.com/ajax/libs/dayjs/1.8.5/dayjs.min.js"></script>
</head>
<body>
<select id="select"></select>
<input type="button" value="go" id="btn_date">
</body>
<script>
//セレクトボックス
var week = [ "日曜日", "月曜日", "火曜日", "水曜日", "木曜日", "金曜日", "土曜日" ];

for(var i = 0; i < week.length; i++) {
  let op = document.createElement("option");
  op.text = week[i];
  op.value = i;
  document.getElementById("select").appendChild(op);
}

window.addEventListener('DOMContentLoaded', function() {
  document.querySelector('#btn_date').addEventListener('click', function(e) {
    var selectedIndex = document.querySelector('select').selectedIndex;
    var day = dayjs();
    var sDay = dayjs().subtract(day.format('d') - selectedIndex, "day");
    var bday = sDay.subtract(1, "week");
    console.log(sDay);
    console.log(bday);

    console.log(week[sDay.format('d') - 0]);
  });
});
</script>
</html>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/11 18:40

    ご教示いただけありがとうございます。
    試させて頂いたところ、意図した動作になりました。dasjsのライブラリーについて知らなかったので勉強になりました。
    お忙しい中、質問に回答して頂きありがとうございました。

    キャンセル

同じタグがついた質問を見る

  • JavaScript

    18713questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。