What should I do here?

Question about the task

Under discussion

I totally don't understand why this solution is wrong. What does the 5th requirement mean if not what I've already done? I'm out of ideas. I'd appreciate your help.

Process the list of threads depending on their state:
1. If the thread isn't running yet, then start it.
2. If the thread is waiting, then interrupt it.
3. If the thread is running, check the isInterrupted flag.
4. If the thread has stopped running, then display its priority on the console.
Use a switch statement.

Requirements:

The processThreads() method take an array of threads as an argument.
If the passed thread is not running, you need to start it.
If the passed thread is waiting, you need to interrupt it.
If the passed thread is running, then you need to check the isInterrupted flag.
If the passed thread has finished running, you need to display its priority on the console.
The processThreads() method must use a switch statement.

package com.codegym.task.task25.task2504; /* A switch for threads */ public class Solution { public static void processThreads(Thread... threads) { for (Thread t : threads){ switch (t.getState()){ case NEW: t.start(); break; case WAITING: case TIMED_WAITING: case BLOCKED: t.interrupt(); break; case RUNNABLE: System.out.println(t.isInterrupted()); break; case TERMINATED: System.out.println(t.getPriority()); break; } } } public static void main(String[] args) { } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

BlueJavaBanana

Level 37

17 October 2020, 16:13

Try putting

t.interrupt();
break;

if the default block of the switch. and remove the

case WAITING:
case TIMED_WAITING:
case BLOCKED

Alternatively Make sure you do each case as a separate statement.

Michał

Level 41 , Krakow, Poland

20 October 2020, 16:38

Didn't work. Unfortunately. I think that requirement refers to the TERMINATED case. Though there's really nothing there to cause any problem. Thanks anyway for your help.

BlueJavaBanana

Level 37

20 October 2020, 17:18

useful

for (Thread t : threads) {

    switch (t.getState()) {

        case NEW:
            t.start();
            break;

        case BLOCKED:
            t.interrupt();
            break;

        case WAITING:
            t.interrupt();
            break;

        case RUNNABLE:
            t.isInterrupted();
            break;

        case TERMINATED:
            System.out.println(t.getPriority());
            break;

        case TIMED_WAITING:
            t.interrupt();
            break;

    }

}

Well try splitting things all up like this...

Michał

Level 41 , Krakow, Poland

20 October 2020, 17:48

Ok, I changed case RUNNABLE. I deleted system.out.println - left just t.isInterrupted(). Don't know why, but it helped.