if文を用いた画像の表示について

Question

前提・実現したいこと

javascriptにて
ボックス内に特定の文字列を入力した状態でボタンをクリックした際に画像１が表示され、
それ以外の文字列が入力されたときは画像２～６うちのどれかがランダムで表示
上記の動きを行いたいのですがうまくいきません。ご教授よろしくお願いいたします。

ソースコード途中経過

(function(){
  'use strict';

var btn = document.getElementById('btn');
var do = document.getElementById('do');
var result = document.getElementById('result');
var reset = document.getElementById('reset');
var img = document.getElementById("image_place");

//ボタンをクリックしたときの処理
btn.addEventListener("click",function(){
  var min = 1 ;
  var max = 5 ;
  var n = Math.floor( Math.random() * (max + 1 - min) ) + min ;
  if(do = "特定の文字列";){
    document.write(' <img src="image/youwin.png" alt="title" width="100" height="100" ')
  }
  else{
    document.write(' <img src="image/youlose[n].png" alt="title" width="100" height="100" ')
  }
   /*var choise = [do1,do2];
   var n = Math.floor(Math.random()*choise.length);*/
    reset.classList.remove("hidden");
  });

reset.addEventListener("click",function(){
 result.textContent = "結果を表示";
 do.value ="";
 do.focus();
});


do.focus();
})();

Answer 1

以下を修正すると動きそうですがいかがでしょうか。

• doは予約語なので変数として利用できない。
• ifのconditionが正しくない。また、doはElementなのでそのまま文字列と比較しても思い通りの結果にはならない。
• youlose[n].pngにある[n]のような書き方で変数の展開はできない。
• もしimgを利用するのであればdocument.writeではなく、setAttributeを使う方が良さそう。

Comments

[m.ts10806]

> if(do = "特定の文字列";){
そもそも↑は構文エラー　というのも追加（変なところに;がある）

・・・あ、「ifのconditionが正しくない」に集約されてますね。

Answer 2

if文に使う比較演算子は==や===です。
=だと、変数に値を代入するという意味です。
値を代入すると、代入した値が戻り値となるので、そのif文は必ず実行されてしまうことになります。

また、doは文字列ではなくElementです。
それが入力欄などであればdo.value、
contenteditable属性を持つ要素であればdo.innerTextで、入力された文字列を取得できます。