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 that you have read and understand our privacy policy and code of conduct.

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