配列、ループ、if文の組み合わせ

Question

配列、ループ、if文をどのように組み合わせるかを理解しようとしています。私は平日の配列を作成しました。特定の日に特定の運動を割り当てる必要があることを指定したいと思います。私は割り当てられた日と運動をプリントアウトしたいと思います。以下のコードからわかるように、weekday.push()メソッドを使用すると、別の日に水泳またはウェイトトレーニングが割り当てられ、ヨガはまったく追加されず、14個の値が作成されます。これにアプローチする方法を教えてください。ありがとう！あなたのコードをModiefied

var weekday = ['Sun', 'Mon', 'Tues', 'Wed', 'Thur', 'Fri', 'Sat']; 
 
var workOut = []; 
 

 
function myExercise() { 
 
    for (var i = 0; i < weekday.length; i++) { 
 
    if (weekday == 'Wed' || 'Mon' || 'Fri') { 
 
     workOut.push('Swimming'); 
 
    } 
 
    if (weekday == 'Tue' || 'Thur' || 'Sat') { 
 
     workOut.push('weight training'); 
 
    } else { 
 
     workOut.push('Yoga' + 'or' + 'power walking'); 
 
    } 
 
    } 
 
} 
 

 
myExercise(); 
 
console.log(workOut);

Answer 1

前の回答で声明を切り替えるが、私は同様に、このいずれかを含むていますが、すでに存在しています。それはしかし、初心者のため理解しやすく、乾燥していないかもしれません。

var weekday = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat']; 
var workOut = []; 

function myExercise() {  
    for (var i = 0; i < weekday.length; i++) { 
     switch(weekday[i]) { 
      case 'Sun': 
       workOut.push('Yoga' + 'or' + 'power walking'); 
       break; 
      case 'Mon': 
       workOut.push('Swimming'); 
       break; 
      case 'Tue': 
       workOut.push('weight training'); 
       break; 
      case 'Wed': 
       workOut.push('Swimming'); 
       break; 
      case 'Thu': 
       workOut.push('weight training'); 
       break; 
      case 'Fri': 
       workOut.push('Swimming'); 
       break; 
      case 'Sat': 
       workOut.push('weight training'); 
       break; 
       default: 
       workOut.push('Sleep!!'); 
     } 
    } 
} 

myExercise(); 
console.log(workOut); 

Answer 2

、スペルミスや条件があったが

var weekday = ['Sun', 'Mon', 'Tues', 'Wed', 'Thur', 'Fri', 'Sat']; 
 
var workOut = []; 
 

 
function myExercise() { 
 
    for (var i = 0; i < weekday.length; i++) { 
 
    console.log(weekday[i]) 
 
    if (weekday[i] == 'Wed' || weekday[i] == 'Mon' || weekday[i] == 'Fri') { 
 
     workOut.push('Swimming'); 
 
    } 
 
    else{ 
 
    if (weekday[i] == 'Tues' || weekday[i] == 'Thur' || weekday[i] == 'Sat') { 
 
     workOut.push('weight training'); 
 
    } else { 
 
     workOut.push('Yoga' + 'or' + 'power walking'); 
 
    } 
 
    } 
 
    } 
 
} 
 

 
myExercise(); 
 
console.log(workOut); 
 
VM280:18

Answer 3

あなたがしようとしたように残念ながら、あなたは短いカットを使用することはできませんが重なりましたあなたのif節で。また、イテレータを忘れた[i]。あなたがIE（includesのサポートはありません）をサポートしたい場合は、代わりにindexOfで仕事ができる

if (['Wed','Mon','Fri'].includes(weekday[i])) 

：通常

if (weekday[i] == 'Wed' || weekday[i] == 'Mon' || weekday[i] == 'Fri') 

のほかに、あなたはまた、次の形式を使用することができます。

これは、多くの値をチェックしたいときに便利です。

注：私はまた、else if（14のエントリの理由である）にifを変更し、Array.prototype.map関数を使用しました。また、スコープに属していないオブジェクトを操作するのは悪い習慣です（ワークアウトはグローバルスコープですが、myExercise関数で操作されます）。目的の結果を返してから単純な割り当てを行う方がよいでしょう。

var weekday = ['Sun', 'Mon', 'Tue', 'Wed', 'Thur', 'Fri', 'Sat']; 
 
var workOut = []; 
 

 
function myExercise() { 
 

 
    return weekday.map(function(day){ 
 
     var workout; 
 
     if (['Mon','Wed','Fri'].includes(day)){ 
 
     workout = "Swimming"; 
 
     } else if (['Tue','Thur','Sat'].includes(day)){ 
 
     workout = "Weight training"; 
 
     } else { 
 
     workout = 'Yoga or power walking'; 
 
     } 
 
     return workout; 
 
    }); 
 
} 
 

 
workOut = myExercise(); 
 
console.log(workOut);

Answer 4

var weekdays = ['Sun', 'Mon', 'Tues', 'Wed', 'Thur', 'Fri', 'Sat']; 
 
var workouts = weekdays.map(day => { 
 
    switch (day) { 
 
    case 'Wed': 
 
    case 'Mon': 
 
    case 'Fri': 
 
     return 'Swimming'; 
 
     break; 
 
    case 'Tues': 
 
    case 'Thur': 
 
    case 'Sat': 
 
     return 'weight training'; 
 
     break; 
 
    default: 
 
     return 'Yoga or power walking'; 
 
     break; 
 
    } 
 
}); 
 

 
console.log(workouts);