I'm trying to search a string for match on an array of words. I tested the code by specifying a word and also an input and it correctly finds it, but for some reason I can't get it to work in general. I checked that the input is a String and the words are extracted from the list as well. Any help on this would be appreciated. Thanks.
def self.wordmatch(input, list)
list.each {|word|
if (input =~ /#{word}/i)
puts "#{input} => #{word}"
return 'MATCH'
end
}
end
This seems to work OK. What you may be getting confused by is that if there is no match then ruby will return the last evaluated expression, which in this case is the list.
I'm not sure what you are wanting to return here in the case of a failure but this will return true/false
def self.wordmatch(input, list)
list.each {|word|
if (word =~ /#{input}/i)
puts "#{input} => #{word}"
return true
end
}
return false
end
The following test code
words = %w[a b c]
matches = %w[a b c d]
matches.each do |match|
puts "Testing #{match} = #{wordmatch(match, words)}"
end
Produces the following results
Testing a = true Testing b = true Testing c = true Testing d = false
EDIT
Following on from the comments this seems to work exactly as advertised.
def wordmatch(input, list)
list.each {|word|
if (input =~ /#{word}/i)
puts "#{input} => #{word}"
return 'MATCH'
end
}
return nil
end
list = ["hate", "indifference", "love", "foo"]
input1 = "I love summer"
input2 = "I endure summer"
puts "Testing #{input1} = #{wordmatch(input1, list)}"
puts "Testing #{input2} = #{wordmatch(input2, list)}"
And produces the following results
I love summer => love Testing I love summer = MATCH Testing I endure summer =
Try this. I think you have them backwards..
def self.wordmatch(input, list)
list.each {|word|
if (word =~ /#{input}/i)
puts "#{input} => #{word}"
return 'MATCH'
end
}
end
http://rubular.com/ is a great site to mess with this stuff.
I would implement it like this:
def wordmatch(input, list)
found = input.scan(/\w+/).find {|w| list.include?(w)}
if found
puts found
return "MATCH"
end
end
It transform your input string to an array of words, and runs Array#find on it to find the first word in the array that is included in the list array.
To make it case insensitive, you might want to add map(&:downcase) before the find if your list contains only lowercase words.
As noted by others, your implementation will return the whole list when a match is not found, since you did not set an explicit return value in that case.
