【Android】CSVを非同期処理して座標を埋め込んだら１つしかマーカーが出なくなった

前回、メインスレッドで緯度経度の入ったｃｓｖファイルを読み込んでgooglemap上に座標を埋め込むことができたのですが、
重かったので,HandleThreadを使って別スレッドで読み込もうとしたら何故かマーカーが一つしか出なくなってしまいました。
HandlerThreadを読み込んで

// CSVの読み込みとマーカー設置.処理が重いので別スレッドで実行.
                    HandlerThread handlerThread = new HandlerThread("AddMarker");
                    handlerThread.start();

                    Handler handler = new Handler(handlerThread.getLooper());

                    handler.post(new Runnable() {
                        @Override
                        public void run() {
                            // AssetManagerの呼び出し
                            AssetManager assetManager = getResources().getAssets();
                            try {
                                // CSVファイルの読み込み
                                InputStream is = assetManager.open("hoge.csv");
                                InputStreamReader inputStreamReader = new InputStreamReader(is);
                                BufferedReader bufferReader = new BufferedReader(inputStreamReader);
                                String line = "";
                                

                                while ((line = bufferReader.readLine()) != null) {
                                        
                                        mMain.mDblLatitude = Double.parseDouble(line.split(",")[0]);
                                        mMain.mDblLongitude = Double.parseDouble(line.split(",")[1]);
                                        // UIスレッドでマーカー設置.
                                        getCsvHandler.sendEmptyMessage(1);
                                }
                                



                                bufferReader.close();
                            } catch (IOException e) {
                                e.printStackTrace();
                                System.out.println("ダメやで");
                            }


                        }
                    });

下のコードをメインスレッドで呼び出しています。


@SuppressLint("HandlerLeak")
    Handler getCsvHandler = new Handler(){
        public void handleMessage(Message msg) {

            addMarker();
        }
    };


        public void addMarker() {
            if (mMap != null) {
                // 表示したマップにマーカーを追加する.

                mMap.addMarker(new MarkerOptions().position(
                        new LatLng(mMain.mDblLatitude, mMain.mDblLongitude))
                .icon(BitmapDescriptorFactory.fromBitmap(resizeMapIcons("book", 200,200))));

            }
        }

悩んでいるのですがよくわかりません。

行動規範の内容に同意します

回答1件

ベストアンサー

おそらくUIスレッドの処理が走るまでにcsvを最後まで読み切ってしまっているのではないでしょうか。mMain.mDblLatitudeとmMain.mDblLongitudeという変数は「1つ」しかなく、csvの行が進むたびに上書き更新されています。getCsvHandler.sendEmptyMessage(1);を呼び出した「瞬間」にaddMarkerが実行されるわけではないので、addMarkerが呼ばれるまでにmDblLatitudeやmDblLongitudeが上書きされてしまえば、それまでに読み込んでいたデータは全て「なかったこと」になります。
読み込んだデータをList等に入れて、addMarkerはそのListから順次読み出していくべきではないでしょうか。

Java
1// mMainのクラスにList<double>型のdataという変数を用意したものとします
2private List<double> data = new ArrayList<>();

Java
1while ((line = bufferReader.readLine()) != null) {
2  String[] values = line.split(",");
3  synchronized(mMain.data) { // addMarker処理と同期を取る
4    mMain.data.add(Double.parseDouble(values[0]));
5    mMain.data.add(Double.parseDouble(values[1]));
6  }
7  // UIスレッドでマーカー設置.
8  getCsvHandler.sendEmptyMessage(1);
9}
10

Java
1public void addMarker() {
2  if (mMap != null) {
3    // 表示したマップにマーカーを追加する.
4    synchronized(mMain.data) { // csv読み込み処理と同期を取る
5      while(!mMain.data.isEmpty()) { // すでに読み込まれているデータが尽きるまで繰り返す
6
7        // dataの先頭2個のデータをlat,lngとして読み込む
8        double lat = mMain.data.remove(0);
9        double lng = mMain.data.remove(0);
10
11        mMap.addMarker(new MarkerOptions().position(new LatLng(lat, lng))
12          .icon(BitmapDescriptorFactory.fromBitmap(resizeMapIcons("book", 200,200))));
13      }
14    }
15  }
16}

投稿2018/01/19 02:09

編集2018/01/23 00:01

masaya_ohashi

総合スコア9206

amiamia

2018/01/22 02:50

ご回答ありがとうございます。ご指摘通りにやったら以下のエラーが出ました。 FATAL EXCEPTION: AddMarker Process: com.example, PID: 10758 java.lang.NullPointerException: Null reference used for synchronization (monitor-enter) どうやらsynchronizedでエラーをはいているようです。。。ちなみに下のサイトを参考にして作っていました。 http://mslgt.hatenablog.com/entry/2015/05/22/011431