Ruby variables in scripts

Question

This seems to work correctly and display the results I want

Dir["/Users/ondrovic/RubyTest/**/*.txt"].each do |i|
    puts i.green if File.readlines(i).grep(/test/).any?
end

Why when I try like this does it not populate info

print "Enter search dir"
src = gets
print "Enter search term"
trm = gets

puts "Looking in #{src} for #{trm}"

Dir["#{src}/**/*.txt"].each do |i|
    puts i.green if File.readlines(i).grep(/"#{trm}"/).any?
end

I have also tried it this way

Dir[src + "/**/*.txt"].each do |i|
   puts i.green if File.readlines(i).grep(/ + trm + /).any?
end

Working code

require 'colorize'
print "Enter search dir\n".green
src = gets.chomp
print "Enter search term\n".blue
trm = gets.chomp

puts "Looking in #{src} for #{trm}"

Dir["#{src}/**/*.txt"].each do |i|
    puts i.green if File.readlines(i).grep(/#{trm}/).any?
end

replace /"#{trm}"/ with /#{trm}/

Aguardientico
– Aguardientico

2014-10-12 01:32:25 +00:00
Commented Oct 12, 2014 at 1:32 — Aguardientico
– Aguardientico, Commented Oct 12, 2014 at 1:32

falsetru · Accepted Answer · 2014-10-12 01:35:59Z

2

The return value of the gets includes trailing newline:

>> gets
asdf
=> "asdf\n"  # <----
>>

Remove it using String#chomp:

print "Enter search dir"
src = gets.chomp
print "Enter search term"
trm = gets.chomp
...

In addition to that, the pattern /"#{trm}"/ includes ". Remove them:

/#{trm}/

Alternatively, you can use Regexp::new:

Regexp.new(trm)

edited Oct 12, 2014 at 1:35

answered Oct 12, 2014 at 1:29

falsetru

371k69 gold badges769 silver badges659 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Ruby variables in scripts

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related