looping all entities

[Pout]

I'm a bit puzzled.

Sketchup.active_model.entities.each{|e|e.erase! if e.valid?}
works but doesn't delete all entities

Sketchup.active_model.entities.to_a.each{|e|e.erase! if e.valid?}
works and deletes all entities

Now, does this mean that to loop all entities in a model i always have to put them in an array first?
I never have run into any problems before using the first method.

Additionaly

Code: Select all

f=0
Sketchup.active_model.entities.each{|ent|
   f+=1
   Sketchup.active_model.selection.clear
   Sketchup.active_model.selection.add ent
   ent.erase!
}
puts f

returns 4095 (on a 11200 entities model)
while

Code: Select all

f=0
Sketchup.active_model.entities.each{|ent|
   f+=1
   Sketchup.active_model.selection.clear
   Sketchup.active_model.selection.add ent
   #ent.erase!
}
puts f

returns 11200 (on a 11200 entities model)

I must be missing something essential here...

[thomthom]

A few points:

1. Bulk operations are much faster than single operations.

Code: Select all

model.entities.erase_entities( array_of_entities ) 

Is faster than

Code: Select all

for entity in array_of_entities
  e.erase!
end

But when you want to clear the entire Enitites collection you can just use

Code: Select all

model.entities.clear! 

2. Never iterate the Entities collection directly when you modify it.

Code: Select all

for entity in model.entities
  e.erase!
end

This will make some entities be skipped as you are modifying the Entities object you iterate.

Instead make an array copy of the entities:

Code: Select all

for entity in model.entities.to_a
  e.erase!
end

This will not skip any entities.


3. Model.entities does not contain every entity in the model - only the root of the model hierarchy. Entities in Groups and Components are stored in the definitions in model.defintions.
But if you want to empty the whole model, model.entities.clear! is still the easiest.
http://www.thomthom.net/thoughts/2012/0 ... -sketchup/

[Pout]

ThomThom

Thanks for the fast reply!
I did the test on a single box (not a group nor instance) and even then it went wrong
So I can assume that Sketchup.active_model.entities.each{|e|
...
}
can always be incorrect even when just checking for something (eg. check if the current entity has a certain value in it's attribute dictionary)?

On point 1, what would be faster

Code: Select all

Sketchup.active_model.entities.to_a.each{|e|
check=e.get_attribute'x','y'
if check==1
e.erase!
end
}

or

Code: Select all

test=[]
Sketchup.active_model.entities.to_a.each{|e|
check=e.get_attribute'x','y'
if check==1
test<<e
end
}
model.entities.erase_entities(test)

[thomthom]

went wrong

What went wrong? Examples?

So I can assume that Sketchup.active_model.entities.each{|e|
...
}
can always be incorrect even when just checking for something?

No, just when you modify it. If you just read states and properties there should be no problems.

On point 1, what would be faster

The second - using .erase_entities.

[Pout]

What went wrong? Examples?

It goes wrong caused by the fact that the entities object is edited while executing the loop. So as you stated.

No, just when you modify it. If you just read states and properties there should be no problems.

Modify (like in push pull) or only when the entity is deleted?

The second - using .erase_entities.

In the second case I could use Sketchup.active_model.entities.each (without the to_a) Correct? And if so would that be faster then with to_a?

Thx again!

[thomthom]

It goes wrong caused by the fact that the entities object is edited while executing the loop. So as you stated.

Yes, this is where you need to use to_a.

Modify (like in push pull) or only when the entity is deleted?

Anything that adds or removed entities to the collection.

Correct? And if so would that be faster then with to_a?

That won't be of much difference. What makes a difference is using bulk methods that affect multiple entities in one operation over methods that affect only one entity at a time.

[Pout]

Excellent information.
Thank you ThomThom, much appreciated!