Reject element from array in efficient way

Question

My array is

 arr = ["wow what", "what anice", "anice day.currently", "day.currently i", "i am", "am in", "in delhi", "delhi but", "but in", "in night", "night i", "i am", "am going", "going to", "to us"]
    arr.each do |el|
     if !el.match('in') && !el.match('is').blank?
      fresh_arr << el
     end

but i have 110k element array and it give 8sec that,s too much time can i do this any another way

Thx

ruby-doc.org/core-2.0/Array.html#method-i-delete_if

ceth
– ceth

2013-03-21 09:29:32 +00:00
Commented Mar 21, 2013 at 9:29 — ceth
– ceth, Commented Mar 21, 2013 at 9:29
String#include? is 2-10 times as fast as String#match.

Holger Just
– Holger Just

2013-03-21 12:29:06 +00:00
Commented Mar 21, 2013 at 12:29 — Holger Just
– Holger Just, Commented Mar 21, 2013 at 12:29

Ryan Bigg · Accepted Answer · 2013-03-21 19:21:58Z

3

Use delete_if

arr.delete_if do |e|
  e.match('in') && e.match('is').blank?
end
arr

edited Mar 21, 2013 at 19:21

answered Mar 21, 2013 at 9:29

Ryan Bigg

108k25 gold badges243 silver badges264 bronze badges

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

1 Comment

Arjan Over a year ago

I think you meant delete_if

Arjan · Accepted Answer · 2013-03-21 09:29:19Z

3

Try this

arr.reject { |i| i.match('in') || i.match('is').blank? }

answered Mar 21, 2013 at 9:29

Arjan

6,2742 gold badges30 silver badges42 bronze badges

Comments

kalifs · Accepted Answer · 2013-03-21 12:21:10Z

0

You can select all elements you need by doing this

arr.select{|el| !el.match('in') && !el.match('is').blank?}

answered Mar 21, 2013 at 12:21

kalifs

1773 bronze badges

Collectives™ on Stack Overflow

Reject element from array in efficient way

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related