[CLOSED] Child.AddTo or Parent.Items.Add

[bayoglu]

Hello,

This is a general question. When building dynamic controls, ParentComponent.Items.Add(ChildComponent) works during page load but not working when called from direct event. ChildComponent.AddTo(ParentComponent) works for both. Is there a best practice or pros/cons for deciding between two alternatives?

Note-1: All in code behind.
Note-2: ParentComponent is defined in the markup, ChildComponent's are dynamically created and added to parent from code behind.

I am asking this to decide the right strategy for refreshing those dynamic child components (tree panel menu etc.)

Thanks.

[Baidaly]

Hello!

I think using

ChildComponent.AddTo(ParentComponent)

is more appropriate for most cases.

I'll ask my colleagues advice you too.

[Daniil]

Hello,

The AddTo method and other "render" methods Render, InsertTo generate JavaScript code which is excessive for initial page loading. Moreover, on some circumstances those generated scripts can cause a JavaScript error. So, these methods should be used during an AJAX request only (DirectEvent/DirectMethod).

During initial page loading please just populate a container's Items collection.

Commonly, I prefer the following template.

container.Items.Add(control);

if (X.IsAjaxRequest)
{
    control.Render();
}

[bayoglu]

That was very informative. I understood the difference well ( I think I did (; ).

Are there any practical reasons to select between two strategies below?

Thanks.

1:

if (X.IsAjaxRequest)
{
    control.AddTo(container);
}
else
{
    container.Items.Add(control);
}

2:

container.Items.Add(control);
 
if (X.IsAjaxRequest)
{
    control.Render();
}

[Daniil]

Those approaches should be the same, just use any you wish.

The second one is 2 code lines less;)

[bayoglu]

Sure it is (;

Thanks Daniil, please mark as closed.

[Daniil]

Please note that if you want to handle DirectEvents of dynamically created controls, you will have to recreate those controls.
http://forums.ext.net/showthread.php?14316#post60155