Question

My javascript function looks like this –

function updateSuccess(aEle) {
    //Here aEle doesn't have the correct id
    percIncrease.innerText = "Success";
}

in realizing it’s bad to get the element this way, what would be a safer way of getting that element?

Some more info –
Here is how I call it from C#..

DropDownList ddl = new DropDownList();
ddl.ID = "ddlData1";

ddl.Attribute.Add("onchange", "updateSuccess('" + ddl.ClientID + "')";

Answer 1

EDIT

As per the edit in your question you just need to do this

ddl.Attribute.Add("onchange", "updateSuccess(this)";

javascript

function updateSuccess(element) {
              alert(element.id);
              document.getElementById(element.id).innerText = "Success";
              //or try below directly
              //element.innerText = "Success";
 }

---

I think you are using javascript with Asp.net for this you can do something like this

Function declration

function updateSuccess(recordID) {
     var percIncrease = document.getElementById(recordID);
     percIncrease.innerText = "Success";
 }

call to function

  updateSuccess('<%= serverelementID.ClientID %>');

or directly like

function updateSuccess() {
     var percIncrease = document.getElementById('<%= serverelementID.ClientID %>');
     percIncrease.innerText = "Success";
 }