質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Q&A

解決済

1回答

2904閲覧

スレッドで実行が止まる理由

grape_ll

総合スコア83

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

0グッド

0クリップ

投稿2020/07/13 04:11

編集2020/07/13 04:21

質問内容

以下のコードで後者の方を実行するとAからとBからを一回ずつしか実行されないのですがその理由はなぜですか?

コード1

Java

1class MyBank{ 2 private String name; 3 private int value; 4 MyBank(String n,int initial_value){ 5 name=n; 6 value=initial_value; 7 } 8 public void transferMoney(MyBank other,int amount) throws InterruptedException{ 9 synchronized(this){ 10 System.out.println("Transferring "+amount+" from "+name); 11 Thread.sleep(100); 12 synchronized(other){ 13 if(amount>value) System.out.println("Not enough money"); 14 else{ 15 other.value=other.value+amount; 16 this.value-=amount; 17 } 18 } 19 } 20 } 21}

コード2

Java

1import java.util.Random; 2class MyBankTest extends Thread{ 3 MyBank b1,b2; 4 Random rand=new Random(); 5 MyBankTest(MyBank b1,MyBank b2){ 6 this.b1=b1; 7 this.b2=b2; 8 } 9 public void run(){ 10 try{ 11 while(true){ 12 b1.transferMoney(b2,rand.nextInt(100)); 13 } 14 }catch(InterruptedException e){ 15 System.out.println(e); 16 } 17 } 18 public static void main(String[] args){ 19 MyBank a=new MyBank("A",1000); 20 MyBank b=new MyBank("B",1000); 21 new MyBankTest(a,b).start(); 22 new MyBankTest(b,a).start(); 23 24 } 25}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

transferMoneyメソッドの中で最初のsynchronizedブロックで自分を掴みます。
次のsynchronizedブロックで、お互いを掴もうとしてますが、もう掴み済みなのでお互いが開放されるのを待っています。
いわゆるデッドロックですね。

コメントに対する追記

自分から金額を引き落として、相手に振り込むまでを1トランザクションとしたいため、
以下のようにすればいいと思います。

Java

1public void transferMoney(MyBank other, int amount) throws InterruptedException { 2 synchronized (MyBank.class) { 3 System.out.println("Transferring " + amount + " from " + name); 4 Thread.sleep(100); 5 if (amount > value) 6 System.out.println("Not enough money"); 7 else { 8 other.value = other.value + amount; 9 this.value -= amount; 10 } 11 } 12}

投稿2020/07/13 05:13

編集2020/07/13 05:33
root_jp

総合スコア4666

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

grape_ll

2020/07/13 05:16

この修正のためにotherのsynchronizedをなくしたのですが対処としては正しいでしょうか
root_jp

2020/07/13 05:33

追記しました。
grape_ll

2020/07/13 06:24

ありがとうございます.参考にさせていただきます.
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問