Lua的for循环与多个条件

Question

我有一个面试问题是让我写一个在Lua语言下列要求的程序：如果数字是3的倍数1之间

• 打印号码100
• 如果数字是5的倍数，则打印'Fizz'而不是数字
• 如果数字是5和3的倍数，则打印'Buzz'而不是数字
• ，打印'FizzBu​​zz'而不是数字

我写了下面：

for i=1,100 do 
    if i%5==0 then 
    if i%3==0 then 
     print("FizzBuzz") 
    else 
     print("Buzz") 
    end 
    elseif i%3==0 then 
    print("Fizz") 
    else 
    print(i) 
    end 
end 

反响并不好，说的代码的质量没有达到他们的标准。我如何才能将以上代码改为最高标准以供将来参考？

重新阅读代码后，我意识到它不易读，所以我确信这是主要原因。也许更少的处理也可能涉及？寻找一些指导。

Answer 1

很难说出这个人询问的内容，但是有几种方法可以使代码更短或更少重复。例如，你可以修改它只需检查i%3一次（与感实性检查所增加的成本），降低print调用次数：

for i=1,100 do 
    local s = i%3==0 and "Fizz" 
    if i%5==0 then s = (s or "") .. "Buzz" end 
    print(s or i) 
end 

您也可包括在第二如果到print：

for i=1,100 do 
    local s = i%3==0 and "Fizz" 
    print(i%5==0 and (s or "") .. "Buzz" or s or i) 
end 

它绝对更短，但有人可能会争辩说它不具有更好的可读性。

您也可避免级联使用FizzBuzz：

for i=1,100 do 
    local s = i%3==0 and "Fizz" 
    print(i%5==0 and (s and "FizzBuzz" or "Buzz") or s or i) 
end 

在实际的采访中，你可以问我，他们会想提高哪些方面的条款。我认为使用print更少的时间是合乎逻辑的步骤，但除此之外，我希望他们指定他们是否更关心清晰度，简洁性，速度或其他。

我也跑所有4个代码片段（原始的一个+ 3中所示）与print通过简单地存储在变量中的值，并且在这里所取代是结果（运行循环亿倍和定时使用的Lua 5.3 os.clock）：

9.515 
10.797 
10.672 
10.047 

也有可能，他们只是希望你能检查15，5，3：

for i=1,100 do 
    print(i%15==0 and "FizzBuzz" or i%3==0 and "Fizz" or i%5==0 and "Buzz" or i) 
end 

Answer 2

IMO，最清晰的代码将是如下：

for n = 1, 100 do 
    local s = (n%3 == 0 and 'Fizz' or '')..(n%5 == 0 and 'Buzz' or '') 
    print(s == '' and n or s) 
end 

Answer 3

对于我来说，最清晰的代码是

for i=1,100 do 
    if i%15==0 then 
    print("FizzBuzz") 
    elseif i%3==0 then 
    print("Fizz") 
    elseif i%5==0 then 
    print("Buzz") 
    else 
    print(i) 
    end 
end 

我猜他们不喜欢你重复测试i%3==0。上述

的代码依赖于我们能够表达A(i) and B(i)简洁。我想，在一般情况下，你不得不做这样的事情，这是明确的，避免了重新计算谓词：

for i=1,100 do 
    local a=A(i) 
    local b=B(i) 
    if a and b then 
    print("FizzBuzz") 
    elseif a then 
    print("Fizz") 
    elseif b then 
    print("Buzz") 
    else 
    print(i) 
    end 
end