jqueryを使いボタン一つで未入力箇所エラー、全角入力エラー、パスワード変数が生成できなくて困ってますが

Question

$('#button1').on('click',function(){
		if ($('#hoge').val() =="" || $('#hoge1').val() =="" ) {
			$('#sms').text("未入力箇所があります");

			}
		var mail_regex1 = new RegExp( '(?:[-!#-\'*+/-9=?A-Z^-~]+\.?(?:\.[-!#-\'*+/-9=?A-Z^-~]+)*|"(?:[!#-\[\]-~]|\\\\[\x09 -~])*")@[-!#-\'*+/-9=?A-Z^-~]+(?:\.[-!#-\'*+/-9=?A-Z^-~]+)*' );
	    var mail_regex2 = new RegExp( '^[^\@]+\@[^\@]+$' );
	    if( address.match( mail_regex1 ) && address.match( mail_regex2 ) ) {
	        // 全角チェック
	        if( address.match( /[^a-zA-Z0-9\!\"\#\$\%\&\'\(\)\=\~\|\-\^\\\@\[\;\:\]\,\.\/\\\<\>\?\_\`\{\+\*\} ]/ ) ) {
	        	return false; }
	        // 末尾TLDチェック（co,jpなどの末尾ミスチェック用）
	        if( !address.match( /\.[a-z]+$/ ) ) { return false; }
	        return true;
	    } else {
	        return false;
	    }

			var seed0 = '0123456789qwertyuiopasdfghjklzxcvbnm';//英数字
			var pwd= '';
			for(var i=0;i<8;i++) {
			var pos =  Math.floor(Math.random() * seed0.length);
			pwd += seed0[pos];
			} /*for*/

			$('<input>').attr({
			    type: 'hidden',
			    id: 'input_id',
			    name:'pwd',
			    value:pwd
			}).appendTo("form");/*''と""ひとつで変数はできなくなる"*/

		});

ボタンを押すと、未入力のテキストボックスがあればエラー、
全角入力なら半角入力を促し、
OKならpwd変数を作りたいのですが ひとまとめにできません。

ひとまとめに行うにはどこを治せばよいでしょうか

パスワードは単体のときはきちんと変数でした

pwd変数がnullになってしまいます

return trueの前に入れてもnullになります
return true 削除してもまだnullです

Answer 1

if( address.match( mail_regex1 ) && address.match( mail_regex2 ) ) {
    // 全角チェック
    if( address.match( /[^a-zA-Z0-9\!\"\#\$\%\&\'\(\)\=\~\|\-\^\\\@\[\;\:\]\,\.\/\\\<\>\?\_\`\{\+\*\} ]/ ) ) {
        return false;
    }
    // 末尾TLDチェック（co,jpなどの末尾ミスチェック用）
    if( !address.match( /\.[a-z]+$/ ) ) {
        return false;
    }
    return true; // <= ここ
} else {
    return false; // <= ここ
}

＃インデントやブラケットの位置は揃えました
ここの条件で return しているので、下半分のロジックは通らないのでは？

Comments

[kentros511]

解析ありがとうございます
trueの前にパスワード変数を入れればいいんですかね

[trick]

hyper-drums-ko さんも言っていますが
ロジック自体が通っていません。
trueの前に入れても、下半分通りませんよ。
多分、return true;を削除すれば問題ないと思います。

今一度、ロジックの流れ方を確認した方が良いと思います。

[kentros511]

そういうことだったんですか
やってみます
ありがとうございました。

[kentros511]

return　trueを削除しましたが　パスワードがまだnullのままです

Answer 2

var address = "[e-mail]";
var mail_regex1 = new RegExp( '(?:[-!#-\'*+/-9=?A-Z^-~]+\.?(?:\.[-!#-\'*+/-9=?A-Z^-~]+)*|"(?:[!#-\[\]-~]|\\\\[\x09 -~])*")@[-!#-\'*+/-9=?A-Z^-~]+(?:\.[-!#-\'*+/-9=?A-Z^-~]+)*' );
var mail_regex2 = new RegExp( '^[^\@]+\@[^\@]+$' );
if( address.match( mail_regex1 ) && address.match( mail_regex2 ) ) {
    
    if( address.match( /[^a-zA-Z0-9\!\"\#\$\%\&\'\(\)\=\~\|\-\^\\\@\[\;\:\]\,\.\/\\\<\>\?\_\`\{\+\*\} ]/ ) ) {
        return false; 
    }
    
    if( !address.match( /\.[a-z]+$/ ) ) { return false; }
} else {
    return false;
}

    var seed0 = '0123456789qwertyuiopasdfghjklzxcvbnm';
    var pwd= '';
    for(var i=0;i<8;i++) {
    var pos =  Math.floor(Math.random() * seed0.length);
        pwd += seed0[pos];
    } 

    alert(pwd);

上記で、pwd は表示されましたよ。
どこかの、return false;
で抜けていませんか？

Comments

[kentros511]

おかげで生成できました