Question

我正在尝试根据用户提供的选项创建密码生成器。我当前的脚本允许用户选择大写，小写，数字和特殊字符。这完美地工作并且生成字符串以达到用户所需的长度，但是在生成时，数字聚集在字符串处，字母在开头处聚类。一个特殊字符分为两个。您对如何改进流程有什么建议吗？

$('document').ready(function() {
$('button').click(function() {
    var lower = "";
    var upper = "";
    var numeric = "";
    var special = "";
    var string_length = "";


    if($('#12').is(':checked')) { string_length = 12; };
    if($('#16').is(':checked')) { string_length = 16; };
    if($('#18').is(':checked')) { string_length = 18; };
    if($('#22').is(':checked')) { string_length = 22; };
    if($('#24').is(':checked')) { string_length = 24; };
    if($('#custom').is(':checked')) { $('#custom').show(); $('#custom').val(); } else { $('#custom').hide(); };

    if($('#ch1').is(':checked')) { lower = "abcdefghijklmnopqrstuvwxyz"; } else { lower = ""; };
    if($('#ch2').is(':checked')) { upper = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; } else { upper = ""; };
    if($('#ch3').is(':checked')) { numeric = "0123456789"; } else { numeric = ""; };
    if($('#ch4').is(':checked')) { special = "!£$%^&*()_-+={};:@~#?/"; } else { special = ""; };

    var chars = lower + upper + numeric + special;

    var randomstring = '';
    var charCount = 0;
    var numCount = 0;

    for (var i=0; i<string_length; i++) {
        if((Math.floor(Math.random() * 2) == 0) && numCount < 3 || charCount >= 5) {
            var rnum = Math.floor(Math.random() * 10);
            randomstring += rnum;
            numCount += 1;
        } else {
            var rnum = Math.floor(Math.random() * chars.length);
            randomstring += chars.substring(rnum,rnum+1);
            charCount += 1;
        }
    }
    $('span.string').html(randomstring);
});
});

选项16长度，小写，大写，数字和特殊字符返回类似 e046pzw％65760294 的内容。

Answer 1

这一行是你的罪魁祸首：

if((Math.floor(Math.random() * 2) == 0) && numCount < 3 || charCount >= 5) {

它说：

前3个字符有超过50/50的机会成为数字。 “then”始终是一个数字，“else”是一个有时取决于选项的数字。
你有5个“其他”选择的字符（这意味着在第8列之后），你将总是有一个数字。

这是因为“＆amp;＆amp;”优先于“||”。如果你想要有50/50的机会使用数字，我建议使用一些括号来包围OR子句。我还提供了另一种方法来做50/50。

if ((Math.random() < 0.5) && (numCount < 3 || charCount >= 5)) {

我不确定为什么你希望数字具有优先权。

Answer 2

另一种解决方案。只差五分钱：

$(function(){

    $('input, select').change(function(){

        var s = $('input[type="checkbox"]:checked').map(function(i, v){
                return v.value;
            }).get().join(''),
            result = '';

        for(var i=0; i < $('#length').val(); i++)
            result += s.charAt(Math.floor(Math.random() * s.length));

        $('#result').val(result);        
    });

});

只是为了给你一些想法。我完全清楚这不会考虑任何“类型计数”。

http://jsfiddle.net/m5y3e/

JQuery字符串'randomizer'聚类字符

2 个答案: