2015-11-01 3 views
0

Я использую AsyncTask для отображения элементов Arraylist в Listview. Но AsyncTask показывает все элементы Arraylist в одном элементе Listview. может ли кто-нибудь предположить, что здесь не так?AsyncTask показывает все элементы списка в одной строке в Listview

Код:

package com.example.ganesh.async; 
import android.os.AsyncTask; 
import android.os.Bundle; 
import android.support.design.widget.FloatingActionButton; 
import android.support.design.widget.Snackbar; 
import android.support.v7.app.AppCompatActivity; 
import android.support.v7.widget.Toolbar; 
import android.view.View; 
import android.view.Menu; 
import android.view.MenuItem; 
import android.widget.ArrayAdapter; 
import android.widget.ListView; 
import android.widget.ProgressBar; 
import android.widget.Toast; 

import java.util.ArrayList; 

public class MainActivity extends AppCompatActivity { 

ProgressBar pb; 
ListView l; 
String s[] = {"MUMBAI, PUNE, DELHI, BANGLORE, KOLKATA, MUMBAI, PUNE, DELHI, BANGLORE, KOLKATA"}; 

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

    l = (ListView) findViewById(R.id.listview); 
    pb = (ProgressBar) findViewById(R.id.progressbar); 
    l.setAdapter(new ArrayAdapter<String>(MainActivity.this, android.R.layout.simple_list_item_1, new ArrayList<String>())); 
    new myTask().execute(); 
} 

class myTask extends AsyncTask{ 

    ArrayAdapter<String> aa; 
    int counter = 0; 
    @Override 
    protected void onPreExecute() { 
     aa = (ArrayAdapter<String>) l.getAdapter(); 
    } 

    @Override 
    protected Void doInBackground(Object[] params) { 
     for(int k=0; k<s.length; k++) 
     { 
      String po = s[k]; 
      publishProgress(po); 
     } 
     return null; 
    } 

    protected void onProgressUpdate(Object[] values) { 
     aa.add(values[0].toString()); 
     counter++; 
     pb.setProgress(((counter/s.length)*10)); 
     Toast.makeText(MainActivity.this," Added "+values[0],Toast.LENGTH_SHORT).show(); 
    } 
    @Override 
    protected void onPostExecute(Object o) { 

    } 
} 
} 

Выход:
Output

ответ

1

Потому что размер вашей строки массива один.

Измените это ....

String s[] = {"MUMBAI, PUNE, DELHI, BANGLORE, KOLKATA, MUMBAI, PUNE, DELHI, BANGLORE, KOLKATA"}; 

это

String s[] = {"MUMBAI", "PUNE", "DELHI", "BANGLORE", "KOLKATA", "MUMBAI", "PUNE", "DELHI", "BANGLORE", "KOLKATA"}; 
+0

OMG да, что хороший глупый mistake.thankyou. – ganesh101

+0

@ganeshIN вы можете отметить этот ответ как принятый. – Ritesh

Смежные вопросы