1

我正在尝试检查一个单词中是否有多个字母(每个单词都是 5 个字母),并且我在一个名为 library 的变量中有一个 8938 个单词的数组。

           function checkLetters(word){
                var constant = 0;
                for(i=0;i<5;i++){
                    for(j=i+1;j<5;j++){
                        if(word.charAt(i) == word.charAt(j)){
                            return false;
                            break;
                        } else {
                            return true;
                        }
                    }
                }
            }
            if(compPrompt == 5 && checkLetters(compChoice) == true){
                console.log(checkLetters(compChoice));
                $('.right').append('<p id="winQuestion">Is "'+compFinalChoice+'" your word?');
                $('.right').append('<div class="yesNo" id="yesQ"><p>Yes</p></div>');
                $('.right').append('<div class="yesNo" id="noQ"><p>No</p></div>');
                $('#yesQ').click(function() {
                    alert('Your word is "'+compFinalChoice+'". You lose!');
                    document.location.reload(true);
                });
                $('#noQ').click(function() {
                    $('.yesNo').remove();
                    $('#winQuestion').remove();
                    var splitCompChoice = compFinalChoice.split('');
                    var constant = 0;
                    var possibleAnswers = [];
                    for(var k=0;k<8939;k++){
                        for(var i=0;i<5;i++){
                            if(library[k].indexOf(splitCompChoice[i]) > -1){
                                constant++;
                                if(constant = 5 && checkLetters(library[k]) == true){
                                    possibleAnswers.push(library[k]);
                                }
                            }
                        }
                });
            }

我的代码有两个问题。一是它正在过去的if(compPrompt == 5 && checkLetters(compChoice) == true){时间checkLetters(compChoice)是不真实的,二是它告诉我library[k]没有定义。

4

2 回答 2

0

在第一个函数上,如果第一个字符不等于第二个字符,则返回 true,为了修复它,您应该更改:

function checkLetters(word){
    var constant = 0;
    for(i=0;i<5;i++){
        for(j=i+1;j<5;j++){
            if(word.charAt(i) == word.charAt(j)){
                return false;
                break;
            } else {
                return true;
            }
        }
     }
}

到:

function checkLetters(word){
    var constant = 0;
    for(i=0;i<5;i++){
        for(j=i+1;j<5;j++){
            if(word.charAt(i) == word.charAt(j)){
                return false;
                break;
            } 
        }
    }
    return true;
}

其次,由于我们看不到library您在此处发布的代码中声明的位置,我相信它可能在某处定义为var(局部变量),这意味着如果您希望能够在其他一些函数中使用它应该通过它。

更新:
关于library- 尝试更改代码如下:

$('#noQ').click(function(library) {
    $('.yesNo').remove();
    $('#winQuestion').remove();
    var splitCompChoice = compFinalChoice.split('');
    var constant = 0;
    var possibleAnswers = [];
    for(var k=0;k<8939;k++){
        for(var i=0;i<5;i++){
            if(library[k].indexOf(splitCompChoice[i]) > -1){
                constant++;
                if(constant = 5 && checkLetters(library[k]) == true){
                    possibleAnswers.push(library[k]);
                }
            }
        }
}(library));
于 2013-08-05T14:17:26.983 回答
0

为了解决当 checkLetters 函数返回 false 时通过 if 语句的问题,我会将语句从

if(compPrompt == 5 && checkLetters(compChoice) == true)

if(compPrompt == 5 && checkLetters(compChoice))
于 2013-08-05T14:22:11.887 回答