1

在我正在制作的 Yahtzee 游戏中,我创建了一个有效的 javascript 函数来检查一个包含 5 个数字的数组中的一个小顺子。我已经对它进行了无止境的测试,我相信它 100% 的时间都有效,但就效率而言,它也可能是有史以来最差的算法。这是它的样子:

function calcSmstraight() {
        var sum = 0;
        var r = new Array();
        var r2 = new Array();
        var counter = 0;
        var temp;
        var bool = false;
        var bool2 = false;
        r[0] = document.getElementById('setKeep1').value;
        r[1] = document.getElementById('setKeep2').value;
        r[2] = document.getElementById('setKeep3').value;
        r[3] = document.getElementById('setKeep4').value;
        r[4] = document.getElementById('setKeep5').value;

        // Move non-duplicates to new array
        r2[0] = r[0];
        for(var i=0; i<r.length; i++) {
            for(var j=0; j<r2.length; j++) {
                if(r[i] == r2[j]) {
                    bool2 = true;   // Already in new list
                }
            }
            // Add to new list if not already in it
            if(!bool2) {
                r2.push(r[i]);
            }
            bool2 = false;
        }
        // Make sure list has at least 4 different numbers
        if(r2.length >= 4) {
            // Sort dice from least to greatest
            while(counter < r2.length) {
                if(r2[counter] > r2[counter+1]) {
                    temp = r2[counter];
                    r2[counter] = r2[counter+1];
                    r2[counter+1] = temp;
                    counter = 0;
                } else {
                    counter++;
                }
            }
            // Check if the dice are in order
            if(((r2[0] == (r2[1]-1)) && (r2[1] == (r2[2]-1)) && (r2[2] == (r2[3]-1)))
                || ((r2[1] == (r2[2]-1)) && (r2[2] == (r2[3]-1)) && (r2[3] == (r2[4]-1)))) {
                bool = true;
            }
        }

        if(bool) {
            // If small straight give 30 points
            sum = 30;
        }

        return sum;
}

我的策略是:

1) Remove duplicates by adding numbers to a new array as they occur

2) Make sure the new array is at least 4 in length (4 different numbers)

3) Sort the array from least to greatest

4) Check if the first 4 OR last 4 (if 5 in length) numbers are in order

我的问题:

有谁知道我可以改进这种方法的方法?这对我来说似乎非常糟糕,但我想不出更好的方法来做到这一点,而且它至少有效。

4

3 回答 3

7

鉴于您正在实现 Yahtzee 游戏,您可能需要测试除小直道之外的其他模式,因此最好在调用函数之前创建值数组,以便您可以在所有测试中使用它们,而不是获取小直测试里面的 DOM 元素的值。

无论如何,这是我想到的第一种方法来测试代表五个六面骰子值的数组中的小直线:

    // assume r is an array with the values from the dice
    r.sort();
    if (/1234|2345|3456/.test(r.join("").replace(/(.)\1/,"$1") {
        // is a small straight
    }

请注意,您可以使用以下代码对数字数组进行排序:

r2.sort(function(a,b){return a-b;});

...但是在您的情况下,数组中的值是字符串,因为它们来自.valueDOM 元素的属性,因此默认字符串排序将与r2.sort(). 无论哪种方式,您都不需要自己的排序例程,因为 JavaScript提供了一个.

编辑:如果您假设您可以像上面那样将五个值作为一个字符串,您可以将所有可能组合的测试实现为一个大的 if/else,如下所示:

r.sort();
r = r.join("");
if (/(.)\1{4}/.test(r)) {
    alert("Five of a Kind");
} else if (/(.)\1{3}/.test(r)) {
    alert("Four of a Kind");
} else if (/(.)\1{2}(.)\2|(.)\3(.)\4{2}/.test(r)) {
    alert("Full House");
} else if (/(.)\1{2}/.test(r)) {
    alert("Three of a Kind");
} else if (/1234|2345|3456/.test( r.replace(/(.)\1/,"$1") ) {
    alert("Small Straight");
} // etc.

演示:http: //jsfiddle.net/4Qzfw/

于 2013-09-04T14:11:33.103 回答
0

为什么不只使用一个六元素布尔数组来指示数字是否存在,然后检查 1-4、2-5 和 3-6 是否全部为真?在伪代码中:

numFlags = array(6);
foreach(dice)
    numFlags[die.value-1] = true;
if(numFlags[0] && numFlags[1] && numFlags[2] && numFlags[3]) return true
//Repeat for 1-4 and 2-5
return false

如果您使用百万面骰子,这将不是一个有用的算法,但对于六面骰子,只有三个可能的小顺子要检查,所以它简单明了。

于 2013-09-04T14:09:11.957 回答
0

我不玩 Yahtzee,但我玩纸牌,看起来算法可能相似。这个用 ActionScript 编写的例程(我的 JavaScript 有点生疏)已经编译但未经测试。它应该接受 5 张牌作为输入,并为大于 3 张牌或对子或更高的顺子返回一条消息。

private function checkCards(card1:int,card2:int,card3:int,card4:int,card5:int):String
{
    // Assumes that the 5 cards have a value between 0-12 (Ace-King)
    //The key to the routine is using the card values as pointers into an array of possible card values.
    var aryCardValues:Array = new Array(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0);
    aryCardValues[card1] += 1;
    aryCardValues[card1] += 1;
    aryCardValues[card1] += 1;
    aryCardValues[card1] += 1;
    aryCardValues[card1] += 1;
    var aryCardNames:Array = new Array("Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Jack", "Queen", "King");
    var strOutMessage:String;
    var intCardCount:int = 0;
    var strSeperator:String;
    var strHighCard:String;
    for (var i:int = 0;i < aryCardValues.length;i++)
    {
        //Check for runs of three of a kind or greater.
        if (aryCardValues[i] >= 2) 
        {
            strOutMessage = strOutMessage + strSeperator + i + "-" + aryCardNames[i] + "s";
            strSeperator = " & ";
        }
        //Check for values in a straight.
        if (aryCardValues[i] > 0)
        {
            intCardCount++;
            if (intCardCount > 3)strHighCard = aryCardNames[i];
        }
        else
        {
            if (intCardCount < 3)intCardCount = 0;
        }
    }
    if (intCardCount > 3) strOutMessage = intCardCount + " run " + strHighCard + " High."
    return strOutMessage;
}

它可能不像上面使用的正则表达式那样简洁,但它可能更具可读性和更容易修改。可以进行的一项更改是传入一组卡片,而不是每张卡片的离散变量。

于 2013-09-05T11:01:56.537 回答