javascript - このIfステートメントを単純化して、コードが重複しないようにします-Javascript

Question

Googleマップのマーカーを作成するために'for'ループ内で実行されるIfステートメントがあります。基本的に、チェックボックスがオンになっていない場合、コードはすべてすべてのマーカーを表示しますが、チェックボックスがオンになっている場合は、2番目のパラメーターをチェックし、この条件に一致するアイテムのコードのみを実行します。

以下のコードは正常に機能しますが、ifステートメントの両方の部分で同じコードを使用する必要があることを意味します。これを達成するためにIfステートメントを構造化するより良い方法がある場合、それを関数に組み込むことは別として？

if(!FriendlyChecked){

    //Code here

} else if(FriendlyChecked && Friendly == "Yes"){

    //Same code here

}

Answer 1

FriendlyCheckedが false の場合、最初の条件が満たされ、コードが実行されます。

したがって、2 番目の条件に達した場合はFriendlyCheckedtrue でなければならないので、それを確認する必要はまったくなく、それを確認するだけで済みますFriendly == "Yes"。

if(!FriendlyChecked || Friendly == "Yes"){
    // code here
}

Answer 2

if( !FriendlyChecked || (FriendlyChecked && Friendly == "Yes") ) {

  // your code

}

!FriendlyChecked || (FriendlyChecked && Friendly == "Yes")FriendlyCheckedどちらかがチェックされますfalse（チェックされていません）

OR FriendlyCheckedは istrueの値FriendlyですYes

Answer 3

これで問題が解決します

 if((!FriendlyChecked) ||((FriendlyChecked)&&(Friendly == "Yes")){

 //Code here

}