随机化列表

Question

我编写了一个函数来从单词列表中随机选择一个单词。这是我的代码。 但它不能选择一个单词，也不能打印。请告诉我我的代码有什么问题。

def long(a: Int, b: Int): String = {
    var a = 5
    var b = 100
    for (i <- args(1)){
        if (i > a && i < b){
            val rand = new Random(System.currentTimeMillis())
            val random_index = rand.nextInt(new_sun .length)
            val result = new_sun(random_index)
            var guess = println("_ " * result.length)
            }
        else{
            println("You have to input the word with length of 5 < 100")
            }
        return i.toString
        }
}

Answer 1

这里有很多错误，以至于很难知道从哪里开始。一次取一个片段：

def long

真的，令人难以置信的糟糕方法的名称！

def long(a: Int, b: Int): String = {
  var a = 5
  var b = 100

您将 a 和 b 作为参数，然后立即隐藏名称以创建变量。这些参数没有什么用。

for (i <- args(1)){
    if (i > a && i < b){
        ...
        }
    else{
        ...
        }
    ...
    }

这不遵循任何公认的范围嵌套模式，在块的左大括号之前留一个空格也是一个很好的做法。更糟糕的是，for 块的右大括号对齐为看起来像 else 子句的右大括号。这是产生不可维护代码的有保证的方法。我是所谓的“一个真正的括号样式”的倡导者，您的代码格式如下：

for (i <- args(1)) {
    if (i > a && i < b) {
        val rand = new Random(System.currentTimeMillis())
        val random_index = rand.nextInt(new_sun .length)
        val result = new_sun(random_index)
        var guess = println("_ " * result.length)
    } else {
        println("You have to input the word with length of 5 < 100")
    }
    return i.toString
}

继续前进...

for (i <- args(1))

什么是 args(1)，它来自哪里？它必须是一个 Option 或某种要在 for 理解中使用的集合，并且其内容必须与 a 和 b 的类型相同>我>一个&&我< b 比较有效。所以我假设 args(1) 返回一个 Option[Int] ，集合似乎不太可能。

val random_index = rand.nextInt(new_sun .length)
val result = new_sun(random_index)

同样的问题。 new_sun 是什么以及它来自哪里。这个名字什么也没告诉我。 new_sun .length 中的空间看起来也很奇怪。

for (i <- args(1)) {
  if (i > a && i < b) {

更好的写法是：

for (i <- args(1) if i > a && i < b) {

或者

args(1) filter (a to b contains _) map { i =>

但遗憾的是，你不能这样做，因为尽管存在保护条件，但值 i 仍被用作返回值。

这也提醒我：

var a = 5
var b = 100
...
if (i > a && i < b) ...
else println("You have to input the word with length of 5 < 100")

这是检查 5 5 5 5 5 5 <我< 100，或者换句话说，6 <= i <= 99。检查与错误消息不匹配。

var guess = println("_ " * result.length)

println 返回Unit。绝对没有可能的理由将此返回值分配给从未使用过的变量。

return i.toString

Scala 中很少需要 return 语句，其中计算的最终表达式将成为返回值。但您不能简单地删除该关键字，因为它会与其他问题相互作用并会停止代码编译。

---

通常，您希望编写代码，以便 if/else 块、for 推导式和其他此类构造不会产生副作用（例如调用 println） 。相反，您应该尝试使这些块计算为 String，然后将计算出的字符串传递给方法末尾的 println。

您还需要更加了解您拥有哪些变量、它们来自哪里以及它们的名称包含多少信息。

并注意你的语法/布局/格式。程序员们为了更少的事情而以宗教方式进行斗争。

Answer 2

那么，为什么不简单地编写类似的代码来随机选择 List[String] 中的一个 String 呢？

def randomString(liste: List[String]) =
  util.Random.shuffle(liste).head

然后您可以创建另一个函数来在屏幕上打印结果

Answer 3

您的 println 不会尝试打印结果。尝试：（

println("result: " + result)

注意“+”是正确的字符串连接运算符，而不是“*”）

每次在内循环中重新创建 rand 也是不必要且低效的。