ruby-on-rails - 为什么初始化变量如此重要？

Question

有人可以向我解释一下，为什么不初始化 first_idx 和 last_idx 会导致代码无法运行？？

当我运行它时，出现此错误“未定义的局部变量或方法 last_idx”。我知道建议总是初始化变量，但我不明白为什么。毕竟 first_idx 和 last_idx 总是会在循环中得到一个值，因为参数 letter 总是出现在字符串中(在这个特定问题中)。

我真的很感激一些(简单的)见解。谢谢!

P.S，我也知道在 Ruby 中使用#index 和#rindex 很容易解决这个问题，但我不允许使用直接的方法来解决它。

def find_for_letter(string, letter)

first_idx = nil
0.upto(string.length - 1) do |idx1|
    if string[idx1] == letter
        first_idx = idx1
        break
    end
end

last_idx = nil
(string.length - 1).downto(0) do |idx2|
    if string[idx2] == letter
        last_idx = idx2
        break
    end
end

if last_idx == first_idx
    return [first_idx]
else
    return [first_idx, last_idx]
end
end



def first_last_indices(word)
    h = {}
    word.chars.each do |char|
        h[char] = find_for_letter(word, char)
    end
    h
end

Answer 1

block 中的变量

来自Ruby Programming Language :

Blocks define a new variable scope: variables created within a block exist only within that block and are undefined outside of the block. Be cautious, however; the local variables in a method are available to any blocks within that method. So if a block assigns a value to a variable that is already defined outside of the block, this does not create a new block-local variable but instead assigns a new value to the already-existing variable.

a = 0

2.times do
  a = 1
end

puts a #=> 1

b = 0

2.times do |i;b| # <- b will stay a block-local variable 
  b = 1
end

puts b #=> 0

2.times do |i|
  c = 1
end

puts c #=> undefined local variable or method `c' for main:Object (NameError)

重构你的代码

迭代字符和索引

这里有一个更小的方法来实现您的目标。它为每个字符保留一个带有 minmax 索引的散列。

默认哈希值是一个空数组。

该方法遍历每个字符(带索引)。

如果 minmax 数组已经包含 2 个值:

• 它用当前索引替换第二个(最大)。
• 否则将当前索引添加到数组。

def first_last_indices(word)
  minmax_hash = Hash.new { |h, k| h[k] = [] }
  word.each_char.with_index do |char, index|
    minmax = minmax_hash[char]
    if minmax.size == 2
      minmax[1] = index
    else
      minmax << index
    end
  end
  minmax_hash
end

p first_last_indices('hello world')
{"h"=>[0], "e"=>[1], "l"=>[2, 9], "o"=>[4, 7], " "=>[5], "w"=>[6], "r"=>[8], "d"=>[10]}

与group_by

还有另一种可能性。它使用 group_by获取每个字符的所有索引，以及 minmax获取第一个和最后一个索引:

def first_last_indices(word)
  word.each_char.with_index
      .group_by{ |c, _| c }.map{ |c, vs|
        [c, vs.map(&:last).minmax.uniq]
      }.to_h
end

p first_last_indices('hello world')
{"h"=>[0], "e"=>[1], "l"=>[2, 9], "o"=>[4, 7], " "=>[5], "w"=>[6], "r"=>[8], "d"=>[10]}