我的&&运算符有什么问题？

Question

我想知道为什么当我输入1982时这会输出false 。 我的&&语句有问题吗？ 我尝试使用!(t==r) ，但是没有用； 由于某种原因，它会一直输出false 。

  def no_repeats?(year)
   out=true
   t=0
   while t<4
     r=0
     while r<4
       if (year[t] == year[r]) && t != r
         out=false
       end
         r+=1

      end
      t+=1
   end

   out
  end

Answer 1

您可能使它变得复杂了很多。

2.2.0-preview1 :001 > load 'no_repeat.rb'
 => true

字符串测试

2.2.0-preview1 :002 > no_repeats?("1981")
 => false
2.2.0-preview1 :003 > no_repeats?("1983")
 => true

测试为整数

2.2.0-preview1 :004 > no_repeats?(1981)
 => false
2.2.0-preview1 :005 > no_repeats?(1983)
 => true

和no_repeat.rb看起来像

  def no_repeats?(year)
    digits = year.to_s.split(//)
    digits.size == digits.uniq.size
  end

编辑：基准

Using Original Post

real    0m0.598s
user    0m0.583s
sys     0m0.015s



Using .split(//)

real    0m1.322s
user    0m1.321s
sys     0m0.000s



Using .chars.to_a

real    0m0.562s
user    0m0.557s
sys     0m0.004s

因此，为了使这个问题更完整，我将基准测试包括在内，每个基准测试使用了40万次。 通过使用split(//) ，您将获得几乎2倍的性能提升。 通过使用chars.to_a ，您将可以保持原来的速度。

def no_repeats?(year)
  digits = year.to_s.chars.to_a
  digits.size == digits.uniq.size
end