为什么这个红宝石代码不返回任何东西？

Question

我正在尝试获取一个数字并返回一个字符串，其中包含任何奇数周围的破折号。此外，字符串不应以破折号开头或结尾。

我写了以下内容，但它没有返回任何内容：

def dasherize_number(num)
  string = num.to_s
  i = 0

  while i<string.length
    if (string[i].to_i % 2) != 0
      string[i] = '-' + string[i] + '-'
    end
    i += 1
  end

  if string[0] == '-'
    string.pop(1)
  end

  if (string.length - 1) == '-'
    string.pop(1)
  end

  string
end

如果我理解正确，它似乎在无限循环；控制台不显示任何输出，除非我刷新，否则不允许我做任何其他事情。我已经按每个字符查看了代码，但我不知道哪里出了问题。

Answer 1

您的代码中有很多逻辑问题。 这里有一些可能对你有用的东西

def dasherize_number(num)
  string = num.to_s
  str_len = string.length

  i = 0

  while i < str_len
    next if string[i] == '-'
    if (string[i].to_i % 2) != 0
      string[i] = '-' + string[i] + '-'
      str_len = string.length
      i += 3
    else
      i += 1
    end
  end

  if string[0] == '-'
    string = string[1..-1]
  end

  if (string[string.length - 1]) == '-'
    string = string[0..-2]
  end

  string.gsub('--', '-')
end

说明

1. 首先，您的 while 循环中有此条件 i < string.length 这是行不通的，因为字符串的长度不断变化。所以我使用了一个变量来存储值并在字符串更新时更新变量。

2. 如果字符串被更新，我们可以确定我们可以跳过接下来的两个索引。 例如：输入的数字 -> 122 那么在第一次迭代之后，字符串将是-1-22 所以我们不想为下一个索引运行相同的条件，因为那将再次是1，因此是无限循环。 （希望你明白）

3. pop 不适用于字符串，因为我们可以使用索引（如数组）来访问字符，但我们不能将pop 用于字符串。

4. 为了确保没有连续的破折号，我使用gsub 将它们替换为单个破折号。

Answer 2

问题似乎出在这部分代码中：

while i<string.length
    if (string[i].to_i % 2) != 0
        string[i] = '-' + string[i] + '-'
    end
    i += 1
end

如果您的 string 包含奇数，它会将其长度增加 2 个字符 (2x-)，但将其增加 1 (i+=1)。

将初始字符串长度分配给 var 并在 while 循环中检查其长度。

string_length = string.length

while i < string_length
    if ((string[i].to_i % 2) != 0)
        string[i] = '-' + string[i] + '-'
    end
    i += 1
end