Javascript email検証についての質問

Question

Javascriptでemailアドレスが有効かどうかを検証するプログラムをif文を使って書いているのですが、

１．
＠と.(ドット)の間に一文字ないとエラーになるというコード

２．
.(ドット)の後に一文字ないとエラーになるというコードが書けずにいます。

else if(email.indexOf("@",n)&&email.indexOf("@",n+1)=== -1){
 return false}

というコードを１の命令をするために書きましたが、動きませんでした。

どなたか分かる方ご回答いただきたいです。
与えられた課題の条件として、indexOf()とlastIndexOf()を使って、検証する必要があるとのことでした。
２のコードは、email.indexOf(.（ドット）, 検索を開始するインデックス番号)で書けそうだとは思ったのですが。ユーザーによってメールアドレスの長さが異なるのでどのインデックス番号を入れればいいのかわかりませんでした。

このプログラムのこれまで書いた全体のコードが以下のものです。

function emailConfirmation() {
let email = prompt("What is your email address?")
if(email.indexOf("@") === -1){
  document.getElementById("emailOutput").innerHTML = "This is invalid email address";  
}else if(email.indexOf("." ===-1&&email.indexOf("@" ===-1)){
  document.getElementById("emailOutput").innerHTML = "This is invalid email address";  
}else if(email.indexOf("@",0)=== -1){
  document.getElementById("emailOutput").innerHTML = "This is invalid email address";  

}
}

Comments

[退会済みユーザー]

コードは ``` と ``` で囲ってください。インデントされて見やすくなるので。インデントされてないコードは質問者さん自身でも読む気がしないのでは？

[jirosian]

失礼しました。修正しました。

[Y.H.]

> }else if(email.indexOf("." ===-1&&email.indexOf("@" ===-1)){
括弧()が無茶苦茶ですが質問記載時のミスですか？

[jirosian]

そうですか、これで正解だと思っていました。正しい書き方を教えていただいてもよろしいですか？

[yoorwm]

(や{は必ず対応する)や}がありますね。

[m.ts10806]

Google検索で出てくるような正規表現のサンプルは使えない縛りとかあるのでしたらteratailで質問するのも同義と思いますが

[Y.H.]

email.indexOf() この引数に
"." ===-1&&email.indexOf("@" ===-1)
が入ってます。

Answer 1

Javascript email検証についての質問

「字句解析」のご質問かもしれませんが、ネイティブに実装された DOM を利用する方法もあります。
（※「字句解析」なら正規表現を使うのが早いと思います）。

ご質問に示された 条件

1. 言語は Javascript
2. emailアドレスが有効かどうかを検証するプログラム
3. if 文を使う

const testEmail = ( ad ) => {
  let elm = Object.assign(document.createElement("input"),{
    type: "email"
  });
  elm.value = ad;
  // return elm.reportValidity()? elm.value: "";
  if( elm.reportValidity() ) {
    return elm.value;
  }
  else {
    // document.getElementById("emailOutput")
    //   .innerHTML = "This is invalid email address";
    return "";
  }
}
console.log( testEmail("[e-mail]")  );

無理矢理ですが、if を使いました。

• MDN:Input
• caniuse:checkvalidity()

ご参考になれば。

Answer 2

１．＠と.(ドット)の間に一文字ないとエラーになるというコード

２．.(ドット)の後に一文字ないとエラーになるというコードが書けずにいます。

if (email.indexOf(".@") !== -1 || email.indexOf("@.") !== -1) { 
    //＠と.(ドット)の間に一文字ない
    //  「".@"がある」または「"@."がある」場合はエラー
    console.log("Error.");
} else if (email.indexOf("..") !== -1 || email.slice(-1) === ".") {
    // .(ドット)の後に一文字ない
    //  「".."がある」か「"."で終わっている」のどちらかがあればエラー
    console.log("Error.");
}

---

本題ではないけどコメントより

}else if(email.indexOf("." ===-1&&email.indexOf("@" ===-1)){

そうですか、これで正解だと思っていました。正しい書き方を教えていただいてもよろしいですか？

本当に？
email.indexOf() この引数に
"." ===-1&&email.indexOf("@" ===-1)
が入ってます。よってこのelse if()ブロックが真になることはありません。

正しい書き方を教えていただいてもよろしいですか？

書き方ではなく考え方を。

とりあえずこのelse ifの条件をエスパーして「"."も"@"も存在しない場合」とした場合、
以下の考え方で記載してください。

• email に "."が存在しない条件はどう記載しますか？
• email に "@"が存在しない条件はどう記載しますか？
• それを && の右辺と左辺に記載するだけです。

Comments

[jirosian]

回答ありがとうございます。
１、２の案をそれぞれしてみました。
一番最初のは通りましたが、二番目のコードをかくと、
有効なメールアドレスも無効になってしまいます。。。

[Y.H.]

すみません、コード間違ってました。
誤： } else if (email.indexOf("..") || email..slice(-1) === ".") {
正： } else if (email.indexOf("..") !== -1 || email.slice(-1) === ".") {