1

Есть класс для потока, код из конструктора касса работает всегда, а run() запускается через раз, в чём может быть причина? В логе пусто.

public class SThread extends Thread
    {
        Thread ScoreThread;
        public SThread()
        {
            ScoreThread = new Thread(this);
            ScoreThread.start();
        }
        public void run()
        {
            while(running)
            {
                if(!onDraw)return;
                try{Thread.sleep(500);}catch (InterruptedException e){e.printStackTrace();}
                score++;
                text = "Score: "+score;
            }
        }
    }

Вызываю вот так

new SThread();
0

1 ответ 1

4

Стартуя поток в конструкторе и передавая this, вы передаёте ссылку на объект, который ещё не полностью сконструирован. Поток стартует раньше, чем конструктор завершится, поэтому результат не предсказуем. Плюс в вашем кейсе лучше использовать Runnable.

public class SThread implements Runnable{
        Thread ScoreThread;
        public SThread(){
            ScoreThread = new Thread(this);
        }
        public void start() {
            ScoreThread.start();
        } 
        public void run(){
            while(running){
                if(!onDraw)return;
                try{Thread.sleep(500);}catch (InterruptedException e){e.printStackTrace();}
                score++;
                text = "Score: "+score;
            }
        }
    }

И запускаем:

    SThread thread = new SThread ();
    thread.start();

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.