javascript - 在 innerHTML 中添加 p 元素后，input[type ="text"] 的值未显示

Question

我的代码有一个奇怪的问题(link 或下面的代码)。

我想创建一个简单的游戏来练习基本的 Javascript。

这个游戏的概念是:给定(输入)8 个插槽。用十六进制数字 (00 - FF) 填充所有插槽。按 Check 按钮，看看您猜到了多少个值。 (用于与您的输入进行比较的数字是使用 getRandomHexValues() 随机生成的)。

在js文件中，函数checkValues()生成8个十六进制数；在 div#lotoResult 中插入 8 个输入(每个包含一个生成的数字)。 div#lotoResult 中的每个输入 都有红色或绿色的边框，具体取决于值是否被正确猜中。然后它添加一个 p 元素，显示有多少错误。

问题是:如果我将 p 元素添加到 div#lotoResult 的 innerHTML 中，input< 中的值/em>s(由 checkValues() 生成)未显示。如果我不添加 p，则会显示元素(但我需要这个 p)。

我想了解问题所在(我已经尝试过，但做不到)。你有什么建议吗？

function checkValues() {
  var inputs = document.getElementsByName("num");
  var inputOk = true;

  for (i = 0; i < inputs.length; ++i) {
    if (inputs[i].value == "") {
      inputOk = false;
      break;
    }
  }

  var lotoResult = document.getElementById("lotoResult");
  lotoResult.style.visibility = "visible";
  if (!inputOk) {
    lotoResult.style.color = "red";
    lotoResult.innerHTML = "!!! Not all slots are filled.";
  } else {
    var genValues = getRandomHexValues();
    for (i = 0; i < inputs.length; ++i) {
      lotoResult.innerHTML += "<input type=\"text\" pattern=\"[a-fA-F0-9]{2}\" name=\"numResult\" readonly />";
    }

    var lotoResultValues = document.getElementsByName('numResult');
    for (i = 0; i < inputs.length; ++i) {
      lotoResultValues[i].value = genValues[i];
    }

    var allValuesTrue = true;
    var nrWrongValues = 0;
    for (i = 0; i < inputs.length; ++i) {
      if (inputs[i].value.toUpperCase() != genValues[i].toUpperCase()) {
        allValuesTrue = false;
        lotoResultValues[i].style.border = "1px solid red";
        nrWrongValues++;
      } else
        lotoResultValues[i].style.border = "1px solid green";
    }

    // If I remove if-else, then the values in input elements will be displayed
    if (!allValuesTrue) {
      lotoResult.style.color = "red";
      lotoResult.innerHTML += "<p>!!! " + nrWrongValues + " wrong values.</p>";
    } else {
      lotoResult.style.color = "green";
      lotoResult.innerHTML += "<p>All elements are correct.</p>";
    }

  }
}


function getRandomHexValues() {
  var length = 2;
  var chars = "0123456789ABCDEF";
  var hex = "";
  var hexNums = new Array();
  var n = 8;

  for (i = 0; i < n; ++i) {
    length = 2;
    hex = "";
    while (length--) hex += chars[Math.floor(Math.random() * 16)];
    hexNums[i] = hex;
  }
  return hexNums;
}

body {
  margin: 10px auto;
  border: 2px solid red;
  width: 800px;
}
header {
  height: 10%;
  color: orange;
  width: 500px;
}
section {
  padding: 10px;
  border: 1px solid blue;
  display: block;
  width: 600px;
  float: left;
  margin: 20px 10px;
}
section p {
  color: red;
  display: inline-block;
  margin: 10px 10px;
}
footer {
  clear: both;
  margin-top: 50px;
}
footer p {
  text-align: center;
}
#S2 h1 {
  margin-left: 10px;
  color: red;
  font-family: sans-serif;
}
#S2 div {
  margin: 0 auto;
  width: 400px;
}
#S2 div input {
  width: 30px;
  border: 1px solid blue;
  color: blue;
}
#S2 p {
  display: block;
  color: blue;
  font-family: monospace;
}
#S2 .buttonSubmit {
  border-radius: 3px;
  border: 1.5px solid blue;
  width: 60px;
  position: relative;
  left: 20px;
  transition: background-color 1s;
}
#S2 .buttonSubmit:hover {
  background: rgba(26, 198, 255, 0.7);
}
#S2 .buttonSubmit:active {
  color: red;
}
#S2 #lotoResult {
  text-align: center;
  margin-top: 5px;
  visibility: hidden;
  color: blue;
  border: 1px solid orange;
  padding: 10px;
}

<header>
  <h1>HelloJavascript</h1>
</header>
<nav>
</nav>
<section id="S2">
  <h1>Game</h1>
  <p>Insert the numbers below:</p>

  <div class="tabs">
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input type="text" pattern="[a-fA-F0-9]{2}" name="num" />
    <input class="buttonSubmit" type="button" onclick="checkValues()" value="Check" />
  </div>
  <div id="lotoResult"></div>
</section>
<footer>
  <p>
    &copy; 2016
  </p>
</footer>

Answer 1

使用 innerHTML 并不是真正的最佳实践。进行“真正的”DOM 操作将解决您的问题:

if (!allValuesTrue) {
    lotoResult.style.color = "red";
    //lotoResult.innerHTML += "<p>!!! " + nrWrongValues + " wrong values.</p>";

    var para = document.createElement('p');
    para.appendChild(document.createTextNode("!!! " + nrWrongValues + " wrong values."));
    lotoResult.appendChild(para);
}