[英]javascript password generator not working
最近,我在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
阶梯,实现使用循环一样。
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;
}
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.