【android】handlerの中と外で変数の値が変わる

Question

お世話になっております。

タイトルの件、マルチスレッド処理に対して深い理解力がないため原因究明ができないでおります。

詳しく説明すると

        //xmlparse開始
        new Thread(new Runnable(){
            @Override
            public void run(){
                try{
                    URL url = new URL("URL" );
                    Log.d("URL",url.toString());
                    HttpURLConnection con = (HttpURLConnection)url.openConnection();
                    String str = InputStreamToString(con.getInputStream());
                    Log.d("xml",str);
                    XmlPullParser xpp = Xml.newPullParser();
                    try{
                        xpp.setInput(new StringReader(str));
                    }catch (XmlPullParserException e){
                        Log.d("XmlPullParser", "Error");
                    }

//                    XML解析
                    try {
                        int eventType;
                        eventType = xpp.getEventType();

                        while (eventType != xpp.END_DOCUMENT) {
                            if((eventType == xpp.START_TAG)){
                                if("Prospect_Date".equals(xpp.getName())){
                                    parts1 = xpp.nextText();
                                }else if("Prospect_Name".equals(xpp.getName())){
                                    parts2 = xpp.nextText();
                                }else if("Prospect_Number".equals(xpp.getName())){

                                }else if ("countNum".equals(xpp.getName())) {
                                    count = Integer.parseInt(xpp.nextText());
                                    handler.post(new Runnable() {
                                        @Override
                                        public void run() {
                                            btn = new Button[count];
                                            txt2 = new String[count];
                                        }
                                    });
                                }
                            }else if(eventType == xpp.END_TAG){
                                if("listDetail".equals(xpp.getName())){
                                    txt = parts1 + " " + parts2;
                                    Log.d("txt1",txt); **←ここは順番通りテキスト１、テキスト２、テキスト３と入ってくる**
                                    handler.post(new Runnable() {
                                        @Override
                                        public void run() {
                                        Log.d("txt2",txt);←なぜかここはテキスト３、テキスト３、テキスト３と入ってくる
                                            Log.d("intVal",String.valueOf(i));
                                            ll2 = new LinearLayout(List.this);
                                            ll2.setOrientation(LinearLayout.HORIZONTAL);

                                            btn[i] = new Button(List.this);
                                            btn[i].setText(txt);
                                            LinearLayout.LayoutParams lllp = new LinearLayout.LayoutParams(
                                                    LinearLayout.LayoutParams.MATCH_PARENT,
                                                    LinearLayout.LayoutParams.WRAP_CONTENT
                                            );
                                            lllp.weight = 1.0f;
                                            ll2.addView(btn[i],lllp);
                                            Button menu = new Button(List.this);
                                            menu.setText("MENU");
                                            ll2.addView(menu);
                                            ll1.addView(ll2);
                                            btn[i].setOnClickListener(new drawerOpenListener());
                                            i ++;
                                            return;
                                        }
                                    });
                                    i2 ++;
                                }

                            }
                            eventType = xpp.next();
                        }
                    }catch (Exception e){

                    }
                }catch (Exception ex){
                    System.out.println(ex);
                }

            }
        }).start();

コード中のコメントで申し訳ないですが要はhandler外で変数txtをLogに出力してもちゃんとループで受け取った順に出力してくれるのですがhandler内で変数txtをLogに出力するとループで最後に帰ってくる値（ここでいうところのテキスト３という文字列）がループ回数分出力されます。

試したこととしましてはデバッグで順に追った場合はhandler内外ともに正常に出力できました。

おそらくhandler内の処理が追い付かず変数の値を上書きしているのかと思いますが解決策がわかりません。
（変数を配列にしたりも試しましたが自分の知識ではうまくいきませんでした）

どうかよろしくお願いします。

Answer 1

txtの宣言箇所が無いですが、たぶんフィールド変数ですよね？
そうすると、処理の流れは下記のようになります。
1-txtをテキスト1に書き換える、ログ出力。
2-handlerに処理1をポストする。
3-txtをテキスト2に書き換える、ログ出力。
4-handlerに処理2をポストする。
5-txtをテキスト3に書き換える、ログ出力。
6-handlerに処理3をポストする。
7-処理1〜3の実行をする。

つまり、7の段階でtxtはテキスト3になっているので全てテキスト3で処理が実行されてしまいます。

Comments

[hanhunhun]

handlerにポストした処理をその都度実行することはできないですか？
素人のような返しですみません

[yona]

仕組み上無理ですね。

txtをローカル変数にし、final修飾子をつければいいですよ。

[hanhunhun]

ご回答有り難うございます！
なんとか解決しました！
今回の件、掘り下げて学習します！

Answer 2

txtがThreadのさらに外にある理由が良くわかりませんね・・・

txtのスコープ狭めてしまってもいいのなら

final String txt = parts1 + " " + parts2;
Log.d("txt1",txt);

handler.post(new Runnable() {
    @Override
    public void run() {
        Log.d("txt2",txt);
    }
});

スコープを変えずに無理やり書くなら

txt = parts1 + " " + parts2;
Log.d("txt1",txt);

handler.post(new Runnable() {
    private String mTxt;
    public Runnable setTxt(String text){
        mTxt = text;
        return this;
    }

    @Override
    public void run() {
        Log.d("txt2",mTxt);
    }

}.setTxt(txt));

こんな感じですかね？
試してないので、動かなかったらごめんなさい

Comments

[hanhunhun]

スコープに関して知識が浅いばかりに過剰反応するあまりメインスレッドに置いとけばどこでも使えるだろという浅はかな考えでした、、、まだまだ理解不足なところが多いです(*_*)
無事解決しました！ありがとうございます！

[abs123]

今回の問題はスコープではなく、finalを付ける付けないの問題になりますね。
この問題を正しく理解するには、内部クラス、匿名クラスの仕組みを知る必要があります。
気が向いたら「内部クラス ローカル変数」あたりで検索してみてください。