TaggedValue created, but TaggedValues doesn't return it?

[Mr Stuff]

This snippet checks to see if an element has a certain tagged value and creates it if it doesn't. It then checks to see the creation was successful but no matter what I do on readback activityScopeInCaseProjectTV is nothing even though the taggedvalue rewally has been created.

Code: [Select]

Session.output("Creating TV named " & asicpTvName & " for " & anEl.Name)
set activityScopeInCaseProjectTV = anEl.TaggedValues.AddNew(asicpTvName,"")
activityScopeInCaseProjectTV.Update
anEl.Update
anEl.Refresh
Repository.AdviseElementChange(anEl.ElementID)
set activityScopeInCaseProjectTV = anEl.TaggedValues.GetByName(asicpTvName)
if activityScopeInCaseProjectTV is nothing then
Session.output("TV named " & asicpTvName & " creation for " & anEl.Name & " failed")
end ifWhy? How can I fix this it?

Thx,

Julian

[qwerty]

I never use GetByName since it does not work on non-unique collections. Try a manual iteration over the elements and see what is returned.

q.

[Mr Stuff]

Can you clarify? What do you mean it doesn't work on non-unique collections?

(Iterate over the elements again? = make it even slower  )

PS If I count the element's tagged values immediately after creation that also returns zero so getAt also failed

[Mr Stuff]

Ok, iterating over the same collection again does allow getbyname to work.

But... has anyone collected a list of known issues of this type - the things that will catch out anyone other than an experienced EA scripter?

[qwerty]

I once maintained a bug list publicly but then gave up since it just was growing and never really shrinking. Just get used to walk over mine fields. Use a stick to not get blast away directly.

q.

[Geert Bellekens]

As an experienced scripter, my advice is to not use GetByName in any situation.
And if you need to iterate a specific collection more then once store it in an ArrayList or Dictonary. That is usually ten times faster then iterating a native EA.Collection.

Geert

[Uffe]

Hi Julian,

In your code, you don't refresh Element.TaggedValues, which you need to do if you add objects to it. AdviseElementChange() is a GUI operation, it does not affect the objects in your code.

Other than that, what Geert said. You have to use Collections when manipulating various project content, but never ever create one yourself, and if you need to do a lot of back-and-forth, cache the contents in a better data structure.


/Uffe

[Mr Stuff]

OK! Thanks for the tips. In hindsight I think maybe element.taggedvaluesrefresh might have been necessary but I'm not doing that now Will certainly consider not using EA iterators too.

I don't suppose that old bug list is still around is it, just for curiosity's sake?

Julian

[Mr Stuff]

Uffe - my reply crossed with yours - thanks!

[qwerty]

https://www.assembla.com/spaces/enterprise-architect/tickets

Also look at the "bug tracker" results from this site's search.

q.

[Mr Stuff]

Thanks qwerty.