grep和不等于在if条件

Question

我要检查，如果“听8581' 出现在check变量，如果它是不，我要添加‘听8581’。

我有验证码：

if (found = check.grep(/Listen 8581/)).empty? 
    run "#{try_sudo} sed -i '$ a\Listen 8581\n' #{ports_file}" 
    puts "OOOps".red 
end 

Answer 1

unless /Listen 8581/.match check 
    # run your code here 
end 

Answer 2

作为一个文体的事情，内部条件赋值给一个变量是一个坏主意，因为它是一个容易打开的bug和维护问题在调试大程序很容易错过逻辑或丢失==。配第一然后检查清晰度：

 
found = check.grep(/Listen 8581/) 
if found.empty? 
    run "#{try_sudo} sed -i '$ a\Listen 8581\n' #{ports_file}" 
    puts "OOOps".red 
end 

我知道这是很有诱惑力，以避免在堆栈中加入一个变量，但如果你的代码被嵌入的方法中，额外的变量就会掉出来的范围，这将是一个无关紧要。

在某些语言（如Perl和C）中，这种写法习惯于这种方式，但这是我在职业生涯中多次遇到问题的原因，也是我避免的结果。