JavaScriptでの入力ﾁｪｯｸについて下記のように､ﾃｷｽﾄﾎﾞｯｸｽとﾃｷｽﾄｴﾘｱを設置し､ submitﾎﾞﾀﾝｸﾘｯｸ時にでJavaScriptによる入力ﾁｪｯｸを行う場合､ …

JavaScriptでの入力ﾁｪｯｸについて

下記のように､ﾃｷｽﾄﾎﾞｯｸｽとﾃｷｽﾄｴﾘｱを設置し､
submitﾎﾞﾀﾝｸﾘｯｸ時にでJavaScriptによる入力ﾁｪｯｸを行う場合､
ﾃｷｽﾄｴﾘｱの方が未入力ﾁｪｯｸに引っ掛かりません｡

どのように修正すべきでしょうか｡ご教示願います｡

<form name="sample" action="" methid="post">
<input type="text" name="test1">
<textarea name="test2" cols="80" rows="4"></textarea>
：
</form>

********************

function InputCheck(){

var flg=0;

if(document.sample.test1.value==""){
msg='ﾃｷｽﾄﾎﾞｯｸｽが入力されていません｡';
flg=1;
}else if(document.sample.test2.value==""){
msg='ﾃｷｽﾄｴﾘｱが入力されていません｡';
flg=1;
}

if(flg){
window.alert(msg);
return false;
}else{
return true;
}
}

上記のﾁｪｯｸ処理部分の記述では｢test1が空白でない場合(else if)にﾃｷｽﾄｴﾘｱのﾁｪｯｸを行う｣となります｡ﾃｷｽﾄとﾃｷｽﾄｴﾘｱ両方のﾁｪｯｸを行い､かつﾀﾞｲｱﾛｸﾞにそれぞれのﾒｯｾｰｼﾞを表示させたい場合は

function InputCheck(){

var flg=0;
var msg="";

if(document.sample.test1.value==""){
msg+='ﾃｷｽﾄﾎﾞｯｸｽが入力されていません｡';
flg=1;
}

if(document.sample.test2.value==""){
msg+='ﾃｷｽﾄｴﾘｱが入力されていません｡';
flg=1;
}

if(flg){
window.alert(msg);
return false;
}else{
return true;
}
}

のようにすればよろしいかと思います｡

function InputCheck(){
var flg=0;
if(document.sample.test1.value==""){
/*
ﾃｷｽﾄﾎﾞｯｸｽが未入力のとき､ﾃｷｽﾄｴﾘｱが入力されていても未入力でも､この部分は実行される｡
ﾃｷｽﾄﾎﾞｯｸｽが入力されているとき､ﾃｷｽﾄｴﾘｱが入力されていても未入力でも､この部分は実行されない｡
 */
msg='ﾃｷｽﾄﾎﾞｯｸｽが入力されていません｡';
flg=1;
}else if(document.sample.test2.value==""){
/*
ﾃｷｽﾄｴﾘｱが未入力のとき､ﾃｷｽﾄﾎﾞｯｸｽが入力されていると､この部分は実行される｡ただし､ﾃｷｽﾄﾎﾞｯｸｽが未入力だと､この部分は実行されない｡
ﾃｷｽﾄｴﾘｱが入力されているとき､ﾃｷｽﾄﾎﾞｯｸｽが入力されていても未入力でも､この部分は実行されない｡
 */
msg='ﾃｷｽﾄｴﾘｱが入力されていません｡';
flg=1;
}
if(flg){
window.alert(msg);
return false;
}else{
return true;
}
}

If文のﾃｷｽﾄｴﾘｱ(test2)のﾁｪｯｸをelse ifで行なっているのがいけません｡
ﾃｷｽﾄﾎﾞｯｸｽ(test1)が未入力の場合､ﾃｷｽﾄｴﾘｱ(test2)の処理がｽﾙｰされてしまいます｡
次のようにすると良いです｡

function InputCheck(){
var flg1=0;
var flg2=0;
if(document.sample.test1.value==""){
msg1='ﾃｷｽﾄﾎﾞｯｸｽが入力されていません｡';
flg1=1;
}
if(document.sample.test2.value==""){
msg2='ﾃｷｽﾄｴﾘｱが入力されていません｡';
flg2=1;
}
if(flg1||flg2){
if(flg1){
window.alert(msg1);
}
if(flg2){
window.alert(msg2);
}
return false;
}
else{
return true;
}
}

どちらも未入力の場合､2回ﾒｯｾｰｼﾞを表示します｡

ﾃｷｽﾄｴﾘｱのばあい､nullを返すことがあるからかも

}else if(document.sample.test2.value=="" || document.sample.test2.value==null){

ように書いてみるとどうでしょうか？

ﾃﾞﾊﾞｯｸﾞ用のﾛｼﾞｯｸを追加して､
document.sample.test2.valueの値を調べたほうが良いと思います｡

たぶんこう動くはずというのでなくて､期待しない値が入ってるかも｡
ﾃｷｽﾄｴﾘｱの場合､未入力でも空白文字(ｽﾍﾟｰｽ)が入ってることも考えられます｡
文字列にTrim処理をかける必要があるかも｡