Well...you already have it in a method...so....since you didn't really specify it's hard to know for sure what you're asking here.
I'll assume that this code is used in other places but for a different drop down? If that is the case, instead of using
you'll want to pass in a drop down as a parameter. Pass it in ByRef instead of ByVal so that it will update it properly. If other things change in the different places you use the code, you'll want to pass those in as parameters too. For example, the values of your SQL parms will probably need to be passed in.
I'm thinking something like this:
Public Sub FillDDL(ByRef ddl As System.Web.UI.WebControls.DropDownList, ByVal strGroupID As String, ByVal strClientID As String)
Although, if you're calling this method on one web page, you'll probably also want to pass in the SQL command as a parm. That way you can set it up once and then re-use it everytime you call it instead of recreating it every time.