引数の値を条件判定させる方法について

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 441

umauman

score 55

下記の結果になるような記述をしたく思っていますが、意図する通りになりません。

希望する動作

「果物A」ボタンを押した場合 ⇒ console.logで「赤りんご」を表示
「果物B」ボタンを押した場合 ⇒ console.logで「青りんご」を表示

試したソース

<button onclick="judgeFruit(apple, red);">果物A</button><br>
<button onclick="judgeFruit(apple, green);">果物B</button>
var apple, red, green;
function judgeFruit(fruit, color){
    if(fruit === apple && color === red){
        console.log('赤りんご');
    } else if (fruit === apple && color === green){
        console.log('青りんご'); //赤りんごと表示されてしまう
    }
};

上記ソースでは両方とも結果が「赤りんご」になってしまいます。
「果物B」ボタンでは引数で「apple(りんご)」と「green(青)」を渡しているにもかかわらず
何故一番目の条件式と合致して「赤りんご」と表示してしまうのか、理解できずにいます。

ちなみにconsole.log(color)やconsole.log(fruit)と記述してみたところ「undefined」となってしまうため
ここら辺に原因があるような気がしてはいるものの、答えに辿りつかず悩んでいます。

また、「var apple, red, green;」も、定義しないとエラーが出てしまうため
加えましたが、なぜ定義が必要なのかもわからずといった感じです…

何か根本的に勘違いをしているように思うのですが、

●何故「console.log(color)」とした場合、結果が「undefined」となってしまうのか
●希望する結果を得るにはどのように記述をすべきか

上記、ご教授いただけないでしょうか?
試したソースは無視していただいて、もし「if」を使用するよりも良い記述の方法等が
ありましたらそちらも教えていただますととても助かります。
どうぞよろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+2

何故「console.log(color)」とした場合、結果が「undefined」となってしまうのか

変数colorは初期化せずに宣言された変数なので、undefinedとなります。

希望する結果を得るにはどのように記述をすべきか

以下のように記述してはいかがでしょうか?

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <title>タイトル</title>
</head>
<body>
<button onclick="judgeFruit('apple', 'red');">果物A</button>
<br>
<button onclick="judgeFruit('apple', 'green');">果物B</button>
<script>
    function judgeFruit(fruit, color) {
        if (fruit === "apple" && color === "red") {
            console.log('赤りんご');
        } else if (fruit === "apple" && color === "green") {
            console.log('青りんご'); //赤りんごと表示されてしまう
        }
    }
</script>
</body>
</html>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/07/16 04:22

    早速ご回答いただきましてありがとうございます。
    値はクォーテーションで囲って文字列として渡せるのですね。知りませんでした。
    とても助かりました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.21%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる