JavaScriptの関数でreturnで値を返す場合の処理について

Question

###前提・実現したいこと
JavaScriptでreturnで値を返す場合の動作について

###発生している問題・エラーメッセージ

JavaScriptで聞きたいことがあります。今ボタンを押したら、入力チェックを行うようにしているのですが、変数を引数として渡して、その変数にいれた値を渡していましたが、別の人から「returnで値を渡して、その値で判断すればいい」と教えて頂きました。
しかし、それを試すと、関数で呼び出した際とif文の時で二度アラートが出ます。私のやり方が間違っているのか、これだとこの方法で合っているのかがよくわかりません。教えて頂けると幸いです。

###旧

//保存ボタンを押した時の処理
function insert\(\){
	check = 0 ;
	input_check\(check\) ;

	if\(check == 0\){
		work_select\(\) ;
	}
}



//入力チェックの処理
function input_check\(check\){
	if\( work_kbn_input\.value == ""\){
		alert\("作業区分は必須入力です。"\) ;
		check = 1 ;
	}else if\( work_name_input\.value == "" \){
		alert\("作業区分名は必須入力です。"\) ;
		check = 1 ;
	}else if\( work_kbn_input\.value\.length != 5\){
		alert\("作業区分に入力する文字数は５文字です。"\) ;
		check = 1 ;
	}else if\( work_kbn_input\.value\.match\(/\[\^0-9A-Z\]/\)\){
		alert\("作業区分に半角英数字大文字以外は入力してはいけません。"\) ;
		check = 1 ;
	}
}

###新

//保存ボタンを押した時の処理
function insert\(\){
	check = 0 ;
	input_check\(\) ;

//	アラートがある関係でこうするとエラーアラートをもう一回出してしまう
	if\( input_check\(\) == 0 \){
		work_select\(\) ;
	}
}



function input_check\(\){
	if\( work_kbn_input\.value == ""\){
		alert\("作業区分は必須入力です。"\) ;
		return 1 ;
	}else if\( work_name_input\.value == "" \){
		alert\("作業区分名は必須入力です。"\) ;
		return 1 ;
	}else if\( work_kbn_input\.value\.length != 5\){
		alert\("作業区分に入力する文字数は５文字です。"\) ;
		return 1 ;
	}else if\( work_kbn_input\.value\.match\(/\[\^0-9A-Z\]/\)\){
		alert\("作業区分に半角英数字大文字以外は入力してはいけません。"\) ;
		return 1 ;;
	}
	return 0 ;
}

Answer 1

サンプルプログラムの趣旨はわかりませんが、引数と戻り値の意味を確実に理解したほうが良いです。

引数：呼び出し元から下請け関数へのデータ受け渡しに使用するもの
戻り値：下請け関数から呼び出し元へのデータ受け渡しに使用するもの

この場合、insertが呼び出し元でinput_checkが下請けとなる関数です。

Answer 2

返り値を判定に利用する場合は、下記のような形で使えば、2回以上判定があっても大丈夫です。
下記の例では変数checkに代入する形にしていますが、不都合であればほかの変数に代入する形にしてください。

//保存ボタンを押した時の処理
function insert(){
    var check = input_check(); // 返り値を変数代入する 

// 返り値を代入した変数を判定に利用する
    if( check == 0 ){
        work_select() ;
    }
}



function input_check(){
    if( work_kbn_input.value == ""){
        alert("作業区分は必須入力です。") ;
        return 1 ;
    }else if( work_name_input.value == "" ){
        alert("作業区分名は必須入力です。") ;
        return 1 ;
    }else if( work_kbn_input.value.length != 5){
        alert("作業区分に入力する文字数は５文字です。") ;
        return 1 ;
    }else if( work_kbn_input.value.match(/[^0-9A-Z]/)){
        alert("作業区分に半角英数字大文字以外は入力してはいけません。") ;
        return 1 ;;
    }
    return 0 ;
}

Answer 3

質問者の求めてる回答とは異なりますが、
このチェック方法だと、作業区分と作業区分名を未入力で保存ボタン押した場合、
アラートで表示されるメッセージって作業区分は必須入力です。だけになると思うのですけど、大丈夫ですか？

関係なくてすいません。

Comments

[think49]

[編集履歴] を読むと分かりますが、質問者が本文を抹消した後にteratail運営が抹消を取り消しています。
2016/12/13 23:02の質問という事もあり、質問者(kurikaraさん)はもう回答を見ていないと思います(いろいろと突っ込みミどころが多い質問ではあるので、気になりますが)。
https://teratail.com/questions/history-questions/58739

[King_of_Flies]

ご丁寧にありがとうございます。
結局解決できたんですかね？気になりますｗ