Zhubham wrote:
> Hi Experts,
>
> I am trying to optimize the following code:
>
> textview[0]=(TextView)findViewById
> (R.id.TextView1);
> textview[1]=(TextView)findViewById(R.id.TextView2);
> textview[2]=(TextView)findViewById(R.id.TextView3);
> textview[3]=(TextView)findViewById(R.id.TextView4);
> textview[4]=(TextView)findViewById(R.id.TextView5);
> textview[5]=(TextView)findViewById(R.id.TextView6);
> textview[6]=(TextView)findViewById(R.id.TextView7);
> textview[7]=(TextView)findViewById(R.id.TextView8);
> textview[8]=(TextView)findViewById(R.id.TextView9);
> textview[9]=(TextView)findViewById(R.id.TextView10);
>
> using:
>
> int j;
> for(int i=0;i<10;i++)
> {
> j = i+1;
> textViewId = "TextView"+j;
> textview[i] =
> (TextView)findViewById(R.id.textViewId);//
> error textViewId not recognized
> }
>
> but this is giving error at the last line.. Can you please help me to
> come out of this.
1. There is nothing named R.id.textViewId in your layout, in all likelihood.
2. Putting things in loops does not optimize the code, from an execution
speed standpoint.
3. You cannot dynamically construct the name of a widget ID in the
manner you are trying.
--
Mark Murphy (a Commons Guy)
http://commonsware.com | http://twitter.com/commonsguy
_The Busy Coder's Guide to Android Development_ Version 2.0 Available!
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google
Groups "Android Developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/android-developers?hl=en
-~----------~----~----~----~------~----~------~--~---
