Q&A
まず何をしたいのか確定しましょう。
実現したいこと
JSの勉強中です。
親と子の勝負で出たトランプの役を出し,
役によって点を持ち、親子比較して勝敗を出したいです。
ご教授していただけると有り難いです。
補足情報
変更いたしました。
了解しました
回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。後から見た人から見て、現在回答されておられる方の回答が「無いコードに対しての指摘」になり、意味がわからなくなります。
回答1件
0
ベストアンサー
『上のpoker()という関数で役名を出したいです。』とのことですが、ご提示のコードでは、pokerが呼び出されていないように見えます。
関数は、まず呼び出されないと動作しません。適切だと思われる場所にpoker();と書いてみてはいかがですか？
投稿2020/03/16 00:44
総合スコア36960
一度入れてみました。しかし動きません。
ワンペアなどの関数が呼び出されていないのが原因でしょうか？
コードをご提示いただけないとなんとも。
＿
当方で試したところ、
numbers is undefined
というエラーが出ました。
コードを見る限り、おそらく
}else if(is_one()){
の部分で、引数を渡さずに関数を呼び出しているのが原因だと思います。
関数定義は
function is_one(numbers, score) {
となっていますから、引数を2つ渡した方がいいでしょう。
チラ見しかしてませんが、今の判定方法だと手がロイヤルストレートフラッシュでもストレートと判定されると思いますよ。
引数渡して一度やってみたので上記のコードみて頂きたいです。
今はエラーが出ていて結果もundefinedになっております。
console,logでjudgementの役名をとるにはどのようにすれば良いでしょうか？？
return 文がないのが原因です。
returnのない関数は、undefined を返します。
もう少し細かく説明すると、たとえば、
function high_card(numbers, score) {
if (numbers === 0) {
return score = 1000
}
}
ですが、引数 numbers に入っているのは配列なので、numbers === 0 は必ず偽になります。
ですから、return文は実行されず、undefined が返されます。
同様に、judgement 関数に書かれた関数呼び出しは、全てundefined が返されますから、これまた return文が実行されずに、undefinedを返します。
そういうわけで、oyaResult には、undefined が入っています。
ですから、
console.log(judgment(oyaResult));
は、
console.log(judgment(undefined));
となり、途中でエラーになっています。
解決方法ですが、役判定の条件文を見直すところから始めてはいかがでしょうか。
細かい説明までありがとうございます。
おかげで理解できました。
役判定の条件分を見直してみます。
ありがとうございました。
頑張ってください。
あなたの回答
tips
プレビュー
