Please what's wrong with this code

Question about the task

A switch for threads

Java Multithreading, Level 5, Lesson 5

Under discussion

I don't know why the requirements are not met

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 thread : threads ){ switch(thread.getState()){ case NEW : thread.start(); break; case RUNNABLE : System.out.println(thread.isInterrupted()); break; case WAITING : thread.interrupt() ; System.out.println(thread.getPriority()); break; case TIMED_WAITING : thread.interrupt() ; System.out.println(thread.getPriority()); break; case BLOCKED : thread.interrupt() ; System.out.println(thread.getPriority()); break; case TERMINATED :System.out.println(thread.getPriority()); } } // Implement this method } public static void main(String[] args) { } }

Comments (6)

Popular
New
Old

You must be signed in to leave a comment

Yiheng Niu

Level 26 , Beijing, China

12 April 2020, 15:09

You only need to "System.out.println(thread.getPriority()); " when the thread is terminated. Because after the thread has finished running, it is terminated. And there should be a "break" after your last case TERMINATED.

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

13 April 2020, 19:03

thanks for your tips , but the task still fails to pass

Yiheng Niu

Level 26 , Beijing, China

14 April 2020, 06:26

I don't know what's your code now after you modified it. Compare the code below with yours:

for (Thread thread : threads) {
            switch (thread.getState()) {
                case NEW:
                    thread.start();
                    break;
                case WAITING:
                    thread.interrupt();
                    break;
                case TIMED_WAITING:
                    thread.interrupt();
                    break;
                case BLOCKED:
                    thread.interrupt();
                    break;
                case RUNNABLE:
                    thread.isInterrupted();
                    break;
                case TERMINATED:
                    System.out.println(thread.getPriority());
                    break;


            }

        }

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

15 April 2020, 15:42

Thank you so much , it really works

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

15 April 2020, 15:47

I think I made a mistake when I was dealing with the Runnable state. switch(thread.getState()){ case NEW : thread.start(); break; case RUNNABLE : System.out.println(thread.isInterrupted()); break; case WAITING : thread.interrupt() ; break; case TIMED_WAITING : thread.interrupt() ; break; case BLOCKED : thread.interrupt() ; break; case TERMINATED :System.out.println(thread.getPriority()); break; }

SOUFIANE DAHIMI

Level 32 , Marrakech, Morocco

15 April 2020, 15:49

The validator misled me, because he was referring to the Terminated state. Thanks again