Toast — как правильно с ним работать?

Question

[kirawa]

Аналогичный код я вешаю на несколько кнопок, меняя названия самих кнопок соответственно. Тут если я зажимаю кнопку выводит текст с её описанием, но если зажимаю другую кнопку, то опять выводит предыдущее описание. В общем нажимаю на btn1 выводит btn1, нажимаю на btn2 выводит btn1, нажимаю опять на bnt2 выводит btn2.

boolean isDown = false;
    Timer timer;
    TimerTask task;
    int count;
 
    public  void start(){
        timer.schedule(task, 2,20);
    }
    public boolean onTouch(final View view, MotionEvent motionEvent) {
        switch (view.getId()){
            case R.id.imgBntAdd:
                switch (motionEvent.getAction()){
                    case MotionEvent.ACTION_DOWN:
                        timer = new Timer();
                        task = new TimerTask() {
                            public void run() {
                                if(isDown){
                                    count++;
                                }
                                 runOnUiThread(new Runnable() {
                                    public void run() {
                                        if(count >= 100){
                                            int[] pos = new int[2];
                                            btnAdd.getLocationInWindow(pos);
                                            String s = btnAdd.getContentDescription().toString();
                                            Toast t =  Toast.makeText(context, s, Toast.LENGTH_SHORT);
                                            t.setGravity(Gravity.TOP | Gravity.LEFT, btnAdd.getRight() + 5 ,pos[1] + 20 );
                                            t.show();
                                            count = 0;
                                            isDown = false;
                                        }
                                    }
                                });
                            }
                        };
                        this.isDown = true;
                        start();
                        btnAdd.setBackgroundColor(color1);
                        this.time1 = System.currentTimeMillis() / 1000;
                        break;
 
                    case MotionEvent.ACTION_UP:
                        btnAdd.setBackgroundColor(color2);
                        this.isDown = false;
                        this.count = 0;
                        this.time2 = System.currentTimeMillis() / 1000;
                        if (time1 + 1 - time2 > 0){
                        startActivityNote();
                        }
                    default:
                        break;
                }
                break;

Помогите разобраться, пожалуйста.

Answer 1

[kirawa]

да не густо, я вообще не понимаю зачем вы это написали

Answer 2

[GavriKos]

@kirawa Объясните, зачем такие сложности в запуске? Может я чего то не понимаю, но к чему таймер, счетчики - это все вроде в тосте реализовано напрямую.

Answer 3

[kirawa]

вот есть кнопка. я на нее нажимаю она подсвечивается, если я её зажал свыше секунды то включается Toast уведомление, иначе переход на другую Activity. А и отпускаю подсветка тоже изчезает.

Comments

[GavriKos]

Пишите ответы при помощи кнопки "комментировать". У вас архитектурно все сделано очень неправильно. Во первых - вы на каждое событие таймера создаете поток, и в нем проверяете таймер - не годится, создавайте поток только если реально нужно показать тост. Второе - как я понял у вас один таймер на две кнопки. А что будет если пользователь ОДНОВРЕМЕННО нажмет обе кнопки?

Я бы делал так - создал свой класс, наследованный от класса кнопки, ему перегрузил бы метод отлова нажатия, в нем запускал таймер, проверял все ли ок, и показывал бы тост. Как разрулить в такой комбинации что пользователь может обе кнопки нажать - не знаю.

[GavriKos]

@kirawa ну и не знаю, как по мне это лишний функционал - вы где то такой видели в приложениях андроидных?

[kirawa]

помогите создать кастомный класс для кнопки, ну или хотя бы дать понять как он будет работать. И как с ним работать
public class Button extends ImageButton implements View.OnTouchListener{
public Button(Context context) {
super(context);
}
@Override
public boolean onTouch(View view, MotionEvent motionEvent) {
switch (motionEvent.getAction()){
case MotionEvent.ACTION_DOWN:

break;
case MotionEvent.ACTION_UP:

break;
}
return true;
}
}
как его использовать, и что он будет возвращать и как это получить

[kirawa]

вот смотрю сюда, но все же неоособо понимаю что делать
public class Button extends ImageButton{

public Button(Context context) {
super(context);
}

@Override
public boolean onKeyPreIme(int keyCode, KeyEvent event) {
if (event.getKeyCode() == KeyEvent.ACTION_UP) {
dispatchKeyEvent(event);
return false;
}else if (event.getKeyCode() == KeyEvent.ACTION_DOWN){
dispatchKeyEvent(event);
}
return super.onKeyPreIme(keyCode, event);
}
}

[kirawa]

не знал я про CountDownTimer;