嵌套循环有问题 (R)

Question

我编写了这个嵌套循环，以便在内部循环中，代码贯穿第一行； 然后，外层更新循环以允许内层运行通过第二行（依此类推）。 数据来自矩阵“supergerador”。 “rodadas”是行大小，“n”是列大小。 “vec”是感兴趣的向量。 先感谢您！

编辑： i, j 最初被分配 i = 1, j = 2

for(e in 1:rodadas) {
  for(f in 1:(n-1)) {
    if(j >= 10) {
      vec[f] = min(supergerador[i, j] - supergerador[i, j - 1], 1 - supergerador[i, j])
    }
    else {
    vec[f] = func(i, j)
    }
    j = j + 1
  }
  i = i + 1
}

func定义为

func = function(i, j) {
  minf = min(supergerador[i, j] - supergerador[i, j - 1], supergerador[i, j + 1] - supergerador[i, j])
  return(minf)
}

作为参考，这是嵌套循环返回的内容。 你可以说它只经过了一行。

> vec
[1] 0.127387378 0.068119707 0.043472981 0.043472981 0.027431603 0.027431603
[7] 0.015739046 0.008010766 0.008010766

Answer 1

我不太确定您打算在这里做什么，但这里有一些建议和代码编辑：

建议：

1. 如果您有一个 for 循环，请为您的子集使用循环索引（尽可能合理），并在合理的情况下避免使用其他索引。
   • 当索引应该被重置但没有被重置时，这可以避免代码混乱和不可预见的错误。
2. 尽可能避免双重子集变量。 例如，如果您多次调用x[i, j] ，请将其存储在一个变量中，然后在您的结果中使用此变量。
3. 单行函数很好，但应该增加代码的可读性。 否则，从效率的角度来看，内联代码是最佳的。

将这些合并到您的代码中，我相信您正在寻找

for(i in 1:rodadas) {
  for(j in 2:n) {
    x1 = supergerador[i, j]
    x2 = supergerador[i, j - 1]
    if(j >= 10) {
      vec[f] = min(x1 - x2, 1 - x1)
    }
    else {
      vec[f] = min(x1 - x2, supergerador[i, j + 1] - x1)
    }
  }
}

在这里，我假设您希望将每一行的列循环到rodadas 。

一旦您对 R 更加熟悉，您应该研究一下矢量化。 对您的问题有了更多了解，我们应该可以很容易地向量化您的第二个 for 循环，删除您的if语句并在 1 次快速扫描中执行计算。 但在那之前，这是一个开始您的编程体验的好地方，并且对for-loops有深刻的理解对于任何语言都至关重要。