編集履歴

回答編集履歴

見づらいので改行位置を修正

2017/10/11 06:40

投稿

スコア9293

test CHANGED Viewed

@@ -44,9 +44,7 @@
 ###ActivityとService間の通信について追記(2017/10/11)
-ご質問の問題の中身とコメントの内容を再考してみますと、どうも混乱させてしまったようですので、
+ご質問の問題の中身とコメントの内容を再考してみますと、どうも混乱させてしまったようですので、追記することで整理させていただきたいと思います。※混乱のひとつは私の回答のせいであります。
-追記することで整理させていただきたいと思います。※混乱のひとつは私の回答のせいであります。
@@ -56,9 +54,7 @@
-この点の回答としては、abs123様のものが適切だと考えています。（K-Tomoya様、これはOKですよね？）
+この点の回答としては、abs123様のものが適切だと考えています。（K-Tomoya様、これはOKですよね？）で、次に、私が最初に回答させてもらった件
-で、次に、私が最初に回答させてもらった件
 > 他のスレッドからTextViewの更新をしている為にエラーが起きています。

当初の回答に不足、不適切な事項があったので、追記。

2017/10/11 06:40

投稿

dodox86

スコア9293

test CHANGED Viewed

@@ -39,3 +39,385 @@
 問題無いかどうかは、SensorEventListenerが違うスレッドで動作していて、
 OnSensorChanged()メソッドがそのスレッドで呼ばれているかどうかによります。
+###ActivityとService間の通信について追記(2017/10/11)
+ご質問の問題の中身とコメントの内容を再考してみますと、どうも混乱させてしまったようですので、
+追記することで整理させていただきたいと思います。※混乱のひとつは私の回答のせいであります。
+当初の質問で示された以下のエラーに関してですが、
+> java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View com.example.kimuratomoya.sensor_db_backup.MainActivity.findViewById(int)' on a null object reference
+この点の回答としては、abs123様のものが適切だと考えています。（K-Tomoya様、これはOKですよね？）
+で、次に、私が最初に回答させてもらった件
+> 他のスレッドからTextViewの更新をしている為にエラーが起きています。
+> TextViewは、この場合MainActivityが動作しているUIスレッドで更新する必要があります。
+ですが、これはこれでその通りなのですが、今回はActivityとService間の通信の話になってくるので、ActivityのUIスレッドだけの話では終わらないです。この点、私の回答が拙速であったこと、お詫びします。
+では結局、Activityが持っているTextViewをService側で発生したデータで更新して表示したいと言う要求はどうしたらいいのかと言うと、簡単には「Activityに`BroadcastReceiver`を持たせ、Serivice側からActivityへIntentで通知する。」との方法が適切かと思います。その為のサンプルを示します。Service内で1秒毎にActivityへIntentを送信し、ActivityではそのIntentから取り出した文字列をTextViewに表示するだけのものです。短いので全部載せます。
+尚、**本サンプルではActivityが停まってもServiceは停まりません**のでご注意ください。
+Activityの、MainActivity.java です。
+```Java
+package examples.products.test24;
+import android.content.BroadcastReceiver;
+import android.content.Context;
+import android.content.Intent;
+import android.content.IntentFilter;
+import android.support.v7.app.AppCompatActivity;
+import android.os.Bundle;
+import android.util.Log;
+import android.widget.TextView;
+public class MainActivity extends AppCompatActivity {
+    private static final String TAG = "Test24.MainActivity";
+    private BroadcastReceiver mReceiver = null;
+    private IntentFilter mIntentFilter = null;
+    private TextView mTextView1 = null;
+    @Override
+    protected void onCreate(Bundle savedInstanceState) {
+        super.onCreate(savedInstanceState);
+        setContentView(R.layout.activity_main);
+        mTextView1 = (TextView) findViewById(R.id.textview1);
+        mReceiver = new BroadcastReceiver() {
+            @Override
+            public void onReceive(Context context, Intent intent) {
+                // このonReceiveでMainServiceからのIntentを受信する。
+                Bundle bundle = intent.getExtras();
+                String message = bundle.getString("message");
+                Log.d(TAG, "Message from MainService: " + message);
+                // TextViewへ文字列をセット
+                mTextView1.setText(message);
+            }
+        };
+        // "TEST24_ACTION" Intentフィルターをセット
+        mIntentFilter = new IntentFilter();
+        mIntentFilter.addAction("TEST24_ACTION");
+        registerReceiver(mReceiver, mIntentFilter);
+        Intent intent = new Intent(getApplication(), MainService.class);
+        startService(intent);
+        // !!!! サンプルの為、MainServiceサービスを停める処理が無いので、注意してください。
+    }
+}
+```
+Serviceの、MainService.javaです。
+```Java
+package examples.products.test24;
+import android.app.Service;
+import android.content.Intent;
+import android.os.IBinder;
+import android.util.Log;
+import java.util.Timer;
+import java.util.TimerTask;
+public class MainService extends Service {
+    private static final String TAG = "Test24.MainService";
+    private Timer mTimer = null;
+    private int mCount = 0;
+    @Override
+    public void onCreate() {
+        super.onCreate();
+    }
+    @Override
+    public int onStartCommand(Intent intent, int flags, int startId) {
+        Log.d(TAG, "onStartCommand():");
+        mTimer = new Timer();
+        mTimer.schedule(new TimerTask() {
+            @Override
+            public void run() {
+                mCount++;
+                // MainActivityへデータを送信
+                String message = String.format("count=%d", mCount);
+                sendBroadcast(message);
+            }
+        }, 0, 1000);
+        return super.onStartCommand(intent, flags, startId);
+    }
+    @Override
+    public void onDestroy() {
+        super.onDestroy();
+        Log.d(TAG, "onDestroy():");
+        // タイマーを停止
+        if (mTimer != null) {
+            mTimer.cancel();
+            mTimer = null;
+        }
+    }
+    @Override
+    public IBinder onBind(Intent intent) {
+        Log.d(TAG, "onBind():");
+        return null;
+    }
+    private void sendBroadcast(String message) {
+        Log.d(TAG, "sendBroadcast: " + message);
+        // IntentをブロードキャストすることでMainActivityへデータを送信
+        Intent intent = new Intent();
+        intent.setAction("TEST24_ACTION");
+        intent.putExtra("message", message);
+        getBaseContext().sendBroadcast(intent);
+    }
+}
+```
+AndroidManifest.xmlです
+```XML
+<?xml version="1.0" encoding="utf-8"?>
+<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    package="examples.products.test24">
+    <application
+        android:allowBackup="true"
+        android:icon="@mipmap/ic_launcher"
+        android:label="@string/app_name"
+        android:roundIcon="@mipmap/ic_launcher_round"
+        android:supportsRtl="true"
+        android:theme="@style/AppTheme">
+        <activity android:name=".MainActivity">
+            <intent-filter>
+                <action android:name="android.intent.action.MAIN" />
+                <category android:name="android.intent.category.LAUNCHER" />
+            </intent-filter>
+        </activity>
+        <service android:name=".MainService"></service>
+    </application>
+</manifest>
+```
+レイアウトのlayout/activity_main.xmlです。
+```XML
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout 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"
+    android:orientation="vertical"
+    tools:context="examples.products.test24.MainActivity">
+    <TextView
+        android:id="@+id/textview1"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="Text#1"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintLeft_toLeftOf="parent"
+        app:layout_constraintRight_toRightOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+</LinearLayout>
+```
+UIスレッドではないスレッドからのTextView等のビューの更新についてですが、先の回答で提示させてもらった以下のクラスメソッドの利用
+- `Handler#post(Runnnable)`
+- `TextView#post(Runnable)`
+- `Activity#runOnUiThread(Runnable)`
+は、これらはActivityで SensorEventListener, LocationListener等を使っていて、イベントがActivityとは違うスレッドで通知される場合に必要になります。今回はServiceでイベントをハンドリングしているので、恐らく問題にはならないと思われます。混乱させてすみません。

注意事項を追記

2017/10/11 06:37

投稿

dodox86

スコア9293

test CHANGED Viewed

@@ -17,3 +17,25 @@
 Activity#runOnUiThread(Runnable) 等の
 メソッドを利用して、その中でTextViewを更新することになると思います。
+**※上記、外している可能性があるので追記させていただきます。**
+ご提示のコードをよく見ると
+public void onSensorChanged() 内で
+new Thread(new Runnable() {
+    @Override
+    public void run() {
+としているので、一見、大丈夫なようにも見えます。
+問題無いかどうかは、SensorEventListenerが違うスレッドで動作していて、
+OnSensorChanged()メソッドがそのスレッドで呼ばれているかどうかによります。