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

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

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

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

Android Studio

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

Q&A

解決済

1回答

9531閲覧

Android Studioで別アクティビティのメソッドを呼び出し戻り値を受け取る方法

DRJV

総合スコア5

Java

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

Android Studio

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

0グッド

0クリップ

投稿2019/11/12 04:55

編集2019/11/12 08:51

前提・実現したいこと

Java初心者です。
Android Studio で継承関係にない別アクティビティのメソッドを呼び出したいです。
また、戻り値としてArrayList<String>を受け取りたいと考えています。
具体的には、メインアクティビティにある、CSVを読み込みArrayListにするメソッドを
マップを表示するアクティビティから呼び出し、
CSVに記載してある緯度経度の数値をマップ上にマーカーとして追加しようとしています。

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

Compilation failed;

該当のソースコード

MainActivity.java

Java

1public class MainActivity extends AppCompatActivity { 2 @Override 3 protected void onCreate(Bundle savedInstanceState) { 4 super.onCreate(savedInstanceState); 5 setContentView(R.layout.activity_main); 6 } 7 8 public ArrayList csvRead(int filename) { 9 ArrayList<ArrayList<String>> arrays = new ArrayList<>(); 10 InputStream is = getResources().openRawResource(filename); 11 BufferedReader reader = new BufferedReader(new InputStreamReader(is)); 12 String line; 13 try { 14 while ((line = reader.readLine()) != null) { 15 //カンマ区切りで1つづつ配列に入れる 16 ArrayList<String> array = new ArrayList<>(); 17 String[] RowData = line.split(","); 18 for (int j = 0; j < RowData.length; j++) { 19 array.add(RowData[j]); 20 } 21 arrays.add(array); 22 //Log.d("array1",""+array+""); 23 } 24 //Log.d("array2",""+arrays+""); 25 reader.close(); 26 } catch (IOException e) { 27 e.printStackTrace(); 28 } 29 return arrays; 30 } 31}

SubActivity.java

Java

1 2public class SubActivity extends FragmentActivity implements OnMapReadyCallback { 3 4 private GoogleMap mMap; 5 6 @Override 7 protected void onCreate(Bundle savedInstanceState) { 8 super.onCreate(savedInstanceState); 9 setContentView(R.layout.activity_all_evac_maps); 10 Log.d("debug", "onCreate()"); 11 // Obtain the SupportMapFragment and get notified when the map is ready to be used. 12 SupportMapFragment mapFragment = (SupportMapFragment) getSupportFragmentManager() 13 .findFragmentById(R.id.map); 14 mapFragment.getMapAsync(this); 15 } 16 17 @Override 18 public void onMapReady(GoogleMap googleMap) { 19 Log.d("debug", "onMapReady()"); 20 mMap = googleMap; 21 22 23//問題の部分。メソッドを呼び出せない 24 Intent intent = new Intent(this, MainActivity.class); 25     intent.getStringArrayListExtra(MainActivity.csvRead(R.raw."ファイル名")); 26//コンパイルエラー 27 28 29//for文でaddMarkerをする予定 30    for (int i=0; i<arrayList.length; i++){ 31 //色々; 32 } 33 } 34} 35

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

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

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

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

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

guest

回答1

0

ベストアンサー

Activityを起動(表示)すること以外でインスタンス化しようとすることはお勧めできません。Activityのインスタンス化は、Androidシステムに任せるべきです。質問者さんのケースではインテントなどを使ってインスタンス化しようとするまでもありません。単純にJavaのメソッドの使い方と同じで、MainActivtyクラスのcsvReadメソッドはpublicstaticメソッドにして、

Java

1public static ArrayList csvRead(int filename) { 2 ArrayList<ArrayList<String>> arrays = new ArrayList<>(); 3 //カンマ区切りで1つづつ配列に入れるメソッド(略) 4 //サブアクティビティから呼び出す場合は問題なく動いているのでメソッドに問題はない。 5 return arrays; 6} 7

AllEvacMapsActivityクラスの内部からは、クラス名.メソッドとのように呼び出せば良いです。

Java

1public void onMapReady(GoogleMap googleMap) { 2 Log.d("debug", "onMapReady()"); 3... 4 // 問題の部分。 5 ArrayList<ArrayList<String>> arrayList 6 = MainActivity.csvRead(R.raw."ファイル名"); 7 8 //for文でaddMarkerをする予定 9 for (int i = 0; i < arrayList.size(); i++) { 10 ; // 色々 11 } 12}

回答後のコメントを受けて追記しました:2019-11-12 16:46

基本的にはMainActivitypublicstaticなメソッドを使う、と言う方針のままです。その内インスタンス変数やメソッドに関わる部分を切り出し、リソースIDを引数とするものと、InputStreamを引数とするものとに分けます。

Java

1 /** 2 * csvReadインスタンスメソッド版 3 * @param filenameId リソースID 4 * @return 5 */ 6 public ArrayList csvRead(int filenameId) { 7 InputStream is = getResources().openRawResource(filenameId); 8 ArrayList arrays = csvRead(is); 9 return arrays; 10 } 11 12 /** 13 * csvReadクラスメソッド版 14 * @param is InputStream 15 * @return 16 */ 17 public static ArrayList csvRead(InputStream is) { 18 ArrayList<ArrayList<String>> arrays = new ArrayList<>(); 19 BufferedReader reader = new BufferedReader(new InputStreamReader(is)); 20 String line; 21 try { 22 while ((line = reader.readLine()) != null) { 23 //カンマ区切りで1つづつ配列に入れる 24 ArrayList<String> array = new ArrayList<>(); 25 String[] RowData = line.split(","); 26 for (int j = 0; j < RowData.length; j++) { 27 array.add(RowData[j]); 28 } 29 arrays.add(array); 30 //Log.d("array1",""+array+""); 31 } 32 //Log.d("array2",""+arrays+""); 33 reader.close(); 34 } catch (IOException e) { 35 e.printStackTrace(); 36 } 37 return arrays; 38 }

これをAllEvacMapsActivityから呼び出すときは、

Java

1 // 問題の部分。 2 // MainActivity.csvReadクラスメソッド版を呼び出す。 3 // ファイルリソースからのInputStreamは自力で取得する 4 InputStream is = getResources().openRawResource(R.raw.datafile); 5 ArrayList arrayList = MainActivity.csvRead(is); 6 7 //for文でaddMarkerをする予定 8 for (int i = 0; i < arrayList.size(); i++) { 9 ; 10 } 11

のようにします。

尚、以下のようにするとコンパイル自体はできますが、**まともに動かないので、やってはいけません。**Activityの生成は、
Androidシステムに任せなければなりません。

Java

1// やってはいけない!! 2MainActivity mainActivity = new MainActivity(); 3arrayList = mainActivity.csvRead(R.raw.datafile); 4

Applicationクラスを経由して複数のActivity間でメソッドや変数を参照できるようにする方法もありますが、今回のケースでは過剰スペックだと思います。まずはJavaのメソッドの取り扱いで何とかするのをお勧めします。更に、MainActivityクラスにcsvReadメソッドを持たせるのではなく、それ用のクラスを別に設けてそちらに任せてしまうようにするのも、設計面で考えるとより適切かもしれません。

投稿2019/11/12 05:44

編集2019/11/12 07:46
dodox86

総合スコア9170

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

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

DRJV

2019/11/12 06:29

ご回答ありがとうございます。 csvReadメソッドをstaticなメソッドにした場合、 csvReadメソッド内にある下記の部分にエラーが発生しました。 InputStream is = getResources().openRawResource(filename) エラー文 Non-static method 'getResources()' cannot be referenced from a static context また、これに対するAndroid Studio のヒントマークはstatic でないメソッドにするということでした・・・。 すみません、ご教授お願い致します。
dodox86

2019/11/12 06:31

正直な感想としては、メソッドの切り出し方に問題があるといえばありますね。csvRead()の全部のコードを、質問文中に追記してもらえますでしょうか。
dodox86

2019/11/12 06:38

> extends Fragment をしているアクティビティではこれで呼び出せたのですが・・・。 尚、getActivity()はFragmentクラスが持っているメソッドなので、FragmentActivityでは呼び出せません。
DRJV

2019/11/12 06:40

失礼いたしました。 追記いたしました。
dodox86

2019/11/12 07:47

追記しましたのでご覧ください。
DRJV

2019/11/12 08:50

メソッドを分割するという発想は全くありませんでした。 拙い質問に対してとても丁寧にご回答いただきありがとうございました! おかげで問題を解決することが出来ました。
dodox86

2019/11/12 08:55

メソッド名が同じで引数が違うようなものはメソッドの「オーバーロード」と言いますが、Javaの標準のAPIにも数多くあります。「なぜ分かれているのか」を考えてみるとその理由や意図が分かってきて、自分のプログラミングにも生かせてくると思います。
dodox86

2019/11/12 09:01

(何度もすみません)今思えば必ずしもメソッド名が同じcsvReadじゃなくても良かったです。要は、インスタンスメソッドとクラスメソッドに分けることで使いやすくしよう、との意図でした。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.53%

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

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

質問する

関連した質問