If I can guess what you are trying to do...
first of all, your i that you have within the <%= %> brackets is going
to refer to your codebehind variables (Not the javascript variable).
It almost seems like you are trying to use reflection or something and
get the ClientID of that item

First of all your javascript is referring to the variable i right
after you declare it but it has no value yet!

Maybe you could use an alternative approach of writing this in your
code behind

WriteScript() //Rough C# Code
{
   int i;
    HtmlControl img; // = Page.FindControl("star"+i); // i not
declared yet
    for(i=1;i<=rating;i++)
    {
        img = Page.FindControl("star"+i);
        img.src = "~/Images/StarOver.gif";
    }
    for(i=rating;i<=5;i++)
    {
// same thing here
//        var img = document.getElementById("<%=star"+i+".ClientID
%>");
//       img.src = "~/Images/StarOff.gif";
    }

// same thing here
//   var img = document.getElementById("<%=star"+i+".ClientID %>");

tada..

Or if you want to do it from javascript not codebehind, just modify
the above to do the following

REsponse.write("<script type='text/javascript'>");
then in the loop you can stick another Response.Write(...)
then end with Response.Write("</script>")

or alternatively, read about RegisterClientScript Here:
http://www.codeproject.com/aspnet/ClientServer.asp

Sameer Alibhai
Sharp Developer
http://www.SharpDeveloper.Net

On May 31, 5:31 am, Zeba <coolz@gmail.com> wrote: