For循环未贯穿长度

时间:2019-03-14 00:55:53

标签: javascript binary decimal converters

我无法弄清楚我做错了什么,这导致我的for循环无法遍历我的长度。我正在尝试使用用户输入将二进制转换为十进制,但是它不起作用。我拥有的是

已编辑

var val = $("txtIn").value;
if (cT[1].checked) {
  var bVal = val;
  if (isNaN(bVal)) {
    alert(val + " is not a number. A number is required to run the program.");
  } else if ((val % 1) !== 0) {
    alert(val + " is not an integer. A whole number is required to run the program.");
  } else if (bVal < 0) {
    alert(val + " is not able to convert. Input must be positive integer.");
  } else {
    convertByArrayB(bVal);
  }
  }
  
  function convertByArrayB(bVal) {
    var r, i, j;

    for (i = 0; i < bVal.length; i++) {
      r = bVal.charAt(i);
      if (r !== '1' && r !== '0') {
        alert("You did not enter a valid binary number. Please try again!");
      }

      var nv = parseInt(r, 2);
      
    }

    $("txtOut").value = nv;

  }

我认为您不需要顶部,但是比后悔更安全。预先感谢您的任何帮助。 (爱这个社区,顺便说一句)

1 个答案:

答案 0 :(得分:1)

您要在循环内更改bVal

bVal = nv;

因此,bVal.length的值在下一次迭代时为undefined,并且循环停止。

调用parseInt()的代码应在循环后 而不是循环内。无需重新分配bVal,它应该解析bVal,而不是r[i]

function convertByArrayB(bVal) {
  var r, i, j;

  for (i = 0; i < bVal.length; i++) {
    r = bVal.charAt(i);
    if (r !== '1' && r !== '0') {
      alert("You did not enter a valid binary number. Please try again!");
      return;
    }
  }
  var nv = parseInt(bVal, 2);

  document.getElementById("txtOut").value = nv;

}

convertByArrayB("101");
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
Result: <input id="txtOut">