回答編集履歴

コード修正

2022/12/04 04:21

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -36,7 +36,7 @@
       @Override
       public void onClick(View v) {
         int c = count ++;
-        log("push Button(" + count + ").");
+        log("push Button(" + c + ").");
         lastExecutor.request(() -> {
           log("process(" + c + ") start.");

java を kotlin に合わせて修正

2022/12/04 04:19

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -23,22 +23,22 @@
     super.onCreate(savedInstanceState);
     setContentView(R.layout.activity_main);
-    LastExecutor lastExecutor = new LastExecutor();
-    Random random = new Random();
     handler = new Handler(Looper.getMainLooper());
     textView = findViewById(R.id.textView);
     scrollView = findViewById(R.id.scrollView);
+    LastExecutor lastExecutor = new LastExecutor();
     Button button = findViewById(R.id.requestButton);
     button.setOnClickListener(new View.OnClickListener() {
+      Random random = new Random();
       int count = 0;
       @Override
       public void onClick(View v) {
-        count ++;
+        int c = count ++;
         log("push Button(" + count + ").");
         lastExecutor.request(() -> {
-          int c = count;
           log("process(" + c + ") start.");
           try {
             Thread.sleep(random.nextInt(3000) + 2000); //2～5秒

kotlin コード追加

2022/12/04 04:13

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -125,3 +125,96 @@
 </androidx.constraintlayout.widget.ConstraintLayout>
 ```
 ![実行してボタンを適当に押した後のスクリーンショット](https://ddjkaamml8q8x.cloudfront.net/questions/2022-12-04/a3b56b41-30bc-455c-9648-0d0869f278ca.png)
+---
+私も AndroidStudio に kotlin 変換してもらって修正したらこんな感じになりました。
+こちらはそれこそ参考ということで…。
+MainActivity.kt
+```kotlin
+import android.os.Bundle
+import android.os.Handler
+import android.os.Looper
+import android.view.View
+import android.widget.Button
+import android.widget.ScrollView
+import android.widget.TextView
+import androidx.appcompat.app.AppCompatActivity
+import java.time.LocalTime
+import java.util.*
+class MainActivity : AppCompatActivity() {
+  private lateinit var handler: Handler
+  private lateinit var textView: TextView
+  private lateinit var scrollView: ScrollView
+  override fun onCreate(savedInstanceState: Bundle?) {
+    super.onCreate(savedInstanceState)
+    setContentView(R.layout.activity_main)
+    handler = Handler(Looper.getMainLooper())
+    textView = findViewById(R.id.textView)
+    scrollView = findViewById(R.id.scrollView)
+    val lastExecutor = LastExecutor()
+    val button: Button = findViewById(R.id.requestButton)
+    button.setOnClickListener(object : View.OnClickListener {
+      val random = Random()
+      var count = 0
+      override fun onClick(v: View) {
+        val c = count++
+        log("push Button($c).")
+        lastExecutor.request {
+          log("process($c) start.")
+          try {
+            Thread.sleep((random.nextInt(3000) + 2000).toLong()) //2～5秒
+          } catch (_: InterruptedException) {
+          }
+          log("process($c) end.")
+        }
+      }
+    })
+  }
+  private fun log(text: String) {
+    handler.post {
+      textView.append("${LocalTime.now()}: $text\n")
+      scrollView.fullScroll(ScrollView.FOCUS_DOWN)
+    }
+  }
+}
+class LastExecutor {
+  private var queue: Runnable? = null
+  private var worker: Thread? = null
+  private val lock = Object()
+  fun request(process: Runnable?) {
+    synchronized(lock) {
+      queue = process
+      lock.notifyAll()
+    }
+    if (worker == null) startWorker()
+  }
+  private fun startWorker() {
+    worker = Thread {
+      try {
+        while (true) {
+          var p: Runnable
+          synchronized(lock) {
+            while (queue == null) lock.wait()
+            p = queue as Runnable
+            queue = null
+          }
+          p.run()
+        }
+      } catch (e: InterruptedException) {
+        e.printStackTrace()
+      }
+    }
+    worker!!.start()
+  }
+}
+```

コードの変数名が頭大文字だったのを小文字化

2022/12/04 03:51

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -23,7 +23,7 @@
     super.onCreate(savedInstanceState);
     setContentView(R.layout.activity_main);
-    LastExecutor LastExecutor = new LastExecutor();
+    LastExecutor lastExecutor = new LastExecutor();
     Random random = new Random();
     handler = new Handler(Looper.getMainLooper());
@@ -37,7 +37,7 @@
       public void onClick(View v) {
         count ++;
         log("push Button(" + count + ").");
-        LastExecutor.request(() -> {
+        lastExecutor.request(() -> {
           int c = count;
           log("process(" + c + ") start.");
           try {

レイアウト貼り忘れ

2022/12/04 03:21

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -1,6 +1,8 @@
 停止も無いやっつけでしかも java で申し訳ないですが、こんな感じは如何でしょう。
 勉強中ということで基本と思われる wait - notify です。
 process が終わるまでに何度ボタンを押しても、最後のが次に実行されます。
+MainActivity.java
 ```java
 import androidx.appcompat.app.AppCompatActivity;
@@ -87,4 +89,39 @@
   }
 }
 ```
+res/layout/activity_main.xml
+```xml
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:context=".MainActivity">
+  <Button
+      android:id="@+id/requestButton"
+      android:layout_width="wrap_content"
+      android:layout_height="wrap_content"
+      android:text="request"
+      app:layout_constraintEnd_toEndOf="parent"
+      app:layout_constraintStart_toStartOf="parent"
+      app:layout_constraintTop_toTopOf="parent" />
+  <ScrollView
+      android:id="@+id/scrollView"
+      android:layout_width="0dp"
+      android:layout_height="0dp"
+      app:layout_constraintBottom_toBottomOf="parent"
+      app:layout_constraintEnd_toEndOf="parent"
+      app:layout_constraintStart_toStartOf="parent"
+      app:layout_constraintTop_toBottomOf="@id/requestButton">
+    <TextView
+        android:id="@+id/textView"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:textAlignment="textStart"
+        android:textSize="20dp" />
+  </ScrollView>
+</androidx.constraintlayout.widget.ConstraintLayout>
+```
 ![実行してボタンを適当に押した後のスクリーンショット](https://ddjkaamml8q8x.cloudfront.net/questions/2022-12-04/a3b56b41-30bc-455c-9648-0d0869f278ca.png)

説明修正

2022/12/03 18:48

投稿

jimbe

スコア13318

test CHANGED Viewed

@@ -1,5 +1,5 @@
 停止も無いやっつけでしかも java で申し訳ないですが、こんな感じは如何でしょう。
-勉強中ということで wait - notify による排他制御です。
+勉強中ということで基本と思われる wait - notify です。
 process が終わるまでに何度ボタンを押しても、最後のが次に実行されます。
 ```java
 import androidx.appcompat.app.AppCompatActivity;