if 语句 - JavaScript 密码生成器不起作用

if statement - javascript password generator not working

本文关键字：不起作用 密码 语句 JavaScript if      更新时间：2023-09-26

>最近我在javascript中启动了一个新项目(随机密码生成器(，但有些东西还没有完成。(我只是js的初学者(

我

有 4 个字符串，根据选中的框，我希望它们连接在一个字符串中，然后从这个字符串中随机选择字母，当然，如果您知道更好的方法，请告诉我，正如我之前所说，我是 js 的初学者。

var uppercase_letters = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'];
var lowercase_letters = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'];
var numbers = ['1', '2', '3', '4', '5', '6', '7', '8', '9', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '0'];
var symbols = ['/', '[', ']', '$', '-', ':', '_', '?', '{', '}', '~', '!', '@', '#', '%', '&', '*', '(', ')', '"', '^', '§', ',', '.', '°', '+', '-', ':'];
var length;
//--------------------
var check1;
var check2;
var check3;
var check4;
var inputnum;
function getvalues() {
  inputnum = document.getElementById("length").value;
  length = inputnum;
  check1 = document.getElementsByClassName("check")[1];
  check2 = document.getElementsByClassName("check")[2];
  check3 = document.getElementsByClassName("check")[3];
  check4 = document.getElementsByClassName("check")[4];
  generatestring();
}
function generatestring() {
  var string;
  if (check1.checked === true) {
    string = string.concat(uppercase_letters)
  };
  if (check2.checked === true) {
    string = string.concat(lowercase_letters)
  };
  if (check3.checked === true) {
    string = string.concat(numbers)
  };
  if (check4.checked === true) {
    string = string.concat(symbols)
  };
  alert("Working?");
  show(string);
}
function show(string) {
  alert(string);
}
<div class='container'>
  <input type='checkbox' class='check' value='uppercase'>uppercase letters
  <br />
  <input type='checkbox' class='check' value='lowercase'>lowercase letters
  <br />
  <input type='checkbox' class='check' value='number'>numbers
  <br />
  <input type='checkbox' class='check' value='symbol'>symbols
  <br />
  <input type='number' id='length' value='length'>
  <br />
  <br />
  <input type='button' id='generate' value='generate' onclick="getvalues()">
</div>

因此，带有

字符串的警报(仅用于测试(没有显示，这就是为什么我添加了带有消息"正在工作？"的警报，而这个警报也没有显示。我不知道为什么...我确信问题是这些"如果"，但我不知道如何以另一种方式做到这一点:(有人可以帮我吗？

可能我做的一切都完全错了，所以请给我一些提示。

谢谢！

  • document.getElementsByClassName计数从 0 开始，而不是 1。所以它应该是 0、1、2 和 3
  • var string undefined.您需要 2 个字符串才能使用concat

工作代码：

var uppercase_letters = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'];
var lowercase_letters = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'];
var numbers = ['1', '2', '3', '4', '5', '6', '7', '8', '9', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '0'];
var symbols = ['/', '[', ']', '$', '-', ':', '_', '?', '{', '}', '~', '!', '@', '#', '%', '&', '*', '(', ')', '"', '^', '§', ',', '.', '°', '+', '-', ':'];
var length;
//--------------------
var check1;
var check2;
var check3;
var check4;
var inputnum;
function getvalues() {
  inputnum = document.getElementById("length").value;
  length = inputnum;
  check1 = document.getElementsByClassName("check")[0];
  check2 = document.getElementsByClassName("check")[1];
  check3 = document.getElementsByClassName("check")[2];
  check4 = document.getElementsByClassName("check")[3];
  generatestring();
}
function generatestring() {
  var string = '';
  if (check1.checked === true) {
    string = string.concat(uppercase_letters)
  };
  if (check2.checked === true) {
    string = string.concat(lowercase_letters)
  };
  if (check3.checked === true) {
    string = string.concat(numbers)
  };
  if (check4.checked === true) {
    string = string.concat(symbols)
  };
  alert("Working?");
  show(string);
}
function show(string) {
  alert(string);
}
<div class='container'>
  <input type='checkbox' class='check' value='uppercase'>uppercase letters
  <br />
  <input type='checkbox' class='check' value='lowercase'>lowercase letters
  <br />
  <input type='checkbox' class='check' value='number'>numbers
  <br />
  <input type='checkbox' class='check' value='symbol'>symbols
  <br />
  <input type='number' id='length' value='length'>
  <br />
  <br />
  <input type='button' id='generate' value='generate' onclick="getvalues()">
</div>

getElementsByClassName 返回一个类似数组的对象，该对象使用从 0 开始的索引。

当您访问各个节点时，您使用了从 1 开始的索引。

check1 = document.getElementsByClassName("check")[1];
check2 = document.getElementsByClassName("check")[2];
check3 = document.getElementsByClassName("check")[3];
check4 = document.getElementsByClassName("check")[4];

由于您只有 4 个复选框，最后一个索引是 3，因此访问 [4] 返回 undefined，随后会引发错误，您可以在浏览器控制台中看到该错误。

虽然@Nick是正确的，但你可以把这个答案看作是增强。

指针，您应该避免全局变量。这是一种不好的做法。

其次，您可以消除梯形图if并使用循环实现相同的梯子。

样本

JSFiddle

var characters = [uppercase_letters, lowercase_letters, numbers, symbols];
.
.
.
var string = "";
var els = document.getElementsByClassName("check");
for (var i = 0; i < els.length; i++) {
  string = (els[i].checked) ? string.concat(characters[i]) : string;
}
相关文章：