他のスレッドに文字列を渡したい

Question

質問内容

以下のコードでは，片方のスレッドでコマンドを受け付け，もう片方でそのコマンドに基づいた実行をするというコードを目指してかきました．求める形としては
Command>add sample1　　としたときに
listに加わってほしいのですが，現段階では以下のように一度改行を挟まないとうまく動いてくれません．これを直すにはどのようにすればよいでしょうか．
また，この内容のプログラムならばスレッドを使う必要はあまり感じられないかもしれませんが，スレッドを使ったやり方でご教授をお願します．

java ThreadList
Command>add sample1
Command>print
[]
Command>add
sample2
add : sample2
Command>print
[sample2]
Command>exit

import java.util.*;
public class ThreadList implements Runnable {

    int id=1;
    //0:inout,1:reader

    static int sw=0;
    static int command_num=0;
    static Object lock;
    LinkedList<String> list=new LinkedList<String>();

    ThreadList(int n,Object lock,LinkedList<String> list){
        this.lock=lock;
        this.list=list;
        id=n;
    }

    public static void main(String[] args){
        Object lock=new Object();
        LinkedList<String> list=new LinkedList<String>();
        (new Thread(new ThreadList(0,lock,list))).start();
        (new Thread(new ThreadList(1,lock,list))).start();

        
    }
    public void run(){
        while(true){
            action();
        }
    }
    void action(){
        synchronized(lock){
            try{
                while(sw==id) lock.wait();
            }catch(InterruptedException e){}

            if(id==0){
                Scanner scan=new Scanner(System.in);
                System.out.printf("Command>");
                String command=scan.nextLine();
                if(command.equals("add")){
                    String content=scan.nextLine();
                    command_num=1;
                    list.add(content);
                }
                else if(command.equals("remove")){
                    String content=scan.nextLine();
                    command_num=2;
                    list.add(content);
                }
                else if(command.equals("print")){
                    command_num=3;
                }
                else if(command.equals("exit")){
                    command_num=4;
                    System.exit(0);
                }

                sw=id;
                lock.notifyAll();

            }
            else if(id==1){
                if(command_num==1){
                    System.out.println("add : "+list.get(list.size()-1));
                }
                else if(command_num==2){
                    System.out.println("remove : "+list.get(list.size()-1));
                    LinkedList<String> set=new LinkedList<String>();
                    Collections.addAll(set,list.get(list.size()-1));
                    list.removeAll(set);
                }
                else if(command_num==3){
                    System.out.println(list);
                }
                else if(command_num==4){
                    System.exit(0);
                }

                sw=id;
                command_num=0;
                lock.notifyAll();

            }
        }
    }
}

Answer 1

ScannerのnextLine()って改行までを1つの塊と扱うので「add sample1」という文字列になっていると思います

splitで切り分けて使うか、
空白文字で切れるnext()を使うといいんじゃないでしょうか

Comments

[grape_ll]

改行まで含まれることを失念していました．
空白文字で切れるようにnext()にしたら解決しました．ご指摘ありがとうございました．

Answer 2

String command=scan.nextLine();

            if(command.equals("add")){
                String content=scan.nextLine();

このコードはどういう動作をしてるのか読んでみましょう。

文字列をコンソールから読み込み、
その文字列が"add"かどうかをチェックし、
文字列をコンソールから読み込む

というそれそのままのコードとなってます。

これをどうにかする必要がありますね

Comments

[grape_ll]

next()にしたら解決しました．ご指摘ありがとうございました．

Answer 3

Scanner scan を ThreadList の static なフィールドにしたらどうなりますか？

 public class ThreadList implements Runnable {
 
+    static Scanner scan = new Scanner(System.in);
     int id=1;
     //0:inout,1:reader
 
====== 
             if(id==0){
-                Scanner scan=new Scanner(System.in);
                 System.out.printf("Command>");

Comments

[grape_ll]

同じ結果になってしまいました．ほかにも改善できそうな点はございますでしょうか．

[grape_ll]

nextLine()をnextにしたら解決しました．
ありがとうございました．