reverse include for array?

Question

I want to check whether any item from the array is part of a string. Something like .include?

I wrote a function for that but I am sure there would be a simpler way.

Could someone suggest how I use less code to achieve the same?

$goes_into_cron_comments=["Edumate_upgrade","login"].map!{|item| item.downcase} 
params={}
params['run']="login_ldap"

def contains(array,string)
        array.each{|item| 
            if string.include?(string)
                return true
            end
        }
        return false
end
#if ($goes_into_cron_comments.include?(params['run'].downcase)) #original condition
if (contains($goes_into_cron_comments,params['run'].downcase))
    puts "contains"
else 
    puts "doesn't contain"
end

Similar question: Find out which words in a large list occur in a small string. — Phrogz
– Phrogz, Commented May 14, 2012 at 4:02

mu is too short · Accepted Answer · 2019-03-17 17:48:02Z

3

There are things you can do that would be nicer. You could use Enumerable#any? rather than iterating by hand:

array.any? { |item| string.include?(item) }

Or you could use Regexp.union:

string =~ Regexp.union(array)
# or if you want a true boolean
!!(string =~ Regexp.union(array))
!(string =~ Regexp.union(array)).nil?

edited Mar 17, 2019 at 17:48

answered May 14, 2012 at 3:49

mu is too short

436k71 gold badges863 silver badges822 bronze badges

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

3 Comments

Phrogz Over a year ago

+1 Came here for the more-efficient regex answer, leaving extra satisfied due to the usage of union.

Radek Over a year ago

Why regex that more-efficient?

mu is too short Over a year ago

@Radek: The regex engine can look at all the elements of array at once so it may be faster for larger arrays.

seph · Accepted Answer · 2012-05-14 03:45:11Z

2

try this:

str = 'hey, this is a string'
arr = %w{one hey may}

arr.any? { |el| str.include? el }
=> true

answered May 14, 2012 at 3:45

seph

6,0963 gold badges23 silver badges19 bronze badges

Comments

ctide · Accepted Answer · 2012-05-14 03:55:56Z

1

I would just do something like this:

def contains?(array,string)
    array.index{|item| string.include?(item)}
end

Index will return a truthy/falsy value, so this is probably the 'cleanest' way to do it. I would also include the ? at the end of your function definition just to make it clearer how it should be used and what result to expect.

edited May 14, 2012 at 3:55

answered May 14, 2012 at 3:43

ctide

5,2571 gold badge31 silver badges26 bronze badges

2 Comments

Radek Over a year ago

I changed my code to if ($goes_into_cron_comments.index{|item| params['run'].downcase.include?(item)}) and I am always getting true regardless if the array contains the correct string. In your code should be ... .include?(item) ...

ctide Over a year ago

Yeah, sorry, I didn't change that bit from your example :)

Collectives™ on Stack Overflow

reverse include for array?

3 Answers 3

3 Comments

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related