0

私はこのコードを持っています.クラッシュを引き起こす可能性のある他の方法から何かが欠けている場合に備えて、その大部分を含めました。私はインスタンス変数としていくつかの文字列といくつかのArrayListsをデラリングしています。ユーザーはいくつかのEditTextに値を入力し、それらは内部に 入りArrayList(allpret,allcant,allprod)ます。そこから各値を取得して入力するので、Strings(pret,prod,cant)それを参照できますcant[1],pret[1][prod1],etc.実行しようとするとクラッシュするため、修正方法がわかりませんcalculeaza()。

public class MainActivity extends Activity implements OnClickListener,
    TextWatcher {

public Button paginanoua;
public Button produsnou;
public Button Salveaza;

float totaltest = 0;
public int k;
public int j;
public int l;
public TextView totalc;
private static final String TAG = "MEDIA";

public List<EditText> allprod = new ArrayList<EditText>();
public List<EditText> allpret = new ArrayList<EditText>();
public List<EditText> allcant = new ArrayList<EditText>();

String[] cant=new String[allcant.size()];
String[] pret=new String[allpret.size()];
String[] prod=new String[allprod.size()];

public void produsnou() {
    final LinearLayout l1 = (LinearLayout) findViewById(R.id.layout1);
    EditText et = new EditText(this);
    et.setHint("Produs");
    l1.addView(et);
    allprod.add(et);
    et.addTextChangedListener(this);
    et.setOnFocusChangeListener(new OnFocusChangeListener() {

        @Override
        public void onFocusChange(View v, boolean hasFocus) {
            // TODO Auto-generated method stub
             if (hasFocus && 
                       l1.getChildAt(l1.getChildCount() - 1) == v) {
                    produsnou();}}
    });

    LinearLayout l2 = (LinearLayout) findViewById(R.id.layout2);
    EditText et2 = new EditText(this);
    et2.setHint("Cantitate");
    et2.setInputType(InputType.TYPE_CLASS_NUMBER
            | InputType.TYPE_NUMBER_FLAG_DECIMAL);

    allcant.add(et2);
    l2.addView(et2);
    et2.addTextChangedListener(this);

    LinearLayout l3 = (LinearLayout) findViewById(R.id.layout3);
    EditText et3 = new EditText(this);
    et3.setHint("Pret");
    et3.setInputType(InputType.TYPE_CLASS_NUMBER
            | InputType.TYPE_NUMBER_FLAG_DECIMAL);
    l3.addView(et3);
    allpret.add(et3);
    et3.addTextChangedListener(this);}



public void calculeaza() {

    totaltest = 0;
    for (int m = 0; m < allprod.size(); m++) {

        prod[m] = allprod.get(m).getText().toString();
        if (prod[m].matches("")) {
        prod[m] = Float.toString(0);}}



    for (int j = 0; j < allcant.size(); j++) {

        cant[j] = allcant.get(j).getText().toString();
        if (cant[j].matches("")) {
            cant[j] = Float.toString(0);}}



    for (int k = 0; k < allpret.size(); k++) {
        pret[k] = allpret.get(k).getText().toString();
        if (pret[k].matches("")) {
            pret[k] = Float.toString(0);}}

    for (int l = 0; l < allpret.size(); l++) {

        Float temp = Float.parseFloat(cant[l]) * Float.parseFloat(pret[l]);
        alltotal.add(temp);
        totaltest = totaltest + temp;

        TextView totalf = (TextView) findViewById(R.id.total);
        totalf.setText(String.format("Total: %.2f", totaltest));}}

次の LogCat を取得します。

06-13 23:03:03.670: E/AndroidRuntime(15089): 致命的な例外: メイン 06-13 23:03:03.670: E/AndroidRuntime(15089): java.lang.ArrayIndexOutOfBoundsException: length=0; index=0 06-13 23:03:03.670: E/AndroidRuntime(15089): com.example.testlayout.MainActivity.calculeaza(MainActivity.java:209) 06-13 23:03:03.670: E/AndroidRuntime(15089) ): com.example.testlayout.MainActivity.onTextChanged(MainActivity.java:283) 06-13 23:03:03.670: E/AndroidRuntime(15089): android.widget.TextView.sendOnTextChanged(TextView.java:7395) 06-13 23:03:03.670: E/AndroidRuntime(15089): android.widget.TextView.handleTextChanged(TextView.java:7454) 06-13 23:03:03.670: E/AndroidRuntime(15089): Android で。 widget.TextView$ChangeWatcher.onTextChanged(TextView.java:9213) 06-13 23:03:03.670: E/AndroidRuntime(15089): android.text.

209行目はこれです：prod[m] = allprod.get(m).getText().toString();

4

2 に答える 2

4

変数は、コンストラクターが開始する前に、出現順に初期化されます。

public List<EditText> allprod = new ArrayList<EditText>();
String[] prod=new String[allprod.size()];

そのため、 whenprodは初期化され、allprod空であり、そのサイズは 0prodです。空の配列もそうであり、その中には何も配置できません。

代わりに、動的なサイズを持つ ArrayList を使用することを意味していた可能性があります (コードの詳細は見ていません)。

于 2013-06-12T20:15:53.343 に答える
3

空のリストのサイズで初期化するため、文字列配列の長さはゼロです。リストを埋めた後、それらを初期化します。calculeazaつまり、メソッドの最初に。

于 2013-06-12T20:15:36.653 に答える