3

我看到我正在处理的一些 Scala 2.7.7 代码存在问题，如果它是用 Java 编写的，则不应该发生这种情况。松散地，代码创建了一堆纸牌玩家并将他们分配给桌子。

class Player(val playerNumber : Int)

class Table (val tableNumber : Int) {
    var players : List[Player]  = List()

    def registerPlayer(player : Player) {
        println("Registering player " + player.playerNumber + " on table " + tableNumber)
        players = player :: players
    }
}

object PlayerRegistrar  {
    def assignPlayersToTables(playSamplesToExecute : Int, playersPerTable:Int) = {
        val numTables = playSamplesToExecute / playersPerTable
        val tables = (1 to numTables).map(new Table(_))
        assert(tables.size == numTables)

        (0 until playSamplesToExecute).foreach {playSample =>
            val tableNumber : Int = playSample % numTables
            tables(tableNumber).registerPlayer(new Player(playSample))
        }
        tables
    }
}

PlayerRegistrar 在桌子之间分配一些玩家。首先，它计算出需要多少张桌子才能将玩家分开并创建他们的列表。

然后在代码的第二部分，它计算出应该将玩家分配到哪个桌子，从列表中拉出该桌子并在该桌子上注册一个新玩家。

桌子上的玩家列表是一个 var，每次调用 registerPlayer() 时都会被覆盖。我通过一个简单的 TestNG 测试检查了它是否正常工作：

@Test def testRegisterPlayer_multiplePlayers() {
    val table = new Table(1)
    (1 to 10).foreach { playerNumber =>
        val player = new Player(playerNumber)
        table.registerPlayer(player)
        assert(table.players.contains(player))
        assert(table.players.length == playerNumber)
    }
}

然后我测试表分配：

  @Test def testAssignPlayerToTables_1table() = {
    val tables = PlayerRegistrar.assignPlayersToTables(10, 10)
    assertEquals(tables.length, 1)
    assertEquals(tables(0).players.length, 10)
}

测试失败并显示“预期：<10> 但为：<0>”。我一直在摸不着头脑，但无法弄清楚为什么 registerPlayer() 没有改变列表中的表格。任何帮助，将不胜感激。

4

2 回答 2

4

原因是在该assignPlayersToTables方法中，您正在创建一个新Table对象。您可以通过在循环中添加一些调试来确认这一点：

val tableNumber : Int = playSample % numTables
println(tables(tableNumber))
tables(tableNumber).registerPlayer(new Player(playSample))

产生类似的东西：

Main$$anon$1$Table@5c73a7ab
Registering player 0 on table 1
Main$$anon$1$Table@21f8c6df
Registering player 1 on table 1
Main$$anon$1$Table@53c86be5
Registering player 2 on table 1

注意每个调用的表的内存地址是如何不同的。

这种行为的原因是 aRange在 Scala 中是非严格的（无论如何，直到 Scala 2.8）。这意味着在需要之前不会评估对范围的调用。因此，您认为您正在返回一个Table对象列表，但实际上您正在返回一个范围，Table每次调用它时都会对其进行评估（实例化一个新对象）。同样，您可以通过添加一些调试来确认这一点：

val tables = (1 to numTables).map(new Table(_))
println(tables)

这给了你：

RangeM(Main$$anon$1$Table@5492bbba)

要执行您想要的操作，toList请在末尾添加一个：

val tables = (1 to numTables).map(new Table(_)).toList
于 2010-01-31T21:14:46.547 回答
2 
val tables = (1 to numTables).map(new Table(_))

这条线似乎造成了所有麻烦 - 映射过度1 to n给你一个RandomAccessSeq.Projection，老实说，我不知道它们是如何工作的，但是有点不那么聪明的初始化技术可以完成这项工作。

var tables: Array[Table] = new Array(numTables)
for (i <- 0 to numTables) tables(i) = new Table(i)

使用第一种初始化方法我无法更改对象（就像你一样），但是使用一个简单的数组似乎一切正常。

于 2010-01-31T21:12:51.557 回答