Delete Item In Array If Conditions Met

[Chris Kalaboukis]

Hi all: Ruby newbie here. Been trying to get this code to work but am
having a tough time of it. Can anyone else please help me out?

I've read from a db into an array, and would like to go through the
array and remove elements which match a specific criteria. Here is the
code that I'm using:


# load array
@array = Capture.findall)

# iterate through array

for x in @array
x.reject!{|x| x.text =~ "something"}
end

 

[Chris Kalaboukis]

Hi all: Ruby newbie here. Been trying to get this code to work but am
having a tough time of it. Can anyone else please help me out?

I've read from a db into an array, and would like to go through the
array and remove elements which match a specific criteria. Here is the
code that I'm using:


# load array
@array = Capture.findall)

# iterate through array

for x in @array
x.reject!{|x| x.text =~ "something"}
end

Sorry: forgot to mention that this breaks with an undefined method
reject!

Thanks in advance!

 

[Mat Brown]

The #reject! method is an iterator; so is the for loop (effectively).
So you're iterating over the array, and then trying to iterate over
individual members of the array. Here's what you want:

@array =3D Capture.findall)
@array.reject! { |item| item.text =3D~ 'something' }

 

[Siep Korteling]

Hi all: Ruby newbie here. Been trying to get this code to work but am
having a tough time of it. Can anyone else please help me out?

I've read from a db into an array, and would like to go through the
array and remove elements which match a specific criteria. Here is the
code that I'm using:


# load array
@array = Capture.findall)

# iterate through array

for x in @array
x.reject!{|x| x.text =~ "something"}
end

reject! does the iterating for you, so you can (must) leave out the for
loop.

hth,

Siep

 

[Tony Arcieri]

[Note: parts of this message were removed to make it a legal post.]

In addition to reject! there's also Array#delete_if

 

[Chris Kalaboukis]

The #reject! method is an iterator; so is the for loop (effectively).
So you're iterating over the array, and then trying to iterate over
individual members of the array. Here's what you want:

@array = Capture.findall)
@array.reject! { |item| item.text =~ 'something' }

Thanks guys that did it!

 

[Albert Schlef]

Thanks guys that did it!

You've better ask your question in the Rails forum. You're using some
ORM (e.g. AcriveRecord, DataMapper, etc.) and need to ask the people who
use it.

Instead of loading all the objects from the database it's more efficient
to pass find() the criterion.