Difference between revisions of "Talk:RemoveAddedForm - FormList"
imported>Chesko |
imported>Dheu |
||
Line 1: | Line 1: | ||
== This method is unreliable == | |||
As of version 1.6, avoid using this method as it appears buggy on some platforms. (See discussion below). AddForm() and Revert() however both work fine, so you can simulate a working method as follows: | |||
<source lang="papyrus"> | |||
; ** Local FormList wont work. Must be created in CK and injected. | |||
; ** CEIF = Custom Empty Injected FormList | |||
Formlist property CEIF auto | |||
Function RemoveForm(FormList source, Form f) | |||
int size = source.GetSize() | |||
if (size > 0) | |||
CEIF.Revert() | |||
int i = 0 | |||
bool notFound = true | |||
Form item | |||
while (i < size && notFound) | |||
item = source.GetAt(i) | |||
if (item != f) | |||
CEIF.AddForm(item) | |||
else | |||
notFound = false | |||
endif | |||
i += 1 | |||
endWhile | |||
if (!notFound) | |||
while (i < size) | |||
CEIF.AddForm(source.GetAt(i)) | |||
i += 1 | |||
endWhile | |||
source.Revert() | |||
size = CEIF.GetSize() | |||
if (size > 0) | |||
i = 0 | |||
while (i < size) | |||
source.AddForm(CEIF.GetAt(i)) | |||
i += 1 | |||
endWhile | |||
endif | |||
endIf | |||
endIf | |||
endFunction | |||
</source> | |||
== RemoveAddedForm bugged? == | == RemoveAddedForm bugged? == | ||
I'm currently trying to manipulate a FormList filled with items using a container. The script is very basic but already gets screwed: | I'm currently trying to manipulate a FormList filled with items using a container. The script is very basic but already gets screwed: |
Revision as of 23:25, 21 July 2012
This method is unreliable
As of version 1.6, avoid using this method as it appears buggy on some platforms. (See discussion below). AddForm() and Revert() however both work fine, so you can simulate a working method as follows:
; ** Local FormList wont work. Must be created in CK and injected.
; ** CEIF = Custom Empty Injected FormList
Formlist property CEIF auto
Function RemoveForm(FormList source, Form f)
int size = source.GetSize()
if (size > 0)
CEIF.Revert()
int i = 0
bool notFound = true
Form item
while (i < size && notFound)
item = source.GetAt(i)
if (item != f)
CEIF.AddForm(item)
else
notFound = false
endif
i += 1
endWhile
if (!notFound)
while (i < size)
CEIF.AddForm(source.GetAt(i))
i += 1
endWhile
source.Revert()
size = CEIF.GetSize()
if (size > 0)
i = 0
while (i < size)
source.AddForm(CEIF.GetAt(i))
i += 1
endWhile
endif
endIf
endIf
endFunction
RemoveAddedForm bugged?
I'm currently trying to manipulate a FormList filled with items using a container. The script is very basic but already gets screwed:
Event OnItemRemoved(Form BaseItem, int iCount, ObjectReference rItem, ObjectReference rContainer)
List.RemoveAddedForm(BaseItem)
EndEvent
Event OnItemAdded(Form BaseItem, int iCount, ObjectReference rItem, ObjectReference rContainer)
List.AddForm(BaseItem)
EndEvent
I used message boxes to track the problem: The items are added to the formlist without problem. The list itself is empty and gets just filled by the OnItemAdded Event, thus every item I want to have removed from the formlist was previously added by this very script to it. RemoveAddedForm just removes the form from the formlist at the very first try in the currently loaded save. That means that the first Item I remove from the container gets removed from the list as well. But then RemoveAddedFrom stops working. It won't even help to open the container again or to refill it completely.
What am I doing wrong? -- M3rvin 09:31, 15 April 2012 (EDT)
This function is indeed potentially bugged. I do not recommend manipulating FormLists at runtime. See this discussion. This function is used a total of 1 time in Bethesda's vanilla scripts (MQ206ThroatoftheWorldTriggerScript) and is probably not stable enough for general use. Chesko (talk) 11:07, 18 July 2012 (EDT)Chesko