質問をすることでしか得られない、回答やアドバイスがある。

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

ただいまの
回答率

90.77%

  • JavaScript

    15343questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    8339questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Javascriptでコード間違いを指摘してください(if構文で画像表示を行いたい)

受付中

回答 3

投稿

  • 評価
  • クリップ 1
  • VIEW 409

web.beginner

score 2

``````###前提・実現したいこと
if構文を使って画像の表示を行いたい
”春”と入力したら指定された画像を表示。
”夏”や”秋”も同様。

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

エラーは出ないのにfunction内が実行されません。
実行して画像表示を行いたい。

該当のソースコード

<script>
function color()
{
var a; //コップaを用意する
var ans=document.getElementById("img");    //コップansを用意し、表示(出力)先とする
a=window.document.myform.a.value-0;           //ボックスaに入力された数字をコップaに入れる

if (a="春" || "Spring") {
ans.src="PCspring.png";
ans.alt="spring";
} else if (a="夏" || "Summer") {
ans.src="PCsummer.png";
ans.alt="summer";
} else if (a="秋" || "Autumn") {
ans.src="PCautumn.png";
ans.alt="autumn";
} else if (a="冬" || "Winter") {
ans.src="PCwinter.png";
ans.alt="winter";
} else {
ans.innerHTML="入力してください";
}
}
</script>
</head>
<body>

<div>
<form name="myform">
入力<br>
<input name="a" type="text">
<input type="button" value="色を見てみる"
onclick="color()"></form>
<div id="img">カラーチャート(画像)が出力</div>
<img id="img" name="img" >
</body>
</html>

試したこと

いろいろ書き替えてみましたが動かないです。。ご指摘お願いします。

補足情報(言語/FW/ツール等のバージョンなど)

なし

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • x_x

    2018/01/16 15:15

    コメントに「ボックスaに入力された数字をコップaに入れる」とありますが、数字ではない文字列と比較しています。どうしたかったのでしょうか?

    キャンセル

  • web.beginner

    2018/01/16 15:18

    すみません、前回作ったものを一部コピーしたため数字というのは間違いで、文字です。

    キャンセル

  • kei344

    2018/01/16 17:43

    質問文のコードはコードブロックで囲んでいただけませんか? ```(バッククオート3つ)で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

    キャンセル

回答 3

+5

idがかぶっています。divのほうを変えるか削除してください。

<div>カラーチャート(画像)が出力</div>

-- 15:42 動くように追記

<div id="ans">カラーチャート(画像)が出力</div>
<img id="img" alt="" />
function color() {
  var img = document.getElementById("img");
  var a = document.myform.a.value;
  img.style.display = 'inline';

  if (a === "春" || a === "Spring") {
    img.src="PCspring.png";
    img.alt="spring";
  } else if (a === "夏" || a === "Summer") {
    img.src="PCsummer.png";
    img.alt="summer";
  } else if (a === "秋" || a === "Autumn") {
    img.src="PCautumn.png";
    img.alt="autumn";
  } else if (a === "冬" || a === "Winter") {
    img.src="PCwinter.png";
    img.alt="winter";
  } else {
    document.getElementById('ans').innerHTML = "入力してください";
    img.style.display = 'none';
  }
}


switchのほうがいいかもしれません・・・

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/16 14:55

    回答ありがとうございます。
    <div id="img">カラーチャート(画像)が出力</div>

    <div>カラーチャート(画像)が出力</div>
    に変更したところ、ボックスに何を入力してもspringと出力されてしまいました。
    やはりif構文が実行されていないor間違いがあるようです。
    この出力部分に画像を表示したいのですが、お分かりになるでしょうか…?
    よろしくお願いいたします。

    キャンセル

  • 2018/01/16 15:01

    比較は===です。
    =一個だと代入になってしまいますね。
    ……でもまだ問題があります

    キャンセル

  • 2018/01/16 15:21

    -0があるため、数値になってしまっています。
    これも削除してみてください。

    キャンセル

  • 2018/01/16 15:29

    回答ありがとうございます。
    画像の表示がされました!数値と認識させてしまっていたためだったのですね。
    あと一点、if文でその他の文字を入力した場合に、”入力してください”と表示させたいのですが、
    表示されない上に画像が残ってしまいます。
    最後のelse文が間違っているのでしょうか?

    キャンセル

  • 2018/01/16 15:34

    ansは画像としてしまったので、divも扱うなら……。回答編集しますね。

    キャンセル

  • 2018/01/16 16:45

    ありがとうございます。
    一度実行されましたが、その後また実行されませんでした。
    もう一度自分で見直してみます。

    キャンセル

+2

a="春"では代入になってしまいます。if (a=="春" || a =="Spring") としましょう(残り3つも同様に書いてください)。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/16 15:10

    回答ありがとうございます。
    代入とはまた別なのですね。。
    ご指摘通り
    if (a=="春" || a =="Spring") に変更しましたが、やはり画像が表示されません。。。

    キャンセル

  • 2018/01/16 15:12

    あとは、「//ボックスaに入力された数字をコップaに入れる」とコメントがついているように、0を引くことで入力値が数値となってしまっています。0を引くのをやめてみましょう。

    キャンセル

  • 2018/01/16 15:26

    ありがとうございます!
    0は数値と判別させるためのものでしたね。。
    画像は出力されました!
    しかしif文でその他の文字を入力した場合に、”入力してください”と表示させたいのですが、
    表示されない上に画像が残ってしまいます。
    最後のelse文に間違いは見当たりますでしょうか。。?

    キャンセル

+2

演算子全般の使い方がハチャメチャです. 
=,==,||,-の使い方について再確認して下さい.
NOTE:
問題となっている部分が多すぎるため, 一つずつ問題を解決するように心がけましょう.

参考
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Operators

function color()
{
var a; //コップaを用意する
var ans=document.getElementById("img");    //コップansを用意し、表示(出力)先とする
//a=window.document.myform.a.value-0;           //ボックスaに入力された数字をコップaに入れる
a=window.document.myform.a.value;           //文字列に-0とするからおかしくなる

if (a=="春" || a=="Spring") {
ans.src="PCspring.png";
ans.alt="spring";
} else if (a=="夏" || a=="Summer") {
ans.src="PCsummer.png";
ans.alt="summer";
} else if (a=="秋" || a=="Autumn") {
ans.src="PCautumn.png";
ans.alt="autumn";
} else if (a=="冬" || a=="Winter") {
ans.src="PCwinter.png";
ans.alt="winter";
} else {
ans.innerHTML="入力してください";
}
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/16 15:09

    回答、参考URLありがとうございます。
    あまりにも初心者過ぎていろいろと混乱してしまいました。

    ご指摘通り直したのですが、画像が表示されません…

    キャンセル

  • 2018/01/16 15:31

    > あまりにも初心者過ぎていろいろと混乱してしまいました。
    初心者だからというのは免罪符にはなりません. わからなければ調べればいいだけの話です. また, 意味もわからずコピペでコードを継ぎ接ぎするのは下策の下策です.

    まずは, どのような仕組みを考え, それに対してどのようなコードを考えたか, そしてどのような結果が得られたかについて正確に相手に伝えられるようにトレーニングしましょう.

    キャンセル

  • 2018/01/16 15:37

    おっしゃる通りです。
    とはいえ意味もわからず継ぎ接ぎしているわけではないのですが、そういった印象を与えてしまう知識のなさ、勉強不足であることに自分自身ショックでした。本質的な理解が不十分だと痛感しております。
    もっとしっかりと勉強します。

    キャンセル

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

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

関連した質問

同じタグがついた質問を見る

  • JavaScript

    15343questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    8339questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。