Почему не проходит валидацию ?

Вопрос по задаче

Решен

Валидатор пишет: "Программа вывела в консоль (на экран) слишком много данных" Что не так ?

1. Измени класс GenerateThread так, чтобы он стал нитью.
2. Создай конструктор GenerateThread, который должен:
2.1. Вызвать конструктор суперкласса с параметром String - номером созданной нити. Используй createdThreadCount.
2.2. Запустить текущую нить.
2.3. Номер первой нити должен начинается с 1.
3. Переопредели метод toString, для этого внутри GenerateThread нажми Alt+Insert -> Override Methods. Начни печатать toString.
3.1. Метод toString должен возвращать № текущей нити и слово " created". Используй getName().

Пример:
8 created

4. Пока количество созданных нитей меньше Solution.count метод run должен:
4.1. Создать новую нить типа GenerateThread.
4.2. Вывести в консоль созданную в пункте 4.1 нить.
5. В итоге должно быть выведено в консоль 15 строк.

Требования:

Класс GenerateThread должен быть унаследован от Thread.
В классе GenerateThread должен быть открытый конструктор без параметров.
Конструктор класса GenerateThread должен увеличивать значение createdThreadCount и передавать его в виде строки в конструктор суперкласса.
Конструктор класса GenerateThread должен запускать нить.
Метод toString класса GenerateThread должен возвращать имя нити и слово " created". Пример: "8 created".
Если количество созданных нитей меньше Solution.count, метод run должен создать новую нить типа GenerateThread.
Если количество созданных нитей меньше Solution.count, метод run должен вывести созданную нить в консоль.
Вывод программы должен соответствовать заданию, показывать, что все 15 нитей были созданы.

package com.javarush.task.task16.task1623; /* Рекурсивное создание нитей */ public class Solution { static int count = 15; static volatile int createdThreadCount; public static void main(String[] args) { System.out.println(new GenerateThread()); } public static class GenerateThread extends Thread{ @Override public String toString() { return getName() + " created"; } public GenerateThread() { super(String.valueOf(++createdThreadCount)); start(); try { join(); }catch (InterruptedException e) {} } @Override public void run() { // System.out.println(this); while (createdThreadCount < count) { System.out.println(new GenerateThread()); } } } }

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Илья Геков

Уровень 40

Master

23 октября 2018, 20:11

А зачем join? Он порядок вывода на консоль меняет. Плюс в мейне логики вывода на консоль не должно быть, вся логика задачи должна быть в самой нити.

Igor

Уровень 24

23 октября 2018, 20:42

Да в общем-то не за чем, это был эксперимент на тему "а вдруг так пройдет", без него тот же самый результат.

Илья Геков

Уровень 40

Master

23 октября 2018, 20:50

По поводу второго замечания поправили?

Igor

Уровень 24

24 октября 2018, 07:05

Пробовал, у меня все правильно отрабатывает, а валидатор зависает: "Программа работала слишком долго и была закрыта!".

package com.javarush.task.task16.task1623;

public class Solution {
    static int count = 15;
    static volatile int createdThreadCount;

    public static void main(String[] args) {
        new GenerateThread();
    }

    public static class GenerateThread extends Thread{
        @Override
        public String toString() {
            return getName() + " created";
        }

        public GenerateThread() {
            super(String.valueOf(++createdThreadCount));
            start();
        }

        @Override
        public void run() {
            System.out.println(this);
            while (createdThreadCount < count) {
                new GenerateThread();
            }
        }
    }
}

Igor

Уровень 24

24 октября 2018, 07:51

решение

Вернул в main вывод, заменил в run() цикл на условие, после этого валидатор принял:

public void run() {
    if (createdThreadCount < count){
        System.out.println(new GenerateThread());
    }
}