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

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

ただいまの
回答率

90.48%

  • Android

    6633questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3792questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

他クラスからTextViewを変更したい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,261

K-Tomoya

score 3

前提・実現したいこと

Androidでアプリ開発をしています。
ServiceからActivityのTextViewに文字列を渡す機能を実装しようとしているのですが
以下のエラーメッセージが発生しました。

センサーが値を取得した時に呼ばれるonSensorChangedの中でサーバーとHTTP通信をするようのスレッドを立てるのですが、そのスレッドの中でMainActivityのテキストビューに文字列を代入しようとしています。

発生している問題・エラーメッセージ

FATAL EXCEPTION: main
Process: com.example.kimuratomoya.sensor_db_backup, PID: 14643
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
at com.example.kimuratomoya.sensor_db_backup.SendDataService.ChangeTextView(SendDataService.java:88)
at com.example.kimuratomoya.sensor_db_backup.SendDataService$3$2.run(SendDataService.java:601)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5253)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:900)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:695)

該当のソースコード

public class MainActivity extends AppCompatActivity {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
〜中略〜
        //成功したHTTPリクエスト数と失敗したHTTPリクエスト数のカウント
        new SendDataService(this);
〜中略〜
public class SendDataService extends Service implements SensorEventListener, LocationListener {
    private static MainActivity mainActivity;
    private Context context = this;
    private Context activitycon;
    public int errornum=0,sucnum=0;
    public TextView success_text, error_text;
〜中略〜

    //引数のないコンストラクタを作らないといけない。
    public SendDataService(){
    }
//MainActivity.javaから受け取ったContextを変数に代入するための関数
    public SendDataService(Context con){
        activitycon = con;
    }

//テキストビューを変更する関数
    public void ChangeTextView(Context con, Integer sucstr, Integer erstr){
        //Context testcon = mainActivity.getBaseContext();
        success_text = (TextView)((com.example.kimuratomoya.sensor_db_backup.MainActivity) activitycon).findViewById(R.id.success_id);
        success_text.setText(sucstr);
        error_text = (TextView)((com.example.kimuratomoya.sensor_db_backup.MainActivity) activitycon).findViewById(R.id.error_id);
        error_text.setText(erstr);
    }
〜中略〜

    public int onStartCommand(Intent intent, int flags, int startId) {
〜中略〜
}
    @Override
    public void onSensorChanged(SensorEvent event) {
〜中略〜
                Log.d("sensor", "スレッド立ち上げ");
                // 通信用のスレッドを起動
                new Thread(new Runnable() {
                    @Override
                    public void run() {
                        try {
                      〜中略〜
                            //テキストビューを変更する関数を呼び出しているところ
                            sucnum += 1;
                            ChangeTextView(activitycon, sucnum, errornum);
                        } catch (Exception e) {
                            try {
                               //テキストビューを変更する関数を呼び出しているところ
                                errornum += 1;
                                ChangeTextView(activitycon, sucnum, errornum);
                                if (conn != null) conn.disconnect();
                            }
〜中略〜
                }).start();
            }
〜中略〜
}

試したこと

public SendDataService(Context con)にMainActivityから直接コンテキストを渡せばテキストビューを変更できるのですが、Service内の他の関数(onSensorChanged)などからも、テキストビューを変更したくて、このようなプログラムになっています。
AplicationContextやActivityContextの違いなども調べたのですが、具体的にどのようにすればいいのかが、わかりません。
ChangeTextView()の中でgetAplicationContextを使ってコンテキストを取得しようとしたりしたのですが、同じエラーが出てしまいました。

補足情報(言語/FW/ツール等のバージョンなど)

Android Studio 2.1.2

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • t_obara

    2017/10/04 12:17

    全体を読み切れておりませんが、intentなどでテキストのみを渡し、TextViewを保有しているアプリがそれをハンドリングしてTextViewに反映させるという方法では問題があるのでしょうか?

    キャンセル

  • K-Tomoya

    2017/10/10 20:37

    それで大丈夫です!今はそのやり方で行き詰まっていて、サービスからstarActivityをしたら、MainActivityのonCreateの中のstartServiceが実行されて、アクティビティとサービスが交互に高速で起動するという状態になってしまっています。どうしたらいいでしょうか?

    キャンセル

回答 2

checkベストアンサー

0

他のスレッドからTextViewの更新をしている為にエラーが起きています。

TextViewは、この場合MainActivityが動作しているUIスレッドで更新する必要があります。

「Android UIスレッド 更新」等でググるとたくさんの情報にヒットしますので見てみてください。
具体的には、
Handler#post(Runnnable)とか、
TextView#post(Runnable)とか、
Activity#runOnUiThread(Runnable) 等の
メソッドを利用して、その中でTextViewを更新することになると思います。

※上記、外している可能性があるので追記させていただきます。
ご提示のコードをよく見ると 
public void onSensorChanged() 内で
new Thread(new Runnable() {
@Override
public void run() {
としているので、一見、大丈夫なようにも見えます。

問題無いかどうかは、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 です。

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です。

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 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 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/10 20:32

    完璧に理解しきれていないので、もしかしたら間違った解釈をしているかもしれませんが、
    Handler#post(Runnnable)、Activity#runOnUiThread(Runnable)は試した時にうまくいかなくて、
    うまくいかなかった理由が、Activityを持っているクラス(MainActivity)とHandlerを持っているクラス(SendDataService)が違うので、Handlerが使えなかった。UIスレッドは、サービスからはアクセスできないため。
    というように解釈し、この方法では無理なのかなと思っていました。
    この考え方は間違っているでしょうか?

    キャンセル

  • 2017/10/11 11:06

    もともとのご質問の回答/Exceptionの原因としてはabs123様のものが適切だったと思いますが、
    Handlerに関してうまくいかなかったのは、そのご理解でおおむね正しいかと思います。
    正確を期すると、クラスが違う為と言うよりは「Handlerのインスタンスを生成したスレッド」で
    run部分が実行される為、MainActivityのスレッドでChangeTextView()が実行されるようにしなければ
    表示されない、とのようになるかと。こちらで書かれている記事が分かり易いかと思いますので、ご紹介させていただきます。
    http://d.hatena.ne.jp/sankumee/20120329/1333021847

    キャンセル

  • 2017/10/11 12:35 編集

    記事まで探していただいて、ありがとうございます!この記事は僕も前に見たのですが、分からないところが幾つかあります。
    記事でいうHogeTaskクラスは僕でいうMainActivityクラスなのでしょうか?
    つまりpublic void execute(Looper toLooper, String data) をMainActivityに書けばいいのでしょうか?そうなると、Looperなど渡さなくても、Stringだけでテキストビューは変えられると思います。
    あとThreadは継承しないといけないのでしょうか?

    そうではなく、Service側にpublic void execute(Looper toLooper, String data) を用意するとしたら、
    public void run() {の中にsuccess_text.setText();を書かなければならず、
    TextView success_text = (TextView)findViewById(R.id.success_id);を書いたら、
    findViewByIdの所に「cannot resolve method findViewById(int)」というエラーが出てしましました。
    stackOverflowにonCreate()に書けと書いてるのですが、サービスの中にonCreate()を作るべきですか?
    onStartCommand()の中に書いても、同じエラーでした。

    手取り足取り教えて頂くようで申し訳ないのですが、よろしくお願い致します。

    キャンセル

  • 2017/10/11 16:00

    (私の回答が混乱を招いた感もありましたので追記しました。そちらもご覧ください)
    HogeTaskはただのクラスなので、MainActivityかと言うと微妙かも。
    むしろ、new HogeTask().execute(toLooper, "hogehage");
    をする呼び出し側が MainActivityな気もします。

    クラスとスレッドは別に考えないと混乱します。スレッドAがクラスAをnewしてメソッドを呼び出せばそのクラスAのインスタンスはスレッドAで動いていますし、スレッドBがクラスAをnewしてメソッドを呼び出せば、そのインスタンスはスレッドBで動いているコードです。

    Service側のコードでMainActivityのインスタンスメソッドfindTextByIdを呼ぶのは無理があります。Serviceの中でMainActivity.onCreate()は呼べない、と言うかonCreateはAndroidのシステムが呼び出してくるものです(コールバック)
    Serviceの中で自作のCreateTextView()は呼べない、と思ってください。Android内部を深く理解していれば方法はあるかもしれませんが、少しハックめいている気がするのでお勧めできないです。

    キャンセル

  • 2017/11/09 15:10

    返信が大変遅くなり申し訳ありません。他の課題で詰まっていたため、アクティビティに取り掛かる時間が取れずに遅くなってしまいました。
    dodox86さんのおかげでアクティビティについて少しは理解することができました!ありがとうございました!

    キャンセル

  • 2017/11/09 15:29

    コメントいただきどうもありがとうざいます。少しでも役に立ってよかったです。

    キャンセル

0

コードがいろいろと省略されているため憶測ですが、
システムがstartServiceメソッドの呼び出しを受けて生成したインスタンスと、
あなたがnewしたことによって生成されたインスタンスの2つ生成されていると考えられます。

サービスとして動いているのはシステムが生成したほうで、
インスタンスの生成にはコンストラクタの引数がないものが使われているはずです。
そのためactivityconはnullのままで、activityconを操作する段階で落ちます。

あなたがnewしたほうは参照の保持すらされていないので
早々にGCで回収されていることでしょう。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/10 20:26

    なるほど!そもそもactivitycon = con;が実行されていなかったのですね。
    コメントありがとうございます

    キャンセル

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

  • ただいまの回答率 90.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Android

    6633questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3792questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。