複合条件&&を併記する場合

Question

基本的なことであれば申し訳ございません。
以下のように
vs1_r
vs1_l
がそれぞれ1文字以上20文字以内を同時にすべて満たす場合trueを返したいのですが、
記載の条件式ではfalse判定になってしまいます。

括弧書き等で分ける等も試してみたのですがうまくいきません。
どの点に問題があるのでしょうか？

vs1_r="a";
vs1_l="ab";
    if (vs1_r.length>1 && vs1_r.length<21 && vs1_l.length>1 && vs1_l.length<21) {
      //do something
    }
コード

Comments

[papinianus]

回答が寄せられてはいますが、質問の書き方として。「記載の条件式ではfalse判定になってしまいます」とのこと。記載の条件式は常にfalseを返すわけではないので、何を与えたらfalseになるか、あるいはvs1_rが○○だったらtrueになってほしいのに、falseになった、みたいなことを意識して書かれることをご提案します。

[pegy]

コメント有難うございます。失礼しました。言われてみれば、仰るとおりです。加筆をしたのですが、
maisumakun様の当初のご指摘とおり、そもそも算数を間違っていたものと思われます。

Answer 1

まず気になったこととして、「1文字以上20文字以内」なのに条件が>1となっているので、1文字のときは意図せず条件を満たさない、ということがあります。

Comments

[pegy]

大変失礼いたしました、ご指摘の通りです。

Answer 2

コピペして、vs1_rとvs1_lに文字列を入れたら動きましたよ。

    var vs1_r = '123';
    var vs1_l = '123';
    
    if (vs1_r.length>1 && vs1_r.length<21 && vs1_l.length>1 && vs1_l.length<21) {
      //do something
      console.log('true')
    }

vs1_rとvs1_lには文字列が入っていますか？

Comments

[yambejp]

var vs1_r = '1'
だと駄目でしょ？

[suzu6]

駄目でしたね。
２文字以上20文字以内になっています。

あー、「1文字以上20文字以内」にしたいという質問でしたか。

[pegy]

申し訳ございません、代入したものが1文字であったのが原因です。
大変失礼いたしました。

[suzu6]

私が勝手に勘違いしておりました。
こちらこそ大変失礼いたしました。

Answer 3

vs1_r.match(/^.{1,20}$/) && vs1_l.match(/^.{1,20}$/)

などでもよいかもしれません
まとめると

[vs1_r,vs1_l].filter(function(x){
  return x.match(/^.{1,20}$/);
}).length==2

Comments

[pegy]

yambejp様
コメント有難うございます。。。正直書き方がおしゃれすぎて（なぜ、不等号なしに条件判定ができるのか、、）今の私では理解ができないため、今日帰宅したらじっくり一つずつ咀嚼してみようと思います。

[papinianus]

正規表現をキーワードに検索してみてください

[think49]

複数行に未対応なので、厳密には要件を満たしていませんね…。