ControlCollection.Remove bug

Abraham Andres Luna · Oct 30, 2006

hello everyone,

this code only removes the first HtmlMeta control even though i loop through
all of the controls:

<%@ Page Language="C#" %>
<script runat="server">
void Page_Load(Object Sender, EventArgs E)
{
foreach (Control ctl in Page.Header.Controls)
if (ctl.GetType().ToString() == "System.Web.UI.HtmlControls.HtmlMeta")
Page.Header.Controls.Remove(ctl);
}
</script>
<html>
<head runat="server">
<title>HtmlMeta Test Page</title>
<meta name="description" content="long description" />
<meta name="keywords" content="keyword list" />
</head>
<body>
Done!
</body>
</html>

thanks for any help with this.

marss · Oct 31, 2006

Abraham Andres Luna Ð½Ð°Ð¿Ð¸ÑÐ°Ð²:

hello everyone,

this code only removes the first HtmlMeta control even though i loop through
all of the controls:

<%@ Page Language="C#" %>
<script runat="server">
void Page_Load(Object Sender, EventArgs E)
{
foreach (Control ctl in Page.Header.Controls)
if (ctl.GetType().ToString() == "System.Web.UI.HtmlControls.HtmlMeta")
Page.Header.Controls.Remove(ctl);
}
</script>
<html>
<head runat="server">
<title>HtmlMeta Test Page</title>
<meta name="description" content="long description" />
<meta name="keywords" content="keyword list" />
</head>
<body>
Done!
</body>
</html>

thanks for any help with this.

"The foreach statement is used to iterate through the collection to get
the desired information, but should not be used to change the contents
of the collection to avoid unpredictable side effects..." (MSDN)

for(int i=Page.Header.Controls.Count-1; i>=0; i--)
if (Page.Header.Controls.GetType().ToString() ==
"System.Web.UI.HtmlControls.HtmlMeta")
Page.Header.Controls.RemoveAt(i);

Abraham Andres Luna · Oct 31, 2006

thanks so much for your reply. i will update my code accordingly.

Abraham Andres Luna ???????:

hello everyone,

this code only removes the first HtmlMeta control even though i loop
through
all of the controls:

<%@ Page Language="C#" %>
<script runat="server">
void Page_Load(Object Sender, EventArgs E)
{
foreach (Control ctl in Page.Header.Controls)
if (ctl.GetType().ToString() ==
"System.Web.UI.HtmlControls.HtmlMeta")
Page.Header.Controls.Remove(ctl);
}
</script>
<html>
<head runat="server">
<title>HtmlMeta Test Page</title>
<meta name="description" content="long description" />
<meta name="keywords" content="keyword list" />
</head>
<body>
Done!
</body>
</html>

thanks for any help with this.

"The foreach statement is used to iterate through the collection to get
the desired information, but should not be used to change the contents
of the collection to avoid unpredictable side effects..." (MSDN)

for(int i=Page.Header.Controls.Count-1; i>=0; i--)
if (Page.Header.Controls.GetType().ToString() ==
"System.Web.UI.HtmlControls.HtmlMeta")
Page.Header.Controls.RemoveAt(i);

Alvin Chooi · Oct 31, 2006

You could only remove first HtmlMEta control is because when you are
deleting the first HtmlMeta Control, the ControlCollection in the
Page.Header will be reduced one, which would immediately out of the
foreach. What you need to do is use the for statement, minus one on
looping index whenever you call the Remove() method.

Hope this helps,

Alvin Chooi
Microsoft ASP.NET Enthusiast
http://alvinzc.blogspot.com

About as basic "Newbie-Question" that you can get.	3	Sep 4, 2023
Loop through record in datareader in aspx for textbox other item	1	Sep 26, 2023
Help with Visual Lightbox: Scripts	2	May 3, 2023
Script stops working when using variables to save time typing...	4	Oct 31, 2022
An unknown bug doesn't allow the quotes app to work. What's the issue?	3	Apr 23, 2023
Why doesn't the function get called?	1	Nov 20, 2023
External html	2	May 13, 2020
HTML Anchor tag not working	2	Dec 15, 2020

ControlCollection.Remove bug

Abraham Andres Luna

marss

Abraham Andres Luna

Alvin Chooi

Ask a Question

Similar Threads

Members online

Forum statistics

Latest Threads