Fill Literal with strings in Page_Load

Radim Motycka

15y2.4k.NET

Hello,

Pls would you advise, I am getting an arraylist from ascx to main aspx.
I am getting values correctly. If I add items into Listbox it fills correctly, but in the Literal there is only last one. Please advice what I am missing.

Many thanks
-------------------------------------------------------------------------------
protected void Page_Load(object sender, EventArgs e)
        {
            members.sendPageMessage += delegate(ArrayList message)
            {
                foreach (string item in message)
                {
                    Literal1.Text = (string)item + "
"; // only last item in arraylist
                    //ListBox1.Items.Add((string)item);   //Works corectly
                }
            };
        }

3 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted answer
As you're using a foreach loop, you're overwriting the value of Literal1.Text each time though the loop so it will end up being set to the last value of the ArrayList. If you intended to add each item on a separate line, then I'd try this:

protected void Page_Load(object sender, EventArgs e)
{
members.sendPageMessage += delegate(ArrayList message)
{
string temp = "";

foreach (string item in message)
{
temp += item + "
";
}

Literal1.Text = temp;
};
}
Radim Motycka15y ago
Many thx, that's it :-)
Vulpes15y ago
The following should also work and is a bit more concise:

protected void Page_Load(object sender, EventArgs e)
{
members.sendPageMessage += delegate(ArrayList message)
{
Literal1.Text = String.Join("
", (string[])message.ToArray(typeof(string)));
};
}