0

for以下のループの何が問題なのか教えていただけますか? プログラムにテキスト内のすべての単語をスキャンしてチェックしてもらいたい - 一連の単語のいずれかに一致することがわかった場合は、それを置き換えます。たとえば、テキスト内の文が「のなちゃん」の場合、単語ごとにスキャンし、「のな」という単語が見つかった場合は、「良い」という単語に置き換えて、次の単語に進みます。

package com.example.split;

import android.os.Bundle;
import android.app.Activity;
import android.view.Menu;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;

public class MainActivity extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        final EditText te1 = (EditText) findViewById(R.id.t1);
        final EditText te2 = (EditText) findViewById(R.id.t2);

        final Button b = (Button) findViewById(R.id.b1);
        b.setOnClickListener(new View.OnClickListener() {

            @Override
            public void onClick(View v) {
                // TODO Auto-generated method stub

                //imva.setImageResource(R.id.b1);

                String t = te1.getText().toString();
                String[] t1 = t.split(" ");
                for (int i = 0; i <= t1.length; i++) {

                    do {
                        if (t1[i].equals("nona")) {
                            String v1 = t1[i];
                            String v2 = " good  ";
                            String a = v1.replace(v1, v2);
                            te2.setText(a);
                        } else if (t1[i].equals("chan")) {
                            String v1 = t1[i];
                            String v3 = " job  ";
                            String a = v1.replace(v1, v3);
                            te2.setText(a);
                        }
                    } while (te1.length() != 0);
                }

            }
        });

    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.main, menu);
        return true;
    }
}
4

2 に答える 2

2

配列の境界を超えています。交換

for (int i = 0 ; i<= t1.length ; i++)

for (int i = 0 ; i < t1.length ; i++)
于 2013-04-01T17:49:50.677 に答える
0

@Reimeusの回答と@Pragnaniのコメントに加えて、どちらもfor()ループが配列の長さを1超えていることは正しいです...

do/whileループは常に無限にループし、アプリがハングするため、ループを削除してください。
置き換えてから、その内容でループします。


また、この行は何もしません:

String a =  v1.replace(v1, v2);

は単純に にインストールv2されるaため、次のように置き換えることができます。

String a = v2;
于 2013-04-01T17:54:38.727 に答える