JavaScriptについて

質問

現在勉強中で星座の判定のプログラムなんですが、consoleに表示させようといていて表示されません。
function seiza(){
if(((mon===3)&&(day>=21))||((mon===4)&&(day<=19))){
return "牡羊座";
}else if(((mon===4)&&(day>=20))||((mon===5)&&(day<=20))){
return "牡牛座";
}else if(((mon===5)&&(day>=21))||((mon===6)&&(day<=21))){
return "双子座";
}else if(((mon===6)&&(day>=22))||((mon===7)&&(day<=22))){
return "蟹座";
}else if(((mon===7)&&(day>=23))||((mon===8)&&(day<=22))){
return "獅子座";
}else if(((mon===8)&&(day>=23))||((mon===9)&&(day<=22))){
return "天秤座";
}else if(((mon===10)&&(day>=24))||((mon===11)&&(day<=22))){
return "蠍座";
}else if(((mon===11)&&(day>=23))||((mon===12)&&(day<=21))){
return "射手座";
}else if(((mon===12)&&(day>=22))||((mon===1)&&(bd<=day))){
return "山羊座";
}else if(((mon===1)&&(day>=21))||((mon===2)&&(day<=18))){
return "水瓶座";
}else if(((mon===2)&&(day>=19))||((mon===3)&&(day<=20))){
return "魚座";
　　　　　　 }else{
return"error";
}　

monとdayは事前にpromptで入力させていて関数で呼び出し、どの数字にしても最後のerrorしか表示されません
このif文は間違っているのでしょうか？またどのようにしたらいいのでしょうか？
前後の文は省略しています。

回答No.5

ushi2015 · ベストアンサー

こんにちは
やはり、
http://qiita.com/PianoScoreJP/items/e43d70ec188c6fed73ed
こちらの問題も有ると思いますので調べてみてはどうでしょうか？

回答No.6

superside0

比較演算子に === を使われていますが
monの実体が文字列　ということはないですか？

"1" == 1 は真ですが
"1" === 1 が　偽　です。

お礼

その通りでした。勉強不足です・・・。
有難うございました！！

回答No.4

ushi2015

こんにちは
No3さんへ
bdとか良く見てなかったです。
後は、monとdayを数値と比較しているのは大丈夫なのでしょうか？

回答No.3

pringlez

とりあえず絶対におかしいところ2つ
・全角空白の混入「 }else{ 」の前に全角空白
・閉じカッコの不足、最後に一つ足りない

おかしそうなところ
山羊座の判定で突然「bd」が出てくる。
「(mon === 1) && (bd <= day)」

どんなエラーが出てるか自分で確認するようにした方がいいですよ。

回答No.2

ushi2015

こんにちは
グローバル変数なら、大丈夫なはずですね。
return"error";
の部分で、monとdayに何がセットされているか表示するように
変更してみてはどうですか？

補足

ありがとうございます。
最後のelseの部分にconsole.log(mon,day)を入れたのですが
しっかり最初に入力したmonとdayが表示されました。
このifに問題があるのでしょうか？

回答No.1

ushi2015

こんにちは
普通、
function seiza()
の括弧内に引数、monとdayが有るのでは？

補足

こんにちは　回答有難うございます。
ご指摘の通り引数を入れたのですが
undefinedと表示されてしまいました。
どうしてなのでしょう？
ちなみにmonとdayはグローバル変数です。