Button supposed to click 5 times through an string array, stop 5 secs and then continue(clicking 5 times) CONSISTENTLY, ERROR

My problem is I cant get my button to consistently click 5 times through a string array which is displayed in a TextView ,
whenever the maxclicks(5) and currentnumber get to 5 it stops working , Ive been trying to create if conditions to work around it, well somehow I have to manipulate my currentnumber to NOT be 5 because IF maxclicks == currentnumber my button is enabled.

In the Code below it stops just afte the first time of clicking 5 times.

so here is the Code :

public class MainActivity extends AppCompatActivity


int currentnumber = 0;
int mod = 5;
TextView display = findViewById(R.id.tx);
Handler handler = new Handler();
int delay = 5000;
int maxclicks = list.length;



@Override
protected void onCreate(Bundle savedInstanceState)
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);



Resources res = getResources();
final String list = res.getStringArray(R.array.xyz);
final Button next_button = findViewById(R.id.next_btn);


((TextView) findViewById(R.id.tx)).setText(list[currentnumber]);

next_button.setOnClickListener(new View.OnClickListener()
@Override
public void onClick(View v)
if(currentnumber == maxclicks)
currentnumber = 0;

if (currentnumber % mod == 0)
next_button.setEnabled(false);
handler.postDelayed(new Runnable()
@Override
public void run()
//the button will unlock after the delay specified
next_button.setEnabled(true);
currentnumber++;


, delay);

else
display.setText(list[currentnumber]);
currentnumber++;


);







1 Answer
1

welcome to SO :) I did my best to understand on your explanation,so this is my solution for your problem and don't forget you can make your vars global to avoid final and one element array thing:

public class MainActivity extends AppCompatActivity
private int currentnumber,mod,delay,Curclicks;
private TextView display;
private Handler handler;
private Button next_button;
@Override
protected void onCreate(Bundle savedInstanceState)
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_setting);

//Binding
display = findViewById(R.id.tx);
next_button = findViewById(R.id.next_button);
//getResources
Resources res = getResources();

//getting the data ready
String list = "1","2","3","4","5","6","7";
//assign vars
handler = new Handler();
currentnumber = 0;
Curclicks=0;
mod = 5;
delay = 5000;
//initial view
next_button.setOnClickListener(new View.OnClickListener()
@Override
public void onClick(View v)

if(currentnumber == list.length)
currentnumber = 0;

if (Curclicks == mod-1)
next_button.setEnabled(false);
display.setText(list[currentnumber]);
currentnumber++;


handler.postDelayed(new Runnable()
@Override
public void run()
//the button will unlock after the delay specified
next_button.setEnabled(true);
Curclicks = 0;

, delay);

else
display.setText(list[currentnumber]);
currentnumber++;


Curclicks++;



);




By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.