Android プログラム中でグローバル変数が使用できない

Question

PCスペック
Windows10 HOME 64bit
メモリ　32GB
プロセッサ　Intel Core i7-8700CPU 3.20GHz
Android Studio 3.4.1(最新ver)
デモ用端末(XPERIA OS:7.0 API 24)

グローバル変数を先に宣言しておいて、クラス内の数か所でその変数を使用したいのですが、
Non-static field 'imageDetail' cannot be referenced from a static context
という赤文でエラーメッセージが出てしまい、困っています。

具体的には、GoogleのCloud Vision API(https://github.com/GoogleCloudPlatform/cloud-vision/tree/master/android/)のコードの中で起こった出来事です。

protected void onPostExecute(String result) {
            MainActivity activity = mActivityWeakReference.get();
            if (activity != null && !activity.isFinishing()) {
                TextView imageDetail = activity.findViewById(R.id.image_details);//imageDetailsを上書きする形になる
                imageDetail.setText(result);//画面に表示
               
            }
        }

この部分のコードで、画像から認識できたすべての「文字」をimageDetail.setText(result);で表示しています。
ここで、imageDetailはonPostExecute内のローカル変数であったため、これをグローバル変数として宣言しなおそうと、下記のように変更しました。

public class MainActivity extends Activity {

private TextView imageDetail;

@Override
    protected void onCreate(Bundle savedInstanceState) {

~~~~~~~~~~~~~~~~~~~~~~~~中略~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

protected void onPostExecute(String result) {
            MainActivity activity = mActivityWeakReference.get();
            if (activity != null && !activity.isFinishing()) {
                imageDetail = activity.findViewById(R.id.image_details);//imageDetailsを上書きする形になる
                imageDetail.setText(result);//画面に表示
                
            }
        }

しかし、このように変えた結果、"Non-static field 'imageDetail' cannot be referenced from a static context"(非静的フィールド 'imageDetail'は静的コンテキストから参照できません)というエラーとなってしまいました。

~~また、2つ目のソースコードで、グローバル変数として宣言した際に、"Variable 'imageDetail' is already defined in the scope"(変数 'imageDetail'はすでにスコープ内で定義されています)と表示され、文字の色が灰色になってしまっています..
~~
↑単に同じものを2回宣言してしまっていただけなので、これは解決。

色々調べてみましたが、解決できずにいます。
これを解決するには、どのような方法があるのでしょうか。

やりたいことは、とにかく、グローバル変数として、クラス内の別の箇所でも利用できるようにしたいです。

追記：onPostExecuteメソッドが入ってるLableDetectionTask extends AsyncTask<Object, Void, String>クラスを載せます

    private static class LableDetectionTask extends AsyncTask<Object, Void, String> {
        private final WeakReference<MainActivity> mActivityWeakReference;
        private Vision.Images.Annotate mRequest;

        LableDetectionTask(MainActivity activity, Vision.Images.Annotate annotate) {
            mActivityWeakReference = new WeakReference<>(activity);
            mRequest = annotate;
        }

        @Override
        protected String doInBackground(Object... params) {
            try {
                Log.d(TAG, "created Cloud Vision request object, sending request");
                BatchAnnotateImagesResponse response = mRequest.execute();
                return convertResponseToString(response);

            } catch (GoogleJsonResponseException e) {               
                Log.d(TAG, "failed to make API request because " + e.getContent());
            } catch (IOException e) {
                Log.d(TAG, "failed to make API request because of other IOException " + e.getMessage());
            }
            return "Cloud Vision API request failed. Check logs for details.";
        }


        protected void onPostExecute(String result) {
            MainActivity activity = mActivityWeakReference.get();
            if (activity != null && !activity.isFinishing()) {
                imageDetail = activity.findViewById(R.id.image_details);
                imageDetail.setText(result);

            }
        }
    }

Comments

[maisumakun]

ご提示のコードに「resultText」が見当たらないようです。

[swordone]

そのエラーが出ている場所はどこですか？

[sekaikan_ozaki]

maisumakun様
大変失礼しました。適当な変数としてresultTextという変数を試しに設定したときのエラーメッセージでしたm(__)m
正しくは、"Non-static field 'imageDetail' cannot be referenced from a static context"でした。"resultText"を"imageDetail"に読み替えていただきたいです。
質問の方も修正します。

[sekaikan_ozaki]

swordone様
2つ目のソースコードのif文内のimageDetailです。

[swordone]

onPostExecuteってことは、AsynkTaskクラス内ですか？それ全体見せてください。

[tmp]

エラーメッセージ通り、変数にstaticつけるか、関数のstaticを削るか。あなたしか、わかりません。あとは、エスパーに！

[sekaikan_ozaki]

swordone様
質問の方に加筆修正させていただきます！

[sekaikan_ozaki]

tmp様
＞エラーメッセージ通り、変数にstaticつけるか、関数のstaticを削るか。
この通りに、どちらかに合わせたところ、エラーが直りました！
ありがとうございます！
こちらでは、ベストアンサーにできないみたいなので、自己解決の方に、この解決法を記入して質問をクローズさせていただきます。

Answer 1

問題のAsynkTask系のクラスが、staticクラスとして宣言されているので、外側のActivityとは無関係にインスタンスを生成できます。
つまり外側のActivityとの紐付けがないので、imageDetailと言われても｢どのimageDetailのことを言っているのか｣がわからないのです。

解決するにはどのimageDetailかがはっきりわかるように書けばいいのです。その方法は、
0. imageDetailをstaticにする。
staticにしてクラス唯一にすればわかります。ただしこの方法はおすすめしません。
0. AsynkTask系のクラスをstaticでなくする。
staticをなくすとこれが内部クラスという扱いになり(それまでは｢ネストされたクラス｣)、外側のクラスと紐付いてインスタンスが存在することになります。この場合は、imageDetailと言われれば紐付いたMainActivityのものだとわかるので解決します。
0. WeakReferenceに入れたMainActivityのimageDetailだと書く。
コンストラクタで渡したMainActivityをWeakReferenceに入れているので、それを取得して、そのメンバのimageDetailだと指定すればできます。ただこのケースはアクセス修飾子の問題をクリアできたかうろ覚えです。

Comments

[sekaikan_ozaki]

swordone様
詳しい回答ありがとうございます！
おっしゃる通り、imageDetailをはっきりさせることで解決できました！
ありがとうございました！

Answer 2

質問への追記・修正の依頼内の、tmp様の提案の通りに、変数にstaticつけるか、関数のstaticを削るかを試したところ、エラーが解消し、解決に至りました。